eventespresso / event-espresso-core

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

Spacing +17 added lines, -17 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,30 +34,30 @@ 
 	 */
 	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 .= ' style="'.$this->_input->html_style().'"';
 		$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;
 	}
 
@@ -68,12 +68,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;
 	}
 
 

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() );
 		}
 

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;
 	}
 

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

Spacing +34 added lines, -34 removed lines

@@ -8,7 +8,7 @@ 
  * stating where the proper subsections should be placed (but usually leaving them to layout
  * their own headers and footers etc).
  */
-abstract class EE_Form_Section_Layout_Base{
+abstract class EE_Form_Section_Layout_Base {
 
 	/**
 	 * Form form section to lay out
@@ -21,7 +21,7 @@ 
 	/**
 	 * 	__construct
 	 */
-	function __construct(){
+	function __construct() {
 		EE_Registry::instance()->load_helper('HTML');
 	}
 
@@ -31,7 +31,7 @@ 
 	 * The form section on which this strategy is to perform
 	 * @param EE_Form_Section_Proper $form
 	 */
-	function _construct_finalize(EE_Form_Section_Proper $form){
+	function _construct_finalize(EE_Form_Section_Proper $form) {
 		$this->_form_section = $form;
 	}
 
@@ -54,28 +54,28 @@ 
 	 * Returns the HTML
 	 * @return string HTML for displaying
 	 */
-	function layout_form(){
+	function layout_form() {
 		$html = '';
 		// layout_form_begin
 		$html .= apply_filters(
-			'FHEE__EE_Form_Section_Layout_Base__layout_form__start__for_' . $this->_form_section->name(),
+			'FHEE__EE_Form_Section_Layout_Base__layout_form__start__for_'.$this->_form_section->name(),
 			$this->layout_form_begin(),
 			$this->_form_section
 		);
 		// layout_form_loop
 		$html .= apply_filters(
-			'FHEE__EE_Form_Section_Layout_Base__layout_form__loop__for_' . $this->_form_section->name(),
+			'FHEE__EE_Form_Section_Layout_Base__layout_form__loop__for_'.$this->_form_section->name(),
 			$this->layout_form_loop(),
 			$this->_form_section
 		);
 		// layout_form_end
 		$html .= apply_filters(
-			'FHEE__EE_Form_Section_Layout_Base__layout_form__end__for_' . $this->_form_section->name(),
+			'FHEE__EE_Form_Section_Layout_Base__layout_form__end__for_'.$this->_form_section->name(),
 			$this->layout_form_end(),
 			$this->_form_section
 		);
 
-		$html = $this->add_form_section_hooks_and_filters( $html );
+		$html = $this->add_form_section_hooks_and_filters($html);
 		return $html;
 	}
 
@@ -84,20 +84,20 @@ 
 	/**
 	 * @return string
 	 */
-	function layout_form_loop(){
+	function layout_form_loop() {
 		$html = '';
-		foreach( $this->_form_section->subsections() as $name => $subsection ){
-			if ( $subsection instanceof EE_Form_Input_Base ){
+		foreach ($this->_form_section->subsections() as $name => $subsection) {
+			if ($subsection instanceof EE_Form_Input_Base) {
 				$html .= apply_filters(
-					'FHEE__EE_Form_Section_Layout_Base__layout_form__loop_for_input_' . $name . '__in_' . $this->_form_section->name(),
-					$this->layout_input( $subsection ),
+					'FHEE__EE_Form_Section_Layout_Base__layout_form__loop_for_input_'.$name.'__in_'.$this->_form_section->name(),
+					$this->layout_input($subsection),
 					$this->_form_section,
 					$subsection
 				);
-			} elseif ( $subsection instanceof EE_Form_Section_Base ){
+			} elseif ($subsection instanceof EE_Form_Section_Base) {
 				$html .= apply_filters(
-					'FHEE__EE_Form_Section_Layout_Base__layout_form__loop_for_non_input_' . $name . '__in_' . $this->_form_section->name(),
-					$this->layout_subsection( $subsection ),
+					'FHEE__EE_Form_Section_Layout_Base__layout_form__loop_for_non_input_'.$name.'__in_'.$this->_form_section->name(),
+					$this->layout_subsection($subsection),
 					$this->_form_section,
 					$subsection
 				);
@@ -145,7 +145,7 @@ 
 	 * @param EE_Form_Section_Base $subsection
 	 * @return string html
 	 */
-	abstract function layout_subsection( $subsection );
+	abstract function layout_subsection($subsection);
 
 
 
@@ -154,10 +154,10 @@ 
 	 * @param EE_Form_Input_Base $input
 	 * @return string
 	 */
-	public function display_label($input){
-		$class = $input->required() ? 'ee-required-label ' . $input->html_label_class() : $input->html_label_class();
-		$label_text = $input->required() ? $input->html_label_text() . '<span class="ee-asterisk">*</span>' : $input->html_label_text();
-		return '<label id="' . $input->html_label_id() . '" class="' . $class . '" style="' . $input->html_label_style() . '" for="' . $input->html_name() . '">' .  $label_text . '</label>';
+	public function display_label($input) {
+		$class = $input->required() ? 'ee-required-label '.$input->html_label_class() : $input->html_label_class();
+		$label_text = $input->required() ? $input->html_label_text().'<span class="ee-asterisk">*</span>' : $input->html_label_text();
+		return '<label id="'.$input->html_label_id().'" class="'.$class.'" style="'.$input->html_label_style().'" for="'.$input->html_name().'">'.$label_text.'</label>';
 	}
 
 
@@ -170,10 +170,10 @@ 
 	 * @param EE_Form_Input_Base $input
 	 * @return string
 	 */
-	public function display_errors($input){
-		if( $input->get_validation_errors() ){
-			return  "<span  id='" . $input->html_id() . "-error' class='ee-error-label red ee-red' for='{$input->html_name()}'>" . $input->get_validation_error_string() . "</span>";
-		}else{
+	public function display_errors($input) {
+		if ($input->get_validation_errors()) {
+			return  "<span  id='".$input->html_id()."-error' class='ee-error-label red ee-red' for='{$input->html_name()}'>".$input->get_validation_error_string()."</span>";
+		} else {
 			return '';
 		}
 	}
@@ -183,10 +183,10 @@ 
 	 * @param EE_Form_Input_Base $input
 	 * @return string
 	 */
-	public function display_help_text( $input ){
-		if ( $input->html_help_text() != '' ) {
+	public function display_help_text($input) {
+		if ($input->html_help_text() != '') {
 			$tag = is_admin() ? 'p' : 'span';
-			return '<' . $tag . ' id="' . $input->html_id() . '-help" class="' . $input->html_help_class() . '" style="' . $input->html_help_style() . '">' . $input->html_help_text() . '</' . $tag . '>';
+			return '<'.$tag.' id="'.$input->html_id().'-help" class="'.$input->html_help_class().'" style="'.$input->html_help_style().'">'.$input->html_help_text().'</'.$tag.'>';
 		}
 		return '';
 	}
@@ -198,13 +198,13 @@ 
 	 * @param string $html
 	 * @return string
 	 */
-	public function add_form_section_hooks_and_filters( $html ){
+	public function add_form_section_hooks_and_filters($html) {
 		// replace dashes and spaces with underscores
-		$hook_name = str_replace( array( '-', ' ' ), '_', $this->_form_section->html_id() );
-		do_action( 'AHEE__Form_Section_Layout__' . $hook_name, $this->_form_section );
-		$html = apply_filters( 'AFEE__Form_Section_Layout__' . $hook_name . '__html', $html, $this->_form_section );
-		$html .= EEH_HTML::nl() . '<!-- AHEE__Form_Section_Layout__' . $hook_name . '__html -->';
-		$html .= EEH_HTML::nl() . '<!-- AFEE__Form_Section_Layout__' . $hook_name . ' -->';
+		$hook_name = str_replace(array('-', ' '), '_', $this->_form_section->html_id());
+		do_action('AHEE__Form_Section_Layout__'.$hook_name, $this->_form_section);
+		$html = apply_filters('AFEE__Form_Section_Layout__'.$hook_name.'__html', $html, $this->_form_section);
+		$html .= EEH_HTML::nl().'<!-- AHEE__Form_Section_Layout__'.$hook_name.'__html -->';
+		$html .= EEH_HTML::nl().'<!-- AFEE__Form_Section_Layout__'.$hook_name.' -->';
 		return $html;
 	}
 

Braces +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
 		//so in case teh IPN is arriving later, let's try to process an IPN!
 		if($_SERVER['REQUEST_METHOD'] == 'POST'){
 			return $this->handle_ipn($_POST, $transaction );
-		}else{
+		} else{
 			return parent::finalize_payment_for( $transaction );
 		}
 	}

core/libraries/form_sections/strategies/normalization/EE_All_Caps_Normalization.strategy.php

Spacing +2 added lines, -2 removed lines

@@ -8,7 +8,7 @@ 
  * @subpackage
  * @author				Mike Nelson
  */
-class EE_All_Caps_Normalization extends EE_Normalization_Strategy_Base{
+class EE_All_Caps_Normalization extends EE_Normalization_Strategy_Base {
 
 	/**
 	 * @param string $value_to_normalize
@@ -24,7 +24,7 @@ 
 	 * @param string $normalized_value
 	 * @return string
 	 */
-	public function unnormalize( $normalized_value ) {
+	public function unnormalize($normalized_value) {
 		return $normalized_value;
 	}