eventespresso / event-espresso-core

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

Spacing +12 added lines, -12 removed lines

@@ -7,21 +7,21 @@ 
  * @subpackage	Expression package is undefined on line 19, column 19 in Templates/Scripting/PHPClass.php.
  * @author				Mike Nelson
  */
-class EE_Text_Validation_Strategy extends EE_Validation_Strategy_Base{
+class EE_Text_Validation_Strategy extends EE_Validation_Strategy_Base {
 
 	protected $_regex = null;
-	public function __construct( $validation_error_message = NULL, $regex = null ) {
+	public function __construct($validation_error_message = NULL, $regex = null) {
 		$this->_regex = $regex;
-		parent::__construct( $validation_error_message );
+		parent::__construct($validation_error_message);
 	}
 
 	/**
 	 * @param $normalized_value
 	 */
 	public function validate($normalized_value) {
-		if( $this->_regex && $normalized_value) {
-			if( ! preg_match( $this->_regex, $normalized_value ) ) {
-				throw new EE_Validation_Error( $this->get_validation_error_message(), 'regex' );
+		if ($this->_regex && $normalized_value) {
+			if ( ! preg_match($this->_regex, $normalized_value)) {
+				throw new EE_Validation_Error($this->get_validation_error_message(), 'regex');
 			}
 		}
 	}
@@ -29,10 +29,10 @@ 
 	/**
 	 * @return array
 	 */
-	function get_jquery_validation_rule_array(){
-		if( $this->_regex !== null ){
-			return array( 'regex'=> $this->regex_js(), 'messages' => array( 'regex' => $this->get_validation_error_message() ) );
-		}else{
+	function get_jquery_validation_rule_array() {
+		if ($this->_regex !== null) {
+			return array('regex'=> $this->regex_js(), 'messages' => array('regex' => $this->get_validation_error_message()));
+		} else {
 			return array();
 		}
 
@@ -46,8 +46,8 @@ 
 	function regex_js() {
 		//first character must be the delimiter
 		$delimeter = $this->_regex[0];
-		$last_occurence_of_delimieter = strrpos($this->_regex, $delimeter );
-		return substr( $this->_regex, 1, $last_occurence_of_delimieter - 1 );
+		$last_occurence_of_delimieter = strrpos($this->_regex, $delimeter);
+		return substr($this->_regex, 1, $last_occurence_of_delimieter - 1);
 	}
 }
 

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/line_item_display/EE_Admin_Table_Registration_Line_Item_Display_Strategy.strategy.php

Indentation +7 added lines, -7 removed lines

@@ -154,13 +154,13 @@
 			$total_label = sprintf( __( 'This registration\'s approximate total %s', 'event_espresso' ), '(' . EE_Registry::instance()->CFG->currency->code . ')' );
 			$total_label .= '<br>';
 			$total_label .= '<p class="ee-footnote-text">'
-			                . sprintf(
-				                __( 'The registrations\' share of the transaction total is approximate because it might not be possible to evenly divide the transaction total among each registration, and so some registrations may need to pay a penny more than others.  This registration\'s final share is actually %1$s%2$s%3$s.', 'event_espresso' ),
-				                '<strong>',
-				                $registration_total,
-				                '</strong>'
-			                )
-			                . '</p>';
+							. sprintf(
+								__( 'The registrations\' share of the transaction total is approximate because it might not be possible to evenly divide the transaction total among each registration, and so some registrations may need to pay a penny more than others.  This registration\'s final share is actually %1$s%2$s%3$s.', 'event_espresso' ),
+								'<strong>',
+								$registration_total,
+								'</strong>'
+							)
+							. '</p>';
 		}
 		$html .= EEH_HTML::th( $total_label, '',  'jst-rght',  '',  ' colspan="3"' );
 		// total th

Spacing +42 added lines, -42 removed lines

@@ -10,7 +10,7 @@ 
  *
  */
 
-class EE_Admin_Table_Registration_Line_Item_Display_Strategy extends EE_Admin_Table_Line_Item_Display_Strategy  {
+class EE_Admin_Table_Registration_Line_Item_Display_Strategy extends EE_Admin_Table_Line_Item_Display_Strategy {
 
 	/**
 	 * Table header for display.
@@ -18,14 +18,14 @@ 
 	 * @param array $options
 	 * @return string
 	 */
-	protected function _table_header( $options ) {
-		$html = EEH_HTML::table( '','', $options['table_css_class'] );
+	protected function _table_header($options) {
+		$html = EEH_HTML::table('', '', $options['table_css_class']);
 		$html .= EEH_HTML::thead();
 		$html .= EEH_HTML::tr();
-		$html .= EEH_HTML::th( __( 'Name', 'event_espresso' ), '', 'jst-left' );
-		$html .= EEH_HTML::th( __( 'Type', 'event_espresso'), '', 'jst-left' );
-		$html .= EEH_HTML::th( __( 'Date(s)', 'event_espresso' ), '', 'jst-left' );
-		$html .= EEH_HTML::th( __( 'Amount', 'event_espresso' ), '', 'jst-cntr' );
+		$html .= EEH_HTML::th(__('Name', 'event_espresso'), '', 'jst-left');
+		$html .= EEH_HTML::th(__('Type', 'event_espresso'), '', 'jst-left');
+		$html .= EEH_HTML::th(__('Date(s)', 'event_espresso'), '', 'jst-left');
+		$html .= EEH_HTML::th(__('Amount', 'event_espresso'), '', 'jst-cntr');
 		$html .= EEH_HTML::tbody();
 		return $html;
 	}
@@ -41,12 +41,12 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	protected function _item_row( EE_Line_Item $line_item, $options = array() ) {
+	protected function _item_row(EE_Line_Item $line_item, $options = array()) {
 		$line_item_related_object = $line_item->get_object();
 		$parent_line_item_related_object = $line_item->parent() instanceof EE_Line_Item ? $line_item->parent()->get_object() : null;
 		// start of row
 		$row_class = $options['odd'] ? 'item odd' : 'item';
-		$html = EEH_HTML::tr( '', '', $row_class );
+		$html = EEH_HTML::tr('', '', $row_class);
 
 
 		//Name Column
@@ -54,48 +54,48 @@ 
 
 		//related object scope.
 		$parent_related_object_name = $parent_line_item_related_object instanceof EEI_Line_Item_Object ? $parent_line_item_related_object->name() : '';
-		$parent_related_object_name = empty( $parent_related_object_name ) && $line_item->parent() instanceof EE_Line_Item ? $line_item->parent()->name() : $parent_related_object_name;
+		$parent_related_object_name = empty($parent_related_object_name) && $line_item->parent() instanceof EE_Line_Item ? $line_item->parent()->name() : $parent_related_object_name;
 		$parent_related_object_link = $parent_line_item_related_object instanceof EEI_Admin_Links ? $parent_line_item_related_object->get_admin_details_link() : '';
 
 
 		$name_html = $line_item_related_object instanceof EEI_Line_Item_Object ? $line_item_related_object->name() : $line_item->name();
-		$name_html = $name_link ? '<a href="' . $name_link . '">' . $name_html . '</a>' : $name_html;
+		$name_html = $name_link ? '<a href="'.$name_link.'">'.$name_html.'</a>' : $name_html;
 		$name_html .= $line_item->is_taxable() ? ' *' : '';
 		//maybe preface with icon?
-		$name_html = $line_item_related_object instanceof EEI_Has_Icon ? $line_item_related_object->get_icon() . $name_html : $name_html;
-		$name_html = '<span class="ee-line-item-name linked">' . $name_html . '</span><br>';
-		$name_html .=  sprintf(
-			_x( '%1$sfor the %2$s: %3$s%4$s', 'eg. "for the Event: My Cool Event"', 'event_espresso'),
+		$name_html = $line_item_related_object instanceof EEI_Has_Icon ? $line_item_related_object->get_icon().$name_html : $name_html;
+		$name_html = '<span class="ee-line-item-name linked">'.$name_html.'</span><br>';
+		$name_html .= sprintf(
+			_x('%1$sfor the %2$s: %3$s%4$s', 'eg. "for the Event: My Cool Event"', 'event_espresso'),
 			'<span class="ee-line-item-related-parent-object">',
-			$line_item->parent() instanceof EE_Line_Item ? $line_item->parent()->OBJ_type() : __( 'Item:', 'event_espresso' ),
-			$parent_related_object_link ? '<a href="' . $parent_related_object_link . '">' . $parent_related_object_name . '</a>' : $parent_related_object_name,
+			$line_item->parent() instanceof EE_Line_Item ? $line_item->parent()->OBJ_type() : __('Item:', 'event_espresso'),
+			$parent_related_object_link ? '<a href="'.$parent_related_object_link.'">'.$parent_related_object_name.'</a>' : $parent_related_object_name,
 			'</span>'
 		);
-		$html .= EEH_HTML::td( $name_html, '', 'jst-left' );
+		$html .= EEH_HTML::td($name_html, '', 'jst-left');
 
 		//Type Column
-		$type_html = $line_item->OBJ_type() ? $line_item->OBJ_type() . '<br />' : '';
+		$type_html = $line_item->OBJ_type() ? $line_item->OBJ_type().'<br />' : '';
 		$code = $line_item_related_object instanceof EEI_Has_Code ? $line_item_related_object->code() : '';
-		$type_html .= ! empty( $code ) ? '<span class="ee-line-item-id">' . sprintf( __( 'Code: %s', 'event_espresso' ), $code ) . '</span>' : '';
-		$html .= EEH_HTML::td( $type_html, '', 'jst-left' );
+		$type_html .= ! empty($code) ? '<span class="ee-line-item-id">'.sprintf(__('Code: %s', 'event_espresso'), $code).'</span>' : '';
+		$html .= EEH_HTML::td($type_html, '', 'jst-left');
 
 		//Date column
 		$datetime_content = '';
-		if ( $line_item_related_object instanceof EE_Ticket ) {
+		if ($line_item_related_object instanceof EE_Ticket) {
 			$datetimes = $line_item_related_object->datetimes();
-			foreach ( $datetimes as $datetime ) {
-				if ( $datetime instanceof EE_Datetime ) {
-					$datetime_content .= $datetime->get_dtt_display_name() . '<br>';
+			foreach ($datetimes as $datetime) {
+				if ($datetime instanceof EE_Datetime) {
+					$datetime_content .= $datetime->get_dtt_display_name().'<br>';
 				}
 			}
 		}
-		$html .= EEH_HTML::td( $datetime_content, '', 'jst-left' );
+		$html .= EEH_HTML::td($datetime_content, '', 'jst-left');
 
 		//Amount Column
-		if ( $line_item->is_percent() ) {
-			$html .= EEH_HTML::td( $line_item->percent() . '%', '', 'jst-rght' );
+		if ($line_item->is_percent()) {
+			$html .= EEH_HTML::td($line_item->percent().'%', '', 'jst-rght');
 		} else {
-			$html .= EEH_HTML::td( $line_item->unit_price_no_code(), '', 'jst-rght' );
+			$html .= EEH_HTML::td($line_item->unit_price_no_code(), '', 'jst-rght');
 		}
 
 
@@ -113,13 +113,13 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	protected function _tax_row( EE_Line_Item $line_item, $options = array() ) {
+	protected function _tax_row(EE_Line_Item $line_item, $options = array()) {
 		// start of row
-		$html = EEH_HTML::tr( '', 'admin-primary-mbox-taxes-tr' );
+		$html = EEH_HTML::tr('', 'admin-primary-mbox-taxes-tr');
 		// name th
-		$html .= EEH_HTML::th(  $line_item->name() . '(' . $line_item->get_pretty( 'LIN_percent' ) . '%)', '',  'jst-rght', '', ' colspan="3"' );
+		$html .= EEH_HTML::th($line_item->name().'('.$line_item->get_pretty('LIN_percent').'%)', '', 'jst-rght', '', ' colspan="3"');
 		// total th
-		$html .= EEH_HTML::th( EEH_Template::format_currency( $line_item->total(), false, false ), '', 'jst-rght' );
+		$html .= EEH_HTML::th(EEH_Template::format_currency($line_item->total(), false, false), '', 'jst-rght');
 		// end of row
 		$html .= EEH_HTML::trx();
 		return $html;
@@ -136,9 +136,9 @@ 
 	 * @param array        $options
 	 * @return mixed
 	 */
-	protected function _total_row( EE_Line_Item $line_item, $options = array() ) {
+	protected function _total_row(EE_Line_Item $line_item, $options = array()) {
 
-		$registration = isset( $options['EE_Registration'] ) ? $options['EE_Registration'] : null;
+		$registration = isset($options['EE_Registration']) ? $options['EE_Registration'] : null;
 		$registration_total = $registration instanceof EE_Registration ? $registration->pretty_final_price() : 0;
 		//if no valid registration object then we're not going to show the approximate text.
 		$total_match = $registration instanceof EE_Registration ? $registration->final_price() == $line_item->total() : true;
@@ -146,26 +146,26 @@ 
 
 
 		// start of row
-		$html = EEH_HTML::tr( '', '', 'admin-primary-mbox-total-tr' );
+		$html = EEH_HTML::tr('', '', 'admin-primary-mbox-total-tr');
 		// Total th label
-		if ( $total_match ) {
-			$total_label = sprintf( __( 'This registration\'s total %s:', 'event_espresso' ), '(' . EE_Registry::instance()->CFG->currency->code . ')' );
+		if ($total_match) {
+			$total_label = sprintf(__('This registration\'s total %s:', 'event_espresso'), '('.EE_Registry::instance()->CFG->currency->code.')');
 		} else {
-			$total_label = sprintf( __( 'This registration\'s approximate total %s', 'event_espresso' ), '(' . EE_Registry::instance()->CFG->currency->code . ')' );
+			$total_label = sprintf(__('This registration\'s approximate total %s', 'event_espresso'), '('.EE_Registry::instance()->CFG->currency->code.')');
 			$total_label .= '<br>';
 			$total_label .= '<p class="ee-footnote-text">'
 			                . sprintf(
-				                __( 'The registrations\' share of the transaction total is approximate because it might not be possible to evenly divide the transaction total among each registration, and so some registrations may need to pay a penny more than others.  This registration\'s final share is actually %1$s%2$s%3$s.', 'event_espresso' ),
+				                __('The registrations\' share of the transaction total is approximate because it might not be possible to evenly divide the transaction total among each registration, and so some registrations may need to pay a penny more than others.  This registration\'s final share is actually %1$s%2$s%3$s.', 'event_espresso'),
 				                '<strong>',
 				                $registration_total,
 				                '</strong>'
 			                )
 			                . '</p>';
 		}
-		$html .= EEH_HTML::th( $total_label, '',  'jst-rght',  '',  ' colspan="3"' );
+		$html .= EEH_HTML::th($total_label, '', 'jst-rght', '', ' colspan="3"');
 		// total th
 
-		$html .= EEH_HTML::th( EEH_Template::format_currency( $line_item->total(), false, false ), '',  'jst-rght' );
+		$html .= EEH_HTML::th(EEH_Template::format_currency($line_item->total(), false, false), '', 'jst-rght');
 		// end of row
 		$html .= EEH_HTML::trx();
 		return $html;

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 +7 added lines, -7 removed lines

@@ -21,9 +21,9 @@ 
 	 * @param string $context - where/how the line items are being displayed
 	 * @param string $other_line_item_display_strategy
 	 */
-	public function __construct( $context = '', $other_line_item_display_strategy = '' ) {
-		$context = strtolower( $context );
-		switch ( $context ) {
+	public function __construct($context = '', $other_line_item_display_strategy = '') {
+		$context = strtolower($context);
+		switch ($context) {
 			case 'invoice' :
 				$this->strategy = new EE_Invoice_Line_Item_Display_Strategy();
 				break;
@@ -35,8 +35,8 @@ 
 				break;
 			default :
 				if (
-					! empty( $other_line_item_display_strategy ) &&
-					class_exists( $other_line_item_display_strategy )
+					! empty($other_line_item_display_strategy) &&
+					class_exists($other_line_item_display_strategy)
 				) {
 					$this->strategy = new  $other_line_item_display_strategy();
 				} else {
@@ -50,8 +50,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);
 	}
 
 

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,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() ) {
 

core/libraries/line_item_filters/EE_Billable_Line_Item_Filter.class.php

Spacing +11 added lines, -12 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 /**
  * Class EE_Billable_Line_Item_Filter
@@ -25,8 +25,8 @@ 
 	 * EE_Billable_Line_Item_Filter constructor.
 	 * @param EE_Registration[] $registrations
 	 */
-	public function __construct( $registrations ) {
-		parent::__construct( $this->_remove_unbillable_registrations( $registrations ) );
+	public function __construct($registrations) {
+		parent::__construct($this->_remove_unbillable_registrations($registrations));
 	}
 
 
@@ -39,25 +39,24 @@ 
 	 * @param EE_Registration[] $registrations
 	 * @return mixed
 	 */
-	protected function _remove_unbillable_registrations( $registrations = array() ) {
-		if ( ! empty( $registrations ) ) {
+	protected function _remove_unbillable_registrations($registrations = array()) {
+		if ( ! empty($registrations)) {
 			// these reg statuses require payment (if event is not free)
 			$requires_payment = EEM_Registration::reg_statuses_that_allow_payment();
-			foreach ( $registrations as $key => $registration ) {
-				if ( ! $registration instanceof EE_Registration ) {
+			foreach ($registrations as $key => $registration) {
+				if ( ! $registration instanceof EE_Registration) {
 					continue;
 				}
 				// are we billing for this registration at this moment ?
-				if ( !
-					$registration->owes_monies_and_can_pay( $requires_payment ) &&
+				if ( ! $registration->owes_monies_and_can_pay($requires_payment) &&
 					! (
 						// free registration with valid reg status
 						$registration->final_price() == 0 &&
-						in_array( $registration->status_ID(), $requires_payment )
+						in_array($registration->status_ID(), $requires_payment)
 					)
 				) {
 					// not billable. remove it
-					unset( $registrations[ $key ] );
+					unset($registrations[$key]);
 				}
 			}
 		}

core/libraries/line_item_filters/EE_Line_Item_Filter_Collection.class.php

Spacing +2 added lines, -2 removed lines

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

core/libraries/line_item_filters/EE_Line_Item_Filter_Processor.class.php

Spacing +18 added lines, -18 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 /**
  * Class EE_Line_Item_Filter_Processor
@@ -55,12 +55,12 @@ 
 	 * @param \EEI_Line_Item                  $grand_total_line_item
 	 * @throws \EE_Error
 	 */
-	public function __construct( EE_Line_Item_Filter_Collection $line_item_filters, EEI_Line_Item $grand_total_line_item ) {
+	public function __construct(EE_Line_Item_Filter_Collection $line_item_filters, EEI_Line_Item $grand_total_line_item) {
 		$this->line_item_filters = $line_item_filters;
-		if ( $grand_total_line_item->type() !== EEM_Line_Item::type_total ) {
-			throw new EE_Error( __( 'A Line Item of the type total is required', 'event_espresso' ) );
+		if ($grand_total_line_item->type() !== EEM_Line_Item::type_total) {
+			throw new EE_Error(__('A Line Item of the type total is required', 'event_espresso'));
 		}
-		$this->grand_total_line_item = $this->clone_and_reset_line_item_tree( $grand_total_line_item );
+		$this->grand_total_line_item = $this->clone_and_reset_line_item_tree($grand_total_line_item);
 	}
 
 
@@ -71,10 +71,10 @@ 
 	 * @param \EEI_Line_Item $line_item
 	 * @return \EEI_Line_Item
 	 */
-	protected function clone_and_reset_line_item_tree( EEI_Line_Item $line_item ) {
-		$cloned_line_item = $this->clone_and_reset_line_item( $line_item );
-		foreach ( $line_item->children() as $child_line_item ) {
-			$cloned_line_item->add_child_line_item( $this->clone_and_reset_line_item_tree( $child_line_item ) );
+	protected function clone_and_reset_line_item_tree(EEI_Line_Item $line_item) {
+		$cloned_line_item = $this->clone_and_reset_line_item($line_item);
+		foreach ($line_item->children() as $child_line_item) {
+			$cloned_line_item->add_child_line_item($this->clone_and_reset_line_item_tree($child_line_item));
 		}
 		return $cloned_line_item;
 	}
@@ -91,16 +91,16 @@ 
 	 * @param \EEI_Line_Item $line_item
 	 * @return \EEI_Line_Item
 	 */
-	protected function clone_and_reset_line_item( EEI_Line_Item $line_item ) {
+	protected function clone_and_reset_line_item(EEI_Line_Item $line_item) {
 		// we don't actually want to work with the original line item, so clone it
 		$cloned_line_item = clone $line_item;
-		$cloned_line_item->set( 'LIN_ID', null );
-		$cloned_line_item->set( 'LIN_parent', null );
+		$cloned_line_item->set('LIN_ID', null);
+		$cloned_line_item->set('LIN_parent', null);
 		$cloned_line_item->clear_related_line_item_cache();
-		foreach( array_keys( EEM_Line_Item::instance()->relation_settings() ) as $relation_name) {
-			$cloned_line_item->clear_cache( $relation_name, null, true );
+		foreach (array_keys(EEM_Line_Item::instance()->relation_settings()) as $relation_name) {
+			$cloned_line_item->clear_cache($relation_name, null, true);
 		}
-		$cloned_line_item->set_allow_persist( false );
+		$cloned_line_item->set_allow_persist(false);
 		return $cloned_line_item;
 	}
 
@@ -113,8 +113,8 @@ 
 	 */
 	public function process() {
 		$this->line_item_filters->rewind();
-		while ( $this->line_item_filters->valid() ) {
-			$this->grand_total_line_item = $this->line_item_filters->current()->process( $this->grand_total_line_item );
+		while ($this->line_item_filters->valid()) {
+			$this->grand_total_line_item = $this->line_item_filters->current()->process($this->grand_total_line_item);
 			$this->line_item_filters->next();
 		}
 		$this->grand_total_line_item->recalculate_total_including_taxes();

core/libraries/line_item_filters/EE_Non_Zero_Line_Item_Filter.class.php

Spacing +18 added lines, -18 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 /**
  * Class EE_Non_Zero_Line_Item_Filter
@@ -31,36 +31,36 @@ 
 	 * @param EEI_Line_Item $line_item
 	 * @return \EEI_Line_Item
 	 */
-	public function process( EEI_Line_Item $line_item ) {
-		$non_zero_line_item = $this->_filter_zero_line_item( $line_item );
-		if ( ! $non_zero_line_item instanceof EEI_Line_Item ) {
+	public function process(EEI_Line_Item $line_item) {
+		$non_zero_line_item = $this->_filter_zero_line_item($line_item);
+		if ( ! $non_zero_line_item instanceof EEI_Line_Item) {
 			return null;
 		}
 		//if this is an event subtotal, we want to only include it if it
 		//has a non-zero total and at least one ticket line item child
-		if( $line_item->children() ) {
+		if ($line_item->children()) {
 			$ticket_or_subtotals_with_tkt_children_count = 0;
-			foreach ( $line_item->children() as $child_line_item ) {
+			foreach ($line_item->children() as $child_line_item) {
 				$code = $child_line_item->code();
-				$child_line_item = $this->process( $child_line_item );
-				if( ! $child_line_item instanceof EEI_Line_Item ) {
-					$line_item->delete_child_line_item( $code );
+				$child_line_item = $this->process($child_line_item);
+				if ( ! $child_line_item instanceof EEI_Line_Item) {
+					$line_item->delete_child_line_item($code);
 					continue;
 				}
 				if (
-					( $child_line_item instanceof EEI_Line_Item &&
+					($child_line_item instanceof EEI_Line_Item &&
 					$child_line_item->type() === EEM_Line_Item::type_line_item &&
-					$child_line_item->OBJ_type() === 'Ticket' ) ||
-					( $child_line_item instanceof EEI_Line_Item &&
-					 $child_line_item->type() === EEM_Line_Item::type_sub_total )
+					$child_line_item->OBJ_type() === 'Ticket') ||
+					($child_line_item instanceof EEI_Line_Item &&
+					 $child_line_item->type() === EEM_Line_Item::type_sub_total)
 				) {
 					$ticket_or_subtotals_with_tkt_children_count++;
 				}
 			}
 			// if this is an event subtotal with NO ticket children
 			// we basically want to ignore it
-			return $this->_filter_zero_subtotal_line_item( $non_zero_line_item, $ticket_or_subtotals_with_tkt_children_count );
-		}else{
+			return $this->_filter_zero_subtotal_line_item($non_zero_line_item, $ticket_or_subtotals_with_tkt_children_count);
+		} else {
 			return $non_zero_line_item;
 		}
 	}
@@ -73,7 +73,7 @@ 
 	 * @param EEI_Line_Item $line_item
 	 * @return EEI_Line_Item
 	 */
-	protected function _filter_zero_line_item( EEI_Line_Item $line_item ) {
+	protected function _filter_zero_line_item(EEI_Line_Item $line_item) {
 		if (
 			$line_item->type() === EEM_Line_Item::type_line_item &&
 			$line_item->OBJ_type() === 'Ticket' &&
@@ -93,7 +93,7 @@ 
 	 * @param int           $ticket_children
 	 * @return \EEI_Line_Item
 	 */
-	protected function _filter_zero_subtotal_line_item( EEI_Line_Item $line_item, $ticket_children = 0 ) {
+	protected function _filter_zero_subtotal_line_item(EEI_Line_Item $line_item, $ticket_children = 0) {
 		if (
 			$line_item->type() === EEM_Line_Item::type_sub_total &&
 			$ticket_children === 0

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/line_item_filters/EE_Single_Registration_Line_Item_Filter.class.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 /**
  * Class EE_Single_Registration_Line_Item_Filter
@@ -21,8 +21,8 @@ 
 	 *
 	 * @param EE_Registration $registration
 	 */
-	public function __construct( $registration ) {
-		parent::__construct( array( $registration ) );
+	public function __construct($registration) {
+		parent::__construct(array($registration));
 	}
 }
 // End of file EE_Single_Registration_Line_Item_Filter.class.php

core/libraries/line_item_filters/EE_Specific_Registrations_Line_Item_Filter.class.php

Indentation +4 added lines, -4 removed lines

@@ -18,10 +18,10 @@
  */
 class EE_Specific_Registrations_Line_Item_Filter extends EE_Line_Item_Filter_Base {
 /**
-	 * array of line item codes and their corresponding quantities for
-	 * registrations that owe money and can pay at this moment
-	 * @type array $_counts_per_line_item_code
-	 */
+ * array of line item codes and their corresponding quantities for
+ * registrations that owe money and can pay at this moment
+ * @type array $_counts_per_line_item_code
+ */
 	protected $_counts_per_line_item_code = array();
 
 	/**

Spacing +40 added lines, -40 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 /**
@@ -36,9 +36,9 @@ 
 	 * EE_Billable_Line_Item_Filter constructor.
 	 * @param EE_Registration[] $registrations
 	 */
-	public function __construct( $registrations ) {
+	public function __construct($registrations) {
 		$this->_registrations = $registrations;
-		$this->_calculate_counts_per_line_item_code( $registrations );
+		$this->_calculate_counts_per_line_item_code($registrations);
 	}
 
 	/**
@@ -46,14 +46,14 @@ 
 	 * @param EE_Registration[] $registrations
 	 * @return void
 	 */
-	protected function _calculate_counts_per_line_item_code( $registrations ) {
-		foreach( $registrations as $registration ) {
-			$line_item_code = EEM_Line_Item::instance()->get_var( EEM_Line_Item::instance()->line_item_for_registration_query_params( $registration, array( 'limit' => 1 ) ), 'LIN_code' );
-			if( $line_item_code ) {
-				if( ! isset( $this->_counts_per_line_item_code[ $line_item_code ] ) ) {
-					$this->_counts_per_line_item_code[ $line_item_code ] = 1;
-				}else{
-					$this->_counts_per_line_item_code[ $line_item_code ]++;
+	protected function _calculate_counts_per_line_item_code($registrations) {
+		foreach ($registrations as $registration) {
+			$line_item_code = EEM_Line_Item::instance()->get_var(EEM_Line_Item::instance()->line_item_for_registration_query_params($registration, array('limit' => 1)), 'LIN_code');
+			if ($line_item_code) {
+				if ( ! isset($this->_counts_per_line_item_code[$line_item_code])) {
+					$this->_counts_per_line_item_code[$line_item_code] = 1;
+				} else {
+					$this->_counts_per_line_item_code[$line_item_code]++;
 				}
 			}
 		}
@@ -67,57 +67,57 @@ 
 	 * @param EEI_Line_Item      $line_item
 	 * @return \EEI_Line_Item
 	 */
-	public function process( EEI_Line_Item $line_item ) {
-		$this->_filter_billable_line_item( $line_item );
-		if( ! $line_item->children() ) {
+	public function process(EEI_Line_Item $line_item) {
+		$this->_filter_billable_line_item($line_item);
+		if ( ! $line_item->children()) {
 			return $line_item;
 		}
 		//the original running total (taking ALL tickets into account)
 		$running_total_of_children = 0;
 		//the new running total (only taking the specified ticket quantities into account)
 		$runnign_total_of_children_under_consideration = 0;
-		foreach ( $line_item->children() as $child_line_item ) {
-			if( $child_line_item->is_percent() ) {
+		foreach ($line_item->children() as $child_line_item) {
+			if ($child_line_item->is_percent()) {
 				$original_li_total = $running_total_of_children * $child_line_item->percent() / 100;
-			}else{
+			} else {
 				$original_li_total = $child_line_item->unit_price() * $child_line_item->quantity();
 			}
 
-			$this->process( $child_line_item );
+			$this->process($child_line_item);
 			/*
 			 * If this line item is a normal line item that isn't for a ticket
 			 * we want to modify its total (and unit price if not a percentage line item)
 			 * so it reflects only that portion of the surcharge/discount shared by these
 			 * registrations
 			 */
-			if( $child_line_item->type() === EEM_Line_Item::type_line_item &&
-					$child_line_item->OBJ_type() !== 'Ticket' ) {
-				if( $running_total_of_children ) {
+			if ($child_line_item->type() === EEM_Line_Item::type_line_item &&
+					$child_line_item->OBJ_type() !== 'Ticket') {
+				if ($running_total_of_children) {
 					$percent_of_running_total = $original_li_total / $running_total_of_children;
 				} else {
 					$percent_of_running_total = 0;
 				}
 
-				$child_line_item->set_total( $runnign_total_of_children_under_consideration * $percent_of_running_total );
-				if( ! $child_line_item->is_percent() ) {
-					$child_line_item->set_unit_price( $child_line_item->total() / $child_line_item->quantity() );
+				$child_line_item->set_total($runnign_total_of_children_under_consideration * $percent_of_running_total);
+				if ( ! $child_line_item->is_percent()) {
+					$child_line_item->set_unit_price($child_line_item->total() / $child_line_item->quantity());
 				}
-			}elseif( $line_item->type() === EEM_Line_Item::type_line_item &&
-					$line_item->OBJ_type() === 'Ticket' ) {
+			}elseif ($line_item->type() === EEM_Line_Item::type_line_item &&
+					$line_item->OBJ_type() === 'Ticket') {
 				//make sure this item's quantity matches its parent
-				if( ! $child_line_item->is_percent() ) {
-					$child_line_item->set_quantity( $line_item->quantity() );
-					$child_line_item->set_total( $child_line_item->unit_price() * $child_line_item->quantity() );
+				if ( ! $child_line_item->is_percent()) {
+					$child_line_item->set_quantity($line_item->quantity());
+					$child_line_item->set_total($child_line_item->unit_price() * $child_line_item->quantity());
 				}
 			}
 			$running_total_of_children += $original_li_total;
 			$runnign_total_of_children_under_consideration += $child_line_item->total();
 		}
-		$line_item->set_total( $runnign_total_of_children_under_consideration );
-		if( $line_item->quantity() ) {
-			$line_item->set_unit_price( $runnign_total_of_children_under_consideration / $line_item->quantity() );
+		$line_item->set_total($runnign_total_of_children_under_consideration);
+		if ($line_item->quantity()) {
+			$line_item->set_unit_price($runnign_total_of_children_under_consideration / $line_item->quantity());
 		} else {
-			$line_item->set_unit_price( 0 );
+			$line_item->set_unit_price(0);
 		}
 		return $line_item;
 	}
@@ -130,18 +130,18 @@ 
 	 * @param EEI_Line_Item $line_item
 	 * @return EEI_Line_Item
 	 */
-	protected function _filter_billable_line_item( EEI_Line_Item $line_item ) {
+	protected function _filter_billable_line_item(EEI_Line_Item $line_item) {
 		// is this a ticket ?
-		if ( $line_item->type() === EEM_Line_Item::type_line_item && $line_item->OBJ_type() == 'Ticket' ) {
+		if ($line_item->type() === EEM_Line_Item::type_line_item && $line_item->OBJ_type() == 'Ticket') {
 			// if this ticket is billable at this moment, then we should have a positive quantity
-			if ( isset( $this->_counts_per_line_item_code[ $line_item->code() ] )) {
+			if (isset($this->_counts_per_line_item_code[$line_item->code()])) {
 				// set quantity based on number of billable registrations for this ticket
-				$quantity = $this->_counts_per_line_item_code[ $line_item->code() ];
+				$quantity = $this->_counts_per_line_item_code[$line_item->code()];
 			} else {
 				$quantity = 0;
 			}
-			$line_item->set_quantity( $quantity );
-			$line_item->set_total( $line_item->unit_price() * $line_item->quantity() );
+			$line_item->set_quantity($quantity);
+			$line_item->set_total($line_item->unit_price() * $line_item->quantity());
 		}
 		return $line_item;
 	}

Braces +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
 			if( $line_item_code ) {
 				if( ! isset( $this->_counts_per_line_item_code[ $line_item_code ] ) ) {
 					$this->_counts_per_line_item_code[ $line_item_code ] = 1;
-				}else{
+				} else{
 					$this->_counts_per_line_item_code[ $line_item_code ]++;
 				}
 			}
@@ -79,7 +79,7 @@ 
 		foreach ( $line_item->children() as $child_line_item ) {
 			if( $child_line_item->is_percent() ) {
 				$original_li_total = $running_total_of_children * $child_line_item->percent() / 100;
-			}else{
+			} else{
 				$original_li_total = $child_line_item->unit_price() * $child_line_item->quantity();
 			}
 
@@ -102,7 +102,7 @@ 
 				if( ! $child_line_item->is_percent() ) {
 					$child_line_item->set_unit_price( $child_line_item->total() / $child_line_item->quantity() );
 				}
-			}elseif( $line_item->type() === EEM_Line_Item::type_line_item &&
+			} elseif( $line_item->type() === EEM_Line_Item::type_line_item &&
 					$line_item->OBJ_type() === 'Ticket' ) {
 				//make sure this item's quantity matches its parent
 				if( ! $child_line_item->is_percent() ) {