Docs and tests for exempt validation actions

docs/en/02_Developer_Guides/03_Forms/01_Validation.md

@@ -158,6 +158,27 @@ reusable and would not be possible within the `CMS` or other automated `UI` but
 			return $this->redirectBack();
 		}
 	}
+	
+## Exempt validation actions
+
+In some cases you might need to disable validation for specific actions. E.g. actions which discard submitted
+data may not need to check the validity of the posted content.
+
+You can disable validation on individual using one of two methods:
+
+
+	:::php
+	$actions = new FieldList(
+		$action = FormAction::create('doSubmitForm', 'Submit')
+	);
+	$form = new Form($controller, 'MyForm', $fields, $actions);
+	
+	// Disable actions on the form action themselves
+	$action->setValidationExempt(true);
+	
+	// Alternatively, you can whitelist individual actions on the form object by name
+	$form->setValidationExemptActions(['doSubmitForm']);
+
 
 ## Server-side validation messages
 

docs/en/04_Changelogs/4.0.0.md

@@ -89,6 +89,7 @@
    * `hasStages` is addded to check if an object has a given stage.
    * `stageTable` is added to get the table for a given class and stage.
  * `ChangeSet` and `ChangeSetItem` have been added for batch publishing of versioned dataobjects.
+ * `FormAction::setValidationExempt` can be used to turn on or off form validation for individual actions
 
 ### Front-end build tooling for CMS interface
 

forms/Form.php

@@ -719,8 +719,12 @@ class Form extends RequestHandler {
 	 * @return bool
 	 */
 	public function actionIsValidationExempt($action) {
-		if ($action->getValidationExempt()) return true;
-		if (in_array($action->actionName(), $this->getValidationExemptActions())) return true;
+		if ($action->getValidationExempt()) {
+			return true;
+		}
+		if (in_array($action->actionName(), $this->getValidationExemptActions())) {
+			return true;
+		}
 		return false;
 	}
 

tests/forms/FormTest.php

@@ -298,6 +298,28 @@ class FormTest extends FunctionalTest {
 			),
 			'Form->sessionMessage() shows up after reloading the form'
 		);
+
+		// Test this same behaviour, but with a form-action exempted via instance
+		$response = $this->submitForm(
+			'Form_Form',
+			'action_doSubmitActionExempt',
+			array(
+				'Email' => 'test@test.com'
+			)
+		);
+
+		// The required message should be empty if validation was skipped
+		$items = $this->cssParser()->getBySelector('#Form_Form_SomeRequiredField_Holder .required');
+		$this->assertEmpty($items);
+
+		// And the session message should show up is submitted successfully
+		$this->assertPartialMatchBySelector(
+			'#Form_Form_error',
+			array(
+				'Validation bypassed!'
+			),
+			'Form->sessionMessage() shows up after reloading the form'
+		);
 	}
 
 	public function testSessionValidationMessage() {
@@ -773,8 +795,10 @@ class FormTest_Controller extends Controller implements TestOnly {
 				new NumericField('Number')
 			),
 			new FieldList(
-				new FormAction('doSubmit'),
-				new FormAction('doSubmitValidationExempt')
+				FormAction::create('doSubmit'),
+				FormAction::create('doSubmitValidationExempt'),
+				FormAction::create('doSubmitActionExempt')
+					->setValidationExempt(true)
 			),
 			new RequiredFields(
 				'Email',
@@ -797,6 +821,11 @@ class FormTest_Controller extends Controller implements TestOnly {
 		return $this->redirectBack();
 	}
 
+	public function doSubmitActionExempt($data, $form, $request) {
+		$form->sessionMessage('Validation bypassed!', 'good');
+		return $this->redirectBack();
+	}
+
 	public function getViewer($action = null) {
 		return new SSViewer('BlankPage');
 	}