eventespresso / event-espresso-core

core/libraries/form_sections/strategies/sensitive_data_removal/EE_No_Sensitive_Data_Removal.strategy.php

Spacing +2 added lines, -2 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_No_Sensitive_Data_Removal extends EE_Sensitive_Data_Removal_Base{
+class EE_No_Sensitive_Data_Removal extends EE_Sensitive_Data_Removal_Base {
 	public function remove_sensitive_data($normalized_value) {
 		return $normalized_value;
 	}

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/sensitive_data_removal/EE_Sensitive_Data_Removal_Base.strategy.php

Spacing +2 added lines, -2 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 @@ 
  *
  * ------------------------------------------------------------------------
  */
-abstract class EE_Sensitive_Data_Removal_Base extends EE_Form_Input_Strategy_Base{
+abstract class EE_Sensitive_Data_Removal_Base extends EE_Form_Input_Strategy_Base {
 	/**
 	 * Removes all the sensitive data from this normalized value.
 	 * For example, should could 'mask' a credit card from 

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/validation/EE_Credit_Card_Validation_Strategy.strategy.php

Spacing +10 added lines, -10 removed lines

@@ -10,16 +10,16 @@ 
  * @since 				4.6
  *
  */
-class EE_Credit_Card_Validation_Strategy extends EE_Text_Validation_Strategy{
+class EE_Credit_Card_Validation_Strategy extends EE_Text_Validation_Strategy {
 
 	/**
 	 * @param null $validation_error_message
 	 */
-	public function __construct( $validation_error_message = NULL ) {
-		if( ! $validation_error_message ){
+	public function __construct($validation_error_message = NULL) {
+		if ( ! $validation_error_message) {
 			$validation_error_message = __("Please enter a valid credit card number", "event_espresso");
 		}
-		parent::__construct( $validation_error_message );
+		parent::__construct($validation_error_message);
 	}
 
 	/**
@@ -29,8 +29,8 @@ 
 	 * @return void
 	 */
 	function validate($normalized_value) {
-		if( $normalized_value && ! $this->verify_is_credit_card($normalized_value)){
-			throw new EE_Validation_Error( $this->get_validation_error_message(), 'required');
+		if ($normalized_value && ! $this->verify_is_credit_card($normalized_value)) {
+			throw new EE_Validation_Error($this->get_validation_error_message(), 'required');
 		}
 	}
 
@@ -41,8 +41,8 @@ 
 	 *
 	 * @return array
 	 */
-	function get_jquery_validation_rule_array(){
-		return array('creditcard'=>true, 'messages' => array( 'creditcard' => $this->get_validation_error_message() ) );
+	function get_jquery_validation_rule_array() {
+		return array('creditcard'=>true, 'messages' => array('creditcard' => $this->get_validation_error_message()));
 	}
 
 	/**
@@ -50,12 +50,12 @@ 
 	 * @param string $card_number
 	 * @return boolean
 	 */
-	private function verify_is_credit_card($card_number){
+	private function verify_is_credit_card($card_number) {
 		//Credit card: All major cards
 		$regex = '~^(?:4[0-9]{12}(?:[0-9]{3})?|5[1-5][0-9]{14}|6011[0-9]{12}|3(?:0[0-5]|[68][0-9])[0-9]{11}|3[47][0-9]{13})$~';
 		if (preg_match($regex, $card_number) || empty($card_number)) {
 			return true;
-		}else{
+		} else {
 			return false;
 		}
 

Braces +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 		$regex = '~^(?:4[0-9]{12}(?:[0-9]{3})?|5[1-5][0-9]{14}|6011[0-9]{12}|3(?:0[0-5]|[68][0-9])[0-9]{11}|3[47][0-9]{13})$~';
 		if (preg_match($regex, $card_number) || empty($card_number)) {
 			return true;
-		}else{
+		} else{
 			return false;
 		}
 

Doc Comments +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 class EE_Credit_Card_Validation_Strategy extends EE_Text_Validation_Strategy{
 
 	/**
-	 * @param null $validation_error_message
+	 * @param boolean|null $validation_error_message
 	 */
 	public function __construct( $validation_error_message = NULL ) {
 		if( ! $validation_error_message ){

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

Spacing +10 added lines, -10 removed lines

@@ -19,20 +19,20 @@ 
 	 */
 	function validate($normalized_value) {
 		parent::validate($normalized_value);
-		if( ! $this->_input instanceof EE_Form_Input_With_Options_Base){
+		if ( ! $this->_input instanceof EE_Form_Input_With_Options_Base) {
 			throw new EE_Error(sprintf(__("Cannot use Enum Validation Strategy with an input that doesn't have options", "event_espresso")));
 		}
 		$enum_options = $this->_input->flat_options();
-		if( $normalized_value === TRUE){
+		if ($normalized_value === TRUE) {
 			$normalized_value = 1;
-		}elseif( $normalized_value === FALSE){
+		}elseif ($normalized_value === FALSE) {
 			$normalized_value = 0;
 		}
-		if( $normalized_value !== NULL && ! array_key_exists( $normalized_value, $enum_options )){
-			throw new EE_Validation_Error( $this->get_validation_error_message(),
+		if ($normalized_value !== NULL && ! array_key_exists($normalized_value, $enum_options)) {
+			throw new EE_Validation_Error($this->get_validation_error_message(),
 				'invalid_enum_value'
 			);
-		}else{
+		} else {
 			return true;
 		}
 	}
@@ -42,15 +42,15 @@ 
 	 * on the allowed options.
 	 * @return string
 	 */
-	public function get_validation_error_message(){
+	public function get_validation_error_message() {
 		$parent_validation_error_message = parent::get_validation_error_message();
-		if( ! $parent_validation_error_message ) {
+		if ( ! $parent_validation_error_message) {
 			$enum_options = $this->_input instanceof EE_Form_Input_With_Options_Base ? $this->_input->flat_options() : '';
 			return sprintf(
 					__("This is not allowed option. Allowed options are %s.", "event_espresso"),
-					implode( ', ', $enum_options )
+					implode(', ', $enum_options)
 				);
-		}else{
+		} else {
 			return $parent_validation_error_message;
 		}
 	}

Braces +3 added lines, -3 removed lines

@@ -25,14 +25,14 @@ 
 		$enum_options = $this->_input->flat_options();
 		if( $normalized_value === TRUE){
 			$normalized_value = 1;
-		}elseif( $normalized_value === FALSE){
+		} elseif( $normalized_value === FALSE){
 			$normalized_value = 0;
 		}
 		if( $normalized_value !== NULL && ! array_key_exists( $normalized_value, $enum_options )){
 			throw new EE_Validation_Error( $this->get_validation_error_message(),
 				'invalid_enum_value'
 			);
-		}else{
+		} else{
 			return true;
 		}
 	}
@@ -50,7 +50,7 @@ 
 					__("This is not allowed option. Allowed options are %s.", "event_espresso"),
 					implode( ', ', $enum_options )
 				);
-		}else{
+		} else{
 			return $parent_validation_error_message;
 		}
 	}

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

Spacing +6 added lines, -6 removed lines

@@ -6,16 +6,16 @@ 
  * @subpackage	Expression package is undefined on line 19, column 19 in Templates/Scripting/PHPClass.php.
  * @author				Mike Nelson
  */
-class EE_Int_Validation_Strategy extends EE_Validation_Strategy_Base{
+class EE_Int_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) {
 			$validation_error_message = __("Only digits are allowed.", "event_espresso");
 		}
-		parent::__construct( $validation_error_message );
+		parent::__construct($validation_error_message);
 	}
 
 
@@ -32,8 +32,8 @@ 
 	/**
 	 * @return array
 	 */
-	function get_jquery_validation_rule_array(){
-		return array('digits'=>true, 'messages' => array( 'digits' => $this->get_validation_error_message() ) );
+	function get_jquery_validation_rule_array() {
+		return array('digits'=>true, 'messages' => array('digits' => $this->get_validation_error_message()));
 	}
 }
 

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

Spacing +9 added lines, -9 removed lines

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

Braces +1 added lines, -1 removed lines

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

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

Spacing +6 added lines, -6 removed lines

@@ -8,7 +8,7 @@ 
  * @since 				4.6
  *
  */
-abstract class EE_Validation_Strategy_Base extends EE_Form_Input_Strategy_Base{
+abstract class EE_Validation_Strategy_Base extends EE_Form_Input_Strategy_Base {
 
 
 	protected $_validation_error_message = '';
@@ -18,8 +18,8 @@ 
 	/**
 	 * @param null $validation_error_message
 	 */
-	public function __construct( $validation_error_message = NULL ) {
-		$this->_validation_error_message = $validation_error_message === NULL ? __( 'Input invalid', 'event_espresso' ) : $validation_error_message ;
+	public function __construct($validation_error_message = NULL) {
+		$this->_validation_error_message = $validation_error_message === NULL ? __('Input invalid', 'event_espresso') : $validation_error_message;
 		parent::__construct();
 	}
 
@@ -34,7 +34,7 @@ 
 	 * @param mixed $normalized_value ready for validation. May very well be NULL (which, unless
 	 * this validation strategy is the 'required' validation strategy, most should be OK with a null, empty string, etc)
 	 */
-	function validate($normalized_value){
+	function validate($normalized_value) {
 		//by default, the validation strategy does no validation. this should be implemented
 	}
 
@@ -58,7 +58,7 @@ 
 	 *
 	 * @return array
 	 */
-	function get_jquery_validation_rule_array(){
+	function get_jquery_validation_rule_array() {
 		return array();
 	}
 
@@ -67,7 +67,7 @@ 
 	 * the input is invalid. Used for both frontend and backend validation.
 	 * @return string
 	 */
-	function get_validation_error_message(){
+	function get_validation_error_message() {
 		return $this->_validation_error_message;
 	}
 }
\ No newline at end of file

core/libraries/line_item_display/EE_Line_Item_Display.class.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_Line_Item_Display
- *
- * Description
- *
- * @package         Event Espresso
- * @subpackage    core
- * @author				Brent Christensen
- * @since		 	   $VID:$
- *
- */
+  *
+  * Class EE_Line_Item_Display
+  *
+  * Description
+  *
+  * @package         Event Espresso
+  * @subpackage    core
+  * @author				Brent Christensen
+  * @since		 	   $VID:$
+  *
+  */
 class EE_Line_Item_Display {
 
 	private $strategy = NULL;

Spacing +6 added lines, -6 removed lines

@@ -18,9 +18,9 @@ 
 	/**
 	 * @param $context - where/how the line items are being displayed
 	 */
-	public function __construct( $context ) {
-		$context = strtolower( $context );
-		switch ( $context ) {
+	public function __construct($context) {
+		$context = strtolower($context);
+		switch ($context) {
 			case 'invoice' :
 				$this->strategy = new EE_Invoice_Line_Item_Display_Strategy();
 				break;
@@ -40,8 +40,8 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	public function display_line_item( EE_Line_Item $line_item, $options = array() ) {
-		return $this->strategy->display_line_item( $line_item, $options );
+	public function display_line_item(EE_Line_Item $line_item, $options = array()) {
+		return $this->strategy->display_line_item($line_item, $options);
 	}
 
 }
@@ -58,7 +58,7 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	public function display_line_item( EE_Line_Item $line_item, $options = array() );
+	public function display_line_item(EE_Line_Item $line_item, $options = array());
 
 }
 // End of file EE_Line_Item_Display.class.php

core/libraries/line_item_display/EE_SPCO_Line_Item_Display_Strategy.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_SPCO_Line_Item_Display_Strategy
- *
- * Description
- *
- * @package         Event Espresso
- * @subpackage    core
- * @author				Brent Christensen
- * @since		 	   $VID:$
- *
- */
+  *
+  * Class EE_SPCO_Line_Item_Display_Strategy
+  *
+  * Description
+  *
+  * @package         Event Espresso
+  * @subpackage    core
+  * @author				Brent Christensen
+  * @since		 	   $VID:$
+  *
+  */
 
 class EE_SPCO_Line_Item_Display_Strategy implements EEI_Line_Item_Display {
 

Doc Comments +8 added lines, -8 removed lines

@@ -19,7 +19,7 @@ 
 	/**
 	 * @param EE_Line_Item $line_item
 	 * @param array        $options
-	 * @return mixed
+	 * @return string
 	 */
 	public function display_line_item( EE_Line_Item $line_item, $options = array() ) {
 
@@ -102,7 +102,7 @@ 
 	 *
 	 * @param EE_Line_Item $line_item
 	 * @param array        $options
-	 * @return mixed
+	 * @return string
 	 */
 	private function _item_row( EE_Line_Item $line_item, $options = array() ) {
 		// start of row
@@ -143,7 +143,7 @@ 
 	 *
 	 * @param EE_Line_Item $line_item
 	 * @param array        $options
-	 * @return mixed
+	 * @return string
 	 */
 	private function _sub_item_row( EE_Line_Item $line_item, $options = array() ) {
 		// start of row
@@ -173,7 +173,7 @@ 
 	 *
 	 * @param EE_Line_Item $line_item
 	 * @param array        $options
-	 * @return mixed
+	 * @return string
 	 */
 	private function _tax_row( EE_Line_Item $line_item, $options = array() ) {
 		// start of row
@@ -200,7 +200,7 @@ 
 	 * @param EE_Line_Item $line_item
 	 * @param string       $text
 	 * @param array        $options
-	 * @return mixed
+	 * @return string
 	 */
 	private function _sub_total_row( EE_Line_Item $line_item, $text = '', $options = array() ) {
 		if ( $line_item->total() ) {
@@ -217,7 +217,7 @@ 
 	 * @param EE_Line_Item $line_item
 	 * @param string       $text
 	 * @param array        $options
-	 * @return mixed
+	 * @return string
 	 */
 	private function _total_row( EE_Line_Item $line_item, $text = '', $options = array() ) {
 		$html = '';
@@ -240,7 +240,7 @@ 
 	 * 	_payments_and_amount_owing_rows
 	 *
 	 * @param EE_Line_Item $line_item
-	 * @return mixed
+	 * @return string
 	 */
 	private function _payments_and_amount_owing_rows( EE_Line_Item $line_item ) {
 		$html = '';
@@ -288,7 +288,7 @@ 
 	 * 	_separator_row
 	 *
 	 * @param array        $options
-	 * @return mixed
+	 * @return string
 	 */
 	private function _separator_row( $options = array() ) {
 		// start of row

Spacing +70 added lines, -70 removed lines

@@ -21,73 +21,73 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	public function display_line_item( EE_Line_Item $line_item, $options = array() ) {
+	public function display_line_item(EE_Line_Item $line_item, $options = array()) {
 
-		EE_Registry::instance()->load_helper( 'Template' );
-		EE_Registry::instance()->load_helper( 'HTML' );
+		EE_Registry::instance()->load_helper('Template');
+		EE_Registry::instance()->load_helper('HTML');
 
 		$html = '';
 		// set some default options and merge with incoming
 		$default_options = array(
-			'show_desc' => TRUE,  // 	TRUE 		FALSE
+			'show_desc' => TRUE, // 	TRUE 		FALSE
 			'odd' => FALSE
 		);
-		$options = array_merge( $default_options, (array)$options );
+		$options = array_merge($default_options, (array) $options);
 
-		switch( $line_item->type() ) {
+		switch ($line_item->type()) {
 
 			case EEM_Line_Item::type_line_item:
 				// item row
-				$html .= $this->_item_row( $line_item, $options );
+				$html .= $this->_item_row($line_item, $options);
 				// got any kids?
-				foreach( $line_item->children() as $child_line_item ) {
-					$this->display_line_item( $child_line_item, $options );
+				foreach ($line_item->children() as $child_line_item) {
+					$this->display_line_item($child_line_item, $options);
 				}
 				break;
 
 			case EEM_Line_Item::type_sub_line_item:
-				$html .= $this->_sub_item_row( $line_item, $options );
+				$html .= $this->_sub_item_row($line_item, $options);
 				break;
 
 			case EEM_Line_Item::type_sub_total:
 				static $sub_total = 0;
 				$child_line_items = $line_item->children();
 				// loop thru children
-				foreach( $child_line_items as $child_line_item ) {
+				foreach ($child_line_items as $child_line_item) {
 					// recursively feed children back into this method
-					$html .= $this->display_line_item( $child_line_item, $options );
+					$html .= $this->display_line_item($child_line_item, $options);
 				}
-				if ( $line_item->total() != $sub_total && count( $child_line_items ) > 1 ) {
-					$html .= $this->_sub_total_row( $line_item, __('Sub-Total', 'event_espresso'), $options );
+				if ($line_item->total() != $sub_total && count($child_line_items) > 1) {
+					$html .= $this->_sub_total_row($line_item, __('Sub-Total', 'event_espresso'), $options);
 					$sub_total = $line_item->total();
 				}
 				break;
 
 			case EEM_Line_Item::type_tax:
-				if ( $this->_show_taxes ) {
-					$html .= $this->_tax_row( $line_item, $options );
+				if ($this->_show_taxes) {
+					$html .= $this->_tax_row($line_item, $options);
 				}
 				break;
 
 			case EEM_Line_Item::type_tax_sub_total:
-				if ( $this->_show_taxes ) {
+				if ($this->_show_taxes) {
 					// loop thru children
-					foreach( $line_item->children() as $child_line_item ) {
+					foreach ($line_item->children() as $child_line_item) {
 						// recursively feed children back into this method
-						$html .= $this->display_line_item( $child_line_item, $options );
+						$html .= $this->display_line_item($child_line_item, $options);
 					}
-					$html .= $this->_total_row( $line_item, __('Tax Total', 'event_espresso'), $options );
+					$html .= $this->_total_row($line_item, __('Tax Total', 'event_espresso'), $options);
 				}
 				break;
 
 			case EEM_Line_Item::type_total:
 				// loop thru children
-				foreach( $line_item->children() as $child_line_item ) {
+				foreach ($line_item->children() as $child_line_item) {
 					// recursively feed children back into this method
-					$html .= $this->display_line_item( $child_line_item, $options );
+					$html .= $this->display_line_item($child_line_item, $options);
 				}
-				$html .= $this->_total_row( $line_item, __('Total', 'event_espresso'), $options );
-				$html .= $this->_payments_and_amount_owing_rows( $line_item );
+				$html .= $this->_total_row($line_item, __('Total', 'event_espresso'), $options);
+				$html .= $this->_payments_and_amount_owing_rows($line_item);
 				break;
 
 		}
@@ -104,10 +104,10 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	private function _item_row( EE_Line_Item $line_item, $options = array() ) {
+	private function _item_row(EE_Line_Item $line_item, $options = array()) {
 		// start of row
 		$row_class = $options['odd'] ? 'item odd' : 'item';
-		$html = EEH_HTML::tr( '', '', $row_class );
+		$html = EEH_HTML::tr('', '', $row_class);
 		// name && desc
 		$name_and_desc = apply_filters(
 			'FHEE__EE_SPCO_Line_Item_Display_Strategy__item_row__name',
@@ -116,21 +116,21 @@ 
 		);
 		$name_and_desc .= apply_filters(
 			'FHEE__EE_SPCO_Line_Item_Display_Strategy__item_row__desc',
-			( $options['show_desc'] ? ' : ' . $line_item->desc() : '' ),
+			($options['show_desc'] ? ' : '.$line_item->desc() : ''),
 			$line_item,
 			$options
 		);
 		// name td
-		$html .= EEH_HTML::td( $name_and_desc, '',  'item_l' );
+		$html .= EEH_HTML::td($name_and_desc, '', 'item_l');
 		// quantity td
-		$html .= EEH_HTML::td( $line_item->quantity(), '',  'item_l jst-rght' );
+		$html .= EEH_HTML::td($line_item->quantity(), '', 'item_l jst-rght');
 		// price td
-		$html .= EEH_HTML::td( $line_item->unit_price_no_code(), '',  'item_c jst-rght' );
+		$html .= EEH_HTML::td($line_item->unit_price_no_code(), '', 'item_c jst-rght');
 		// total td
-		$total = EEH_Template::format_currency( $line_item->total(), false, false );
+		$total = EEH_Template::format_currency($line_item->total(), false, false);
 		$total .= $line_item->is_taxable() ? '*' : '';
 		$this->_show_taxes = $line_item->is_taxable() ? TRUE : $this->_show_taxes;
-		$html .= EEH_HTML::td( $total, '',  'item_r jst-rght' );
+		$html .= EEH_HTML::td($total, '', 'item_r jst-rght');
 		// end of row
 		$html .= EEH_HTML::trx();
 		return $html;
@@ -145,22 +145,22 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	private function _sub_item_row( EE_Line_Item $line_item, $options = array() ) {
+	private function _sub_item_row(EE_Line_Item $line_item, $options = array()) {
 		// start of row
-		$html = EEH_HTML::tr( '', 'item sub-item-row' );
+		$html = EEH_HTML::tr('', 'item sub-item-row');
 		// name && desc
 		$name_and_desc = $line_item->name();
-		$name_and_desc .= $options['show_desc'] ? ' : ' . $line_item->desc() : '';
+		$name_and_desc .= $options['show_desc'] ? ' : '.$line_item->desc() : '';
 		// name td
-		$html .= EEH_HTML::td( $name_and_desc, '',  'item_l sub-item' );
+		$html .= EEH_HTML::td($name_and_desc, '', 'item_l sub-item');
 		// discount/surcharge td
-		if ( $line_item->is_percent() ) {
-			$html .= EEH_HTML::td( $line_item->percent() . '%', '',  'item_c' );
+		if ($line_item->is_percent()) {
+			$html .= EEH_HTML::td($line_item->percent().'%', '', 'item_c');
 		} else {
-			$html .= EEH_HTML::td( $line_item->unit_price_no_code(), '',  'item_c jst-rght' );
+			$html .= EEH_HTML::td($line_item->unit_price_no_code(), '', 'item_c jst-rght');
 		}
 		// total td
-		$html .= EEH_HTML::td( EEH_Template::format_currency( $line_item->total(), false, false ), '',  'item_r jst-rght' );
+		$html .= EEH_HTML::td(EEH_Template::format_currency($line_item->total(), false, false), '', 'item_r jst-rght');
 		// end of row
 		$html .= EEH_HTML::trx();
 		return $html;
@@ -175,18 +175,18 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	private function _tax_row( EE_Line_Item $line_item, $options = array() ) {
+	private function _tax_row(EE_Line_Item $line_item, $options = array()) {
 		// start of row
-		$html = EEH_HTML::tr( '', 'item sub-item tax-total' );
+		$html = EEH_HTML::tr('', 'item sub-item tax-total');
 		// name && desc
 		$name_and_desc = $line_item->name();
-		$name_and_desc .= $options['show_desc'] ? ' : ' . $line_item->desc() : '';
+		$name_and_desc .= $options['show_desc'] ? ' : '.$line_item->desc() : '';
 		// name td
-		$html .= EEH_HTML::td( $name_and_desc, '',  'item_l sub-item', '', ' colspan="2"' );
+		$html .= EEH_HTML::td($name_and_desc, '', 'item_l sub-item', '', ' colspan="2"');
 		// percent td
-		$html .= EEH_HTML::td( $line_item->percent() . '%', '',  'item_c', '' );
+		$html .= EEH_HTML::td($line_item->percent().'%', '', 'item_c', '');
 		// total td
-		$html .= EEH_HTML::td( EEH_Template::format_currency( $line_item->total(), false, false ), '',  'item_r jst-rght' );
+		$html .= EEH_HTML::td(EEH_Template::format_currency($line_item->total(), false, false), '', 'item_r jst-rght');
 		// end of row
 		$html .= EEH_HTML::trx();
 		return $html;
@@ -202,9 +202,9 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	private function _sub_total_row( EE_Line_Item $line_item, $text = '', $options = array() ) {
-		if ( $line_item->total() ) {
-			return $this->_total_row( $line_item, $text, $options);
+	private function _sub_total_row(EE_Line_Item $line_item, $text = '', $options = array()) {
+		if ($line_item->total()) {
+			return $this->_total_row($line_item, $text, $options);
 		}
 		return '';
 	}
@@ -219,15 +219,15 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	private function _total_row( EE_Line_Item $line_item, $text = '', $options = array() ) {
+	private function _total_row(EE_Line_Item $line_item, $text = '', $options = array()) {
 		$html = '';
-		if ( $line_item->total() ) {
+		if ($line_item->total()) {
 			// start of row
-			$html = EEH_HTML::tr( '', '', 'total_tr odd' );
+			$html = EEH_HTML::tr('', '', 'total_tr odd');
 			// total td
-			$html .= EEH_HTML::td( $text, '',  'total_currency total jst-rght',  '',  ' colspan="3"' );
+			$html .= EEH_HTML::td($text, '', 'total_currency total jst-rght', '', ' colspan="3"');
 			// total td
-			$html .= EEH_HTML::td( EEH_Template::format_currency( $line_item->total(), false, false ), '',  'total jst-rght' );
+			$html .= EEH_HTML::td(EEH_Template::format_currency($line_item->total(), false, false), '', 'total jst-rght');
 			// end of row
 			$html .= EEH_HTML::trx();
 		}
@@ -242,38 +242,38 @@ 
 	 * @param EE_Line_Item $line_item
 	 * @return mixed
 	 */
-	private function _payments_and_amount_owing_rows( EE_Line_Item $line_item ) {
+	private function _payments_and_amount_owing_rows(EE_Line_Item $line_item) {
 		$html = '';
-		$transaction = EEM_Transaction::instance()->get_one_by_ID( $line_item->TXN_ID() );
-		if ( $transaction instanceof EE_Transaction ) {
+		$transaction = EEM_Transaction::instance()->get_one_by_ID($line_item->TXN_ID());
+		if ($transaction instanceof EE_Transaction) {
 			$payments = $transaction->approved_payments();
-			if ( ! empty( $payments )) {
+			if ( ! empty($payments)) {
 				$owing = $line_item->total();
-				foreach ( $payments as $payment ) {
-					if ( $payment instanceof EE_Payment ) {
+				foreach ($payments as $payment) {
+					if ($payment instanceof EE_Payment) {
 						$owing = $owing - $payment->amount();
 						$payment_desc = sprintf(
 							__('Payment%1$s Received: %2$s', 'event_espresso'),
-							$payment->txn_id_chq_nmbr() != '' ? ' ' . $payment->txn_id_chq_nmbr() : '',
+							$payment->txn_id_chq_nmbr() != '' ? ' '.$payment->txn_id_chq_nmbr() : '',
 							$payment->timestamp()
 						);
 						// start of row
-						$html .= EEH_HTML::tr( '', '', 'total_tr odd' );
+						$html .= EEH_HTML::tr('', '', 'total_tr odd');
 						// payment desc
-						$html .= EEH_HTML::td( $payment_desc, '',  '',  '',  ' colspan="3"' );
+						$html .= EEH_HTML::td($payment_desc, '', '', '', ' colspan="3"');
 						// total td
-						$html .= EEH_HTML::td( EEH_Template::format_currency( $payment->amount(), false, false ), '',  'total jst-rght' );
+						$html .= EEH_HTML::td(EEH_Template::format_currency($payment->amount(), false, false), '', 'total jst-rght');
 						// end of row
 						$html .= EEH_HTML::trx();
 					}
 				}
-				if ( $line_item->total() ) {
+				if ($line_item->total()) {
 					// start of row
-					$html .= EEH_HTML::tr( '', '', 'total_tr odd' );
+					$html .= EEH_HTML::tr('', '', 'total_tr odd');
 					// total td
-					$html .= EEH_HTML::td( __('Amount Owing', 'event_espresso'), '',  'total_currency total jst-rght',  '',  ' colspan="3"' );
+					$html .= EEH_HTML::td(__('Amount Owing', 'event_espresso'), '', 'total_currency total jst-rght', '', ' colspan="3"');
 					// total td
-					$html .= EEH_HTML::td( EEH_Template::format_currency( $owing, false, false ), '',  'total jst-rght' );
+					$html .= EEH_HTML::td(EEH_Template::format_currency($owing, false, false), '', 'total jst-rght');
 					// end of row
 					$html .= EEH_HTML::trx();
 				}
@@ -290,9 +290,9 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	private function _separator_row( $options = array() ) {
+	private function _separator_row($options = array()) {
 		// start of row
-		$html = EEH_HTML::tr( EEH_HTML::td( '<hr>', '',  '',  '',  ' colspan="4"' ));
+		$html = EEH_HTML::tr(EEH_HTML::td('<hr>', '', '', '', ' colspan="4"'));
 		return $html;
 	}