FIX for #4417: Ensuring ->removeValidation() is defined on instances …

…of Validator. Setup new API for enabling/disabling validation. Documentation and better type handling.

forms/Validator.php

@@ -4,7 +4,7 @@
  * This validation class handles all form and custom form validation through
  * the use of Required fields.
  *
- * Relies on javascript for client-side validation, and marking fields after serverside validation.
+ * Relies on javascript for client-side validation, and marking fields after server-side validation.
  *
  * Acts as a visitor to individual form fields.
  *
@@ -21,7 +21,12 @@ abstract class Validator extends Object {
 	/**
 	 * @var array $errors
 	 */
-	protected $errors;
+	protected $errors = array();
+
+	/**
+	 * @var bool
+	 */
+	protected $enabled = true;
 
 	/**
 	 * @param Form $form
@@ -37,13 +42,12 @@ public function setForm($form) {
 	/**
 	 * Returns any errors there may be.
 	 *
-	 * @return null|array
+	 * @return array
 	 */
 	public function validate() {
-		$this->errors = null;
-
-		$this->php($this->form->getData());
-
+		// Reset errors array (if already set).
+		$this->errors = array();
+		if ($this->enabled) $this->php($this->form->getData());
 		return $this->errors;
 	}
 
@@ -81,8 +85,10 @@ public function getErrors() {
 	}
 
 	/**
-	 * @param string $fieldName
-	 * @param array $data
+	 * Ensures the specified field is not empty.
+	 *
+	 * @param	string	$fieldName	The name of the valid you wish to ensure is not empty.
+	 * @param	array	$data		An array of field values you want to check against.
 	 */
 	public function requireField($fieldName, $data) {
 		if(is_array($data[$fieldName]) && count($data[$fieldName])) {
@@ -116,9 +122,33 @@ public function fieldIsRequired($fieldName) {
 	}
 
 	/**
+	 * Performs actual validation. Data from the form is passed into this method.
+	 *
 	 * @param array $data
 	 *
 	 * @return mixed
 	 */
 	abstract public function php($data);
+
+	/**
+	 * Ensures your validator will not perform any validation at all, in case the current user doesn't have access to
+	 * edit any fields (for example).
+	 *
+	 * @param bool	$enabled
+	 * @return static
+	 */
+	public function setEnabled($enabled) {
+		$this->enabled = (bool) $enabled;
+		return $this;
+	}
+
+	/**
+	 * An alias to ->setEnabled(false) for backward compatibility.
+	 *
+	 * @return static
+	 */
+	public function removeValidation() {
+		$this->setEnabled(false);
+		return $this;
+	}
 }