Merge branch '4'

.travis.yml

@@ -4,9 +4,6 @@ sudo: false
 
 language: php
 
- # Support PHP 5.3
-dist: precise
-
 matrix:
   include:
     - php: 5.3

client/src/bundles/FieldEditor.js

@@ -92,19 +92,11 @@
 							$newField.find('.col-Title input').focus();
 						}
 
-						// animate the row positioning (add the first class)
+						$newField.addClass('flashBackground');
-						if (document.createElement('div').style.animationName !== void 0) {
-							$newField.addClass('newField');
-						}
-
-						// Once the animation has completed
-						setTimeout(function () {
-							$newField.removeClass('newField').addClass('flashBackground');
 						$(".cms-content-fields").scrollTop($(".cms-content-fields")[0].scrollHeight);
 						if($groupEnd) {
 							$groupEnd.css('visibility', 'visible');
 						}
-						}, 500);
 					});
 				});
 			},

client/src/bundles/Recipient.js

@@ -5,12 +5,11 @@
 (function ($) {
 	$(document).ready(function () {
 
+        var sendPlain = $('input[name="SendPlain"]');
 		var recipient = {
 			// Some fields are only visible when HTML email are being sent.
 			updateFormatSpecificFields: function () {
-				var sendPlainChecked = $('#SendPlain')
+				var sendPlainChecked = sendPlain.is(':checked');
-					.find('input[type="checkbox"]')
-					.is(':checked');
 
 				$(".field.toggle-html-only")[sendPlainChecked ? 'hide' : 'show']();
 				$(".field.toggle-plain-only")[sendPlainChecked ? 'show' : 'hide']();
@@ -27,7 +26,7 @@
 				}
 			});
 
-			$('#SendPlain').entwine({
+			sendPlain.entwine({
 				onchange: function () {
 					recipient.updateFormatSpecificFields();
 				}

client/src/styles/userforms-cms.scss

@@ -2,11 +2,6 @@
  * Animations
  */
 
-@keyframes rowSlide {
-	0% {top: 20%;}
-	100% {top: 80%;}
-}
-
 @keyframes flashBackground {
 	0% {background-color: white;}
 	10% {background-color: #dcfedd;}
@@ -44,11 +39,6 @@
 					min-height: 46px;
 				}
 
-				&.newField {
-					position: fixed;
-					animation: rowSlide .5s ease forwards;
-				}
-
 				&.flashBackground {
 					animation: flashBackground 2s linear;
 				}

code/model/editableformfields/EditableCheckboxGroupField.php

@@ -18,8 +18,9 @@ class EditableCheckboxGroupField extends EditableMultipleOptionField
 
     public function getFormField()
     {
-        $field = new UserFormsCheckboxSetField($this->Name, $this->EscapedTitle, $this->getOptionsMap());
+        $field = UserFormsCheckboxSetField::create($this->Name, $this->EscapedTitle, $this->getOptionsMap())
-        $field->setFieldHolderTemplate('UserFormsMultipleOptionField_holder');
+            ->setFieldHolderTemplate('UserFormsMultipleOptionField_holder')
+            ->setTemplate('UserFormsCheckboxSetField');
 
         // Set the default checked items
         $defaultCheckedItems = $this->getDefaultOptions();

code/model/editableformfields/EditableFileField.php

@@ -45,10 +45,14 @@ class EditableFileField extends EditableFormField
             )
         );
 
-        $fields->addFieldToTab("Root.Main", new LiteralField("FileUploadWarning",
+        $fields->addFieldToTab("Root.Main", new LiteralField(
-            "<p class=\"message notice\">" . _t("UserDefinedForm.FileUploadWarning",
+            "FileUploadWarning",
-                "Files uploaded through this field could be publicly accessible if the exact URL is known")
+            "<p class=\"message notice\">"
-            . "</p>"), "Type");
+            . _t(
+                "UserDefinedForm.FileUploadWarning",
+                "Files uploaded through this field could be publicly accessible if the exact URL is known"
+            ) . "</p>"
+        ), "Type");
 
         $fields->addFieldToTab(
             'Root.Main',
@@ -57,6 +61,8 @@ class EditableFileField extends EditableFormField
                 ->setDescription("Note: Maximum php allowed size is {$this->getPHPMaxFileSizeMB()} MB")
         );
 
+        $fields->removeByName('Default');
+
         return $fields;
     }
 

code/model/editableformfields/EditableRadioField.php

@@ -28,8 +28,9 @@ class EditableRadioField extends EditableMultipleOptionField
 
     public function getFormField()
     {
-        $field = OptionsetField::create($this->Name, $this->EscapedTitle, $this->getOptionsMap());
+        $field = OptionsetField::create($this->Name, $this->EscapedTitle, $this->getOptionsMap())
-        $field->setFieldHolderTemplate('UserFormsMultipleOptionField_holder');
+            ->setFieldHolderTemplate('UserFormsMultipleOptionField_holder')
+            ->setTemplate('UserFormsOptionSetField');
 
         // Set default item
         $defaultOption = $this->getDefaultOptions()->first();

code/model/recipients/UserDefinedForm_EmailRecipient.php

@@ -261,10 +261,7 @@ class UserDefinedForm_EmailRecipient extends DataObject
                 ->addExtraClass('toggle-html-only'),
             TextareaField::create('EmailBody', _t('UserDefinedForm.EMAILBODY', 'Body'))
                 ->addExtraClass('toggle-plain-only'),
-            LiteralField::create(
+            LiteralField::create('EmailPreview', $preview)
-                'EmailPreview',
-                '<div id="EmailPreview" class="field toggle-html-only">' . $preview . '</div>'
-            )
         ));
 
         $fields->fieldByName('Root.EmailContent')->setTitle(_t('UserDefinedForm_EmailRecipient.EMAILCONTENTTAB', 'Email Content'));
@@ -421,7 +418,10 @@ class UserDefinedForm_EmailRecipient extends DataObject
      */
     public function getEmailBodyContent()
     {
-        return $this->SendPlain ? $this->EmailBody : $this->EmailBodyHtml;
+        if ($this->SendPlain) {
+            return DBField::create_field('HTMLText', $this->EmailBody)->NoHTML();
+        }
+        return DBField::create_field('HTMLText', $this->EmailBodyHtml)->RAW();
     }
 
     /**

code/model/submissions/SubmittedFileField.php

@@ -55,7 +55,7 @@ class SubmittedFileField extends SubmittedFormField
     {
         if ($file = $this->UploadedFile()) {
             if (trim($file->getFilename(), '/') != trim(ASSETS_DIR, '/')) {
-                return $this->UploadedFile()->URL;
+                return $this->UploadedFile()->AbsoluteLink();
             }
         }
     }

code/model/submissions/SubmittedForm.php

@@ -75,15 +75,20 @@ class SubmittedForm extends DataObject
                 )
             );
 
-            $values = new GridField(
+            $values = GridField::create(
                 'Values',
                 'SubmittedFormField',
                 $self->Values()->sort('Created', 'ASC')
             );
 
+            $exportColumns = array(
+                'Title' => 'Title',
+                'ExportValue' => 'Value'
+            );
+
             $config = new GridFieldConfig();
             $config->addComponent(new GridFieldDataColumns());
-            $config->addComponent(new GridFieldExportButton());
+            $config->addComponent(new GridFieldExportButton('after', $exportColumns));
             $config->addComponent(new GridFieldPrintButton());
             $values->setConfig($config);
 

templates/Includes/UserForm.ss

@@ -9,13 +9,16 @@
 	<p id="{$FormName}_error" class="message $MessageType" aria-hidden="true" style="display: none;"></p>
 <% end_if %>
 
+<% if $Legend %>
     <fieldset>
-	<% if $Legend %><legend>$Legend</legend><% end_if %>
+        <legend>$Legend</legend>
-	<% loop $Fields %>
+        <% include UserFormFields %>
-		$FieldHolder
-	<% end_loop %>
-	<div class="clear"><!-- --></div>
     </fieldset>
+<% else %>
+    <div class="userform-fields">
+        <% include UserFormFields %>
+    </div>
+<% end_if %>
 
 <% if $Steps.Count > 1 %>
 	<% include UserFormStepNav %>

templates/Includes/UserFormFields.ss

@@ -0,0 +1,5 @@
+<% loop $Fields %>
+    $FieldHolder
+<% end_loop %>
+<div class="clear"><!-- --></div>
+

templates/Includes/UserFormStepErrors.ss

@@ -1,8 +1,6 @@
 <% if $Steps.Count > 1 %>
-	<fieldset class="error-container form-wide-errors" aria-hidden="true" style="display: none;">
+    <div class="error-container form-wide-errors" aria-hidden="true" style="display: none;">
-		<div>
         <h4></h4>
         <ul class="error-list"></ul>
     </div>
-	</fieldset>
 <% end_if %>

templates/UserFormsOptionSetField.ss

@@ -0,0 +1,7 @@
+<% loop $Options %>
+    <div class="$Class">
+        <input id="$ID" class="radio" name="$Name" type="radio" value="$Value.ATT"<% if $isChecked %>
+               checked<% end_if %><% if $isDisabled %> disabled<% end_if %> <% if $Up.Required %>required<% end_if %> />
+        <label for="$ID">$Title</label>
+    </div>
+<% end_loop %>

templates/forms/UserFormsCheckboxSetField.ss

@@ -0,0 +1,11 @@
+<% if $Options.Count %>
+    <% loop $Options %>
+        <div class="$Class">
+            <input id="$ID" class="checkbox" name="$Name" type="checkbox" value="$Value.ATT"<% if $isChecked %>
+                   checked="checked"<% end_if %><% if $isDisabled %> disabled="disabled"<% end_if %> />
+            <label for="$ID">$Title</label>
+        </div>
+    <% end_loop %>
+<% else %>
+    <p>No options available</p>
+<% end_if %>

templates/forms/UserFormsLiteralField_holder.ss

@@ -1,5 +1,5 @@
 <div id="$Name" class="field<% if $extraClass %> $extraClass<% end_if %>">
-    <% if $Title %><label class="left">$Title</label><% end_if %>
+    <% if $Title %><p class="left">$Title</p><% end_if %>
     <div class="middleColumn">
         <% loop $FieldList %>
             $Field

templates/forms/UserFormsStepField.ss

@@ -1,15 +1,15 @@
-<fieldset id="$Name" class="form-step $extraClass" data-title="$Title">
+<div id="$Name" class="form-step $extraClass" data-title="$Title">
 	<% if $Form.DisplayErrorMessagesAtTop %>
-		<fieldset class="error-container" aria-hidden="true" style="display: none;">
+		<div class="error-container" aria-hidden="true" style="display: none;">
 			<div>
 				<h4></h4>
 				<ul class="error-list"></ul>
 			</div>
-		</fieldset>
+		</div>
 	<% end_if %>
 
 	<% loop $Children %>
 		$FieldHolder
 	<% end_loop %>
 
-</fieldset>
+</div>

tests/EditableCheckboxGroupFieldTest.php

@@ -0,0 +1,15 @@
+<?php
+
+class EditableCheckboxGroupFieldTest extends SapphireTest
+{
+    protected static $fixture_file = 'EditableFormFieldTest.yml';
+
+    /**
+     * Tests that this element is rendered with a custom template
+     */
+    public function testRenderedWithCustomTemplate()
+    {
+        $checkboxGroup = $this->objFromFixture('EditableCheckboxGroupField', 'checkbox-group');
+        $this->assertEquals('UserFormsCheckboxSetField', $checkboxGroup->getFormField()->getTemplate());
+    }
+}

tests/EditableRadioFieldTest.php

@@ -0,0 +1,15 @@
+<?php
+
+class EditableRadioFieldTest extends SapphireTest
+{
+    protected static $fixture_file = 'EditableFormFieldTest.yml';
+
+    /**
+     * Tests that this element is rendered with a custom template
+     */
+    public function testRenderedWithCustomTemplate()
+    {
+        $radio = $this->objFromFixture('EditableRadioField', 'radio-field');
+        $this->assertEquals('UserFormsOptionSetField', $radio->getFormField()->getTemplate());
+    }
+}

tests/model/recipients/UserDefinedForm_EmailRecipientTest.php

@@ -0,0 +1,24 @@
+<?php
+
+class UserDefinedForm_EmailRecipientTest extends SapphireTest
+{
+    protected static $fixture_file = 'UserDefinedForm_EmailRecipientTest.yml';
+
+    public function testShortcodesAreRenderedInEmailPreviewContent()
+    {
+        $page = $this->objFromFixture('SiteTree', 'about_us');
+
+        $recipient = UserDefinedForm_EmailRecipient::create();
+        $recipient->SendPlain = false;
+        $recipient->EmailBodyHtml = '<p>Some email content. About us: [sitetree_link,id=' . $page->ID . '].</p>';
+
+        $result = $recipient->getEmailBodyContent();
+        $this->assertContains('/about-us/', $result);
+
+        $recipient->SendPlain = true;
+        $recipient->EmailBody = 'Some email content. About us: [sitetree_link,id=' . $page->ID . '].';
+
+        $result = $recipient->getEmailBodyContent();
+        $this->assertContains('/about-us/', $result);
+    }
+}

tests/model/recipients/UserDefinedForm_EmailRecipientTest.yml

@@ -0,0 +1,4 @@
+SiteTree:
+  about_us:
+    Title: About Us
+    URLSegment: about-us