eventespresso / event-espresso-core

core/libraries/form_sections/strategies/validation/EE_Full_HTML_Validation_Strategy.strategy.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      */
     public function __construct($validation_error_message = null)
     {
-        if (! $validation_error_message) {
+        if ( ! $validation_error_message) {
             $validation_error_message = sprintf(
                 esc_html__('Only the following HTML tags are allowed:%1$s%2$s', "event_espresso"),
                 '<br />',

Indentation +52 added lines, -52 removed lines

@@ -13,62 +13,62 @@
  */
 class EE_Full_HTML_Validation_Strategy extends EE_Validation_Strategy_Base
 {
-    /**
-     * @param null $validation_error_message
-     */
-    public function __construct($validation_error_message = null)
-    {
-        if (! $validation_error_message) {
-            $validation_error_message = sprintf(
-                esc_html__('Only the following HTML tags are allowed:%1$s%2$s', "event_espresso"),
-                '<br />',
-                $this->get_list_of_allowed_tags()
-            );
-        }
-        parent::__construct($validation_error_message);
-    }
+	/**
+	 * @param null $validation_error_message
+	 */
+	public function __construct($validation_error_message = null)
+	{
+		if (! $validation_error_message) {
+			$validation_error_message = sprintf(
+				esc_html__('Only the following HTML tags are allowed:%1$s%2$s', "event_espresso"),
+				'<br />',
+				$this->get_list_of_allowed_tags()
+			);
+		}
+		parent::__construct($validation_error_message);
+	}
 
 
-    /**
-     * get_list_of_allowed_tags
-     *
-     * generates and returns a string that lists the top-level HTML tags that are allowable for this input
-     *
-     * @return string
-     */
-    public function get_list_of_allowed_tags()
-    {
-        $tags_we_allow = $this->getAllowedTags();
-        ksort($tags_we_allow);
-        return implode(', ', array_keys($tags_we_allow));
-    }
+	/**
+	 * get_list_of_allowed_tags
+	 *
+	 * generates and returns a string that lists the top-level HTML tags that are allowable for this input
+	 *
+	 * @return string
+	 */
+	public function get_list_of_allowed_tags()
+	{
+		$tags_we_allow = $this->getAllowedTags();
+		ksort($tags_we_allow);
+		return implode(', ', array_keys($tags_we_allow));
+	}
 
 
-    /**
-     * Returns an array whose keys are allowed tags and values are an array of allowed attributes
-     *
-     * @return array
-     */
-    protected function getAllowedTags()
-    {
-        global $allowedposttags;
-        return array_merge_recursive(
-            $allowedposttags,
-            EEH_HTML::get_simple_tags()
-        );
-    }
+	/**
+	 * Returns an array whose keys are allowed tags and values are an array of allowed attributes
+	 *
+	 * @return array
+	 */
+	protected function getAllowedTags()
+	{
+		global $allowedposttags;
+		return array_merge_recursive(
+			$allowedposttags,
+			EEH_HTML::get_simple_tags()
+		);
+	}
 
 
-    /**
-     * @param $normalized_value
-     * @throws \EE_Validation_Error
-     */
-    public function validate($normalized_value)
-    {
-        parent::validate($normalized_value);
-        $normalized_value_sans_tags = wp_kses("$normalized_value", $this->getAllowedTags());
-        if (strlen($normalized_value) > strlen($normalized_value_sans_tags)) {
-            throw new EE_Validation_Error($this->get_validation_error_message(), 'complex_html_tags');
-        }
-    }
+	/**
+	 * @param $normalized_value
+	 * @throws \EE_Validation_Error
+	 */
+	public function validate($normalized_value)
+	{
+		parent::validate($normalized_value);
+		$normalized_value_sans_tags = wp_kses("$normalized_value", $this->getAllowedTags());
+		if (strlen($normalized_value) > strlen($normalized_value_sans_tags)) {
+			throw new EE_Validation_Error($this->get_validation_error_message(), 'complex_html_tags');
+		}
+	}
 }

core/libraries/form_sections/strategies/validation/EE_Credit_Card_Validation_Strategy.strategy.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      */
     public function __construct($validation_error_message = null)
     {
-        if (! $validation_error_message) {
+        if ( ! $validation_error_message) {
             $validation_error_message = esc_html__("Please enter a valid credit card number", "event_espresso");
         }
         parent::__construct(

Indentation +15 added lines, -15 removed lines

@@ -13,19 +13,19 @@
  */
 class EE_Credit_Card_Validation_Strategy extends EE_Text_Validation_Strategy
 {
-    /**
-     * @param null $validation_error_message
-     */
-    public function __construct($validation_error_message = null)
-    {
-        if (! $validation_error_message) {
-            $validation_error_message = esc_html__("Please enter a valid credit card number", "event_espresso");
-        }
-        parent::__construct(
-            $validation_error_message,
-            // @codingStandardsIgnoreStart
-            '~^(?:4[0-9]{12}(?:[0-9]{3})?|5[1-5][0-9]{14}|2[2-7][0-9]{14}|6011[0-9]{12}|3(?:0[0-5]|[68][0-9])[0-9]{11}|3[47][0-9]{13})$~'
-            // @codingStandardsIgnoreEnd
-        );
-    }
+	/**
+	 * @param null $validation_error_message
+	 */
+	public function __construct($validation_error_message = null)
+	{
+		if (! $validation_error_message) {
+			$validation_error_message = esc_html__("Please enter a valid credit card number", "event_espresso");
+		}
+		parent::__construct(
+			$validation_error_message,
+			// @codingStandardsIgnoreStart
+			'~^(?:4[0-9]{12}(?:[0-9]{3})?|5[1-5][0-9]{14}|2[2-7][0-9]{14}|6011[0-9]{12}|3(?:0[0-5]|[68][0-9])[0-9]{11}|3[47][0-9]{13})$~'
+			// @codingStandardsIgnoreEnd
+		);
+	}
 }

core/libraries/form_sections/strategies/validation/EE_Enum_Validation_Strategy.strategy.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
     public function validate($normalized_value)
     {
         parent::validate($normalized_value);
-        if (! $this->_input instanceof EE_Form_Input_With_Options_Base) {
+        if ( ! $this->_input instanceof EE_Form_Input_With_Options_Base) {
             throw new EE_Error(sprintf(esc_html__("Cannot use Enum Validation Strategy with an input that doesn't have options", "event_espresso")));
         }
         $enum_options = $this->_input->flat_options();
@@ -49,7 +49,7 @@ 
     public function get_validation_error_message()
     {
         $parent_validation_error_message = parent::get_validation_error_message();
-        if (! $parent_validation_error_message) {
+        if ( ! $parent_validation_error_message) {
             $enum_options = $this->_input instanceof EE_Form_Input_With_Options_Base ? $this->_input->flat_options() : '';
             return sprintf(
                 esc_html__("This is not allowed option. Allowed options are %s.", "event_espresso"),

Indentation +46 added lines, -46 removed lines

@@ -11,51 +11,51 @@
  */
 class EE_Enum_Validation_Strategy extends EE_Validation_Strategy_Base
 {
-    /**
-     * Check that the value is in the allowed list
-     * @param $normalized_value
-     * @throws EE_Error
-     * @throws EE_Validation_Error
-     * @return boolean
-     */
-    public function validate($normalized_value)
-    {
-        parent::validate($normalized_value);
-        if (! $this->_input instanceof EE_Form_Input_With_Options_Base) {
-            throw new EE_Error(sprintf(esc_html__("Cannot use Enum Validation Strategy with an input that doesn't have options", "event_espresso")));
-        }
-        $enum_options = $this->_input->flat_options();
-        if ($normalized_value === true) {
-            $normalized_value = 1;
-        } elseif ($normalized_value === false) {
-            $normalized_value = 0;
-        }
-        if ($normalized_value !== null && ! array_key_exists($normalized_value, $enum_options)) {
-            throw new EE_Validation_Error(
-                $this->get_validation_error_message(),
-                'invalid_enum_value'
-            );
-        } else {
-            return true;
-        }
-    }
+	/**
+	 * Check that the value is in the allowed list
+	 * @param $normalized_value
+	 * @throws EE_Error
+	 * @throws EE_Validation_Error
+	 * @return boolean
+	 */
+	public function validate($normalized_value)
+	{
+		parent::validate($normalized_value);
+		if (! $this->_input instanceof EE_Form_Input_With_Options_Base) {
+			throw new EE_Error(sprintf(esc_html__("Cannot use Enum Validation Strategy with an input that doesn't have options", "event_espresso")));
+		}
+		$enum_options = $this->_input->flat_options();
+		if ($normalized_value === true) {
+			$normalized_value = 1;
+		} elseif ($normalized_value === false) {
+			$normalized_value = 0;
+		}
+		if ($normalized_value !== null && ! array_key_exists($normalized_value, $enum_options)) {
+			throw new EE_Validation_Error(
+				$this->get_validation_error_message(),
+				'invalid_enum_value'
+			);
+		} else {
+			return true;
+		}
+	}
 
-    /**
-     * If we are using the default validation error message, make it dynamic based
-     * on the allowed options.
-     * @return string
-     */
-    public function get_validation_error_message()
-    {
-        $parent_validation_error_message = parent::get_validation_error_message();
-        if (! $parent_validation_error_message) {
-            $enum_options = $this->_input instanceof EE_Form_Input_With_Options_Base ? $this->_input->flat_options() : '';
-            return sprintf(
-                esc_html__("This is not allowed option. Allowed options are %s.", "event_espresso"),
-                implode(', ', $enum_options)
-            );
-        } else {
-            return $parent_validation_error_message;
-        }
-    }
+	/**
+	 * If we are using the default validation error message, make it dynamic based
+	 * on the allowed options.
+	 * @return string
+	 */
+	public function get_validation_error_message()
+	{
+		$parent_validation_error_message = parent::get_validation_error_message();
+		if (! $parent_validation_error_message) {
+			$enum_options = $this->_input instanceof EE_Form_Input_With_Options_Base ? $this->_input->flat_options() : '';
+			return sprintf(
+				esc_html__("This is not allowed option. Allowed options are %s.", "event_espresso"),
+				implode(', ', $enum_options)
+			);
+		} else {
+			return $parent_validation_error_message;
+		}
+	}
 }

core/libraries/form_sections/strategies/validation/EE_Required_Validation_Strategy.strategy.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      */
     public function __construct($validation_error_message = null)
     {
-        if (! $validation_error_message) {
+        if ( ! $validation_error_message) {
             $validation_error_message = esc_html__("This field is required.", "event_espresso");
         }
         parent::__construct($validation_error_message);

Indentation +44 added lines, -44 removed lines

@@ -11,54 +11,54 @@
  */
 class EE_Required_Validation_Strategy extends EE_Validation_Strategy_Base
 {
-    /**
-     * @param string $validation_error_message
-     */
-    public function __construct($validation_error_message = null)
-    {
-        if (! $validation_error_message) {
-            $validation_error_message = esc_html__("This field is required.", "event_espresso");
-        }
-        parent::__construct($validation_error_message);
-    }
+	/**
+	 * @param string $validation_error_message
+	 */
+	public function __construct($validation_error_message = null)
+	{
+		if (! $validation_error_message) {
+			$validation_error_message = esc_html__("This field is required.", "event_espresso");
+		}
+		parent::__construct($validation_error_message);
+	}
 
 
 
-    /**
-     * just checks the field isn't blank, provided the requirement conditions
-     * indicate this input is still required
-     *
-     * @param $normalized_value
-     * @return bool
-     * @throws \EE_Error
-     * @throws \EE_Validation_Error
-     */
-    public function validate($normalized_value)
-    {
-        if (
-            $normalized_value === ''
-            || $normalized_value === null
-            || $normalized_value === array()
-        ) {
-            throw new EE_Validation_Error($this->get_validation_error_message(), 'required');
-        } else {
-            return true;
-        }
-    }
+	/**
+	 * just checks the field isn't blank, provided the requirement conditions
+	 * indicate this input is still required
+	 *
+	 * @param $normalized_value
+	 * @return bool
+	 * @throws \EE_Error
+	 * @throws \EE_Validation_Error
+	 */
+	public function validate($normalized_value)
+	{
+		if (
+			$normalized_value === ''
+			|| $normalized_value === null
+			|| $normalized_value === array()
+		) {
+			throw new EE_Validation_Error($this->get_validation_error_message(), 'required');
+		} else {
+			return true;
+		}
+	}
 
 
 
-    /**
-     * @return array
-     * @throws \EE_Error
-     */
-    public function get_jquery_validation_rule_array()
-    {
-        return array(
-            'required' => true,
-            'messages' => array(
-                'required' => $this->get_validation_error_message()
-            )
-        );
-    }
+	/**
+	 * @return array
+	 * @throws \EE_Error
+	 */
+	public function get_jquery_validation_rule_array()
+	{
+		return array(
+			'required' => true,
+			'messages' => array(
+				'required' => $this->get_validation_error_message()
+			)
+		);
+	}
 }

core/libraries/form_sections/strategies/validation/EE_Model_Matching_Query_Validation_Strategy.strategy.php

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function __construct($validation_error_message = null, $model_name = '', $query_params = array(), $input_field_name = '')
     {
-        if (! EE_Registry::instance()->is_model_name($model_name)) {
+        if ( ! EE_Registry::instance()->is_model_name($model_name)) {
             throw new EE_Error(sprintf(esc_html__('You must provide a valid model object ', 'event_espresso'), $model_name));
         }
         $this->_model = EE_Registry::instance()->load_model($model_name);
@@ -59,8 +59,8 @@ 
             return true;
         }
         $combined_query_params = $this->get_query_params();
-        $combined_query_params[0][ $this->treat_input_as_field() ] = $normalized_value;
-        if (! $this->get_model()->exists($combined_query_params)) {
+        $combined_query_params[0][$this->treat_input_as_field()] = $normalized_value;
+        if ( ! $this->get_model()->exists($combined_query_params)) {
             throw new EE_Validation_Error($this->get_validation_error_message(), 'no_matching_model_object');
         }
     }

Indentation +71 added lines, -71 removed lines

@@ -13,86 +13,86 @@
  */
 class EE_Model_Matching_Query_Validation_Strategy extends EE_Validation_Strategy_Base
 {
-    /**
-     *
-     * @var EEM_Base
-     */
-    protected $_model;
-    protected $_query_params;
-    protected $_input_field_name;
+	/**
+	 *
+	 * @var EEM_Base
+	 */
+	protected $_model;
+	protected $_query_params;
+	protected $_input_field_name;
 
 
 
-    /**
-     * @param string $validation_error_message
-     * @param string $model_name  name of an EEM_Base model
-     * @param array  $query_params     @see https://github.com/eventespresso/event-espresso-core/tree/master/docs/G--Model-System/model-query-params.md
-     * @param string $input_field_name the input will be treated as this field's value
-     * @throws \EE_Error
-     */
-    public function __construct($validation_error_message = null, $model_name = '', $query_params = array(), $input_field_name = '')
-    {
-        if (! EE_Registry::instance()->is_model_name($model_name)) {
-            throw new EE_Error(sprintf(esc_html__('You must provide a valid model object ', 'event_espresso'), $model_name));
-        }
-        $this->_model = EE_Registry::instance()->load_model($model_name);
-        $this->_query_params = $query_params;
-        if (empty($input_field_name)) {
-            $input_field_name = $this->_model->primary_key_name();
-        }
-        $this->_input_field_name = $input_field_name;
-        parent::__construct($validation_error_message);
-    }
+	/**
+	 * @param string $validation_error_message
+	 * @param string $model_name  name of an EEM_Base model
+	 * @param array  $query_params     @see https://github.com/eventespresso/event-espresso-core/tree/master/docs/G--Model-System/model-query-params.md
+	 * @param string $input_field_name the input will be treated as this field's value
+	 * @throws \EE_Error
+	 */
+	public function __construct($validation_error_message = null, $model_name = '', $query_params = array(), $input_field_name = '')
+	{
+		if (! EE_Registry::instance()->is_model_name($model_name)) {
+			throw new EE_Error(sprintf(esc_html__('You must provide a valid model object ', 'event_espresso'), $model_name));
+		}
+		$this->_model = EE_Registry::instance()->load_model($model_name);
+		$this->_query_params = $query_params;
+		if (empty($input_field_name)) {
+			$input_field_name = $this->_model->primary_key_name();
+		}
+		$this->_input_field_name = $input_field_name;
+		parent::__construct($validation_error_message);
+	}
 
 
 
-    /**
-     * @param $normalized_value
-     * @return bool|void
-     * @throws \EE_Error
-     * @throws \EE_Validation_Error
-     */
-    public function validate($normalized_value)
-    {
-        if (empty($normalized_value)) {
-            return true;
-        }
-        $combined_query_params = $this->get_query_params();
-        $combined_query_params[0][ $this->treat_input_as_field() ] = $normalized_value;
-        if (! $this->get_model()->exists($combined_query_params)) {
-            throw new EE_Validation_Error($this->get_validation_error_message(), 'no_matching_model_object');
-        }
-    }
+	/**
+	 * @param $normalized_value
+	 * @return bool|void
+	 * @throws \EE_Error
+	 * @throws \EE_Validation_Error
+	 */
+	public function validate($normalized_value)
+	{
+		if (empty($normalized_value)) {
+			return true;
+		}
+		$combined_query_params = $this->get_query_params();
+		$combined_query_params[0][ $this->treat_input_as_field() ] = $normalized_value;
+		if (! $this->get_model()->exists($combined_query_params)) {
+			throw new EE_Validation_Error($this->get_validation_error_message(), 'no_matching_model_object');
+		}
+	}
 
-    /**
-     * Gets the model used for querying
-     * @return EEM_Base
-     */
-    public function get_model()
-    {
-        return $this->_model;
-    }
+	/**
+	 * Gets the model used for querying
+	 * @return EEM_Base
+	 */
+	public function get_model()
+	{
+		return $this->_model;
+	}
 
-    /**
-     * Returns query params used for model query
-     * @return array
-     */
-    public function get_query_params()
-    {
-        return (array) $this->_query_params;
-    }
+	/**
+	 * Returns query params used for model query
+	 * @return array
+	 */
+	public function get_query_params()
+	{
+		return (array) $this->_query_params;
+	}
 
-    /**
-     * Gets the name of the field that will be used for lookup.
-     * eg it could be "EVT_name", meaning that if there is a model object in
-     * the database that has that event name, and matching the other query parameters
-     * on this strategy, the input will pass validation server-side
-     * @return string
-     */
-    public function treat_input_as_field()
-    {
-        return $this->_input_field_name;
-    }
+	/**
+	 * Gets the name of the field that will be used for lookup.
+	 * eg it could be "EVT_name", meaning that if there is a model object in
+	 * the database that has that event name, and matching the other query parameters
+	 * on this strategy, the input will pass validation server-side
+	 * @return string
+	 */
+	public function treat_input_as_field()
+	{
+		return $this->_input_field_name;
+	}
 }
 
 // End of file EE_FUll_HTML_Validation_Strategy.strategy.php

core/libraries/form_sections/strategies/validation/EE_Plaintext_Validation_Strategy.strategy.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      */
     public function __construct($validation_error_message = null)
     {
-        if (! $validation_error_message) {
+        if ( ! $validation_error_message) {
             $validation_error_message = esc_html__("HTML tags are not permitted in this field", "event_espresso");
         }
         parent::__construct($validation_error_message);

Indentation +23 added lines, -23 removed lines

@@ -13,28 +13,28 @@
  */
 class EE_Plaintext_Validation_Strategy extends EE_Validation_Strategy_Base
 {
-    /**
-     * @param null $validation_error_message
-     */
-    public function __construct($validation_error_message = null)
-    {
-        if (! $validation_error_message) {
-            $validation_error_message = esc_html__("HTML tags are not permitted in this field", "event_espresso");
-        }
-        parent::__construct($validation_error_message);
-    }
+	/**
+	 * @param null $validation_error_message
+	 */
+	public function __construct($validation_error_message = null)
+	{
+		if (! $validation_error_message) {
+			$validation_error_message = esc_html__("HTML tags are not permitted in this field", "event_espresso");
+		}
+		parent::__construct($validation_error_message);
+	}
 
-    /**
-     * @param string|null $normalized_value
-     * @throws EE_Validation_Error
-     */
-    public function validate($normalized_value)
-    {
-        $normalized_value = (string) $normalized_value;
-        $no_tags = wp_strip_all_tags($normalized_value);
-        if (strlen($no_tags) < strlen(trim($normalized_value))) {
-            throw new EE_Validation_Error($this->get_validation_error_message(), 'no_html_tags');
-        }
-        parent::validate($normalized_value);
-    }
+	/**
+	 * @param string|null $normalized_value
+	 * @throws EE_Validation_Error
+	 */
+	public function validate($normalized_value)
+	{
+		$normalized_value = (string) $normalized_value;
+		$no_tags = wp_strip_all_tags($normalized_value);
+		if (strlen($no_tags) < strlen(trim($normalized_value))) {
+			throw new EE_Validation_Error($this->get_validation_error_message(), 'no_html_tags');
+		}
+		parent::validate($normalized_value);
+	}
 }

core/libraries/form_sections/strategies/validation/EE_Conditionally_Required_Validation_Strategy.strategy.php

Spacing +11 added lines, -11 removed lines

@@ -28,7 +28,7 @@ 
      */
     public function __construct($validation_error_message = null, $requirement_conditions = array())
     {
-        if (! $validation_error_message) {
+        if ( ! $validation_error_message) {
             $validation_error_message = esc_html__("This field is required.", "event_espresso");
         }
         $this->set_requirement_conditions($requirement_conditions);
@@ -116,7 +116,7 @@ 
     {
         $requirement_value = '';
         $conditions = $this->get_requirement_conditions();
-        if (! is_array($conditions)) {
+        if ( ! is_array($conditions)) {
             EE_Error::throw_exception_if_debugging(
                 sprintf(
                     esc_html__('Input requirement conditions must be an array. You provided %1$s', 'event_espresso'),
@@ -141,7 +141,7 @@ 
         }
         foreach ($conditions as $input_path => $op_and_value) {
             $input = $this->_input->find_section_from_path($input_path);
-            if (! $input instanceof EE_Form_Input_Base) {
+            if ( ! $input instanceof EE_Form_Input_Base) {
                 EE_Error::throw_exception_if_debugging(
                     sprintf(
                         esc_html__('Error encountered while setting requirement condition for input %1$s. The path %2$s does not correspond to a valid input', 'event_espresso'),
@@ -154,9 +154,9 @@ 
                 );
                 return false;
             }
-            list( $op, $value ) = $this->_validate_op_and_value($op_and_value);
+            list($op, $value) = $this->_validate_op_and_value($op_and_value);
             // ok now the jquery dependency expression depends on the input's display strategy.
-            if (! $input->get_display_strategy() instanceof EE_Select_Display_Strategy) {
+            if ( ! $input->get_display_strategy() instanceof EE_Select_Display_Strategy) {
                 EE_Error::throw_exception_if_debugging(
                     sprintf(
                         esc_html__('Required Validation Strategy can only depend on another input which uses the EE_Select_Display_Strategy, but you specified a field "%1$s" that uses display strategy "%2$s". If you need others, please add support for it! The related input is %3$s', 'event_espresso'),
@@ -169,7 +169,7 @@ 
                     __LINE__
                 );
             }
-            $requirement_value = $input->html_id(true) . ' option[value="' . $value . '"]:selected';
+            $requirement_value = $input->html_id(true).' option[value="'.$value.'"]:selected';
         }
         return $requirement_value;
     }
@@ -190,7 +190,7 @@ 
         $conditions = $this->get_requirement_conditions();
         foreach ($conditions as $input_path => $op_and_value) {
             $input = $this->_input->find_section_from_path($input_path);
-            if (! $input instanceof EE_Form_Input_Base) {
+            if ( ! $input instanceof EE_Form_Input_Base) {
                 EE_Error::throw_exception_if_debugging(
                     sprintf(
                         esc_html__('Error encountered while setting requirement condition for input %1$s. The path %2$s does not correspond to a valid input', 'event_espresso'),
@@ -203,13 +203,13 @@ 
                 );
                 return false;
             }
-            list( $op, $value ) = $this->_validate_op_and_value($op_and_value);
+            list($op, $value) = $this->_validate_op_and_value($op_and_value);
             switch ($op) {
                 case '=':
                 default:
                     $meets_all_requirements = $input->normalized_value() === $value;
             }
-            if (! $meets_all_requirements) {
+            if ( ! $meets_all_requirements) {
                 break;
             }
         }
@@ -228,7 +228,7 @@ 
      */
     protected function _validate_op_and_value($op_and_value)
     {
-        if (! isset($op_and_value[0], $op_and_value[1])) {
+        if ( ! isset($op_and_value[0], $op_and_value[1])) {
                 EE_Error::throw_exception_if_debugging(
                     sprintf(
                         esc_html__('Required Validation Strategy conditions array\'s value must be an array with two elements: an operator, and a value. It didn\'t. The related input is %1$s', 'event_espresso'),
@@ -252,6 +252,6 @@ 
                 __LINE__
             );
         }
-            return array( $operator, $value );
+            return array($operator, $value);
     }
 }

Indentation +222 added lines, -222 removed lines

@@ -11,246 +11,246 @@
  */
 class EE_Conditionally_Required_Validation_Strategy extends EE_Validation_Strategy_Base
 {
-    /**
-     * Array describing conditions necessary to make the input required.
-     * This is used to derive a jquery dependency expression (see http://jqueryvalidation.org/required-method)
-     * or jquery callback; and server-side logic to determine if the field is necessary.
-     * @var array
-     */
-    protected $requirement_conditions;
+	/**
+	 * Array describing conditions necessary to make the input required.
+	 * This is used to derive a jquery dependency expression (see http://jqueryvalidation.org/required-method)
+	 * or jquery callback; and server-side logic to determine if the field is necessary.
+	 * @var array
+	 */
+	protected $requirement_conditions;
 
 
 
-    /**
-     * @param string $validation_error_message
-     * @param array $requirement_conditions
-     */
-    public function __construct($validation_error_message = null, $requirement_conditions = array())
-    {
-        if (! $validation_error_message) {
-            $validation_error_message = esc_html__("This field is required.", "event_espresso");
-        }
-        $this->set_requirement_conditions($requirement_conditions);
-        parent::__construct($validation_error_message);
-    }
+	/**
+	 * @param string $validation_error_message
+	 * @param array $requirement_conditions
+	 */
+	public function __construct($validation_error_message = null, $requirement_conditions = array())
+	{
+		if (! $validation_error_message) {
+			$validation_error_message = esc_html__("This field is required.", "event_espresso");
+		}
+		$this->set_requirement_conditions($requirement_conditions);
+		parent::__construct($validation_error_message);
+	}
 
 
 
-    /**
-     * just checks the field isn't blank, provided the requirement conditions
-     * indicate this input is still required
-     *
-     * @param $normalized_value
-     * @return bool
-     * @throws \EE_Error
-     * @throws \EE_Validation_Error
-     */
-    public function validate($normalized_value)
-    {
-        if (
-            (
-                $normalized_value === ''
-                || $normalized_value === null
-                || $normalized_value === array()
-            )
-            && $this->_input_is_required_server_side()
-        ) {
-            throw new EE_Validation_Error($this->get_validation_error_message(), 'required');
-        } else {
-            return true;
-        }
-    }
+	/**
+	 * just checks the field isn't blank, provided the requirement conditions
+	 * indicate this input is still required
+	 *
+	 * @param $normalized_value
+	 * @return bool
+	 * @throws \EE_Error
+	 * @throws \EE_Validation_Error
+	 */
+	public function validate($normalized_value)
+	{
+		if (
+			(
+				$normalized_value === ''
+				|| $normalized_value === null
+				|| $normalized_value === array()
+			)
+			&& $this->_input_is_required_server_side()
+		) {
+			throw new EE_Validation_Error($this->get_validation_error_message(), 'required');
+		} else {
+			return true;
+		}
+	}
 
 
 
-    /**
-     * @return array
-     * @throws \EE_Error
-     */
-    public function get_jquery_validation_rule_array()
-    {
-        return array(
-            'required' => $this->_get_jquery_requirement_value(),
-            'messages' => array(
-                'required' => $this->get_validation_error_message()
-            )
-        );
-    }
+	/**
+	 * @return array
+	 * @throws \EE_Error
+	 */
+	public function get_jquery_validation_rule_array()
+	{
+		return array(
+			'required' => $this->_get_jquery_requirement_value(),
+			'messages' => array(
+				'required' => $this->get_validation_error_message()
+			)
+		);
+	}
 
-    /**
-     * Sets the "required conditions". This should be an array, its top-level key
-     * is the name of a field, its value is an array. This 2nd level array has two items:
-     * the first is the operator (for now only '=' is accepted), and teh 2nd argument is the
-     * the value the field should be in order to make the field required.
-     * Eg array( 'payment_type' => array( '=', 'credit_card' ).
-     *
-     * @param array $requirement_conditions
-     */
-    public function set_requirement_conditions($requirement_conditions)
-    {
-        $this->requirement_conditions = (array) $requirement_conditions;
-    }
+	/**
+	 * Sets the "required conditions". This should be an array, its top-level key
+	 * is the name of a field, its value is an array. This 2nd level array has two items:
+	 * the first is the operator (for now only '=' is accepted), and teh 2nd argument is the
+	 * the value the field should be in order to make the field required.
+	 * Eg array( 'payment_type' => array( '=', 'credit_card' ).
+	 *
+	 * @param array $requirement_conditions
+	 */
+	public function set_requirement_conditions($requirement_conditions)
+	{
+		$this->requirement_conditions = (array) $requirement_conditions;
+	}
 
-    /**
-     * Gets the array that describes when the related input should be required.
-     * see set_requirement_conditions for a description of how it should be formatted
-     * @return array
-     */
-    public function get_requirement_conditions()
-    {
-        return $this->requirement_conditions;
-    }
+	/**
+	 * Gets the array that describes when the related input should be required.
+	 * see set_requirement_conditions for a description of how it should be formatted
+	 * @return array
+	 */
+	public function get_requirement_conditions()
+	{
+		return $this->requirement_conditions;
+	}
 
 
 
-    /**
-     * gets jQuery dependency expression used for client-side validation
-     * Its possible this could also return a javascript callback used for determining
-     * if the input is required or not. That is not yet implemented, however.
-     *
-     * @return string see http://jqueryvalidation.org/required-method for format
-     * @throws \EE_Error
-     */
-    protected function _get_jquery_requirement_value()
-    {
-        $requirement_value = '';
-        $conditions = $this->get_requirement_conditions();
-        if (! is_array($conditions)) {
-            EE_Error::throw_exception_if_debugging(
-                sprintf(
-                    esc_html__('Input requirement conditions must be an array. You provided %1$s', 'event_espresso'),
-                    $this->_input->name()
-                ),
-                __FILE__,
-                __FUNCTION__,
-                __LINE__
-            );
-            return true;
-        }
-        if (count($conditions) > 1) {
-            EE_Error::throw_exception_if_debugging(
-                sprintf(
-                    esc_html__('Required Validation Strategy does not yet support multiple conditions. You should add it! The related input is %1$s', 'event_espresso'),
-                    $this->_input->name()
-                ),
-                __FILE__,
-                __FUNCTION__,
-                __LINE__
-            );
-        }
-        foreach ($conditions as $input_path => $op_and_value) {
-            $input = $this->_input->find_section_from_path($input_path);
-            if (! $input instanceof EE_Form_Input_Base) {
-                EE_Error::throw_exception_if_debugging(
-                    sprintf(
-                        esc_html__('Error encountered while setting requirement condition for input %1$s. The path %2$s does not correspond to a valid input', 'event_espresso'),
-                        $this->_input->name(),
-                        $input_path
-                    ),
-                    __FILE__,
-                    __FUNCTION__,
-                    __LINE__
-                );
-                return false;
-            }
-            list( $op, $value ) = $this->_validate_op_and_value($op_and_value);
-            // ok now the jquery dependency expression depends on the input's display strategy.
-            if (! $input->get_display_strategy() instanceof EE_Select_Display_Strategy) {
-                EE_Error::throw_exception_if_debugging(
-                    sprintf(
-                        esc_html__('Required Validation Strategy can only depend on another input which uses the EE_Select_Display_Strategy, but you specified a field "%1$s" that uses display strategy "%2$s". If you need others, please add support for it! The related input is %3$s', 'event_espresso'),
-                        $input->name(),
-                        get_class($input->get_display_strategy()),
-                        $this->_input->name()
-                    ),
-                    __FILE__,
-                    __FUNCTION__,
-                    __LINE__
-                );
-            }
-            $requirement_value = $input->html_id(true) . ' option[value="' . $value . '"]:selected';
-        }
-        return $requirement_value;
-    }
+	/**
+	 * gets jQuery dependency expression used for client-side validation
+	 * Its possible this could also return a javascript callback used for determining
+	 * if the input is required or not. That is not yet implemented, however.
+	 *
+	 * @return string see http://jqueryvalidation.org/required-method for format
+	 * @throws \EE_Error
+	 */
+	protected function _get_jquery_requirement_value()
+	{
+		$requirement_value = '';
+		$conditions = $this->get_requirement_conditions();
+		if (! is_array($conditions)) {
+			EE_Error::throw_exception_if_debugging(
+				sprintf(
+					esc_html__('Input requirement conditions must be an array. You provided %1$s', 'event_espresso'),
+					$this->_input->name()
+				),
+				__FILE__,
+				__FUNCTION__,
+				__LINE__
+			);
+			return true;
+		}
+		if (count($conditions) > 1) {
+			EE_Error::throw_exception_if_debugging(
+				sprintf(
+					esc_html__('Required Validation Strategy does not yet support multiple conditions. You should add it! The related input is %1$s', 'event_espresso'),
+					$this->_input->name()
+				),
+				__FILE__,
+				__FUNCTION__,
+				__LINE__
+			);
+		}
+		foreach ($conditions as $input_path => $op_and_value) {
+			$input = $this->_input->find_section_from_path($input_path);
+			if (! $input instanceof EE_Form_Input_Base) {
+				EE_Error::throw_exception_if_debugging(
+					sprintf(
+						esc_html__('Error encountered while setting requirement condition for input %1$s. The path %2$s does not correspond to a valid input', 'event_espresso'),
+						$this->_input->name(),
+						$input_path
+					),
+					__FILE__,
+					__FUNCTION__,
+					__LINE__
+				);
+				return false;
+			}
+			list( $op, $value ) = $this->_validate_op_and_value($op_and_value);
+			// ok now the jquery dependency expression depends on the input's display strategy.
+			if (! $input->get_display_strategy() instanceof EE_Select_Display_Strategy) {
+				EE_Error::throw_exception_if_debugging(
+					sprintf(
+						esc_html__('Required Validation Strategy can only depend on another input which uses the EE_Select_Display_Strategy, but you specified a field "%1$s" that uses display strategy "%2$s". If you need others, please add support for it! The related input is %3$s', 'event_espresso'),
+						$input->name(),
+						get_class($input->get_display_strategy()),
+						$this->_input->name()
+					),
+					__FILE__,
+					__FUNCTION__,
+					__LINE__
+				);
+			}
+			$requirement_value = $input->html_id(true) . ' option[value="' . $value . '"]:selected';
+		}
+		return $requirement_value;
+	}
 
 
 
-    /**
-     * Returns whether or not this input is required based on the _requirement_conditions
-     * (not whether or not the input passes validation. That's for the validate method
-     * to decide)
-     *
-     * @return boolean
-     * @throws \EE_Error
-     */
-    protected function _input_is_required_server_side()
-    {
-        $meets_all_requirements = true;
-        $conditions = $this->get_requirement_conditions();
-        foreach ($conditions as $input_path => $op_and_value) {
-            $input = $this->_input->find_section_from_path($input_path);
-            if (! $input instanceof EE_Form_Input_Base) {
-                EE_Error::throw_exception_if_debugging(
-                    sprintf(
-                        esc_html__('Error encountered while setting requirement condition for input %1$s. The path %2$s does not correspond to a valid input', 'event_espresso'),
-                        $this->_input->name(),
-                        $input_path
-                    ),
-                    __FILE__,
-                    __FUNCTION__,
-                    __LINE__
-                );
-                return false;
-            }
-            list( $op, $value ) = $this->_validate_op_and_value($op_and_value);
-            switch ($op) {
-                case '=':
-                default:
-                    $meets_all_requirements = $input->normalized_value() === $value;
-            }
-            if (! $meets_all_requirements) {
-                break;
-            }
-        }
-        return $meets_all_requirements;
-    }
+	/**
+	 * Returns whether or not this input is required based on the _requirement_conditions
+	 * (not whether or not the input passes validation. That's for the validate method
+	 * to decide)
+	 *
+	 * @return boolean
+	 * @throws \EE_Error
+	 */
+	protected function _input_is_required_server_side()
+	{
+		$meets_all_requirements = true;
+		$conditions = $this->get_requirement_conditions();
+		foreach ($conditions as $input_path => $op_and_value) {
+			$input = $this->_input->find_section_from_path($input_path);
+			if (! $input instanceof EE_Form_Input_Base) {
+				EE_Error::throw_exception_if_debugging(
+					sprintf(
+						esc_html__('Error encountered while setting requirement condition for input %1$s. The path %2$s does not correspond to a valid input', 'event_espresso'),
+						$this->_input->name(),
+						$input_path
+					),
+					__FILE__,
+					__FUNCTION__,
+					__LINE__
+				);
+				return false;
+			}
+			list( $op, $value ) = $this->_validate_op_and_value($op_and_value);
+			switch ($op) {
+				case '=':
+				default:
+					$meets_all_requirements = $input->normalized_value() === $value;
+			}
+			if (! $meets_all_requirements) {
+				break;
+			}
+		}
+		return $meets_all_requirements;
+	}
 
 
 
-    /**
-     * Verifies this is an array with keys 0 and 1, where key 0 is a usable
-     * operator (initially just '=') and key 1 is something that can be cast to a string
-     *
-     * @param array $op_and_value
-     * @return array
-     * @throws \EE_Error
-     */
-    protected function _validate_op_and_value($op_and_value)
-    {
-        if (! isset($op_and_value[0], $op_and_value[1])) {
-                EE_Error::throw_exception_if_debugging(
-                    sprintf(
-                        esc_html__('Required Validation Strategy conditions array\'s value must be an array with two elements: an operator, and a value. It didn\'t. The related input is %1$s', 'event_espresso'),
-                        $this->_input->name()
-                    ),
-                    __FILE__,
-                    __FUNCTION__,
-                    __LINE__
-                );
-        }
-            $operator = $op_and_value[0];
-            $value = (string) $op_and_value[1];
-        if ($operator !== '=') {
-            EE_Error::throw_exception_if_debugging(
-                sprintf(
-                    esc_html__('Required Validation Strategy conditions can currently only use the equals operator. If you need others, please add support for it! The related input is %1$s', 'event_espresso'),
-                    $this->_input->name()
-                ),
-                __FILE__,
-                __FUNCTION__,
-                __LINE__
-            );
-        }
-            return array( $operator, $value );
-    }
+	/**
+	 * Verifies this is an array with keys 0 and 1, where key 0 is a usable
+	 * operator (initially just '=') and key 1 is something that can be cast to a string
+	 *
+	 * @param array $op_and_value
+	 * @return array
+	 * @throws \EE_Error
+	 */
+	protected function _validate_op_and_value($op_and_value)
+	{
+		if (! isset($op_and_value[0], $op_and_value[1])) {
+				EE_Error::throw_exception_if_debugging(
+					sprintf(
+						esc_html__('Required Validation Strategy conditions array\'s value must be an array with two elements: an operator, and a value. It didn\'t. The related input is %1$s', 'event_espresso'),
+						$this->_input->name()
+					),
+					__FILE__,
+					__FUNCTION__,
+					__LINE__
+				);
+		}
+			$operator = $op_and_value[0];
+			$value = (string) $op_and_value[1];
+		if ($operator !== '=') {
+			EE_Error::throw_exception_if_debugging(
+				sprintf(
+					esc_html__('Required Validation Strategy conditions can currently only use the equals operator. If you need others, please add support for it! The related input is %1$s', 'event_espresso'),
+					$this->_input->name()
+				),
+				__FILE__,
+				__FUNCTION__,
+				__LINE__
+			);
+		}
+			return array( $operator, $value );
+	}
 }

core/libraries/form_sections/strategies/validation/EE_Float_Validation_Strategy.strategy.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
      */
     public function __construct($validation_error_message = null)
     {
-        if (! $validation_error_message) {
+        if ( ! $validation_error_message) {
             $validation_error_message = sprintf(esc_html__("Only numeric characters, commas, periods, and spaces, please!", "event_espresso"));
         }
         parent::__construct($validation_error_message);
@@ -42,6 +42,6 @@ 
      */
     public function get_jquery_validation_rule_array()
     {
-        return array('number' => true, 'messages' => array( 'number' => $this->get_validation_error_message() ) );
+        return array('number' => true, 'messages' => array('number' => $this->get_validation_error_message()));
     }
 }

Indentation +26 added lines, -26 removed lines

@@ -11,36 +11,36 @@
  */
 class EE_Float_Validation_Strategy extends EE_Validation_Strategy_Base
 {
-    /**
-     * @param null $validation_error_message
-     */
-    public function __construct($validation_error_message = null)
-    {
-        if (! $validation_error_message) {
-            $validation_error_message = sprintf(esc_html__("Only numeric characters, commas, periods, and spaces, please!", "event_espresso"));
-        }
-        parent::__construct($validation_error_message);
-    }
+	/**
+	 * @param null $validation_error_message
+	 */
+	public function __construct($validation_error_message = null)
+	{
+		if (! $validation_error_message) {
+			$validation_error_message = sprintf(esc_html__("Only numeric characters, commas, periods, and spaces, please!", "event_espresso"));
+		}
+		parent::__construct($validation_error_message);
+	}
 
 
 
-    /**
-     *
-     * @param $normalized_value
-     * @return bool
-     */
-    public function validate($normalized_value)
-    {
-        // errors should have been detected by the normalization strategy
-    }
+	/**
+	 *
+	 * @param $normalized_value
+	 * @return bool
+	 */
+	public function validate($normalized_value)
+	{
+		// errors should have been detected by the normalization strategy
+	}
 
 
 
-    /**
-     * @return array
-     */
-    public function get_jquery_validation_rule_array()
-    {
-        return array('number' => true, 'messages' => array( 'number' => $this->get_validation_error_message() ) );
-    }
+	/**
+	 * @return array
+	 */
+	public function get_jquery_validation_rule_array()
+	{
+		return array('number' => true, 'messages' => array( 'number' => $this->get_validation_error_message() ) );
+	}
 }

core/libraries/form_sections/strategies/validation/EE_Simple_HTML_Validation_Strategy.strategy.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
      */
     public function __construct($validation_error_message = null)
     {
-        if (! $validation_error_message) {
+        if ( ! $validation_error_message) {
             $allowedtags = $this->_get_allowed_tags();
             $validation_error_message = sprintf(esc_html__("Only simple HTML tags are allowed. Eg, %s", "event_espresso"), implode(",", array_keys($allowedtags)));
         }
@@ -62,7 +62,7 @@ 
     {
         $allowedtags = $this->_get_allowed_tags();
         parent::validate($normalized_value);
-        $normalized_value_sans_tags =  wp_kses("$normalized_value", $allowedtags);
+        $normalized_value_sans_tags = wp_kses("$normalized_value", $allowedtags);
         if (strlen($normalized_value) > strlen($normalized_value_sans_tags)) {
             throw new EE_Validation_Error($this->get_validation_error_message(), 'complex_html_tags');
         }

Indentation +44 added lines, -44 removed lines

@@ -13,57 +13,57 @@
  */
 class EE_Simple_HTML_Validation_Strategy extends EE_Validation_Strategy_Base
 {
-    /**
-     * @param null $validation_error_message
-     */
-    public function __construct($validation_error_message = null)
-    {
-        if (! $validation_error_message) {
-            $allowedtags = $this->_get_allowed_tags();
-            $validation_error_message = sprintf(esc_html__("Only simple HTML tags are allowed. Eg, %s", "event_espresso"), implode(",", array_keys($allowedtags)));
-        }
-        parent::__construct($validation_error_message);
-    }
+	/**
+	 * @param null $validation_error_message
+	 */
+	public function __construct($validation_error_message = null)
+	{
+		if (! $validation_error_message) {
+			$allowedtags = $this->_get_allowed_tags();
+			$validation_error_message = sprintf(esc_html__("Only simple HTML tags are allowed. Eg, %s", "event_espresso"), implode(",", array_keys($allowedtags)));
+		}
+		parent::__construct($validation_error_message);
+	}
 
 
 
-    /**
-     * get tags allowed
-     */
-    protected function _get_allowed_tags()
-    {
-        return EEH_HTML::get_simple_tags();
-    }
+	/**
+	 * get tags allowed
+	 */
+	protected function _get_allowed_tags()
+	{
+		return EEH_HTML::get_simple_tags();
+	}
 
 
 
-    /**
-     * add_more_tags
-     *
-     * generates and returns a string that lists the top-level HTML tags that are allowable for this input
-     *
-     * @return string
-     */
-    public function get_list_of_allowed_tags()
-    {
-        $allowed_tags = $this->_get_allowed_tags();
-        ksort($allowed_tags);
-        return implode(', ', array_keys($allowed_tags));
-    }
+	/**
+	 * add_more_tags
+	 *
+	 * generates and returns a string that lists the top-level HTML tags that are allowable for this input
+	 *
+	 * @return string
+	 */
+	public function get_list_of_allowed_tags()
+	{
+		$allowed_tags = $this->_get_allowed_tags();
+		ksort($allowed_tags);
+		return implode(', ', array_keys($allowed_tags));
+	}
 
 
 
-    /**
-     * @param $normalized_value
-     * @throws \EE_Validation_Error
-     */
-    public function validate($normalized_value)
-    {
-        $allowedtags = $this->_get_allowed_tags();
-        parent::validate($normalized_value);
-        $normalized_value_sans_tags =  wp_kses("$normalized_value", $allowedtags);
-        if (strlen($normalized_value) > strlen($normalized_value_sans_tags)) {
-            throw new EE_Validation_Error($this->get_validation_error_message(), 'complex_html_tags');
-        }
-    }
+	/**
+	 * @param $normalized_value
+	 * @throws \EE_Validation_Error
+	 */
+	public function validate($normalized_value)
+	{
+		$allowedtags = $this->_get_allowed_tags();
+		parent::validate($normalized_value);
+		$normalized_value_sans_tags =  wp_kses("$normalized_value", $allowedtags);
+		if (strlen($normalized_value) > strlen($normalized_value_sans_tags)) {
+			throw new EE_Validation_Error($this->get_validation_error_message(), 'complex_html_tags');
+		}
+	}
 }