eventespresso / event-espresso-core

core/libraries/form_sections/inputs/EE_Select_Input.input.php

Spacing +6 added lines, -6 removed lines

@@ -1,4 +1,4 @@ 
-<?php  if ( ! defined( 'EVENT_ESPRESSO_VERSION' )) { exit( 'No direct script access allowed' ); }
+<?php  if ( ! defined('EVENT_ESPRESSO_VERSION')) { exit('No direct script access allowed'); }
 /**
  * Class EE_Select_Input
  *
@@ -10,22 +10,22 @@ 
  * @since 				4.6
  *
  */
-class EE_Select_Input extends EE_Form_Input_With_Options_Base{
+class EE_Select_Input extends EE_Form_Input_With_Options_Base {
 
 	/**
 	 * @param array $answer_options
 	 * @param array $input_settings
 	 */
-	public function __construct( $answer_options, $input_settings = array() ) {
-		$this->_set_display_strategy( new EE_Select_Display_Strategy( $answer_options ) );
+	public function __construct($answer_options, $input_settings = array()) {
+		$this->_set_display_strategy(new EE_Select_Display_Strategy($answer_options));
 		$this->_add_validation_strategy(
 			new EE_Enum_Validation_Strategy(
-				isset( $input_settings['validation_error_message'] )
+				isset($input_settings['validation_error_message'])
 					? $input_settings['validation_error_message']
 					: null
 			)
 		);
-		parent::__construct( $answer_options, $input_settings );
+		parent::__construct($answer_options, $input_settings);
 	}
 
 }
\ No newline at end of file

core/libraries/form_sections/inputs/EE_Form_Input_Base.input.php

Indentation +1 added lines, -2 removed lines

@@ -810,8 +810,7 @@
 	 * (If that doesn't exist, we also check for this subsection's name
 	 * at the TOP LEVEL of the request data. Eg $_REQUEST['form_input_1'].)
 	 * This function finds its value in the form.
-
-*
+	 *
 *@param array $req_data
 	 * @return mixed whatever the raw value of this form section is in the request data
 	 * @throws \EE_Error

Braces +7 added lines, -7 removed lines

@@ -243,7 +243,7 @@ 
 					$this->html_id()
 				)
 			);
-		}else{
+		} else{
 			return $this->_display_strategy;
 		}
 	}
@@ -458,14 +458,14 @@ 
 			if ( $validation_strategy instanceof EE_Validation_Strategy_Base ) {
 				try{
 					$validation_strategy->validate($this->normalized_value());
-				}catch(EE_Validation_Error $e){
+				} catch(EE_Validation_Error $e){
 					$this->add_validation_error($e);
 				}
 			}
 		}
 		if( $this->get_validation_errors()){
 			return false;
-		}else{
+		} else{
 			return true;
 		}
 	}
@@ -828,7 +828,7 @@ 
 		if( isset( $matches[ 1 ] ) && is_array( $matches[ 1 ] ) ){
 			$name_parts = $matches[ 1 ];
 			array_unshift($name_parts, $before_any_brackets);
-		}else{
+		} else{
 			$name_parts = array( $before_any_brackets );
 		}
 		// now get the value for the input
@@ -853,13 +853,13 @@ 
 		if( isset( $req_data[ $first_part_to_consider ] ) ){
 			if( empty($html_name_parts ) ){
 				return $req_data[ $first_part_to_consider ];
-			}else{
+			} else{
 				return $this->_find_form_data_for_this_section_using_name_parts(
 					$html_name_parts,
 					$req_data[ $first_part_to_consider ]
 				);
 			}
-		}else{
+		} else{
 			return NULL;
 		}
 	}
@@ -880,7 +880,7 @@ 
 		$checked_value = $this->find_form_data_for_this_section( $req_data );
 		if( $checked_value !== null ){
 			return TRUE;
-		}else{
+		} else{
 			return FALSE;
 		}
 	}

Spacing +148 added lines, -148 removed lines

@@ -8,7 +8,7 @@ 
  * @subpackage
  * @author				Mike Nelson
  */
-abstract class EE_Form_Input_Base extends EE_Form_Section_Validatable{
+abstract class EE_Form_Input_Base extends EE_Form_Section_Validatable {
 
 	/**
 	 * the input's name attribute
@@ -143,54 +143,54 @@ 
 	 *  @type EE_Validation_Strategy_Base[]  $validation_strategies
 	 * }
 	 */
-	public function __construct( $input_args = array() ){
-		$input_args = (array) apply_filters( 'FHEE__EE_Form_Input_Base___construct__input_args', $input_args, $this );
+	public function __construct($input_args = array()) {
+		$input_args = (array) apply_filters('FHEE__EE_Form_Input_Base___construct__input_args', $input_args, $this);
 		// the following properties must be cast as arrays
-		if ( isset( $input_args['validation_strategies'] ) ) {
-			foreach ( (array) $input_args['validation_strategies'] as $validation_strategy ) {
-				if ( $validation_strategy instanceof EE_Validation_Strategy_Base ) {
-					$this->_validation_strategies[ get_class( $validation_strategy ) ] = $validation_strategy;
+		if (isset($input_args['validation_strategies'])) {
+			foreach ((array) $input_args['validation_strategies'] as $validation_strategy) {
+				if ($validation_strategy instanceof EE_Validation_Strategy_Base) {
+					$this->_validation_strategies[get_class($validation_strategy)] = $validation_strategy;
 				}
 			}
-			unset( $input_args['validation_strategies'] );
+			unset($input_args['validation_strategies']);
 		}
 		// loop thru incoming options
-		foreach( $input_args as $key => $value ) {
+		foreach ($input_args as $key => $value) {
 			// add underscore to $key to match property names
-			$_key = '_' . $key;
-			if ( property_exists( $this, $_key )) {
+			$_key = '_'.$key;
+			if (property_exists($this, $_key)) {
 				$this->{$_key} = $value;
 			}
 		}
 		// ensure that "required" is set correctly
 		$this->set_required(
-			$this->_required, isset( $input_args[ 'required_validation_error_message' ] )
-				? $input_args[ 'required_validation_error_message' ]
+			$this->_required, isset($input_args['required_validation_error_message'])
+				? $input_args['required_validation_error_message']
 				: null
 		);
 
 		//$this->_html_name_specified = isset( $input_args['html_name'] ) ? TRUE : FALSE;
 
 		$this->_display_strategy->_construct_finalize($this);
-		foreach( $this->_validation_strategies as $validation_strategy ){
+		foreach ($this->_validation_strategies as $validation_strategy) {
 			$validation_strategy->_construct_finalize($this);
 		}
 
-		if( ! $this->_normalization_strategy){
+		if ( ! $this->_normalization_strategy) {
 			$this->_normalization_strategy = new EE_Text_Normalization();
 		}
 		$this->_normalization_strategy->_construct_finalize($this);
 
 		//at least we can use the normalization strategy to populate the default
-		if( isset( $input_args[ 'default' ] ) ) {
-			$this->set_default( $input_args[ 'default' ] );
+		if (isset($input_args['default'])) {
+			$this->set_default($input_args['default']);
 		}
 
-		if( ! $this->_sensitive_data_removal_strategy){
+		if ( ! $this->_sensitive_data_removal_strategy) {
 			$this->_sensitive_data_removal_strategy = new EE_No_Sensitive_Data_Removal();
 		}
 		$this->_sensitive_data_removal_strategy->_construct_finalize($this);
-		parent::__construct( $input_args );
+		parent::__construct($input_args);
 	}
 
 
@@ -201,11 +201,11 @@ 
 	 *
 	 * @throws \EE_Error
 	 */
-	protected function _set_default_html_name_if_empty(){
-		if( ! $this->_html_name){
+	protected function _set_default_html_name_if_empty() {
+		if ( ! $this->_html_name) {
 			$this->_html_name = $this->name();
-			if( $this->_parent_section && $this->_parent_section instanceof EE_Form_Section_Proper){
-				$this->_html_name = $this->_parent_section->html_name_prefix() . "[{$this->name()}]";
+			if ($this->_parent_section && $this->_parent_section instanceof EE_Form_Section_Proper) {
+				$this->_html_name = $this->_parent_section->html_name_prefix()."[{$this->name()}]";
 			}
 		}
 	}
@@ -219,10 +219,10 @@ 
 	 */
 	public function _construct_finalize($parent_form_section, $name) {
 		parent::_construct_finalize($parent_form_section, $name);
-		if( $this->_html_label === null && $this->_html_label_text === null ){
-			$this->_html_label_text = ucwords( str_replace("_"," ",$name));
+		if ($this->_html_label === null && $this->_html_label_text === null) {
+			$this->_html_label_text = ucwords(str_replace("_", " ", $name));
 		}
-		do_action( 'AHEE__EE_Form_Input_Base___construct_finalize__end', $this, $parent_form_section, $name );
+		do_action('AHEE__EE_Form_Input_Base___construct_finalize__end', $this, $parent_form_section, $name);
 	}
 
 	 /**
@@ -230,9 +230,9 @@ 
 	  * @return EE_Display_Strategy_Base
 	  * @throws EE_Error
 	  */
-	protected function _get_display_strategy(){
+	protected function _get_display_strategy() {
 		$this->ensure_construct_finalized_called();
-		if( ! $this->_display_strategy || ! $this->_display_strategy instanceof EE_Display_Strategy_Base){
+		if ( ! $this->_display_strategy || ! $this->_display_strategy instanceof EE_Display_Strategy_Base) {
 			throw new EE_Error(
 				sprintf(
 					__(
@@ -243,7 +243,7 @@ 
 					$this->html_id()
 				)
 			);
-		}else{
+		} else {
 			return $this->_display_strategy;
 		}
 	}
@@ -251,7 +251,7 @@ 
 	 * Sets the display strategy.
 	 * @param EE_Display_Strategy_Base $strategy
 	 */
-	protected function _set_display_strategy(EE_Display_Strategy_Base $strategy){
+	protected function _set_display_strategy(EE_Display_Strategy_Base $strategy) {
 		$this->_display_strategy = $strategy;
 	}
 
@@ -259,7 +259,7 @@ 
 	 * Sets the sanitization strategy
 	 * @param EE_Normalization_Strategy_Base $strategy
 	 */
-	protected function _set_normalization_strategy(EE_Normalization_Strategy_Base $strategy){
+	protected function _set_normalization_strategy(EE_Normalization_Strategy_Base $strategy) {
 		$this->_normalization_strategy = $strategy;
 	}
 
@@ -285,14 +285,14 @@ 
 	 * Gets the display strategy for this input
 	 * @return EE_Display_Strategy_Base
 	 */
-	public function get_display_strategy(){
+	public function get_display_strategy() {
 		return $this->_display_strategy;
 	}
 	/**
 	 * Overwrites the display strategy
 	 * @param EE_Display_Strategy_Base $display_strategy
 	 */
-	public function set_display_strategy($display_strategy){
+	public function set_display_strategy($display_strategy) {
 		$this->_display_strategy = $display_strategy;
 		$this->_display_strategy->_construct_finalize($this);
 	}
@@ -300,14 +300,14 @@ 
 	 * Gets the normalization strategy set on this input
 	 * @return EE_Normalization_Strategy_Base
 	 */
-	public function get_normalization_strategy(){
+	public function get_normalization_strategy() {
 		return $this->_normalization_strategy;
 	}
 	/**
 	 * Overwrites the normalization strategy
 	 * @param EE_Normalization_Strategy_Base $normalization_strategy
 	 */
-	public function set_normalization_strategy($normalization_strategy){
+	public function set_normalization_strategy($normalization_strategy) {
 		$this->_normalization_strategy = $normalization_strategy;
 		$this->_normalization_strategy->_construct_finalize($this);
 	}
@@ -316,7 +316,7 @@ 
 	 * Returns all teh validation strategies which apply to this field, numerically indexed
 	 * @return EE_Validation_Strategy_Base[]
 	 */
-	public function get_validation_strategies(){
+	public function get_validation_strategies() {
 		return $this->_validation_strategies;
 	}
 
@@ -327,8 +327,8 @@ 
 	 * @param EE_Validation_Strategy_Base $validation_strategy
 	 * @return void
 	 */
-	protected function _add_validation_strategy( EE_Validation_Strategy_Base $validation_strategy ){
-		$validation_strategy->_construct_finalize( $this );
+	protected function _add_validation_strategy(EE_Validation_Strategy_Base $validation_strategy) {
+		$validation_strategy->_construct_finalize($this);
 		$this->_validation_strategies[] = $validation_strategy;
 	}
 
@@ -339,8 +339,8 @@ 
 	 * @param EE_Validation_Strategy_Base $validation_strategy
 	 * @return void
 	 */
-	public function add_validation_strategy( EE_Validation_Strategy_Base $validation_strategy ) {
-		$this->_add_validation_strategy( $validation_strategy );
+	public function add_validation_strategy(EE_Validation_Strategy_Base $validation_strategy) {
+		$this->_add_validation_strategy($validation_strategy);
 	}
 
 
@@ -350,13 +350,13 @@ 
 	 *
 	 * @param string $validation_strategy_classname
 	 */
-	public function remove_validation_strategy( $validation_strategy_classname ) {
-		foreach( $this->_validation_strategies as $key => $validation_strategy ){
-			if(
+	public function remove_validation_strategy($validation_strategy_classname) {
+		foreach ($this->_validation_strategies as $key => $validation_strategy) {
+			if (
 				$validation_strategy instanceof $validation_strategy_classname
-				|| is_subclass_of( $validation_strategy, $validation_strategy_classname )
+				|| is_subclass_of($validation_strategy, $validation_strategy_classname)
 			) {
-				unset( $this->_validation_strategies[ $key ] );
+				unset($this->_validation_strategies[$key]);
 			}
 		}
 	}
@@ -369,12 +369,12 @@ 
 	 * @param array $validation_strategy_classnames
 	 * @return bool
 	 */
-	public function has_validation_strategy( $validation_strategy_classnames ) {
-		$validation_strategy_classnames = is_array( $validation_strategy_classnames )
+	public function has_validation_strategy($validation_strategy_classnames) {
+		$validation_strategy_classnames = is_array($validation_strategy_classnames)
 			? $validation_strategy_classnames
-			: array( $validation_strategy_classnames );
-		foreach( $this->_validation_strategies as $key => $validation_strategy ){
-			if( in_array( $key, $validation_strategy_classnames ) ) {
+			: array($validation_strategy_classnames);
+		foreach ($this->_validation_strategies as $key => $validation_strategy) {
+			if (in_array($key, $validation_strategy_classnames)) {
 				return true;
 			}
 		}
@@ -387,7 +387,7 @@ 
 	 * Gets the HTML
 	 * @return string
 	 */
-	public function get_html(){
+	public function get_html() {
 		return $this->_parent_section->get_html_for_input($this);
 	}
 
@@ -401,7 +401,7 @@ 
 	 * @return string
 	 * @throws \EE_Error
 	 */
-	public function get_html_for_input(){
+	public function get_html_for_input() {
 		
 		return  $this->_get_display_strategy()->display();
 	}
@@ -412,7 +412,7 @@ 
 	 * @return string
 	 */
 	public function html_other_attributes() {
-		return ! empty( $this->_html_other_attributes ) ? ' ' . $this->_html_other_attributes : '';
+		return ! empty($this->_html_other_attributes) ? ' '.$this->_html_other_attributes : '';
 	}
 
 
@@ -420,7 +420,7 @@ 
 	/**
 	 * @param string $html_other_attributes
 	 */
-	public function set_html_other_attributes( $html_other_attributes ) {
+	public function set_html_other_attributes($html_other_attributes) {
 		$this->_html_other_attributes = $html_other_attributes;
 	}
 
@@ -429,7 +429,7 @@ 
 	 * according to the form section's layout strategy
 	 * @return string
 	 */
-	public function get_html_for_label(){
+	public function get_html_for_label() {
 		return $this->_parent_section->get_layout_strategy()->display_label($this);
 	}
 	/**
@@ -437,7 +437,7 @@ 
 	 * according to the form section's layout strategy
 	 * @return string
 	 */
-	public function get_html_for_errors(){
+	public function get_html_for_errors() {
 		return $this->_parent_section->get_layout_strategy()->display_errors($this);
 	}
 	/**
@@ -445,7 +445,7 @@ 
 	 * according to the form section's layout strategy
 	 * @return string
 	 */
-	public function get_html_for_help(){
+	public function get_html_for_help() {
 		return $this->_parent_section->get_layout_strategy()->display_help_text($this);
 	}
 	/**
@@ -454,18 +454,18 @@ 
 	 * @return boolean
 	 */
 	protected function _validate() {
-		foreach($this->_validation_strategies as $validation_strategy){
-			if ( $validation_strategy instanceof EE_Validation_Strategy_Base ) {
-				try{
+		foreach ($this->_validation_strategies as $validation_strategy) {
+			if ($validation_strategy instanceof EE_Validation_Strategy_Base) {
+				try {
 					$validation_strategy->validate($this->normalized_value());
-				}catch(EE_Validation_Error $e){
+				} catch (EE_Validation_Error $e) {
 					$this->add_validation_error($e);
 				}
 			}
 		}
-		if( $this->get_validation_errors()){
+		if ($this->get_validation_errors()) {
 			return false;
-		}else{
+		} else {
 			return true;
 		}
 	}
@@ -479,8 +479,8 @@ 
 	 * @param string $value
 	 * @return null|string
 	 */
-	private function _sanitize( $value ) {
-		return $value !== null ? stripslashes( html_entity_decode( trim( $value ) ) ) : null;
+	private function _sanitize($value) {
+		return $value !== null ? stripslashes(html_entity_decode(trim($value))) : null;
 	}
 
 
@@ -494,25 +494,25 @@ 
 	 * @return boolean whether or not there was an error
 	 * @throws \EE_Error
 	 */
-	protected function _normalize( $req_data ) {
+	protected function _normalize($req_data) {
 		//any existing validation errors don't apply so clear them
 		$this->_validation_errors = array();
 		try {
-			$raw_input = $this->find_form_data_for_this_section( $req_data );
+			$raw_input = $this->find_form_data_for_this_section($req_data);
 			//super simple sanitization for now
-			if ( is_array( $raw_input )) {
+			if (is_array($raw_input)) {
 				$raw_value = array();
-				foreach( $raw_input as $key => $value ) {
-					$raw_value[ $key ] = $this->_sanitize( $value );
+				foreach ($raw_input as $key => $value) {
+					$raw_value[$key] = $this->_sanitize($value);
 				}
-				$this->_set_raw_value( $raw_value );
+				$this->_set_raw_value($raw_value);
 			} else {
-				$this->_set_raw_value( $this->_sanitize( $raw_input ) );
+				$this->_set_raw_value($this->_sanitize($raw_input));
 			}
 			//we want to mostly leave the input alone in case we need to re-display it to the user
-			$this->_set_normalized_value( $this->_normalization_strategy->normalize( $this->raw_value() ) );
-		} catch ( EE_Validation_Error $e ) {
-			$this->add_validation_error( $e );
+			$this->_set_normalized_value($this->_normalization_strategy->normalize($this->raw_value()));
+		} catch (EE_Validation_Error $e) {
+			$this->add_validation_error($e);
 		}
 	}
 
@@ -521,7 +521,7 @@ 
 	/**
 	 * @return string
 	 */
-	public function html_name(){
+	public function html_name() {
 		$this->_set_default_html_name_if_empty();
 		return $this->_html_name;
 	}
@@ -531,8 +531,8 @@ 
 	/**
 	 * @return string
 	 */
-	public function html_label_id(){
-		return ! empty( $this->_html_label_id ) ? $this->_html_label_id : $this->_html_id . '-lbl';
+	public function html_label_id() {
+		return ! empty($this->_html_label_id) ? $this->_html_label_id : $this->_html_id.'-lbl';
 	}
 
 
@@ -540,7 +540,7 @@ 
 	/**
 	 * @return string
 	 */
-	public function html_label_class(){
+	public function html_label_class() {
 		return $this->_html_label_class;
 	}
 
@@ -549,7 +549,7 @@ 
 	/**
 	 * @return string
 	 */
-	public function html_label_style(){
+	public function html_label_style() {
 		return $this->_html_label_style;
 	}
 
@@ -558,7 +558,7 @@ 
 	/**
 	 * @return string
 	 */
-	public function html_label_text(){
+	public function html_label_text() {
 		return $this->_html_label_text;
 	}
 
@@ -567,7 +567,7 @@ 
 	/**
 	 * @return string
 	 */
-	public function html_help_text(){
+	public function html_help_text() {
 		return $this->_html_help_text;
 	}
 
@@ -576,7 +576,7 @@ 
 	/**
 	 * @return string
 	 */
-	public function html_help_class(){
+	public function html_help_class() {
 		return $this->_html_help_class;
 	}
 
@@ -585,7 +585,7 @@ 
 	/**
 	 * @return string
 	 */
-	public function html_help_style(){
+	public function html_help_style() {
 		return $this->_html_style;
 	}
 	/**
@@ -598,7 +598,7 @@ 
 	 * in which case, we would have stored the malicious content to our database.
 	 * @return string
 	 */
-	public function raw_value(){
+	public function raw_value() {
 		return $this->_raw_value;
 	}
 	/**
@@ -606,15 +606,15 @@ 
 	 * it escapes all html entities
 	 * @return string
 	 */
-	public function raw_value_in_form(){
-		return htmlentities($this->raw_value(),ENT_QUOTES, 'UTF-8');
+	public function raw_value_in_form() {
+		return htmlentities($this->raw_value(), ENT_QUOTES, 'UTF-8');
 	}
 	/**
 	 * returns the value after it's been sanitized, and then converted into it's proper type
 	 * in PHP. Eg, a string, an int, an array,
 	 * @return mixed
 	 */
-	public function normalized_value(){
+	public function normalized_value() {
 		return $this->_normalized_value;
 	}
 
@@ -624,7 +624,7 @@ 
 	 * the best thing to display
 	 * @return string
 	 */
-	public function pretty_value(){
+	public function pretty_value() {
 		return $this->_normalized_value;
 	}
 	/**
@@ -643,19 +643,19 @@ 
 		  }</code>
 	 * @return array
 	 */
-	public function get_jquery_validation_rules(){
+	public function get_jquery_validation_rules() {
 		$jquery_validation_js = array();
 		$jquery_validation_rules = array();
-		foreach($this->get_validation_strategies() as $validation_strategy){
+		foreach ($this->get_validation_strategies() as $validation_strategy) {
 			$jquery_validation_rules = array_replace_recursive(
 				$jquery_validation_rules,
 				$validation_strategy->get_jquery_validation_rule_array()
 			);
 		}
 
-		if(! empty($jquery_validation_rules)){
-			foreach( $this->get_display_strategy()->get_html_input_ids( true ) as $html_id_with_pound_sign ) {
-				$jquery_validation_js[ $html_id_with_pound_sign ] = $jquery_validation_rules;
+		if ( ! empty($jquery_validation_rules)) {
+			foreach ($this->get_display_strategy()->get_html_input_ids(true) as $html_id_with_pound_sign) {
+				$jquery_validation_js[$html_id_with_pound_sign] = $jquery_validation_rules;
 			}
 		}
 		return $jquery_validation_js;
@@ -667,16 +667,16 @@ 
 	 * @param mixed $value
 	 * @return void
 	 */
-	public function set_default($value){
-		$this->_set_normalized_value( $value );
-		$this->_set_raw_value( $value );
+	public function set_default($value) {
+		$this->_set_normalized_value($value);
+		$this->_set_raw_value($value);
 	}
 	
 	/**
 	 * Sets the normalized value on this input
 	 * @param mixed $value
 	 */
-	protected function _set_normalized_value( $value ) {
+	protected function _set_normalized_value($value) {
 		$this->_normalized_value = $value;
 	}
 	
@@ -684,8 +684,8 @@ 
 	 * Sets the raw value on this input (ie, exactly as the user submitted it)
 	 * @param mixed $value
 	 */
-	protected function _set_raw_value( $value ) {
-		$this->_raw_value = $this->_normalization_strategy->unnormalize( $value );
+	protected function _set_raw_value($value) {
+		$this->_raw_value = $this->_normalization_strategy->unnormalize($value);
 	}
 
 	/**
@@ -693,7 +693,7 @@ 
 	 * @param string $label
 	 * @return void
 	 */
-	public function set_html_label_text($label){
+	public function set_html_label_text($label) {
 		$this->_html_label_text = $label;
 	}
 
@@ -707,13 +707,13 @@ 
 	 * @param boolean $required boolean
 	 * @param null    $required_text
 	 */
-	public function set_required($required = true, $required_text = NULL ){
-		$required = filter_var( $required, FILTER_VALIDATE_BOOLEAN  );
+	public function set_required($required = true, $required_text = NULL) {
+		$required = filter_var($required, FILTER_VALIDATE_BOOLEAN);
 		//whether $required is a string or a boolean, we want to add a required validation strategy
-		if ( $required ) {
-			$this->_add_validation_strategy( new EE_Required_Validation_Strategy( $required_text ) );
+		if ($required) {
+			$this->_add_validation_strategy(new EE_Required_Validation_Strategy($required_text));
 		} else {
-			$this->remove_validation_strategy( 'EE_Required_Validation_Strategy' );
+			$this->remove_validation_strategy('EE_Required_Validation_Strategy');
 		}
 		$this->_required = $required;
 	}
@@ -721,7 +721,7 @@ 
 	 * Returns whether or not this field is required
 	 * @return boolean
 	 */
-	public function required(){
+	public function required() {
 		return $this->_required;
 	}
 
@@ -730,7 +730,7 @@ 
 	/**
 	 * @param string $required_css_class
 	 */
-	public function set_required_css_class( $required_css_class ) {
+	public function set_required_css_class($required_css_class) {
 		$this->_required_css_class = $required_css_class;
 	}
 
@@ -749,7 +749,7 @@ 
 	 * Sets the help text, in case
 	 * @param string $text
 	 */
-	public function set_html_help_text($text){
+	public function set_html_help_text($text) {
 		$this->_html_help_text = $text;
 	}
 	/**
@@ -761,9 +761,9 @@ 
 	public function clean_sensitive_data() {
 		//if we do ANY kind of sensitive data removal on this, then just clear out the raw value
 		//if we need more logic than this we'll make a strategy for it
-		if( $this->_sensitive_data_removal_strategy &&
-				! $this->_sensitive_data_removal_strategy instanceof EE_No_Sensitive_Data_Removal ){
-			$this->_set_raw_value( null );
+		if ($this->_sensitive_data_removal_strategy &&
+				! $this->_sensitive_data_removal_strategy instanceof EE_No_Sensitive_Data_Removal) {
+			$this->_set_raw_value(null);
 		}
 		//and clean the normalized value according to the appropriate strategy
 		$this->_set_normalized_value(
@@ -780,10 +780,10 @@ 
 	 * @param string $button_size
 	 * @param string $other_attributes
 	 */
-	public function set_button_css_attributes( $primary = TRUE, $button_size = '', $other_attributes = '' ) {
+	public function set_button_css_attributes($primary = TRUE, $button_size = '', $other_attributes = '') {
 		$button_css_attributes = 'button';
 		$button_css_attributes .= $primary === TRUE ? ' button-primary' : ' button-secondary';
-		switch ( $button_size ) {
+		switch ($button_size) {
 			case 'xs' :
 			case 'extra-small' :
 				$button_css_attributes .= ' button-xs';
@@ -804,8 +804,8 @@ 
 			default :
 				$button_css_attributes .= '';
 		}
-		$this->_button_css_attributes .= ! empty( $other_attributes )
-			? $button_css_attributes . ' ' . $other_attributes
+		$this->_button_css_attributes .= ! empty($other_attributes)
+			? $button_css_attributes.' '.$other_attributes
 			: $button_css_attributes;
 	}
 
@@ -815,7 +815,7 @@ 
 	 * @return string
 	 */
 	public function button_css_attributes() {
-		if ( empty( $this->_button_css_attributes )) {
+		if (empty($this->_button_css_attributes)) {
 			$this->set_button_css_attributes();
 		}
 		return $this->_button_css_attributes;
@@ -837,26 +837,26 @@ 
 	 * @return mixed whatever the raw value of this form section is in the request data
 	 * @throws \EE_Error
 	 */
-	public function find_form_data_for_this_section( $req_data ){
+	public function find_form_data_for_this_section($req_data) {
 		// break up the html name by "[]"
-		if ( strpos( $this->html_name(), '[' ) !== FALSE ) {
-			$before_any_brackets = substr( $this->html_name(), 0, strpos($this->html_name(), '[') );
+		if (strpos($this->html_name(), '[') !== FALSE) {
+			$before_any_brackets = substr($this->html_name(), 0, strpos($this->html_name(), '['));
 		} else {
 			$before_any_brackets = $this->html_name();
 		}
 		// grab all of the segments
-		preg_match_all('~\[([^]]*)\]~',$this->html_name(), $matches);
-		if( isset( $matches[ 1 ] ) && is_array( $matches[ 1 ] ) ){
-			$name_parts = $matches[ 1 ];
+		preg_match_all('~\[([^]]*)\]~', $this->html_name(), $matches);
+		if (isset($matches[1]) && is_array($matches[1])) {
+			$name_parts = $matches[1];
 			array_unshift($name_parts, $before_any_brackets);
-		}else{
-			$name_parts = array( $before_any_brackets );
+		} else {
+			$name_parts = array($before_any_brackets);
 		}
 		// now get the value for the input
 		$value = $this->_find_form_data_for_this_section_using_name_parts($name_parts, $req_data);
 		// check if this thing's name is at the TOP level of the request data
-		if( $value === null && isset( $req_data[ $this->name() ] ) ){
-			$value = $req_data[ $this->name() ];
+		if ($value === null && isset($req_data[$this->name()])) {
+			$value = $req_data[$this->name()];
 		}
 		return $value;
 	}
@@ -869,18 +869,18 @@ 
 	 * @param array $req_data
 	 * @return array | NULL
 	 */
-	public function _find_form_data_for_this_section_using_name_parts($html_name_parts, $req_data){
-		$first_part_to_consider = array_shift( $html_name_parts );
-		if( isset( $req_data[ $first_part_to_consider ] ) ){
-			if( empty($html_name_parts ) ){
-				return $req_data[ $first_part_to_consider ];
-			}else{
+	public function _find_form_data_for_this_section_using_name_parts($html_name_parts, $req_data) {
+		$first_part_to_consider = array_shift($html_name_parts);
+		if (isset($req_data[$first_part_to_consider])) {
+			if (empty($html_name_parts)) {
+				return $req_data[$first_part_to_consider];
+			} else {
 				return $this->_find_form_data_for_this_section_using_name_parts(
 					$html_name_parts,
-					$req_data[ $first_part_to_consider ]
+					$req_data[$first_part_to_consider]
 				);
 			}
-		}else{
+		} else {
 			return NULL;
 		}
 	}
@@ -894,14 +894,14 @@ 
 	 * @return boolean
 	 * @throws \EE_Error
 	 */
-	public function form_data_present_in($req_data = NULL){
-		if( $req_data === NULL ){
+	public function form_data_present_in($req_data = NULL) {
+		if ($req_data === NULL) {
 			$req_data = $_POST;
 		}
-		$checked_value = $this->find_form_data_for_this_section( $req_data );
-		if( $checked_value !== null ){
+		$checked_value = $this->find_form_data_for_this_section($req_data);
+		if ($checked_value !== null) {
 			return TRUE;
-		}else{
+		} else {
 			return FALSE;
 		}
 	}
@@ -912,8 +912,8 @@ 
 	 * @param array $form_other_js_data
 	 * @return array
 	 */
-	public function get_other_js_data( $form_other_js_data = array() ) {
-		$form_other_js_data = $this->get_other_js_data_from_strategies( $form_other_js_data );
+	public function get_other_js_data($form_other_js_data = array()) {
+		$form_other_js_data = $this->get_other_js_data_from_strategies($form_other_js_data);
 		return $form_other_js_data;
 	}
 
@@ -926,10 +926,10 @@ 
 	 * @param array $form_other_js_data
 	 * @return array
 	 */
-	public function get_other_js_data_from_strategies( $form_other_js_data = array() ) {
-		$form_other_js_data = $this->get_display_strategy()->get_other_js_data( $form_other_js_data );
-		foreach( $this->get_validation_strategies() as $validation_strategy ) {
-			$form_other_js_data = $validation_strategy->get_other_js_data( $form_other_js_data );
+	public function get_other_js_data_from_strategies($form_other_js_data = array()) {
+		$form_other_js_data = $this->get_display_strategy()->get_other_js_data($form_other_js_data);
+		foreach ($this->get_validation_strategies() as $validation_strategy) {
+			$form_other_js_data = $validation_strategy->get_other_js_data($form_other_js_data);
 		}
 		return $form_other_js_data;
 	}
@@ -938,7 +938,7 @@ 
 	 * Override parent because we want to give our strategies an opportunity to enqueue some js and css
 	 * @return void
 	 */
-	public function enqueue_js(){
+	public function enqueue_js() {
 		//ask our display strategy and validation strategies if they have js to enqueue
 		$this->enqueue_js_from_strategies();
 	}
@@ -949,7 +949,7 @@ 
 	 */
 	public function enqueue_js_from_strategies() {
 		$this->get_display_strategy()->enqueue_js();
-		foreach( $this->get_validation_strategies() as $validation_strategy ) {
+		foreach ($this->get_validation_strategies() as $validation_strategy) {
 			$validation_strategy->enqueue_js();
 		}
 	}

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

Spacing +8 added lines, -8 removed lines

@@ -8,18 +8,18 @@ 
  * @since 				4.6
  *
  */
-class EE_Required_Validation_Strategy extends EE_Validation_Strategy_Base{
+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 ){
+	public function __construct($validation_error_message = null) {
+		if ( ! $validation_error_message) {
 			$validation_error_message = __("This field is required.", "event_espresso");
 		}
-		parent::__construct( $validation_error_message );
+		parent::__construct($validation_error_message);
 	}
 
 
@@ -34,13 +34,13 @@ 
 	 * @throws \EE_Validation_Error
 	 */
 	public function validate($normalized_value) {
-		if(
+		if (
 			$normalized_value === ''
 			|| $normalized_value === null
 			|| $normalized_value === array()
 		) {
-			throw new EE_Validation_Error( $this->get_validation_error_message(), 'required');
-		}else{
+			throw new EE_Validation_Error($this->get_validation_error_message(), 'required');
+		} else {
 			return true;
 		}
 	}
@@ -51,7 +51,7 @@ 
 	 * @return array
 	 * @throws \EE_Error
 	 */
-	public function get_jquery_validation_rule_array(){
+	public function get_jquery_validation_rule_array() {
 		return array(
 			'required'=> true,
 			'messages' => array(

Braces +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 			|| $normalized_value === array()
 		) {
 			throw new EE_Validation_Error( $this->get_validation_error_message(), 'required');
-		}else{
+		} else{
 			return true;
 		}
 	}

caffeinated/admin/new/pricing/templates/event_tickets_datetime_edit_row.template.php

Spacing +2 added lines, -2 removed lines

@@ -41,8 +41,8 @@
 		</div>
 		<span data-context="datetime" data-datetime-row="<?php echo $dtt_row; ?>" class="<?php echo $clone_icon; ?> clickable"></span>
 		<span data-context="datetime" data-datetime-row="<?php echo $dtt_row; ?>" class="<?php echo $trash_icon; ?> clickable"<?php echo $show_trash; ?>></span>
-		<?php if ( $reg_list_url !== '' ) : ?>
-		<a href="<?php echo $reg_list_url; ?>" title="<?php _e( 'View registrations for this datetime.', 'event_espresso' );?>" style="text-decoration: none;">
+		<?php if ($reg_list_url !== '') : ?>
+		<a href="<?php echo $reg_list_url; ?>" title="<?php _e('View registrations for this datetime.', 'event_espresso'); ?>" style="text-decoration: none;">
 			<span data-context="datetime" data-datetime-row="<?php echo $dtt_row; ?>" class="dashicons dashicons-groups clickable"></span>
 		</a>
 		<?php endif; ?>

caffeinated/admin/extend/registrations/Extend_EE_Registrations_List_Table.class.php

Indentation +53 added lines, -53 removed lines

@@ -30,7 +30,7 @@ 
 
 	/**
 	 * 		REG_date
-	*/
+	 */
 	function column_REG_date(EE_Registration $item){
 
 		//Build row actions
@@ -60,60 +60,60 @@ 
 		$remove_defaults = array('default_where_conditions' => 'none');
 		$ticket = $item->ticket();
 		$datetimes = $ticket instanceof EE_Ticket ? $ticket->datetimes($remove_defaults) : array();
-	    $EVT_ID = $item->event_ID();
-	    $datetime_string = '';
-	    foreach( $datetimes as $datetime ){
-		    if (
-			    EE_Registry::instance()->CAP->current_user_can(
-				    'ee_read_checkin',
-				    'espresso_registrations_registration_checkins',
-				    $item->ID()
-			    )
-		    ) {
-			    // open "a" tag and "href"
-			    $datetime_string .= '<a href="';
-			    // checkin URL
-			    $datetime_string .= EE_Admin_Page::add_query_args_and_nonce(
-				    array(
-					    'action'   => 'event_registrations',
-					    'event_id' => $EVT_ID,
-					    'DTT_ID'   => $datetime->ID(),
-				    ),
-				    REG_ADMIN_URL
-			    );
-			    // close "href"
-			    $datetime_string .= '"';
-			    // open "title" tag
-			    $datetime_string .= ' title="';
-			    // link title text
-			    $datetime_string .= esc_attr__( 'View Checkins for this Event', 'event_espresso' );
-			    // close "title" tag and end of "a" tag opening
-			    $datetime_string .= '">';
-			    // link text
-			    $datetime_string .= $datetime->get_i18n_datetime( 'DTT_EVT_start' );
-			    // close "a" tag
-			    $datetime_string .= '</a>';
-		    } else {
-			    $datetime_string .= $datetime->get_i18n_datetime( 'DTT_EVT_start' );
-		    }
-		    // add a "View Registrations" link that filters list by event AND datetime
-		    $datetime_string .= $this->row_actions(
-			    array(
-				    'event_datetime_filter' => '<a href="' . EE_Admin_Page::add_query_args_and_nonce(
-						    array( 'event_id' => $EVT_ID, 'datetime_id' => $datetime->ID() ),
-						    REG_ADMIN_URL
-					    ) . '" title="' . sprintf(
-						    esc_attr__(
-							    'Filter this list to only show registrations for this datetime %s',
-							    'event_espresso'
-						    ),
-						    $datetime->name()
-					    ) . '">' . __( 'View Registrations', 'event_espresso' ) . '</a>'
-			    )
-		    );
+		$EVT_ID = $item->event_ID();
+		$datetime_string = '';
+		foreach( $datetimes as $datetime ){
+			if (
+				EE_Registry::instance()->CAP->current_user_can(
+					'ee_read_checkin',
+					'espresso_registrations_registration_checkins',
+					$item->ID()
+				)
+			) {
+				// open "a" tag and "href"
+				$datetime_string .= '<a href="';
+				// checkin URL
+				$datetime_string .= EE_Admin_Page::add_query_args_and_nonce(
+					array(
+						'action'   => 'event_registrations',
+						'event_id' => $EVT_ID,
+						'DTT_ID'   => $datetime->ID(),
+					),
+					REG_ADMIN_URL
+				);
+				// close "href"
+				$datetime_string .= '"';
+				// open "title" tag
+				$datetime_string .= ' title="';
+				// link title text
+				$datetime_string .= esc_attr__( 'View Checkins for this Event', 'event_espresso' );
+				// close "title" tag and end of "a" tag opening
+				$datetime_string .= '">';
+				// link text
+				$datetime_string .= $datetime->get_i18n_datetime( 'DTT_EVT_start' );
+				// close "a" tag
+				$datetime_string .= '</a>';
+			} else {
+				$datetime_string .= $datetime->get_i18n_datetime( 'DTT_EVT_start' );
+			}
+			// add a "View Registrations" link that filters list by event AND datetime
+			$datetime_string .= $this->row_actions(
+				array(
+					'event_datetime_filter' => '<a href="' . EE_Admin_Page::add_query_args_and_nonce(
+							array( 'event_id' => $EVT_ID, 'datetime_id' => $datetime->ID() ),
+							REG_ADMIN_URL
+						) . '" title="' . sprintf(
+							esc_attr__(
+								'Filter this list to only show registrations for this datetime %s',
+								'event_espresso'
+							),
+							$datetime->name()
+						) . '">' . __( 'View Registrations', 'event_espresso' ) . '</a>'
+				)
+			);
 		}
 		return $datetime_string;
-    }
+	}
 
 
 

Spacing +16 added lines, -16 removed lines

@@ -31,20 +31,20 @@ 
 	/**
 	 * 		REG_date
 	*/
-	function column_REG_date(EE_Registration $item){
+	function column_REG_date(EE_Registration $item) {
 
 		//Build row actions
 		$actions = array();
 
 		//Build row actions
-		$check_in_url = EE_Admin_Page::add_query_args_and_nonce( array( 'action'=>'event_registrations', 'event_id'=>$item->event_ID() ), REG_ADMIN_URL );
-		$actions['check_in'] = EE_Registry::instance()->CAP->current_user_can( 'ee_read_checkin', 'espresso_registrations_registration_checkins', $item->ID() ) ? '
-			<a href="'.$check_in_url.'" title="' . esc_attr__( 'The Check-In List allows you to easily toggle check-in status for this event', 'event_espresso' ) . '">' . __( 'View Check-ins', 'event_espresso' ) . '</a>' : __( 'View Check-ins', 'event_espresso' );
+		$check_in_url = EE_Admin_Page::add_query_args_and_nonce(array('action'=>'event_registrations', 'event_id'=>$item->event_ID()), REG_ADMIN_URL);
+		$actions['check_in'] = EE_Registry::instance()->CAP->current_user_can('ee_read_checkin', 'espresso_registrations_registration_checkins', $item->ID()) ? '
+			<a href="'.$check_in_url.'" title="'.esc_attr__('The Check-In List allows you to easily toggle check-in status for this event', 'event_espresso').'">'.__('View Check-ins', 'event_espresso').'</a>' : __('View Check-ins', 'event_espresso');
 
-		$view_lnk_url = EE_Admin_Page::add_query_args_and_nonce( array( 'action'=>'view_transaction', 'TXN_ID'=>$item->transaction()->ID() ), TXN_ADMIN_URL );
-		$REG_date = EE_Regisry::instance()->CAP->current_user_can( 'ee_read_transaction', 'espresso_transactions_view_transaction' ) ?  '<a href="'.$view_lnk_url.'" title="' . esc_attr__( 'View Transaction Details', 'event_espresso' ) . '">' . $item->get_i18n_datetime( 'REG_date' ) . '</a>' : $item->get_i18n_datetime( 'REG_date' );
+		$view_lnk_url = EE_Admin_Page::add_query_args_and_nonce(array('action'=>'view_transaction', 'TXN_ID'=>$item->transaction()->ID()), TXN_ADMIN_URL);
+		$REG_date = EE_Regisry::instance()->CAP->current_user_can('ee_read_transaction', 'espresso_transactions_view_transaction') ? '<a href="'.$view_lnk_url.'" title="'.esc_attr__('View Transaction Details', 'event_espresso').'">'.$item->get_i18n_datetime('REG_date').'</a>' : $item->get_i18n_datetime('REG_date');
 
-		return sprintf('%1$s %2$s', $REG_date, $this->row_actions($actions) );
+		return sprintf('%1$s %2$s', $REG_date, $this->row_actions($actions));
 
 	}
 
@@ -56,13 +56,13 @@ 
 	 * @param \EE_Registration $item
 	 * @return string
 	 */
-   	public function column_DTT_EVT_start(EE_Registration $item){
+   	public function column_DTT_EVT_start(EE_Registration $item) {
 		$remove_defaults = array('default_where_conditions' => 'none');
 		$ticket = $item->ticket();
 		$datetimes = $ticket instanceof EE_Ticket ? $ticket->datetimes($remove_defaults) : array();
 	    $EVT_ID = $item->event_ID();
 	    $datetime_string = '';
-	    foreach( $datetimes as $datetime ){
+	    foreach ($datetimes as $datetime) {
 		    if (
 			    EE_Registry::instance()->CAP->current_user_can(
 				    'ee_read_checkin',
@@ -86,29 +86,29 @@ 
 			    // open "title" tag
 			    $datetime_string .= ' title="';
 			    // link title text
-			    $datetime_string .= esc_attr__( 'View Checkins for this Event', 'event_espresso' );
+			    $datetime_string .= esc_attr__('View Checkins for this Event', 'event_espresso');
 			    // close "title" tag and end of "a" tag opening
 			    $datetime_string .= '">';
 			    // link text
-			    $datetime_string .= $datetime->get_i18n_datetime( 'DTT_EVT_start' );
+			    $datetime_string .= $datetime->get_i18n_datetime('DTT_EVT_start');
 			    // close "a" tag
 			    $datetime_string .= '</a>';
 		    } else {
-			    $datetime_string .= $datetime->get_i18n_datetime( 'DTT_EVT_start' );
+			    $datetime_string .= $datetime->get_i18n_datetime('DTT_EVT_start');
 		    }
 		    // add a "View Registrations" link that filters list by event AND datetime
 		    $datetime_string .= $this->row_actions(
 			    array(
-				    'event_datetime_filter' => '<a href="' . EE_Admin_Page::add_query_args_and_nonce(
-						    array( 'event_id' => $EVT_ID, 'datetime_id' => $datetime->ID() ),
+				    'event_datetime_filter' => '<a href="'.EE_Admin_Page::add_query_args_and_nonce(
+						    array('event_id' => $EVT_ID, 'datetime_id' => $datetime->ID()),
 						    REG_ADMIN_URL
-					    ) . '" title="' . sprintf(
+					    ).'" title="'.sprintf(
 						    esc_attr__(
 							    'Filter this list to only show registrations for this datetime %s',
 							    'event_espresso'
 						    ),
 						    $datetime->name()
-					    ) . '">' . __( 'View Registrations', 'event_espresso' ) . '</a>'
+					    ).'">'.__('View Registrations', 'event_espresso').'</a>'
 			    )
 		    );
 		}

Braces +3 added lines, -1 removed lines

@@ -1,4 +1,6 @@
-<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed');
+<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
+}
 /**
  * Event Espresso
  *

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

Spacing +11 added lines, -11 removed lines

@@ -10,17 +10,17 @@ 
  * @since 				4.6
  *
  */
-class EE_Simple_HTML_Validation_Strategy extends EE_Validation_Strategy_Base{
+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 ){
+	public function __construct($validation_error_message = NULL) {
+		if ( ! $validation_error_message) {
 			$allowedtags = $this->_get_allowed_tags();
-			$validation_error_message = sprintf( __( "Only simple HTML tags are allowed. Eg, %s", "event_espresso" ), implode( ",", array_keys( $allowedtags ) ) );
+			$validation_error_message = sprintf(__("Only simple HTML tags are allowed. Eg, %s", "event_espresso"), implode(",", array_keys($allowedtags)));
 		}
-		parent::__construct( $validation_error_message );
+		parent::__construct($validation_error_message);
 	}
 
 
@@ -43,8 +43,8 @@ 
 	 */
 	public function get_list_of_allowed_tags() {
 		$allowed_tags = $this->_get_allowed_tags();
-		ksort( $allowed_tags );
-		return implode( ', ', array_keys( $allowed_tags ) );
+		ksort($allowed_tags);
+		return implode(', ', array_keys($allowed_tags));
 	}
 
 
@@ -55,10 +55,10 @@ 
 	 */
 	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' );
+		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');
 		}
 	}
 }
\ No newline at end of file

shortcodes/espresso_txn_page/EES_Espresso_Txn_Page.shortcode.php

Indentation +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
 				$gateway = $payment_method->type_obj()->get_gateway();
 				if (
 					$gateway instanceof EE_Offsite_Gateway
-				    && $gateway->handle_IPN_in_this_request(
+					&& $gateway->handle_IPN_in_this_request(
 						\EE_Registry::instance()->REQ->params(),
 						true
 					)

Spacing +17 added lines, -17 removed lines

@@ -1,4 +1,4 @@ 
-<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) {exit('No direct script access allowed');}
+<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) {exit('No direct script access allowed'); }
 /**
  * EES_Espresso_Txn_Page
  *
@@ -27,7 +27,7 @@ 
 	 *  @return 	void
 	 */
 	public static function set_hooks() {
-		add_action( 'wp_loaded', array( 'EES_Espresso_Txn_Page', 'set_definitions' ), 2 );
+		add_action('wp_loaded', array('EES_Espresso_Txn_Page', 'set_definitions'), 2);
 	}
 
 	/**
@@ -48,8 +48,8 @@ 
 	 *  @return 	void
 	 */
 	public static function set_definitions() {
-		define( 'TXN_PAGE_ASSETS_URL', plugin_dir_url( __FILE__ ) . 'assets' . DS );
-		define( 'TXN_PAGE_TEMPLATES_PATH', str_replace( '\\', DS, plugin_dir_path( __FILE__ )) . 'templates' . DS );
+		define('TXN_PAGE_ASSETS_URL', plugin_dir_url(__FILE__).'assets'.DS);
+		define('TXN_PAGE_TEMPLATES_PATH', str_replace('\\', DS, plugin_dir_path(__FILE__)).'templates'.DS);
 	}
 
 
@@ -66,21 +66,21 @@ 
 	 * @throws \Exception
 	 * @throws \EE_Error
 	 */
-	public function run( WP $WP ) {
+	public function run(WP $WP) {
 		$this->_current_txn = null;
-		if ( EE_Registry::instance()->REQ->is_set('e_reg_url_link' )){
+		if (EE_Registry::instance()->REQ->is_set('e_reg_url_link')) {
 			/** @var EEM_Transaction $EEM_Transaction */
-			$EEM_Transaction = EE_Registry::instance()->load_model( 'Transaction' );
+			$EEM_Transaction = EE_Registry::instance()->load_model('Transaction');
 			$this->_current_txn = $EEM_Transaction->get_transaction_from_reg_url_link();
 		}
-		if ( $this->_current_txn instanceof EE_Transaction ) {
+		if ($this->_current_txn instanceof EE_Transaction) {
 			$payment_method = null;
-			$payment_method_slug = EE_Registry::instance()->REQ->get( 'ee_payment_method', null );
-			if( $payment_method_slug ) {
-				$payment_method = EEM_Payment_Method::instance()->get_one_by_slug( $payment_method_slug );
+			$payment_method_slug = EE_Registry::instance()->REQ->get('ee_payment_method', null);
+			if ($payment_method_slug) {
+				$payment_method = EEM_Payment_Method::instance()->get_one_by_slug($payment_method_slug);
 			}
 
-			if ( $payment_method instanceof EE_Payment_Method && $payment_method->is_off_site() ) {
+			if ($payment_method instanceof EE_Payment_Method && $payment_method->is_off_site()) {
 				$gateway = $payment_method->type_obj()->get_gateway();
 				if (
 					$gateway instanceof EE_Offsite_Gateway
@@ -90,12 +90,12 @@ 
 					)
 				) {
 					/** @type EE_Payment_Processor $payment_processor */
-					$payment_processor = EE_Registry::instance()->load_core( 'Payment_Processor' );
-					$payment_processor->process_ipn( $_REQUEST, $this->_current_txn, $payment_method );
+					$payment_processor = EE_Registry::instance()->load_core('Payment_Processor');
+					$payment_processor->process_ipn($_REQUEST, $this->_current_txn, $payment_method);
 				}
 			}
 			//allow gateways to add a filter to stop rendering the page
-			if( apply_filters( 'FHEE__EES_Espresso_Txn_Page__run__exit', FALSE ) ){
+			if (apply_filters('FHEE__EES_Espresso_Txn_Page__run__exit', FALSE)) {
 				exit;
 			}
 		}
@@ -112,8 +112,8 @@ 
 	 *  @param		array 	$attributes
 	 *  @return 	string
 	 */
-	public function process_shortcode( $attributes = array() ) {
-		return __( 'This is the Event Espresso Transactions page. This page receives instant payment notification (IPN) requests and should have a status of published, but should not be easily accessible by site visitors. Do not add it to your website\'s navigation menu or link to it from another page. Also, do not delete it or change its status to private.', 'event_espresso' );
+	public function process_shortcode($attributes = array()) {
+		return __('This is the Event Espresso Transactions page. This page receives instant payment notification (IPN) requests and should have a status of published, but should not be easily accessible by site visitors. Do not add it to your website\'s navigation menu or link to it from another page. Also, do not delete it or change its status to private.', 'event_espresso');
 	}
 
 

core/libraries/form_sections/payment_methods/EE_Billing_Attendee_Info_Form.form.php

Spacing +51 added lines, -51 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 /**
@@ -14,31 +14,31 @@ 
  * @author				Mike Nelson
  *
  */
-class EE_Billing_Attendee_Info_Form extends EE_Billing_Info_Form{
+class EE_Billing_Attendee_Info_Form extends EE_Billing_Info_Form {
 
 	/**
 	 *
 	 * @param EE_Payment_Method $payment_method
 	 * @param array $options_array @see EE_Form_Section_Proper::__construct()
 	 */
-	public function __construct( EE_Payment_Method $payment_method, $options_array= array()){
+	public function __construct(EE_Payment_Method $payment_method, $options_array = array()) {
 		$options_array['subsections'] = array_merge(
 			array(
-				'first_name' 	=> new EE_Text_Input( array( 'required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-fname', 'html_label_text' => __( 'First Name', 'event_espresso' ) ) ),
-				'last_name'		=> new EE_Text_Input( array( 'required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-lname', 'html_label_text' => __( 'Last Name', 'event_espresso' ) ) ),
-				'email'				=> new EE_Email_Input( array( 'required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-email', 'html_label_text' => __( 'Email', 'event_espresso' ) ) ),
-				'address'			=> new EE_Text_Input( array( 'html_label_text'=>  __( 'Address', 'event_espresso'), 'required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-address' )),
-				'address2'		=> new EE_Text_Input( array( 'html_label_text'=> __( 'Address 2', 'event_espresso'), 'html_class' => 'ee-billing-qstn ee-billing-qstn-address2' )),
-				'city'					=> new EE_Text_Input( array( 'required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-city', 'html_label_text' => __( 'City', 'event_espresso' ) ) ),
-				'state' 				=> apply_filters( 'FHEE__EE_Billing_Attendee_Info_Form__state_field', new EE_State_Select_Input( NULL, array( 'required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-state', 'html_label_text' => __( 'State', 'event_espresso' ) ) ) ),
-				'country' 			=> apply_filters( 'FHEE__EE_Billing_Attendee_Info_Form__country_field', new EE_Country_Select_Input( NULL, array( 'required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-country', 'html_label_text' => __( 'Country', 'event_espresso' ) ) ) ),
-				'zip'					=> new EE_Text_Input( array( 'required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-zip', 'html_label_text' => __( 'Zip', 'event_espresso' ) ) ),
-				'phone'			=> new EE_Text_Input( array( 'html_class' => 'ee-billing-qstn ee-billing-qstn-phone', 'html_label_text' => __( 'Phone', 'event_espresso' ) )),
+				'first_name' 	=> new EE_Text_Input(array('required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-fname', 'html_label_text' => __('First Name', 'event_espresso'))),
+				'last_name'		=> new EE_Text_Input(array('required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-lname', 'html_label_text' => __('Last Name', 'event_espresso'))),
+				'email'				=> new EE_Email_Input(array('required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-email', 'html_label_text' => __('Email', 'event_espresso'))),
+				'address'			=> new EE_Text_Input(array('html_label_text'=>  __('Address', 'event_espresso'), 'required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-address')),
+				'address2'		=> new EE_Text_Input(array('html_label_text'=> __('Address 2', 'event_espresso'), 'html_class' => 'ee-billing-qstn ee-billing-qstn-address2')),
+				'city'					=> new EE_Text_Input(array('required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-city', 'html_label_text' => __('City', 'event_espresso'))),
+				'state' 				=> apply_filters('FHEE__EE_Billing_Attendee_Info_Form__state_field', new EE_State_Select_Input(NULL, array('required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-state', 'html_label_text' => __('State', 'event_espresso')))),
+				'country' 			=> apply_filters('FHEE__EE_Billing_Attendee_Info_Form__country_field', new EE_Country_Select_Input(NULL, array('required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-country', 'html_label_text' => __('Country', 'event_espresso')))),
+				'zip'					=> new EE_Text_Input(array('required'=>TRUE, 'html_class' => 'ee-billing-qstn ee-billing-qstn-zip', 'html_label_text' => __('Zip', 'event_espresso'))),
+				'phone'			=> new EE_Text_Input(array('html_class' => 'ee-billing-qstn ee-billing-qstn-phone', 'html_label_text' => __('Phone', 'event_espresso'))),
 			),
-			isset( $options_array['subsections'] ) ? $options_array['subsections'] : array()
+			isset($options_array['subsections']) ? $options_array['subsections'] : array()
 		);
 
-		parent::__construct( $payment_method, $options_array );
+		parent::__construct($payment_method, $options_array);
 
 	}
 
@@ -46,11 +46,11 @@ 
 	 * Sets the defaults for the billing form according to the attendee's details
 	 * @param EE_Attendee $attendee
 	 */
-	public function populate_from_attendee( $attendee ){
+	public function populate_from_attendee($attendee) {
 		$attendee = EEM_Attendee::instance()->ensure_is_obj($attendee);
 		/** @var $attendee EE_Attendee */
 		$this->populate_defaults( 
-			apply_filters( 'FHEE__EE_Billing_Attendee_Info_Form__populate_from_attendee',
+			apply_filters('FHEE__EE_Billing_Attendee_Info_Form__populate_from_attendee',
 				array(
 					'first_name'=>$attendee->fname(),
 					'last_name'=>$attendee->lname(),
@@ -78,48 +78,48 @@ 
 	 * @param \EE_Attendee $attendee - the attendee object to copy details to
 	 * @return \EE_Attendee
 	 */
-	public function copy_billing_form_data_to_attendee( EE_Attendee $attendee ){
+	public function copy_billing_form_data_to_attendee(EE_Attendee $attendee) {
 		// grab billing form data
 		$data = $this->valid_data();
 		// copy first_name
-		if ( ! empty( $data['first_name'] )) {
-			$attendee->set_fname( $data['first_name'] );
+		if ( ! empty($data['first_name'])) {
+			$attendee->set_fname($data['first_name']);
 		}
 		// copy last_name
-		if ( ! empty( $data['last_name'] )) {
-			$attendee->set_lname( $data['last_name'] );
+		if ( ! empty($data['last_name'])) {
+			$attendee->set_lname($data['last_name']);
 		}
 		// copy email
-		if ( ! empty( $data['email'] )) {
-			$attendee->set_email( $data['email'] );
+		if ( ! empty($data['email'])) {
+			$attendee->set_email($data['email']);
 		}
 		// copy address
-		if ( ! empty( $data['address'] )) {
-			$attendee->set_address( $data['address'] );
+		if ( ! empty($data['address'])) {
+			$attendee->set_address($data['address']);
 		}
 		// copy address2
-		if ( ! empty( $data['address2'] )) {
-			$attendee->set_address2( $data['address2'] );
+		if ( ! empty($data['address2'])) {
+			$attendee->set_address2($data['address2']);
 		}
 		// copy city
-		if ( ! empty( $data['city'] )) {
-			$attendee->set_city( $data['city'] );
+		if ( ! empty($data['city'])) {
+			$attendee->set_city($data['city']);
 		}
 		// copy state
-		if ( ! empty( $data['state'] )) {
-			$attendee->set_state( $data['state'] );
+		if ( ! empty($data['state'])) {
+			$attendee->set_state($data['state']);
 		}
 		// copy country
-		if ( ! empty( $data['country'] )) {
-			$attendee->set_country( $data['country'] );
+		if ( ! empty($data['country'])) {
+			$attendee->set_country($data['country']);
 		}
 		// copy zip
-		if ( ! empty( $data['zip'] )) {
-			$attendee->set_zip( $data['zip'] );
+		if ( ! empty($data['zip'])) {
+			$attendee->set_zip($data['zip']);
 		}
 		// copy phone
-		if ( ! empty( $data['phone'] )) {
-			$attendee->set_phone( $data['phone'] );
+		if ( ! empty($data['phone'])) {
+			$attendee->set_phone($data['phone']);
 		}
 		return $attendee;
 	}
@@ -130,20 +130,20 @@ 
 	 * uses info from the billing form to create a new attendee
 	 * @return \EE_Attendee
 	 */
-	public function create_attendee_from_billing_form_data(){
+	public function create_attendee_from_billing_form_data() {
 		// grab billing form data
 		$data = $this->valid_data();
-		return EE_Attendee::new_instance( array(
-			'ATT_fname' 		=> ! empty( $data['first_name'] ) ? $data['first_name'] : '',
-			'ATT_lname' 		=> ! empty( $data['last_name'] ) ? $data['last_name'] : '',
-			'ATT_email' 		=> ! empty( $data['email'] ) ? $data['email'] : '',
-			'ATT_address' 		=> ! empty( $data['address'] ) ? $data['address'] : '',
-			'ATT_address2' 	=> ! empty( $data['address2'] ) ? $data['address2'] : '',
-			'ATT_city' 			=> ! empty( $data['city'] ) ? $data['city'] : '',
-			'STA_ID' 				=> ! empty( $data['state'] ) ? $data['state'] : '',
-			'CNT_ISO' 			=> ! empty( $data['country'] ) ? $data['country'] : '',
-			'ATT_zip' 				=> ! empty( $data['zip'] ) ? $data['zip'] : '',
-			'ATT_phone' 		=> ! empty( $data['phone'] ) ? $data['phone'] : '',
+		return EE_Attendee::new_instance(array(
+			'ATT_fname' 		=> ! empty($data['first_name']) ? $data['first_name'] : '',
+			'ATT_lname' 		=> ! empty($data['last_name']) ? $data['last_name'] : '',
+			'ATT_email' 		=> ! empty($data['email']) ? $data['email'] : '',
+			'ATT_address' 		=> ! empty($data['address']) ? $data['address'] : '',
+			'ATT_address2' 	=> ! empty($data['address2']) ? $data['address2'] : '',
+			'ATT_city' 			=> ! empty($data['city']) ? $data['city'] : '',
+			'STA_ID' 				=> ! empty($data['state']) ? $data['state'] : '',
+			'CNT_ISO' 			=> ! empty($data['country']) ? $data['country'] : '',
+			'ATT_zip' 				=> ! empty($data['zip']) ? $data['zip'] : '',
+			'ATT_phone' 		=> ! empty($data['phone']) ? $data['phone'] : '',
 		));
 	}
 }

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

Spacing +38 added lines, -38 removed lines

@@ -8,7 +8,7 @@ 
  * @since 				4.6
  *
  */
-class EE_Conditionally_Required_Validation_Strategy extends EE_Validation_Strategy_Base{
+class EE_Conditionally_Required_Validation_Strategy extends EE_Validation_Strategy_Base {
 
 	/**
 	 * Array describing conditions necessary to make the input required.
@@ -24,12 +24,12 @@ 
 	 * @param string $validation_error_message
 	 * @param array $requirement_conditions
 	 */
-	public function __construct( $validation_error_message = null, $requirement_conditions = array() ) {
-		if( ! $validation_error_message ){
+	public function __construct($validation_error_message = null, $requirement_conditions = array()) {
+		if ( ! $validation_error_message) {
 			$validation_error_message = __("This field is required.", "event_espresso");
 		}
-		$this->set_requirement_conditions( $requirement_conditions );
-		parent::__construct( $validation_error_message );
+		$this->set_requirement_conditions($requirement_conditions);
+		parent::__construct($validation_error_message);
 	}
 
 
@@ -44,7 +44,7 @@ 
 	 * @throws \EE_Validation_Error
 	 */
 	public function validate($normalized_value) {
-		if(
+		if (
 			(
 				$normalized_value === ''
 				|| $normalized_value === null
@@ -52,8 +52,8 @@ 
 			)
 			&& $this->_input_is_required_server_side()
 		) {
-			throw new EE_Validation_Error( $this->get_validation_error_message(), 'required');
-		}else{
+			throw new EE_Validation_Error($this->get_validation_error_message(), 'required');
+		} else {
 			return true;
 		}
 	}
@@ -64,7 +64,7 @@ 
 	 * @return array
 	 * @throws \EE_Error
 	 */
-	public function get_jquery_validation_rule_array(){
+	public function get_jquery_validation_rule_array() {
 		return array(
 			'required'=> $this->_get_jquery_requirement_value(),
 			'messages' => array(
@@ -82,7 +82,7 @@ 
 	 *
 	 * @param array $requirement_conditions
 	 */
-	public function set_requirement_conditions( $requirement_conditions ) {
+	public function set_requirement_conditions($requirement_conditions) {
 		$this->requirement_conditions = (array) $requirement_conditions;
 	}
 
@@ -108,10 +108,10 @@ 
 	protected function _get_jquery_requirement_value() {
 		$requirement_value = '';
 		$conditions = $this->get_requirement_conditions();
-		if( ! is_array( $conditions ) ){
+		if ( ! is_array($conditions)) {
 			EE_Error::throw_exception_if_debugging(
 				sprintf(
-					__( 'Input requirement conditions must be an array. You provided %1$s', 'event_espresso' ),
+					__('Input requirement conditions must be an array. You provided %1$s', 'event_espresso'),
 					$this->_input->name()
 				),
 				__FILE__,
@@ -120,10 +120,10 @@ 
 			);
 			return true;
 		}
-		if( count( $conditions ) > 1 ) {
+		if (count($conditions) > 1) {
 			EE_Error::throw_exception_if_debugging(
 				sprintf(
-					__( 'Required Validation Strategy does not yet support multiple conditions. You should add it! The related input is %1$s', 'event_espresso' ),
+					__('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__,
@@ -131,12 +131,12 @@ 
 				__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 ) {
+		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(
-						__( 'Error encountered while setting requirement condition for input %1$s. The path %2$s does not correspond to a valid input'),
+						__('Error encountered while setting requirement condition for input %1$s. The path %2$s does not correspond to a valid input'),
 						$this->_input->name(),
 						$input_path
 					),
@@ -146,14 +146,14 @@ 
 				);
 				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(
-						__( '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' ),
+						__('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() ),
+						get_class($input->get_display_strategy()),
 						$this->_input->name()
 					),
 					__FILE__,
@@ -161,7 +161,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;
 	}
@@ -179,12 +179,12 @@ 
 	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 ) {
+		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(
-						__( 'Error encountered while setting requirement condition for input %1$s. The path %2$s does not correspond to a valid input'),
+						__('Error encountered while setting requirement condition for input %1$s. The path %2$s does not correspond to a valid input'),
 						$this->_input->name(),
 						$input_path
 					),
@@ -194,13 +194,13 @@ 
 				);
 				return false;
 			}
-			list( $op, $value ) = $this->_validate_op_and_value( $op_and_value );
-			switch( $op ) {
+			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;
 			}
 		}
@@ -217,11 +217,11 @@ 
 	 * @return array
 	 * @throws \EE_Error
 	 */
-	protected function _validate_op_and_value( $op_and_value ) {
-		if( ! isset( $op_and_value[ 0 ], $op_and_value[ 1 ] ) ) {
+	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(
-						__( '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' ),
+						__('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__,
@@ -229,12 +229,12 @@ 
 					__LINE__
 				);
 			}
-			$operator = $op_and_value[ 0 ];
-			$value = (string) $op_and_value[ 1 ];
-			if( $operator !== '=' ) {
+			$operator = $op_and_value[0];
+			$value = (string) $op_and_value[1];
+			if ($operator !== '=') {
 				EE_Error::throw_exception_if_debugging(
 					sprintf(
-						__( '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' ),
+						__('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__,
@@ -242,7 +242,7 @@ 
 					__LINE__
 				);
 			}
-			return array( $operator, $value );
+			return array($operator, $value);
 
 	}
 }

Braces +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 			&& $this->_input_is_required_server_side()
 		) {
 			throw new EE_Validation_Error( $this->get_validation_error_message(), 'required');
-		}else{
+		} else{
 			return true;
 		}
 	}