Merge branch '3.1' into 4.0

code/forms/UserForm.php

@@ -166,4 +166,12 @@ class UserForm extends Form {
 
 		return $attrs;
 	}
+
+    /**
+     * @return string
+     */
+    public function getButtonText() {
+        return $this->config()->get('button_text');
+    }
+
 }

code/model/UserDefinedForm.php

@@ -311,6 +311,7 @@ SQL;
 	public function getCMSValidator() {
 		return new UserFormValidator();
 	}
+
 }
 
 /**

code/model/editableformfields/EditableFormHeading.php

@@ -56,7 +56,8 @@ class EditableFormHeading extends EditableFormField {
 	}
 
 	public function getFormField() {
-		$labelField = new HeaderField($this->Name, $this->EscapedTitle, $this->Level);
+		$labelField = HeaderField::create($this->EscapedTitle)
+            ->setHeadingLevel($this->Level);
 		$labelField->addExtraClass('FormHeading');
 		$labelField->setAttribute('data-id', $this->Name);
 		$this->doUpdateFormField($labelField);

code/model/editableformfields/EditableOption.php

@@ -150,4 +150,15 @@ class EditableOption extends DataObject {
         }
         return $value;
     }
+
+    /**
+     *
+     */
+    protected function onBeforeWrite() {
+        if (!$this->Sort) {
+            $this->Sort = EditableOption::get()->max('Sort') + 1;
+        }
+
+        parent::onBeforeWrite();
+    }
 }

docs/en/installation.md

@@ -84,3 +84,12 @@ UserDefinedForm:
 ````
 
 Any SilverStripe templates placed in your `email_template_directory` directory will be available for use with submission emails.
+
+### Custom Multi-Step button Text
+
+If you want to change the button text when using the Multi-Step/Page Break feature, simply add the following to your `config.yml`:
+
+````
+UserForm:
+  button_text: 'Your Text Here'
+````

javascript/UserForm.js

@@ -468,7 +468,7 @@ jQuery(function ($) {
 		this.$buttons.each(function (i, stepButton) {
 			$(stepButton).on('click', function (e) {
 				e.preventDefault();
-				self.$el.trigger('userform.progress.changestep', [parseInt($(this).text(), 10)]);
+				self.$el.trigger('userform.progress.changestep', [parseInt($(this).data('step'), 10)]);
 			});
 		});
 
@@ -552,7 +552,7 @@ jQuery(function ($) {
 			var $element = $(element),
 				$item = $element.parent();
 
-			if (parseInt($element.text(), 10) === stepNumber && $element.is(':visible')) {
+			if (parseInt($element.data('step'), 10) === stepNumber && $element.is(':visible')) {
 				$item.addClass('current viewed');
 				$element.removeAttr('disabled');
 

templates/Includes/UserFormProgress.ss

@@ -9,7 +9,7 @@
 				<% loop $Steps %>
 				<li class="step-button-wrapper<% if $First %> current<% end_if %>" data-for="$Name">
 					<%-- Remove js-align class to remove javascript positioning --%>
-					<button class="step-button-jump js-align" disabled="disabled">$Pos</button>
+					<button class="step-button-jump js-align" disabled="disabled" data-step="$Pos"><% if $Top.ButtonText %>$Top.ButtonText <% end_if %>$Pos</button>
 				</li>
 				<% end_loop %>
 			</ul>