eventespresso / event-espresso-core

core/libraries/line_item_display/EE_Admin_Table_Line_Item_Display_Strategy.strategy.php

Indentation +377 added lines, -377 removed lines

@@ -13,381 +13,381 @@
 class EE_Admin_Table_Line_Item_Display_Strategy implements EEI_Line_Item_Display
 {
 
-    /**
-     * whether to display the taxes row or not
-     *
-     * @type bool $_show_taxes
-     */
-    protected $_show_taxes = false;
-
-    /**
-     * html for any tax rows
-     *
-     * @type string $_show_taxes
-     */
-    protected $_taxes_html = '';
-
-
-    /**
-     * total amount including tax we can bill for at this time
-     *
-     * @type float $_grand_total
-     */
-    protected $_grand_total = 0.00;
-
-
-    /**
-     * @return float
-     */
-    public function grand_total()
-    {
-        return $this->_grand_total;
-    }
-
-
-    /**
-     * This is used to output a single
-     *
-     * @param EE_Line_Item $line_item
-     * @param array        $options
-     * @return mixed
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function display_line_item(EE_Line_Item $line_item, $options = [])
-    {
-        $html = '';
-        // set some default options and merge with incoming
-        $default_options = [
-            'odd'                => true,
-            'use_table_wrapper'  => true,
-            'table_css_class'    => 'admin-primary-mbox-tbl',
-            'taxes_tr_css_class' => 'admin-primary-mbox-taxes-tr',
-            'total_tr_css_class' => 'admin-primary-mbox-total-tr',
-        ];
-        $options         = array_merge($default_options, (array) $options);
-
-        switch ($line_item->type()) {
-            case EEM_Line_Item::type_line_item:
-                // item row
-                $html .= $this->_item_row($line_item, $options);
-                break;
-
-            case EEM_Line_Item::type_sub_line_item:
-                // currently not showing sub-items
-                // $html .= $this->_sub_item_row($line_item, $options);
-                break;
-
-            case EEM_Line_Item::type_sub_total:
-                if ($line_item->quantity() === 0) {
-                    return $html;
-                }
-                // loop through children
-                $child_line_items = $line_item->children();
-                // loop through children
-                foreach ($child_line_items as $child_line_item) {
-                    // recursively feed children back into this method
-                    $html .= $this->display_line_item($child_line_item, $options);
-                }
-                // currently not showing subtotal row
-                // $html .= $this->_sub_total_row($line_item, $options);
-                break;
-
-            case EEM_Line_Item::type_tax:
-                if ($this->_show_taxes) {
-                    $this->_taxes_html .= $this->_tax_row($line_item);
-                }
-                break;
-
-            case EEM_Line_Item::type_tax_sub_total:
-                foreach ($line_item->children() as $child_line_item) {
-                    if ($child_line_item->type() == EEM_Line_Item::type_tax) {
-                        $this->display_line_item($child_line_item, $options);
-                    }
-                }
-                break;
-
-            case EEM_Line_Item::type_total:
-                // determine whether to display taxes or not
-                $this->_show_taxes = $line_item->get_total_tax() > 0;
-                // get all child line items
-                $children = $line_item->children();
-
-                // loop thru all non-tax child line items
-                foreach ($children as $child_line_item) {
-                    $html .= $this->display_line_item($child_line_item, $options);
-                }
-
-                $html .= $this->_taxes_html;
-                $html .= $this->_total_row($line_item);
-                if ($options['use_table_wrapper']) {
-                    $html = $this->_table_header($options) . $html . $this->_table_footer();
-                }
-                break;
-        }
-
-        return $html;
-    }
-
-
-    /**
-     * Table header for display.
-     *
-     * @param array $options array of options for the table.
-     * @return string
-     * @since   4.8
-     */
-    protected function _table_header(array $options)
-    {
-        $html = EEH_HTML::table('', '', $options['table_css_class']);
-        $html .= EEH_HTML::thead();
-        $html .= EEH_HTML::tr();
-        $html .= EEH_HTML::th(esc_html__('Name', 'event_espresso'), '', 'jst-left');
-        $html .= EEH_HTML::th(esc_html__('Type', 'event_espresso'), '', 'jst-left');
-        $html .= EEH_HTML::th(esc_html__('Amount', 'event_espresso'), '', 'jst-cntr');
-        $html .= EEH_HTML::th(esc_html__('Qty', 'event_espresso'), '', 'jst-cntr');
-        $html .= EEH_HTML::th(esc_html__('Line Total', 'event_espresso'), '', 'jst-cntr');
-        $html .= EEH_HTML::tbody();
-        return $html;
-    }
-
-
-    /**
-     * Table footer for display
-     *
-     * @return string
-     * @since 4.8
-     */
-    protected function _table_footer()
-    {
-        return EEH_HTML::tbodyx() . EEH_HTML::tablex();
-    }
-
-
-    /**
-     *    _item_row
-     *
-     * @param EE_Line_Item $line_item
-     * @param array        $options
-     * @return mixed
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    protected function _item_row(EE_Line_Item $line_item, $options = [])
-    {
-        $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);
-
-        // Name Column
-        $name_link = $line_item_related_object instanceof EEI_Admin_Links
-            ? $line_item_related_object->get_admin_details_link()
-            : '';
-
-        // 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_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 .= $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'),
-            '<span class="ee-line-item-related-parent-object">',
-            $line_item->parent() instanceof EE_Line_Item
-                ? $line_item->parent()->OBJ_type_i18n()
-                : esc_html__('Item:', 'event_espresso'),
-            $parent_related_object_link
-                ? '<a href="' . $parent_related_object_link . '">' . $parent_related_object_name . '</a>'
-                : $parent_related_object_name,
-            '</span>'
-        );
-
-        $name_html = apply_filters(
-            'FHEE__EE_Admin_Table_Line_Item_Display_Strategy___item_row__name_html',
-            $name_html,
-            $line_item,
-            $options
-        );
-
-        $html .= EEH_HTML::td($name_html, '', 'jst-left');
-
-        // Type Column
-        $type_html = $line_item->OBJ_type() ? $line_item->OBJ_type_i18n() : '';
-        $type_html .= $this->_get_cancellations($line_item);
-        $type_html .= $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(esc_html__('Code: %s', 'event_espresso'), $code) . '</span>'
-            : '';
-        $html      .= EEH_HTML::td($type_html, '', 'jst-left');
-
-        // Amount Column
-        $html .= $line_item->is_percent()
-            ? EEH_HTML::td($line_item->percent() . '%', '', 'jst-rght')
-            : EEH_HTML::td($line_item->unit_price_no_code(), '', 'jst-rght');
-
-        // QTY column
-        $html .= EEH_HTML::td($line_item->quantity(), '', 'jst-rght');
-
-        // total column
-        $html .= EEH_HTML::td(
-            EEH_Template::format_currency($line_item->total(), false, false),
-            '',
-            'jst-rght'
-        );
-
-        // finish things off and return
-        $html .= EEH_HTML::trx();
-        return $html;
-    }
-
-
-    /**
-     *    _get_cancellations
-     *
-     * @param EE_Line_Item $line_item
-     * @return string
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    protected function _get_cancellations(EE_Line_Item $line_item)
-    {
-        $html          = '';
-        $cancellations = $line_item->get_cancellations();
-        $cancellation  = reset($cancellations);
-        if ($cancellation instanceof EE_Line_Item) {
-            $html .= ' <span class="ee-line-item-id">';
-            $html .= sprintf(
-                _n(
-                    '(%1$s Cancellation)',
-                    '(%1$s Cancellations)',
-                    $cancellation->quantity(),
-                    'event_espresso'
-                ),
-                $cancellation->quantity()
-            );
-            $html .= '</span>';
-        }
-        return $html;
-    }
-
-
-    // /**
-    //  *  _sub_item_row
-    //  *
-    //  * @param EE_Line_Item $line_item
-    //  * @param array        $options
-    //  * @return mixed
-    //  */
-    // protected function _sub_item_row(EE_Line_Item $line_item, $options = [])
-    // {
-    //     // for now we're not showing sub-items
-    //     return '';
-    // }
-
-
-    /**
-     *  _tax_row
-     *
-     * @param EE_Line_Item $line_item
-     * @return mixed
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    protected function _tax_row(EE_Line_Item $line_item)
-    {
-        // start of row
-        $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="4"'
-        );
-        // total th
-        $html .= EEH_HTML::th(
-            EEH_Template::format_currency($line_item->total(), false, false),
-            '',
-            'jst-rght'
-        );
-        // end of row
-        $html .= EEH_HTML::trx();
-        return $html;
-    }
-
-
-    // /**
-    //  *  _total_row
-    //  *
-    //  * @param EE_Line_Item $line_item
-    //  * @param string       $text
-    //  * @param array        $options
-    //  * @return mixed
-    //  */
-    // protected function _sub_total_row(EE_Line_Item $line_item, $text = '', $options = [])
-    // {
-    //     // currently not showing subtotal row
-    //     return '';
-    // }
-
-
-    /**
-     *  _total_row
-     *
-     * @param EE_Line_Item $line_item
-     * @return mixed
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    protected function _total_row(EE_Line_Item $line_item)
-    {
-        // start of row
-        $html = EEH_HTML::tr('', '', 'admin-primary-mbox-total-tr');
-        // Total th label
-        $total_label =
-            sprintf(
-                esc_html__('Transaction Total %s', 'event_espresso'),
-                '(' . EE_Registry::instance()->CFG->currency->code . ')'
-            );
-        $html        .= EEH_HTML::th($total_label, '', 'jst-rght', '', ' colspan="4"');
-        // total th
-
-        $html .= EEH_HTML::th(
-            EEH_Template::format_currency($line_item->total(), false, false),
-            '',
-            'jst-rght'
-        );
-        // end of row
-        $html .= EEH_HTML::trx();
-        return $html;
-    }
+	/**
+	 * whether to display the taxes row or not
+	 *
+	 * @type bool $_show_taxes
+	 */
+	protected $_show_taxes = false;
+
+	/**
+	 * html for any tax rows
+	 *
+	 * @type string $_show_taxes
+	 */
+	protected $_taxes_html = '';
+
+
+	/**
+	 * total amount including tax we can bill for at this time
+	 *
+	 * @type float $_grand_total
+	 */
+	protected $_grand_total = 0.00;
+
+
+	/**
+	 * @return float
+	 */
+	public function grand_total()
+	{
+		return $this->_grand_total;
+	}
+
+
+	/**
+	 * This is used to output a single
+	 *
+	 * @param EE_Line_Item $line_item
+	 * @param array        $options
+	 * @return mixed
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function display_line_item(EE_Line_Item $line_item, $options = [])
+	{
+		$html = '';
+		// set some default options and merge with incoming
+		$default_options = [
+			'odd'                => true,
+			'use_table_wrapper'  => true,
+			'table_css_class'    => 'admin-primary-mbox-tbl',
+			'taxes_tr_css_class' => 'admin-primary-mbox-taxes-tr',
+			'total_tr_css_class' => 'admin-primary-mbox-total-tr',
+		];
+		$options         = array_merge($default_options, (array) $options);
+
+		switch ($line_item->type()) {
+			case EEM_Line_Item::type_line_item:
+				// item row
+				$html .= $this->_item_row($line_item, $options);
+				break;
+
+			case EEM_Line_Item::type_sub_line_item:
+				// currently not showing sub-items
+				// $html .= $this->_sub_item_row($line_item, $options);
+				break;
+
+			case EEM_Line_Item::type_sub_total:
+				if ($line_item->quantity() === 0) {
+					return $html;
+				}
+				// loop through children
+				$child_line_items = $line_item->children();
+				// loop through children
+				foreach ($child_line_items as $child_line_item) {
+					// recursively feed children back into this method
+					$html .= $this->display_line_item($child_line_item, $options);
+				}
+				// currently not showing subtotal row
+				// $html .= $this->_sub_total_row($line_item, $options);
+				break;
+
+			case EEM_Line_Item::type_tax:
+				if ($this->_show_taxes) {
+					$this->_taxes_html .= $this->_tax_row($line_item);
+				}
+				break;
+
+			case EEM_Line_Item::type_tax_sub_total:
+				foreach ($line_item->children() as $child_line_item) {
+					if ($child_line_item->type() == EEM_Line_Item::type_tax) {
+						$this->display_line_item($child_line_item, $options);
+					}
+				}
+				break;
+
+			case EEM_Line_Item::type_total:
+				// determine whether to display taxes or not
+				$this->_show_taxes = $line_item->get_total_tax() > 0;
+				// get all child line items
+				$children = $line_item->children();
+
+				// loop thru all non-tax child line items
+				foreach ($children as $child_line_item) {
+					$html .= $this->display_line_item($child_line_item, $options);
+				}
+
+				$html .= $this->_taxes_html;
+				$html .= $this->_total_row($line_item);
+				if ($options['use_table_wrapper']) {
+					$html = $this->_table_header($options) . $html . $this->_table_footer();
+				}
+				break;
+		}
+
+		return $html;
+	}
+
+
+	/**
+	 * Table header for display.
+	 *
+	 * @param array $options array of options for the table.
+	 * @return string
+	 * @since   4.8
+	 */
+	protected function _table_header(array $options)
+	{
+		$html = EEH_HTML::table('', '', $options['table_css_class']);
+		$html .= EEH_HTML::thead();
+		$html .= EEH_HTML::tr();
+		$html .= EEH_HTML::th(esc_html__('Name', 'event_espresso'), '', 'jst-left');
+		$html .= EEH_HTML::th(esc_html__('Type', 'event_espresso'), '', 'jst-left');
+		$html .= EEH_HTML::th(esc_html__('Amount', 'event_espresso'), '', 'jst-cntr');
+		$html .= EEH_HTML::th(esc_html__('Qty', 'event_espresso'), '', 'jst-cntr');
+		$html .= EEH_HTML::th(esc_html__('Line Total', 'event_espresso'), '', 'jst-cntr');
+		$html .= EEH_HTML::tbody();
+		return $html;
+	}
+
+
+	/**
+	 * Table footer for display
+	 *
+	 * @return string
+	 * @since 4.8
+	 */
+	protected function _table_footer()
+	{
+		return EEH_HTML::tbodyx() . EEH_HTML::tablex();
+	}
+
+
+	/**
+	 *    _item_row
+	 *
+	 * @param EE_Line_Item $line_item
+	 * @param array        $options
+	 * @return mixed
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	protected function _item_row(EE_Line_Item $line_item, $options = [])
+	{
+		$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);
+
+		// Name Column
+		$name_link = $line_item_related_object instanceof EEI_Admin_Links
+			? $line_item_related_object->get_admin_details_link()
+			: '';
+
+		// 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_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 .= $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'),
+			'<span class="ee-line-item-related-parent-object">',
+			$line_item->parent() instanceof EE_Line_Item
+				? $line_item->parent()->OBJ_type_i18n()
+				: esc_html__('Item:', 'event_espresso'),
+			$parent_related_object_link
+				? '<a href="' . $parent_related_object_link . '">' . $parent_related_object_name . '</a>'
+				: $parent_related_object_name,
+			'</span>'
+		);
+
+		$name_html = apply_filters(
+			'FHEE__EE_Admin_Table_Line_Item_Display_Strategy___item_row__name_html',
+			$name_html,
+			$line_item,
+			$options
+		);
+
+		$html .= EEH_HTML::td($name_html, '', 'jst-left');
+
+		// Type Column
+		$type_html = $line_item->OBJ_type() ? $line_item->OBJ_type_i18n() : '';
+		$type_html .= $this->_get_cancellations($line_item);
+		$type_html .= $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(esc_html__('Code: %s', 'event_espresso'), $code) . '</span>'
+			: '';
+		$html      .= EEH_HTML::td($type_html, '', 'jst-left');
+
+		// Amount Column
+		$html .= $line_item->is_percent()
+			? EEH_HTML::td($line_item->percent() . '%', '', 'jst-rght')
+			: EEH_HTML::td($line_item->unit_price_no_code(), '', 'jst-rght');
+
+		// QTY column
+		$html .= EEH_HTML::td($line_item->quantity(), '', 'jst-rght');
+
+		// total column
+		$html .= EEH_HTML::td(
+			EEH_Template::format_currency($line_item->total(), false, false),
+			'',
+			'jst-rght'
+		);
+
+		// finish things off and return
+		$html .= EEH_HTML::trx();
+		return $html;
+	}
+
+
+	/**
+	 *    _get_cancellations
+	 *
+	 * @param EE_Line_Item $line_item
+	 * @return string
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	protected function _get_cancellations(EE_Line_Item $line_item)
+	{
+		$html          = '';
+		$cancellations = $line_item->get_cancellations();
+		$cancellation  = reset($cancellations);
+		if ($cancellation instanceof EE_Line_Item) {
+			$html .= ' <span class="ee-line-item-id">';
+			$html .= sprintf(
+				_n(
+					'(%1$s Cancellation)',
+					'(%1$s Cancellations)',
+					$cancellation->quantity(),
+					'event_espresso'
+				),
+				$cancellation->quantity()
+			);
+			$html .= '</span>';
+		}
+		return $html;
+	}
+
+
+	// /**
+	//  *  _sub_item_row
+	//  *
+	//  * @param EE_Line_Item $line_item
+	//  * @param array        $options
+	//  * @return mixed
+	//  */
+	// protected function _sub_item_row(EE_Line_Item $line_item, $options = [])
+	// {
+	//     // for now we're not showing sub-items
+	//     return '';
+	// }
+
+
+	/**
+	 *  _tax_row
+	 *
+	 * @param EE_Line_Item $line_item
+	 * @return mixed
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	protected function _tax_row(EE_Line_Item $line_item)
+	{
+		// start of row
+		$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="4"'
+		);
+		// total th
+		$html .= EEH_HTML::th(
+			EEH_Template::format_currency($line_item->total(), false, false),
+			'',
+			'jst-rght'
+		);
+		// end of row
+		$html .= EEH_HTML::trx();
+		return $html;
+	}
+
+
+	// /**
+	//  *  _total_row
+	//  *
+	//  * @param EE_Line_Item $line_item
+	//  * @param string       $text
+	//  * @param array        $options
+	//  * @return mixed
+	//  */
+	// protected function _sub_total_row(EE_Line_Item $line_item, $text = '', $options = [])
+	// {
+	//     // currently not showing subtotal row
+	//     return '';
+	// }
+
+
+	/**
+	 *  _total_row
+	 *
+	 * @param EE_Line_Item $line_item
+	 * @return mixed
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	protected function _total_row(EE_Line_Item $line_item)
+	{
+		// start of row
+		$html = EEH_HTML::tr('', '', 'admin-primary-mbox-total-tr');
+		// Total th label
+		$total_label =
+			sprintf(
+				esc_html__('Transaction Total %s', 'event_espresso'),
+				'(' . EE_Registry::instance()->CFG->currency->code . ')'
+			);
+		$html        .= EEH_HTML::th($total_label, '', 'jst-rght', '', ' colspan="4"');
+		// total th
+
+		$html .= EEH_HTML::th(
+			EEH_Template::format_currency($line_item->total(), false, false),
+			'',
+			'jst-rght'
+		);
+		// end of row
+		$html .= EEH_HTML::trx();
+		return $html;
+	}
 }

Spacing +12 added lines, -12 removed lines

@@ -65,7 +65,7 @@ 
             'taxes_tr_css_class' => 'admin-primary-mbox-taxes-tr',
             'total_tr_css_class' => 'admin-primary-mbox-total-tr',
         ];
-        $options         = array_merge($default_options, (array) $options);
+        $options = array_merge($default_options, (array) $options);
 
         switch ($line_item->type()) {
             case EEM_Line_Item::type_line_item:
@@ -121,7 +121,7 @@ 
                 $html .= $this->_taxes_html;
                 $html .= $this->_total_row($line_item);
                 if ($options['use_table_wrapper']) {
-                    $html = $this->_table_header($options) . $html . $this->_table_footer();
+                    $html = $this->_table_header($options).$html.$this->_table_footer();
                 }
                 break;
         }
@@ -160,7 +160,7 @@ 
      */
     protected function _table_footer()
     {
-        return EEH_HTML::tbodyx() . EEH_HTML::tablex();
+        return EEH_HTML::tbodyx().EEH_HTML::tablex();
     }
 
 
@@ -205,15 +205,15 @@ 
             ? $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
+            ? $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 = '<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'),
@@ -222,7 +222,7 @@ 
                 ? $line_item->parent()->OBJ_type_i18n()
                 : esc_html__('Item:', 'event_espresso'),
             $parent_related_object_link
-                ? '<a href="' . $parent_related_object_link . '">' . $parent_related_object_name . '</a>'
+                ? '<a href="'.$parent_related_object_link.'">'.$parent_related_object_name.'</a>'
                 : $parent_related_object_name,
             '</span>'
         );
@@ -244,13 +244,13 @@ 
             ? $line_item_related_object->code()
             : '';
         $type_html .= ! empty($code)
-            ? '<span class="ee-line-item-id">' . sprintf(esc_html__('Code: %s', 'event_espresso'), $code) . '</span>'
+            ? '<span class="ee-line-item-id">'.sprintf(esc_html__('Code: %s', 'event_espresso'), $code).'</span>'
             : '';
         $html      .= EEH_HTML::td($type_html, '', 'jst-left');
 
         // Amount Column
         $html .= $line_item->is_percent()
-            ? EEH_HTML::td($line_item->percent() . '%', '', 'jst-rght')
+            ? EEH_HTML::td($line_item->percent().'%', '', 'jst-rght')
             : EEH_HTML::td($line_item->unit_price_no_code(), '', 'jst-rght');
 
         // QTY column
@@ -327,7 +327,7 @@ 
         $html = EEH_HTML::tr('', 'admin-primary-mbox-taxes-tr');
         // name th
         $html .= EEH_HTML::th(
-            $line_item->name() . '(' . $line_item->get_pretty('LIN_percent') . '%)',
+            $line_item->name().'('.$line_item->get_pretty('LIN_percent').'%)',
             '',
             'jst-rght',
             '',
@@ -376,9 +376,9 @@ 
         $total_label =
             sprintf(
                 esc_html__('Transaction Total %s', 'event_espresso'),
-                '(' . EE_Registry::instance()->CFG->currency->code . ')'
+                '('.EE_Registry::instance()->CFG->currency->code.')'
             );
-        $html        .= EEH_HTML::th($total_label, '', 'jst-rght', '', ' colspan="4"');
+        $html .= EEH_HTML::th($total_label, '', 'jst-rght', '', ' colspan="4"');
         // total th
 
         $html .= EEH_HTML::th(

core/domain/services/checkout/extra_txn_fees/line_items/ExtraTxnFeesForLineItemsHandler.php

Indentation +77 added lines, -77 removed lines

@@ -18,87 +18,87 @@
 {
 
 
-    /**
-     * @var ExtraTxnFeesDistributionStrategyInterface
-     */
-    protected $extra_fees_strategy;
+	/**
+	 * @var ExtraTxnFeesDistributionStrategyInterface
+	 */
+	protected $extra_fees_strategy;
 
 
-    /**
-     * @return ExtraTxnFeesDistributionStrategyInterface
-     * @throws DomainException
-     */
-    private function getExtraTxnFeesDistributionStrategy()
-    {
-        if (! $this->extra_fees_strategy instanceof ExtraTxnFeesDistributionStrategyInterface) {
-            switch ($this->getDistributionStrategy()) {
-                case ExtraTxnFeesHandler::STRATEGY_PRIMARY_REGISTRANT_ONLY:
-                    $extra_fees_strategy = new ApplyExtraFeesToPrimaryRegistrantOnly();
-                    break;
-                case ExtraTxnFeesHandler::STRATEGY_DISTRIBUTE_EVENLY:
-                    $extra_fees_strategy = new ApplyExtraFeesToAllRegistrationsEvenly();
-                    break;
-                default:
-                    $extra_fees_strategy = new ApplyExtraFeesToPrimaryRegistrantOnly();
-            }
-            $this->validateExtraTxnFeesDistributionStrategy(
-                $extra_fees_strategy,
-                ExtraTxnFeesDistributionStrategyInterface::class
-            );
-            $this->extra_fees_strategy = $extra_fees_strategy;
-        }
-        return $this->extra_fees_strategy;
-    }
+	/**
+	 * @return ExtraTxnFeesDistributionStrategyInterface
+	 * @throws DomainException
+	 */
+	private function getExtraTxnFeesDistributionStrategy()
+	{
+		if (! $this->extra_fees_strategy instanceof ExtraTxnFeesDistributionStrategyInterface) {
+			switch ($this->getDistributionStrategy()) {
+				case ExtraTxnFeesHandler::STRATEGY_PRIMARY_REGISTRANT_ONLY:
+					$extra_fees_strategy = new ApplyExtraFeesToPrimaryRegistrantOnly();
+					break;
+				case ExtraTxnFeesHandler::STRATEGY_DISTRIBUTE_EVENLY:
+					$extra_fees_strategy = new ApplyExtraFeesToAllRegistrationsEvenly();
+					break;
+				default:
+					$extra_fees_strategy = new ApplyExtraFeesToPrimaryRegistrantOnly();
+			}
+			$this->validateExtraTxnFeesDistributionStrategy(
+				$extra_fees_strategy,
+				ExtraTxnFeesDistributionStrategyInterface::class
+			);
+			$this->extra_fees_strategy = $extra_fees_strategy;
+		}
+		return $this->extra_fees_strategy;
+	}
 
 
-    /**
-     * @param EEI_Line_Item $line_item
-     * @param EEI_Line_Item $child_line_item
-     * @param float         $original_li_total
-     * @param float         $running_total_for_all_tickets
-     * @param float         $running_total_for_specific_ticket
-     * @param bool          $is_primary_registrant
-     * @throws EE_Error
-     */
-    public function adjustUnitPriceForNonTicketLineItem(
-        EEI_Line_Item $line_item,
-        EEI_Line_Item $child_line_item,
-        $original_li_total,
-        $running_total_for_all_tickets,
-        $running_total_for_specific_ticket,
-        $is_primary_registrant
-    ) {
-        $extra_fees_strategy = $this->getExtraTxnFeesDistributionStrategy();
-        $extra_fees_strategy->adjustUnitPriceForNonTicketLineItem(
-            $line_item,
-            $child_line_item,
-            $original_li_total,
-            $running_total_for_all_tickets,
-            $running_total_for_specific_ticket,
-            $is_primary_registrant,
-            $this->total_reg_count,
-            $this->context
-        );
-    }
+	/**
+	 * @param EEI_Line_Item $line_item
+	 * @param EEI_Line_Item $child_line_item
+	 * @param float         $original_li_total
+	 * @param float         $running_total_for_all_tickets
+	 * @param float         $running_total_for_specific_ticket
+	 * @param bool          $is_primary_registrant
+	 * @throws EE_Error
+	 */
+	public function adjustUnitPriceForNonTicketLineItem(
+		EEI_Line_Item $line_item,
+		EEI_Line_Item $child_line_item,
+		$original_li_total,
+		$running_total_for_all_tickets,
+		$running_total_for_specific_ticket,
+		$is_primary_registrant
+	) {
+		$extra_fees_strategy = $this->getExtraTxnFeesDistributionStrategy();
+		$extra_fees_strategy->adjustUnitPriceForNonTicketLineItem(
+			$line_item,
+			$child_line_item,
+			$original_li_total,
+			$running_total_for_all_tickets,
+			$running_total_for_specific_ticket,
+			$is_primary_registrant,
+			$this->total_reg_count,
+			$this->context
+		);
+	}
 
 
-    /**
-     * @param EEI_Line_Item $line_item
-     * @param EEI_Line_Item $child_line_item
-     * @param bool          $is_primary_registrant
-     */
-    public function adjustUnitPriceAndQtyForTicketLineItem(
-        EEI_Line_Item $line_item,
-        EEI_Line_Item $child_line_item,
-        $is_primary_registrant
-    ) {
-        $extra_fees_strategy = $this->getExtraTxnFeesDistributionStrategy();
-        $extra_fees_strategy->adjustUnitPriceAndQtyForTicketLineItem(
-            $line_item,
-            $child_line_item,
-            $is_primary_registrant,
-            $this->total_reg_count,
-            $this->context
-        );
-    }
+	/**
+	 * @param EEI_Line_Item $line_item
+	 * @param EEI_Line_Item $child_line_item
+	 * @param bool          $is_primary_registrant
+	 */
+	public function adjustUnitPriceAndQtyForTicketLineItem(
+		EEI_Line_Item $line_item,
+		EEI_Line_Item $child_line_item,
+		$is_primary_registrant
+	) {
+		$extra_fees_strategy = $this->getExtraTxnFeesDistributionStrategy();
+		$extra_fees_strategy->adjustUnitPriceAndQtyForTicketLineItem(
+			$line_item,
+			$child_line_item,
+			$is_primary_registrant,
+			$this->total_reg_count,
+			$this->context
+		);
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
      */
     private function getExtraTxnFeesDistributionStrategy()
     {
-        if (! $this->extra_fees_strategy instanceof ExtraTxnFeesDistributionStrategyInterface) {
+        if ( ! $this->extra_fees_strategy instanceof ExtraTxnFeesDistributionStrategyInterface) {
             switch ($this->getDistributionStrategy()) {
                 case ExtraTxnFeesHandler::STRATEGY_PRIMARY_REGISTRANT_ONLY:
                     $extra_fees_strategy = new ApplyExtraFeesToPrimaryRegistrantOnly();

core/domain/services/checkout/extra_txn_fees/line_items/ExtraTxnFeesDistributionStrategyInterface.php

Indentation +34 added lines, -34 removed lines

@@ -6,40 +6,40 @@
 
 interface ExtraTxnFeesDistributionStrategyInterface
 {
-    /**
-     * @param EEI_Line_Item $line_item
-     * @param EEI_Line_Item $child_line_item
-     * @param float         $original_li_total
-     * @param float         $running_total_for_all_tickets
-     * @param float         $running_total_for_specific_ticket
-     * @param bool          $is_primary_registrant
-     * @param int           $total_reg_count
-     * @param string        $context
-     */
-    public function adjustUnitPriceForNonTicketLineItem(
-        EEI_Line_Item $line_item,
-        EEI_Line_Item $child_line_item,
-        $original_li_total,
-        $running_total_for_all_tickets,
-        $running_total_for_specific_ticket,
-        $is_primary_registrant,
-        $total_reg_count,
-        $context
-    );
+	/**
+	 * @param EEI_Line_Item $line_item
+	 * @param EEI_Line_Item $child_line_item
+	 * @param float         $original_li_total
+	 * @param float         $running_total_for_all_tickets
+	 * @param float         $running_total_for_specific_ticket
+	 * @param bool          $is_primary_registrant
+	 * @param int           $total_reg_count
+	 * @param string        $context
+	 */
+	public function adjustUnitPriceForNonTicketLineItem(
+		EEI_Line_Item $line_item,
+		EEI_Line_Item $child_line_item,
+		$original_li_total,
+		$running_total_for_all_tickets,
+		$running_total_for_specific_ticket,
+		$is_primary_registrant,
+		$total_reg_count,
+		$context
+	);
 
 
-    /**
-     * @param EEI_Line_Item $line_item
-     * @param EEI_Line_Item $child_line_item
-     * @param bool          $is_primary_registrant
-     * @param int           $total_reg_count
-     * @param string        $context
-     */
-    public function adjustUnitPriceAndQtyForTicketLineItem(
-        EEI_Line_Item $line_item,
-        EEI_Line_Item $child_line_item,
-        $is_primary_registrant,
-        $total_reg_count,
-        $context
-    );
+	/**
+	 * @param EEI_Line_Item $line_item
+	 * @param EEI_Line_Item $child_line_item
+	 * @param bool          $is_primary_registrant
+	 * @param int           $total_reg_count
+	 * @param string        $context
+	 */
+	public function adjustUnitPriceAndQtyForTicketLineItem(
+		EEI_Line_Item $line_item,
+		EEI_Line_Item $child_line_item,
+		$is_primary_registrant,
+		$total_reg_count,
+		$context
+	);
 }

core/domain/services/checkout/extra_txn_fees/line_items/ApplyExtraFeesToAllRegistrationsEvenly.php

Indentation +54 added lines, -54 removed lines

@@ -17,62 +17,62 @@
 {
 
 
-    /**
-     * @param EEI_Line_Item $line_item
-     * @param EEI_Line_Item $child_line_item
-     * @param float         $original_li_total
-     * @param float         $running_total_for_all_tickets
-     * @param float         $running_total_for_specific_ticket
-     * @param bool          $is_primary_registrant
-     * @param int           $total_reg_count
-     * @param string        $context
-     * @throws EE_Error
-     */
-    public function adjustUnitPriceForNonTicketLineItem(
-        EEI_Line_Item $line_item,
-        EEI_Line_Item $child_line_item,
-        $original_li_total,
-        $running_total_for_all_tickets,
-        $running_total_for_specific_ticket,
-        $is_primary_registrant,
-        $total_reg_count,
-        $context
-    ) {
-        $percent_of_running_total = $running_total_for_all_tickets
-            ? $original_li_total / $running_total_for_all_tickets
-            : 0;
-        $line_item->set_total($running_total_for_specific_ticket * $percent_of_running_total);
-        $line_item->set_total($running_total_for_specific_ticket);
+	/**
+	 * @param EEI_Line_Item $line_item
+	 * @param EEI_Line_Item $child_line_item
+	 * @param float         $original_li_total
+	 * @param float         $running_total_for_all_tickets
+	 * @param float         $running_total_for_specific_ticket
+	 * @param bool          $is_primary_registrant
+	 * @param int           $total_reg_count
+	 * @param string        $context
+	 * @throws EE_Error
+	 */
+	public function adjustUnitPriceForNonTicketLineItem(
+		EEI_Line_Item $line_item,
+		EEI_Line_Item $child_line_item,
+		$original_li_total,
+		$running_total_for_all_tickets,
+		$running_total_for_specific_ticket,
+		$is_primary_registrant,
+		$total_reg_count,
+		$context
+	) {
+		$percent_of_running_total = $running_total_for_all_tickets
+			? $original_li_total / $running_total_for_all_tickets
+			: 0;
+		$line_item->set_total($running_total_for_specific_ticket * $percent_of_running_total);
+		$line_item->set_total($running_total_for_specific_ticket);
 
-        if ($context === ExtraTxnFeesHandler::CONTEXT_ONE_REGISTRATION) {
-            $new_unit_price = $original_li_total / $total_reg_count;
-            $child_line_item->set_unit_price($new_unit_price);
-            $child_line_item->set_quantity(1);
-            $child_line_item->set_total($new_unit_price);
-        }
+		if ($context === ExtraTxnFeesHandler::CONTEXT_ONE_REGISTRATION) {
+			$new_unit_price = $original_li_total / $total_reg_count;
+			$child_line_item->set_unit_price($new_unit_price);
+			$child_line_item->set_quantity(1);
+			$child_line_item->set_total($new_unit_price);
+		}
 
-        if (! $line_item->is_percent()) {
-            $line_item->set_unit_price($line_item->total() / $line_item->quantity());
-        }
-    }
+		if (! $line_item->is_percent()) {
+			$line_item->set_unit_price($line_item->total() / $line_item->quantity());
+		}
+	}
 
 
-    /**
-     * @param EEI_Line_Item $line_item
-     * @param EEI_Line_Item $child_line_item
-     * @param bool          $is_primary_registrant
-     * @param int           $total_reg_count
-     * @param string        $context
-     * @since   $VID:$
-     */
-    public function adjustUnitPriceAndQtyForTicketLineItem(
-        EEI_Line_Item $line_item,
-        EEI_Line_Item $child_line_item,
-        $is_primary_registrant,
-        $total_reg_count,
-        $context
-    ) {
-        $child_line_item->set_quantity($line_item->quantity());
-        $child_line_item->set_total($child_line_item->unit_price() * $child_line_item->quantity());
-    }
+	/**
+	 * @param EEI_Line_Item $line_item
+	 * @param EEI_Line_Item $child_line_item
+	 * @param bool          $is_primary_registrant
+	 * @param int           $total_reg_count
+	 * @param string        $context
+	 * @since   $VID:$
+	 */
+	public function adjustUnitPriceAndQtyForTicketLineItem(
+		EEI_Line_Item $line_item,
+		EEI_Line_Item $child_line_item,
+		$is_primary_registrant,
+		$total_reg_count,
+		$context
+	) {
+		$child_line_item->set_quantity($line_item->quantity());
+		$child_line_item->set_total($child_line_item->unit_price() * $child_line_item->quantity());
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
             $child_line_item->set_total($new_unit_price);
         }
 
-        if (! $line_item->is_percent()) {
+        if ( ! $line_item->is_percent()) {
             $line_item->set_unit_price($line_item->total() / $line_item->quantity());
         }
     }

core/domain/services/checkout/extra_txn_fees/line_items/ApplyExtraFeesToPrimaryRegistrantOnly.php

Indentation +51 added lines, -51 removed lines

@@ -16,59 +16,59 @@
 class ApplyExtraFeesToPrimaryRegistrantOnly implements ExtraTxnFeesDistributionStrategyInterface
 {
 
-    /**
-     * @param EEI_Line_Item $line_item
-     * @param EEI_Line_Item $child_line_item
-     * @param float         $original_li_total
-     * @param float         $running_total_for_all_tickets
-     * @param float         $running_total_for_specific_ticket
-     * @param bool          $is_primary_registrant
-     * @param int           $total_reg_count
-     * @param string        $context
-     * @throws EE_Error
-     */
-    public function adjustUnitPriceForNonTicketLineItem(
-        EEI_Line_Item $line_item,
-        EEI_Line_Item $child_line_item,
-        $original_li_total,
-        $running_total_for_all_tickets,
-        $running_total_for_specific_ticket,
-        $is_primary_registrant,
-        $total_reg_count,
-        $context
-    ) {
-        $percent_of_running_total = $running_total_for_all_tickets
-            ? $original_li_total / $running_total_for_all_tickets
-            : 0;
-        $line_item->set_total($running_total_for_specific_ticket * $percent_of_running_total);
+	/**
+	 * @param EEI_Line_Item $line_item
+	 * @param EEI_Line_Item $child_line_item
+	 * @param float         $original_li_total
+	 * @param float         $running_total_for_all_tickets
+	 * @param float         $running_total_for_specific_ticket
+	 * @param bool          $is_primary_registrant
+	 * @param int           $total_reg_count
+	 * @param string        $context
+	 * @throws EE_Error
+	 */
+	public function adjustUnitPriceForNonTicketLineItem(
+		EEI_Line_Item $line_item,
+		EEI_Line_Item $child_line_item,
+		$original_li_total,
+		$running_total_for_all_tickets,
+		$running_total_for_specific_ticket,
+		$is_primary_registrant,
+		$total_reg_count,
+		$context
+	) {
+		$percent_of_running_total = $running_total_for_all_tickets
+			? $original_li_total / $running_total_for_all_tickets
+			: 0;
+		$line_item->set_total($running_total_for_specific_ticket * $percent_of_running_total);
 
-        if ($context === ExtraTxnFeesHandler::CONTEXT_ONE_REGISTRATION && ! $is_primary_registrant) {
-            $child_line_item->set_unit_price(0);
-            $child_line_item->set_quantity(0);
-            $child_line_item->set_total(0);
-        }
+		if ($context === ExtraTxnFeesHandler::CONTEXT_ONE_REGISTRATION && ! $is_primary_registrant) {
+			$child_line_item->set_unit_price(0);
+			$child_line_item->set_quantity(0);
+			$child_line_item->set_total(0);
+		}
 
-        if (! $line_item->is_percent()) {
-            $line_item->set_unit_price($line_item->total() / $line_item->quantity());
-        }
-    }
+		if (! $line_item->is_percent()) {
+			$line_item->set_unit_price($line_item->total() / $line_item->quantity());
+		}
+	}
 
 
-    /**
-     * @param EEI_Line_Item $line_item
-     * @param EEI_Line_Item $child_line_item
-     * @param bool          $is_primary_registrant
-     * @param int           $total_reg_count
-     * @param string        $context
-     */
-    public function adjustUnitPriceAndQtyForTicketLineItem(
-        EEI_Line_Item $line_item,
-        EEI_Line_Item $child_line_item,
-        $is_primary_registrant,
-        $total_reg_count,
-        $context
-    ) {
-        $child_line_item->set_quantity($line_item->quantity());
-        $child_line_item->set_total($child_line_item->unit_price() * $child_line_item->quantity());
-    }
+	/**
+	 * @param EEI_Line_Item $line_item
+	 * @param EEI_Line_Item $child_line_item
+	 * @param bool          $is_primary_registrant
+	 * @param int           $total_reg_count
+	 * @param string        $context
+	 */
+	public function adjustUnitPriceAndQtyForTicketLineItem(
+		EEI_Line_Item $line_item,
+		EEI_Line_Item $child_line_item,
+		$is_primary_registrant,
+		$total_reg_count,
+		$context
+	) {
+		$child_line_item->set_quantity($line_item->quantity());
+		$child_line_item->set_total($child_line_item->unit_price() * $child_line_item->quantity());
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
             $child_line_item->set_total(0);
         }
 
-        if (! $line_item->is_percent()) {
+        if ( ! $line_item->is_percent()) {
             $line_item->set_unit_price($line_item->total() / $line_item->quantity());
         }
     }

core/domain/services/checkout/extra_txn_fees/spco/ExtraTxnFeesDistributionStrategyInterface.php

Indentation +5 added lines, -5 removed lines

@@ -6,9 +6,9 @@
 
 interface ExtraTxnFeesDistributionStrategyInterface
 {
-    /**
-     * @param float $extra_fees
-     * @param EE_Registration[] $registrations
-     */
-    public function applyExtraFeesToRegistrants($extra_fees, array $registrations);
+	/**
+	 * @param float $extra_fees
+	 * @param EE_Registration[] $registrations
+	 */
+	public function applyExtraFeesToRegistrants($extra_fees, array $registrations);
 }

core/domain/services/checkout/extra_txn_fees/spco/ApplyExtraFeesToAllRegistrationsEvenly.php

Indentation +18 added lines, -18 removed lines

@@ -15,22 +15,22 @@
  */
 class ApplyExtraFeesToAllRegistrationsEvenly implements ExtraTxnFeesDistributionStrategyInterface
 {
-    /**
-     * @param float             $extra_fees
-     * @param EE_Registration[] $registrations
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function applyExtraFeesToRegistrants($extra_fees, array $registrations)
-    {
-        // divide remaining fees by number of registrations and apply to each
-        $extra_payment = $extra_fees / count($registrations);
-        foreach ($registrations as $registration) {
-            if ($registration instanceof EE_Registration) {
-                $new_registration_total = $registration->final_price() + $extra_payment;
-                $registration->set_final_price($new_registration_total);
-                $registration->save();
-            }
-        }
-    }
+	/**
+	 * @param float             $extra_fees
+	 * @param EE_Registration[] $registrations
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function applyExtraFeesToRegistrants($extra_fees, array $registrations)
+	{
+		// divide remaining fees by number of registrations and apply to each
+		$extra_payment = $extra_fees / count($registrations);
+		foreach ($registrations as $registration) {
+			if ($registration instanceof EE_Registration) {
+				$new_registration_total = $registration->final_price() + $extra_payment;
+				$registration->set_final_price($new_registration_total);
+				$registration->save();
+			}
+		}
+	}
 }

core/domain/services/checkout/extra_txn_fees/spco/ExtraTxnFeesForRegistrantsHandler.php

Indentation +44 added lines, -44 removed lines

@@ -17,51 +17,51 @@
 class ExtraTxnFeesForRegistrantsHandler extends ExtraTxnFeesHandler
 {
 
-    /**
-     * @var ExtraTxnFeesDistributionStrategyInterface
-     */
-    protected $extra_fees_strategy;
+	/**
+	 * @var ExtraTxnFeesDistributionStrategyInterface
+	 */
+	protected $extra_fees_strategy;
 
 
-    /**
-     * @return void
-     * @throws DomainException
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function applyExtraFeesToRegistrants()
-    {
-        $this->calculateExtraFeesForRegistrations($this->registrations);
-        // if there's money owing after all the individual registration fees have been subtracted
-        if ($this->extra_fees > 0) {
-            // then first decide whether to spread extra fees across all registrations,
-            // or assign everything to the primary registrant
-            $extra_fees_strategy = $this->getExtraTxnFeesDistributionStrategy();
-            $extra_fees_strategy->applyExtraFeesToRegistrants($this->extra_fees, $this->registrations);
-        }
-    }
+	/**
+	 * @return void
+	 * @throws DomainException
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function applyExtraFeesToRegistrants()
+	{
+		$this->calculateExtraFeesForRegistrations($this->registrations);
+		// if there's money owing after all the individual registration fees have been subtracted
+		if ($this->extra_fees > 0) {
+			// then first decide whether to spread extra fees across all registrations,
+			// or assign everything to the primary registrant
+			$extra_fees_strategy = $this->getExtraTxnFeesDistributionStrategy();
+			$extra_fees_strategy->applyExtraFeesToRegistrants($this->extra_fees, $this->registrations);
+		}
+	}
 
-    /**
-     * @return ExtraTxnFeesDistributionStrategyInterface
-     * @throws DomainException
-     */
-    private function getExtraTxnFeesDistributionStrategy()
-    {
-        if (! $this->extra_fees_strategy instanceof ExtraTxnFeesDistributionStrategyInterface) {
-            switch ($this->getDistributionStrategy()) {
-                case ExtraTxnFeesHandler::STRATEGY_PRIMARY_REGISTRANT_ONLY:
-                    $extra_fees_strategy = new ApplyExtraFeesToPrimaryRegistrantOnly();
-                    break;
-                case ExtraTxnFeesHandler::STRATEGY_DISTRIBUTE_EVENLY:
-                default:
-                    $extra_fees_strategy = new ApplyExtraFeesToAllRegistrationsEvenly();
-            }
-            $this->validateExtraTxnFeesDistributionStrategy(
-                $extra_fees_strategy,
-                ExtraTxnFeesDistributionStrategyInterface::class
-            );
-            $this->extra_fees_strategy = $extra_fees_strategy;
-        }
-        return $this->extra_fees_strategy;
-    }
+	/**
+	 * @return ExtraTxnFeesDistributionStrategyInterface
+	 * @throws DomainException
+	 */
+	private function getExtraTxnFeesDistributionStrategy()
+	{
+		if (! $this->extra_fees_strategy instanceof ExtraTxnFeesDistributionStrategyInterface) {
+			switch ($this->getDistributionStrategy()) {
+				case ExtraTxnFeesHandler::STRATEGY_PRIMARY_REGISTRANT_ONLY:
+					$extra_fees_strategy = new ApplyExtraFeesToPrimaryRegistrantOnly();
+					break;
+				case ExtraTxnFeesHandler::STRATEGY_DISTRIBUTE_EVENLY:
+				default:
+					$extra_fees_strategy = new ApplyExtraFeesToAllRegistrationsEvenly();
+			}
+			$this->validateExtraTxnFeesDistributionStrategy(
+				$extra_fees_strategy,
+				ExtraTxnFeesDistributionStrategyInterface::class
+			);
+			$this->extra_fees_strategy = $extra_fees_strategy;
+		}
+		return $this->extra_fees_strategy;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
      */
     private function getExtraTxnFeesDistributionStrategy()
     {
-        if (! $this->extra_fees_strategy instanceof ExtraTxnFeesDistributionStrategyInterface) {
+        if ( ! $this->extra_fees_strategy instanceof ExtraTxnFeesDistributionStrategyInterface) {
             switch ($this->getDistributionStrategy()) {
                 case ExtraTxnFeesHandler::STRATEGY_PRIMARY_REGISTRANT_ONLY:
                     $extra_fees_strategy = new ApplyExtraFeesToPrimaryRegistrantOnly();

core/domain/services/checkout/extra_txn_fees/spco/ApplyExtraFeesToPrimaryRegistrantOnly.php

Indentation +16 added lines, -16 removed lines

@@ -15,20 +15,20 @@
  */
 class ApplyExtraFeesToPrimaryRegistrantOnly implements ExtraTxnFeesDistributionStrategyInterface
 {
-    /**
-     * @param float             $extra_fees
-     * @param EE_Registration[] $registrations
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function applyExtraFeesToRegistrants($extra_fees, array $registrations)
-    {
-        foreach ($registrations as $registration) {
-            if ($registration instanceof EE_Registration && $registration->is_primary_registrant()) {
-                $primary_registrant_total = $registration->final_price() + $extra_fees;
-                $registration->set_final_price($primary_registrant_total);
-                $registration->save();
-            }
-        }
-    }
+	/**
+	 * @param float             $extra_fees
+	 * @param EE_Registration[] $registrations
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function applyExtraFeesToRegistrants($extra_fees, array $registrations)
+	{
+		foreach ($registrations as $registration) {
+			if ($registration instanceof EE_Registration && $registration->is_primary_registrant()) {
+				$primary_registrant_total = $registration->final_price() + $extra_fees;
+				$registration->set_final_price($primary_registrant_total);
+				$registration->save();
+			}
+		}
+	}
 }