eventespresso / event-espresso-core

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

Spacing +16 added lines, -16 removed lines

@@ -8,7 +8,7 @@ 
  * @subpackage
  * @author				Mike Nelson
  */
-class EE_Select_Multi_Model_Input extends EE_Select_Multiple_Input{
+class EE_Select_Multi_Model_Input extends EE_Select_Multiple_Input {
 
 
 	protected $_naming_method;
@@ -24,18 +24,18 @@ 
 	 *     example: 	if the class were an EE_Event, this could be slug(), description(), name() (default)
 	 * }
 	 */
-	public function __construct( $answer_options = array(), $input_settings = array() ) {
-		if( isset( $input_settings['naming_method'] )){
-			$this->set_option_naming_method( $input_settings['naming_method'] );
+	public function __construct($answer_options = array(), $input_settings = array()) {
+		if (isset($input_settings['naming_method'])) {
+			$this->set_option_naming_method($input_settings['naming_method']);
 		}
-		parent::__construct( $answer_options, $input_settings );
+		parent::__construct($answer_options, $input_settings);
 	}
 
 	/**
 	 * Sets the method name which will be called when outputting the options list
 	 * @param string $method
 	 */
-	public function set_option_naming_method($method){
+	public function set_option_naming_method($method) {
 		$this->_naming_method = $method;
 	}
 
@@ -46,16 +46,16 @@ 
 	 * @param EE_Base_Class[] $answer_options
 	 * @return null|void
 	 */
-	public function set_select_options( $answer_options = array() ) {
+	public function set_select_options($answer_options = array()) {
 		//convert the model objects to select from into normal select options
 		$select_options = array();
-		foreach( $answer_options as $model_obj ){
-			if( $this->_naming_method ){
-				$display_value = call_user_func( array( $model_obj, $this->_naming_method ));
-			}else{
+		foreach ($answer_options as $model_obj) {
+			if ($this->_naming_method) {
+				$display_value = call_user_func(array($model_obj, $this->_naming_method));
+			} else {
 				$display_value = $model_obj->name();
 			}
-			$select_options[ $model_obj->ID() ] = $display_value;
+			$select_options[$model_obj->ID()] = $display_value;
 		}
 		parent::set_select_options($select_options);
 	}
@@ -67,12 +67,12 @@ 
 	 * into the format EE_Select_Multiple expects
 	 * @param EE_Base_Class[]|array $values
 	 */
-	public function set_default($values){
+	public function set_default($values) {
 		$defaults_as_simple_ids = array();
-		foreach($values as $key => $value){
-			if($value instanceof EE_Base_Class){
+		foreach ($values as $key => $value) {
+			if ($value instanceof EE_Base_Class) {
 				$defaults_as_simple_ids[] = $value->ID();
-			}else{
+			} else {
 				$defaults_as_simple_ids[] = $value;
 			}
 		}

Braces +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
 		foreach( $answer_options as $model_obj ){
 			if( $this->_naming_method ){
 				$display_value = call_user_func( array( $model_obj, $this->_naming_method ));
-			}else{
+			} else{
 				$display_value = $model_obj->name();
 			}
 			$select_options[ $model_obj->ID() ] = $display_value;
@@ -72,7 +72,7 @@ 
 		foreach($values as $key => $value){
 			if($value instanceof EE_Base_Class){
 				$defaults_as_simple_ids[] = $value->ID();
-			}else{
+			} else{
 				$defaults_as_simple_ids[] = $value;
 			}
 		}

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

Spacing +5 added lines, -5 removed lines

@@ -6,17 +6,17 @@
  * @subpackage
  * @author				Mike Nelson
  */
-class EE_Select_Multiple_Input extends EE_Form_Input_With_Options_Base{
+class EE_Select_Multiple_Input extends EE_Form_Input_With_Options_Base {
 
 	/**
 	 * @param array | EE_Question_Option[] $answer_options
 	 * @param array $input_settings
 	 */
-	public function __construct( $answer_options, $input_settings = array() ) {
-		$this->_set_display_strategy( new EE_Select_Multiple_Display_Strategy() );
-		$this->_add_validation_strategy( new EE_Many_Valued_Validation_Strategy( array( new EE_Enum_Validation_Strategy( isset( $input_settings[ 'validation_error_message' ] ) ? $input_settings[ 'validation_error_message' ] : NULL ) )));
+	public function __construct($answer_options, $input_settings = array()) {
+		$this->_set_display_strategy(new EE_Select_Multiple_Display_Strategy());
+		$this->_add_validation_strategy(new EE_Many_Valued_Validation_Strategy(array(new EE_Enum_Validation_Strategy(isset($input_settings['validation_error_message']) ? $input_settings['validation_error_message'] : NULL))));
 		$this->_multiple_selections = TRUE;
-		parent::__construct( $answer_options, $input_settings );
+		parent::__construct($answer_options, $input_settings);
 	}
 
 }

core/libraries/form_sections/inputs/EE_State_Select_Input.php

Spacing +10 added lines, -10 removed lines

@@ -6,19 +6,19 @@ 
  * @subpackage
  * @author				Mike Nelson
  */
-class EE_State_Select_Input extends EE_Select_Input{
+class EE_State_Select_Input extends EE_Select_Input {
 
 	/**
 	 * @param array $state_options
 	 * @param array $input_settings
 	 */
-	function __construct( $state_options, $input_settings = array() ){
+	function __construct($state_options, $input_settings = array()) {
 		$state_options = apply_filters(
 			'FHEE__EE_State_Select_Input____construct__state_options',
-			$this->get_state_answer_options( $state_options ),
+			$this->get_state_answer_options($state_options),
 			$this
 		);
-		parent::__construct( $state_options, $input_settings );
+		parent::__construct($state_options, $input_settings);
 	}
 
 
@@ -29,17 +29,17 @@ 
 	 * @param array $state_options
 	 * @return array
 	 */
-	public function get_state_answer_options( $state_options = NULL ){
+	public function get_state_answer_options($state_options = NULL) {
 		// if passed something that is NOT an array
-		if ( ! is_array( $state_options )) {
+		if ( ! is_array($state_options)) {
 			// get possibly cached list of states
 			$states = EEM_State::instance()->get_all_active_states();
-			if ( ! empty( $states )) {
+			if ( ! empty($states)) {
 				//set the default
 				$state_options[''][''] = '';
-				foreach( $states as $state ){
-					if ( $state instanceof EE_State ) {
-						$state_options[ $state->country()->name() ][ $state->ID() ] = $state->name();
+				foreach ($states as $state) {
+					if ($state instanceof EE_State) {
+						$state_options[$state->country()->name()][$state->ID()] = $state->name();
 					}
 				}
 			} else {

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

Spacing +3 added lines, -3 removed lines

@@ -8,15 +8,15 @@
  *
  * This input has a default validation strategy of plaintext (which can be removed after construction)
  */
-class EE_Submit_Input extends EE_Form_Input_Base{
+class EE_Submit_Input extends EE_Form_Input_Base {
 
 	/**
 	 * @param array $options
 	 */
-	function __construct($options = array()){
+	function __construct($options = array()) {
 		$this->_set_display_strategy(new EE_Submit_Input_Display_Strategy());
 		$this->_set_normalization_strategy(new EE_Text_Normalization());
-		$this->_add_validation_strategy( new EE_Plaintext_Validation_Strategy() );
+		$this->_add_validation_strategy(new EE_Plaintext_Validation_Strategy());
 		parent::__construct($options);
 	}
 }
\ No newline at end of file

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

Spacing +4 added lines, -4 removed lines

@@ -6,14 +6,14 @@
  * @subpackage
  * @author				Mike Nelson
  */
-class EE_Yes_No_Input extends EE_Select_Input{
+class EE_Yes_No_Input extends EE_Select_Input {
 
 	/**
 	 * @param array $options
 	 */
-	function __construct( $options = array()){
-		$select_options = array(true=>  __("Yes", "event_espresso"),false=>  __("No", "event_espresso"));
+	function __construct($options = array()) {
+		$select_options = array(true=>  __("Yes", "event_espresso"), false=>  __("No", "event_espresso"));
 
-		parent::__construct($select_options,$options);
+		parent::__construct($select_options, $options);
 	}
 }

core/libraries/form_sections/strategies/display/EE_Hidden_Display_Strategy.strategy.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION'))
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('No direct script access allowed');
 
 /**
@@ -25,12 +25,12 @@ 
  *
  * ------------------------------------------------------------------------
  */
-class EE_Hidden_Display_Strategy extends EE_Display_Strategy_Base{
+class EE_Hidden_Display_Strategy extends EE_Display_Strategy_Base {
 	/**
 	 * 
 	 * @return string of html to display the HIDDEN field
 	 */
-	function display(){
+	function display() {
 		$input = $this->_input;		
 		return "<input type='hidden' id='{$input->html_id()}' name='{$input->html_name()}' class='{$input->html_class()}' style='{$input->html_style()}' value='{$input->raw_value_in_form()}'/>";
 	}

Braces +2 added lines, -1 removed lines

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

core/libraries/form_sections/strategies/display/EE_Select_Multiple_Display_Strategy.strategy.php

Braces +3 added lines, -2 removed lines

@@ -1,7 +1,8 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION'))
+if (!defined('EVENT_ESPRESSO_VERSION')) {
 	exit('No direct script access allowed');
+}
 
 /**
  * Event Espresso
@@ -53,7 +54,7 @@ 
 		EEH_HTML::indent( 1, 'select' );
 		if( EEH_Array::is_multi_dimensional_array( $this->_input->options() )){
 			throw new EE_Error(sprintf(__("Select multiple display strategy does not allow for nested arrays of options.", "event_espresso")));
-		}else{
+		} else{
 			$html.=$this->_display_options( $this->_input->options() );
 		}
 

Spacing +18 added lines, -18 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION'))
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('No direct script access allowed');
 
 /**
@@ -25,7 +25,7 @@ 
  *
  * ------------------------------------------------------------------------
  */
-class EE_Select_Multiple_Display_Strategy extends EE_Select_Display_Strategy{
+class EE_Select_Multiple_Display_Strategy extends EE_Select_Display_Strategy {
 
 	/**
 	 *
@@ -34,31 +34,31 @@ 
 	 */
 	public function display() {
 
-		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(__('Cannot use Select Multiple Display Strategy with an input that doesn\'t have options', "event_espresso")));
 		}
 
-		$html = EEH_HTML::nl( 0, 'select' );
+		$html = EEH_HTML::nl(0, 'select');
 		$html .= '<select multiple';
-		$html .= ' id="' . $this->_input->html_id() . '"';
-		$html .= ' name="' . $this->_input->html_name() . '[]"';
-		$class = $this->_input->required() ? $this->_input->required_css_class() . ' ' . $this->_input->html_class() : $this->_input->html_class();
-		$html .= ' class="' . $class . '"';
+		$html .= ' id="'.$this->_input->html_id().'"';
+		$html .= ' name="'.$this->_input->html_name().'[]"';
+		$class = $this->_input->required() ? $this->_input->required_css_class().' '.$this->_input->html_class() : $this->_input->html_class();
+		$html .= ' class="'.$class.'"';
 		// add html5 required
 		$html .= $this->_input->required() ? ' required' : '';
-		$html .= ' style="' . $this->_input->html_style() . '"';
-		$html .= ' ' . $this->_input->other_html_attributes();
+		$html .= ' style="'.$this->_input->html_style().'"';
+		$html .= ' '.$this->_input->other_html_attributes();
 		$html .= '>';
 
 		EE_Registry::instance()->load_helper('Array');
-		EEH_HTML::indent( 1, 'select' );
-		if( EEH_Array::is_multi_dimensional_array( $this->_input->options() )){
+		EEH_HTML::indent(1, 'select');
+		if (EEH_Array::is_multi_dimensional_array($this->_input->options())) {
 			throw new EE_Error(sprintf(__("Select multiple display strategy does not allow for nested arrays of options.", "event_espresso")));
-		}else{
-			$html.=$this->_display_options( $this->_input->options() );
+		} else {
+			$html .= $this->_display_options($this->_input->options());
 		}
 
-		$html.= EEH_HTML::nl( -1, 'select' ) . "</select>";
+		$html .= EEH_HTML::nl( -1, 'select' )."</select>";
 		return $html;
 	}
 
@@ -69,12 +69,12 @@ 
 	 * @param string|int $value unnormalized value option (string)
 	 * @return boolean
 	 */
-	protected function _check_if_option_selected( $value ){
+	protected function _check_if_option_selected($value) {
 		$selected_options = $this->_input->raw_value();
-		if ( empty( $selected_options )){
+		if (empty($selected_options)) {
 			return FALSE;
 		}
-		return in_array( $value, $selected_options ) ? TRUE : FALSE;
+		return in_array($value, $selected_options) ? TRUE : FALSE;
 	}
 
 

core/libraries/form_sections/strategies/display/EE_Submit_Input_Display_Strategy.strategy.php

Spacing +6 added lines, -6 removed lines

@@ -1,15 +1,15 @@
 <?php
-class EE_Submit_Input_Display_Strategy extends EE_Display_Strategy_Base{
+class EE_Submit_Input_Display_Strategy extends EE_Display_Strategy_Base {
 	/**
 	 *
 	 * @return string of html to display the input
 	 */
-	function display(){
+	function display() {
 		$html = '<input type="submit" ';
-		$html .= 'value="' . $this->_input->raw_value_in_form() . '" ';
-		$html .= 'id="' . $this->_input->html_id() . '-submit" ';
-		$html .= 'class="' . $this->_input->html_class() . ' ' . $this->_input->button_css_attributes() . '" ';
-		$html .= 'style="' . $this->_input->html_style() . '" ';
+		$html .= 'value="'.$this->_input->raw_value_in_form().'" ';
+		$html .= 'id="'.$this->_input->html_id().'-submit" ';
+		$html .= 'class="'.$this->_input->html_class().' '.$this->_input->button_css_attributes().'" ';
+		$html .= 'style="'.$this->_input->html_style().'" ';
 		$html .= $this->_input->other_html_attributes();
 		$html .= '/>';
 		return $html;

core/libraries/form_sections/strategies/layout/EE_Fieldset_Section_Layout.strategy.php

Indentation +11 added lines, -11 removed lines

@@ -1,16 +1,16 @@
 <?php if ( ! defined('EVENT_ESPRESSO_VERSION')) { exit('No direct script access allowed'); }
  /**
- *
- * Class EE_Fieldset_Section_Layout
- *
- * Description
- *
- * @package         Event Espresso
- * @subpackage    core
- * @author				Brent Christensen
- * @since		 	   $VID:$
- *
- */
+  *
+  * Class EE_Fieldset_Section_Layout
+  *
+  * Description
+  *
+  * @package         Event Espresso
+  * @subpackage    core
+  * @author				Brent Christensen
+  * @since		 	   $VID:$
+  *
+  */
 class EE_Fieldset_Section_Layout extends EE_Div_Per_Section_Layout{
 
 	/**

Spacing +11 added lines, -11 removed lines

@@ -11,7 +11,7 @@ 
  * @since		 	   $VID:$
  *
  */
-class EE_Fieldset_Section_Layout extends EE_Div_Per_Section_Layout{
+class EE_Fieldset_Section_Layout extends EE_Div_Per_Section_Layout {
 
 	/**
 	 * legend_class
@@ -32,10 +32,10 @@ 
 	 *
 	 * @param array $options
 	 */
-	function __construct( $options = array() ){
-		foreach( $options as $key => $value ) {
-			$key = '_' . $key;
-			if ( property_exists( $this, $key )) {
+	function __construct($options = array()) {
+		foreach ($options as $key => $value) {
+			$key = '_'.$key;
+			if (property_exists($this, $key)) {
 				$this->$key = $value;
 			}
 		}
@@ -47,8 +47,8 @@ 
 	 * @return string
 	 */
 	public function layout_form_begin() {
-		$html = EEH_HTML::nl(1) . '<fieldset id="' . $this->_form_section->html_id() . '" class="' . $this->_form_section->html_class() . '" style="' . $this->_form_section->html_style() . '">';
-		$html .= '<legend class="' . $this->legend_class() . '">' . $this->legend_text() . '</legend>';
+		$html = EEH_HTML::nl(1).'<fieldset id="'.$this->_form_section->html_id().'" class="'.$this->_form_section->html_class().'" style="'.$this->_form_section->html_style().'">';
+		$html .= '<legend class="'.$this->legend_class().'">'.$this->legend_text().'</legend>';
 		return $html;
 	}
 
@@ -58,8 +58,8 @@ 
 	 * closing div tag for a form
 	 * @return string
 	 */
-	public function layout_form_end(){
-		return EEH_HTML::nl(-1) . '</fieldset>';
+	public function layout_form_end() {
+		return EEH_HTML::nl(-1).'</fieldset>';
 	}
 
 
@@ -67,7 +67,7 @@ 
 	/**
 	 * @param string $legend_class
 	 */
-	public function set_legend_class( $legend_class ) {
+	public function set_legend_class($legend_class) {
 		$this->_legend_class = $legend_class;
 	}
 
@@ -85,7 +85,7 @@ 
 	/**
 	 * @param string $legend_text
 	 */
-	public function set_legend_text( $legend_text ) {
+	public function set_legend_text($legend_text) {
 		$this->_legend_text = $legend_text;
 	}