eventespresso / event-espresso-core

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

Spacing +16 added lines, -16 removed lines

@@ -8,7 +8,7 @@ 
  * @since                4.6
  *
  */
-class EE_Admin_File_Uploader_Display_Strategy extends EE_Display_Strategy_Base{
+class EE_Admin_File_Uploader_Display_Strategy extends EE_Display_Strategy_Base {
 
 	/**
 	 * return EE_Admin_File_Uploader_Display_Strategy
@@ -29,10 +29,10 @@ 
 	 * Besides that, no special consideration should be required to make the media uploader appear, besides having
 	 * this input displayed.
 	 */
-	static function enqueue_scripts(){
+	static function enqueue_scripts() {
 		wp_enqueue_media();
 		wp_enqueue_script('media-upload');
-		wp_enqueue_script('ee-payments',EE_GLOBAL_ASSETS_URL.'scripts/ee-media-uploader.js');
+		wp_enqueue_script('ee-payments', EE_GLOBAL_ASSETS_URL.'scripts/ee-media-uploader.js');
 	}
 
 
@@ -42,20 +42,20 @@ 
 	 * @return string of html to display the field
 	 */
 
-	function display(){
+	function display() {
 
-		EE_Registry::instance()->load_helper( 'HTML' );
+		EE_Registry::instance()->load_helper('HTML');
 		// the actual input
 		$input = '<input type="text" size="34" ';
-		$input .= 'name="' . $this->_input->html_name() . '" ';
-		$input .= $this->_input->html_class() != '' ? 'class="large-text ee_media_url ' . $this->_input->html_class() . '" ' : 'class="large-text ee_media_url" ';
-		$input .= 'value="' . $this->_input->raw_value_in_form() . '">';
+		$input .= 'name="'.$this->_input->html_name().'" ';
+		$input .= $this->_input->html_class() != '' ? 'class="large-text ee_media_url '.$this->_input->html_class().'" ' : 'class="large-text ee_media_url" ';
+		$input .= 'value="'.$this->_input->raw_value_in_form().'">';
 		// image uploader
-		$uploader = EEH_HTML::link( '#', '<img src="' . admin_url( 'images/media-button-image.gif' ) . '" >', __( 'click to add an image', 'event_espresso' ), '', 'ee_media_upload' );
+		$uploader = EEH_HTML::link('#', '<img src="'.admin_url('images/media-button-image.gif').'" >', __('click to add an image', 'event_espresso'), '', 'ee_media_upload');
 		//only attempt to show the image if it at least exists
-		$image = $this->src_exists( $this->_input->raw_value() ) ? EEH_HTML::br(2) . EEH_HTML::img( $this->_input->raw_value(), __( 'logo', 'event_espresso' ), '', 'ee_media_image' ) : '';
+		$image = $this->src_exists($this->_input->raw_value()) ? EEH_HTML::br(2).EEH_HTML::img($this->_input->raw_value(), __('logo', 'event_espresso'), '', 'ee_media_image') : '';
 		// html string
-		return EEH_HTML::div( $input . EEH_HTML::nbsp() . $uploader . $image, '', 'ee_media_uploader_area' );
+		return EEH_HTML::div($input.EEH_HTML::nbsp().$uploader.$image, '', 'ee_media_uploader_area');
 	}
 
 
@@ -66,11 +66,11 @@ 
 	 * @param string $src
 	 * @return boolean
 	 */
-	protected function src_exists( $src ){
-		$results = wp_remote_head( $src );
-		if( is_array( $results) && ! $results instanceof WP_Error){
-			return strpos($results['headers']['content-type'], "image" ) !== FALSE;
-		}else{
+	protected function src_exists($src) {
+		$results = wp_remote_head($src);
+		if (is_array($results) && ! $results instanceof WP_Error) {
+			return strpos($results['headers']['content-type'], "image") !== FALSE;
+		} else {
 			return FALSE;
 		}
 	}

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/display/EE_Display_Strategy_Base.strategy.php

Spacing +6 added lines, -6 removed lines

@@ -8,7 +8,7 @@ 
  * @since               	4.6
  *
  */
-abstract class EE_Display_Strategy_Base extends EE_Form_Input_Strategy_Base{
+abstract class EE_Display_Strategy_Base extends EE_Form_Input_Strategy_Base {
 	/**
 	 * returns HTML and javascript related to the displaying of this input
 	 * @return string
@@ -24,10 +24,10 @@ 
 	 * @param string $chars - exact string of characters to remove
 	 * @return string
 	 */
-	protected function _remove_chars( $string = '', $chars = '-' ) {
-		$char_length = strlen( $chars ) * -1;
+	protected function _remove_chars($string = '', $chars = '-') {
+		$char_length = strlen($chars) * -1;
 		// if last three characters of string is  " - ", then remove it
-		return substr( $string, $char_length ) == $chars ? substr( $string, 0, $char_length ) : $string;
+		return substr($string, $char_length) == $chars ? substr($string, 0, $char_length) : $string;
 	}
 
 
@@ -39,8 +39,8 @@ 
 	 * @param string $chars - exact string of characters to be added to end of string
 	 * @return string
 	 */
-	protected function _append_chars( $string = '', $chars = '-' ) {
-		return  $this->_remove_chars( $string, $chars ) . $chars;
+	protected function _append_chars($string = '', $chars = '-') {
+		return  $this->_remove_chars($string, $chars).$chars;
 	}
 
 

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@
 <?php if ( ! defined('EVENT_ESPRESSO_VERSION')) { exit('No direct script access allowed'); }
 /**
- * Class EE_Display_Strategy_Base
- *
- * @package 			Event Espresso
- * @subpackage    core
- * @author 				Mike Nelson, Brent Christensen
- * @since               	4.6
- *
- */
+	 * Class EE_Display_Strategy_Base
+	 *
+	 * @package 			Event Espresso
+	 * @subpackage    core
+	 * @author 				Mike Nelson, Brent Christensen
+	 * @since               	4.6
+	 *
+	 */
 abstract class EE_Display_Strategy_Base extends EE_Form_Input_Strategy_Base{
 	/**
 	 * returns HTML and javascript related to the displaying of this input

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_Radio_Button_Display_Strategy.strategy.php

Spacing +20 added lines, -20 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_Radio_Button_Display_Strategy
@@ -11,43 +11,43 @@ 
  * @since 				$VID:$
  *
  */
-class EE_Radio_Button_Display_Strategy extends EE_Display_Strategy_Base{
+class EE_Radio_Button_Display_Strategy extends EE_Display_Strategy_Base {
 
 	/**
 	 *
 	 * @throws EE_Error
 	 * @return string of html to display the field
 	 */
-	function display(){
-		if ( ! $this->_input instanceof EE_Form_Input_With_Options_Base ){
-			throw new EE_Error(sprintf(__('Can not use Radio Button Display Strategy with an input that doesn\'t have options', 'event_espresso' )));
+	function display() {
+		if ( ! $this->_input instanceof EE_Form_Input_With_Options_Base) {
+			throw new EE_Error(sprintf(__('Can not use Radio Button Display Strategy with an input that doesn\'t have options', 'event_espresso')));
 		}
 
 		$this->_input->set_label_sizes();
 		$label_size_class = $this->_input->get_label_size_class();
 		$html = '';
-		foreach( $this->_input->options() as $value => $display_text ){
-			$value = $this->_input->get_normalization_strategy()->unnormalize( $value );
+		foreach ($this->_input->options() as $value => $display_text) {
+			$value = $this->_input->get_normalization_strategy()->unnormalize($value);
 
-			$html_id = $this->_append_chars( $this->_input->html_id(), '-' ) . sanitize_key( $value );
-			$html .= EEH_HTML::nl( 0, 'radio' );
-			$html .= '<label for="' . $html_id . '"';
-			$html .= ' id="' . $html_id . '-lbl"';
-			$html .= ' class="ee-radio-label-after' . $label_size_class . '">';
-			$html .= EEH_HTML::nl( 1, 'radio' );
-			$html .= '<input id="' . $html_id . '"';
-			$html .= ' name="' . $this->_input->html_name() . '"';
-			$html .= ' class="' . $this->_input->html_class() . '"';
-			$html .= ' style="' . $this->_input->html_style() . '"';
+			$html_id = $this->_append_chars($this->_input->html_id(), '-').sanitize_key($value);
+			$html .= EEH_HTML::nl(0, 'radio');
+			$html .= '<label for="'.$html_id.'"';
+			$html .= ' id="'.$html_id.'-lbl"';
+			$html .= ' class="ee-radio-label-after'.$label_size_class.'">';
+			$html .= EEH_HTML::nl(1, 'radio');
+			$html .= '<input id="'.$html_id.'"';
+			$html .= ' name="'.$this->_input->html_name().'"';
+			$html .= ' class="'.$this->_input->html_class().'"';
+			$html .= ' style="'.$this->_input->html_style().'"';
 			$html .= ' type="radio"';
-			$html .= ' value="' . esc_attr( $value ) . '"';
+			$html .= ' value="'.esc_attr($value).'"';
 			$html .= $this->_input->raw_value() === $value ? ' checked="checked"' : '';
 			$html .= '>&nbsp;';
 			$html .= $display_text;
-			$html .= EEH_HTML::nl( -1, 'radio' ) . '</label>';
+			$html .= EEH_HTML::nl( -1, 'radio' ).'</label>';
 
 		}
-		$html .= EEH_HTML::div( '', '', 'clear-float' );
+		$html .= EEH_HTML::div('', '', 'clear-float');
 		$html .= EEH_HTML::divx();
 		return $html;
 	}

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_Text_Area_Display_Strategy.strategy.php

Spacing +4 added lines, -4 removed lines

@@ -1,16 +1,16 @@
 <?php
-class EE_Text_Area_Display_Strategy extends EE_Display_Strategy_Base{
+class EE_Text_Area_Display_Strategy extends EE_Display_Strategy_Base {
 	/**
 	 *
 	 * @return string of html to display the field
 	 */
-	function display(){
+	function display() {
 		$input = $this->_input;
 		$raw_value = maybe_serialize($input->raw_value());
-		if( $input instanceof EE_Text_Area_Input ) {
+		if ($input instanceof EE_Text_Area_Input) {
 			$rows = $input->get_rows();
 			$cols = $input->get_cols();
-		}else{
+		} else {
 			$rows = 4;
 			$cols = 20;
 		}

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/display/EE_Text_Input_Display_Strategy.strategy.php

Spacing +11 added lines, -11 removed lines

@@ -11,7 +11,7 @@ 
  * @since               	4.6
  *
  */
-class EE_Text_Input_Display_Strategy extends EE_Display_Strategy_Base{
+class EE_Text_Input_Display_Strategy extends EE_Display_Strategy_Base {
 	/**
 	 * The html "type" attribute value. default is "text"
 	 * @var string
@@ -23,7 +23,7 @@ 
 	/**
 	 * @param string $type
 	 */
-	function __construct( $type = 'text' ) {
+	function __construct($type = 'text') {
 		$this->_type = $type;
 		parent::__construct();
 	}
@@ -31,23 +31,23 @@ 
 	 * Gets the html "type" attribute's value
 	 * @return string
 	 */
-	function get_type(){
+	function get_type() {
 		return $this->_type;
 	}
 	/**
 	 *
 	 * @return string of html to display the field
 	 */
-	function display(){
-		$input = '<input type="'. $this->get_type() .'"';
-		$input .= ' name="' . $this->_input->html_name() . '"';
-		$input .= ' id="' . $this->_input->html_id() . '"';
-		$class = $this->_input->required() ? $this->_input->required_css_class() . ' ' . $this->_input->html_class() : $this->_input->html_class();
-		$input .= ' class="' . $class . '"';
+	function display() {
+		$input = '<input type="'.$this->get_type().'"';
+		$input .= ' name="'.$this->_input->html_name().'"';
+		$input .= ' id="'.$this->_input->html_id().'"';
+		$class = $this->_input->required() ? $this->_input->required_css_class().' '.$this->_input->html_class() : $this->_input->html_class();
+		$input .= ' class="'.$class.'"';
 		// add html5 required
 		$input .= $this->_input->required() ? ' required' : '';
-		$input .= ' value="' . $this->_input->raw_value_in_form() . '"';
-		$input .= ' style="' . $this->_input->html_style() . '"';
+		$input .= ' value="'.$this->_input->raw_value_in_form().'"';
+		$input .= ' style="'.$this->_input->html_style().'"';
 		$input .= $this->_input->html_other_attributes();
 		$input .= '/>';
 		return $input;

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

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@
 <?php
 /**
- * Abstract parent class for all form layouts. Mostly just contains a reference to the form
- * we are to lay out.
- * Form layouts should add HTML content for each form section (eg a header and footer)
- * for the form section, and dictate how to layout all the inputs and proper subsections
- * (laying out where to put the input's label, the actual input widget, and its errors; and
- * stating where the proper subsections should be placed (but usually leaving them to layout
- * their own headers and footers etc).
- */
+	 * Abstract parent class for all form layouts. Mostly just contains a reference to the form
+	 * we are to lay out.
+	 * Form layouts should add HTML content for each form section (eg a header and footer)
+	 * for the form section, and dictate how to layout all the inputs and proper subsections
+	 * (laying out where to put the input's label, the actual input widget, and its errors; and
+	 * 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{
 
 	/**