eventespresso / event-espresso-core

core/domain/services/capabilities/user_caps/UserCapabilities.php

Spacing +6 added lines, -6 removed lines

@@ -36,10 +36,10 @@ 
      */
     protected function hasEntityCap(string $cap, string $context, $ID): bool
     {
-        if (! isset($this->caps[ $cap ][ $context ][ $ID ])) {
-            $this->caps[ $cap ][ $context ][ $ID ] = $this->capabilities->current_user_can($cap, $context, $ID);
+        if ( ! isset($this->caps[$cap][$context][$ID])) {
+            $this->caps[$cap][$context][$ID] = $this->capabilities->current_user_can($cap, $context, $ID);
         }
-        return $this->caps[ $cap ][ $context ][ $ID ];
+        return $this->caps[$cap][$context][$ID];
     }
 
 
@@ -50,9 +50,9 @@ 
      */
     protected function hasGlobalCap(string $cap, string $context): bool
     {
-        if (! isset($this->caps[ $cap ][ $context ])) {
-            $this->caps[ $cap ][ $context ] = $this->capabilities->current_user_can($cap, $context);
+        if ( ! isset($this->caps[$cap][$context])) {
+            $this->caps[$cap][$context] = $this->capabilities->current_user_can($cap, $context);
         }
-        return $this->caps[ $cap ][ $context ];
+        return $this->caps[$cap][$context];
     }
 }

Indentation +41 added lines, -41 removed lines

@@ -6,45 +6,45 @@
 
 abstract class UserCapabilities
 {
-    private EE_Capabilities $capabilities;
-
-    private array $caps = [];
-
-
-    /**
-     * @param EE_Capabilities $capabilities
-     */
-    public function __construct(EE_Capabilities $capabilities)
-    {
-        $this->capabilities = $capabilities;
-    }
-
-
-    /**
-     * @param string     $cap
-     * @param string     $context
-     * @param int|string $ID
-     * @return bool
-     */
-    protected function hasEntityCap(string $cap, string $context, $ID): bool
-    {
-        if (! isset($this->caps[ $cap ][ $context ][ $ID ])) {
-            $this->caps[ $cap ][ $context ][ $ID ] = $this->capabilities->current_user_can($cap, $context, $ID);
-        }
-        return $this->caps[ $cap ][ $context ][ $ID ];
-    }
-
-
-    /**
-     * @param string $cap
-     * @param string $context
-     * @return bool
-     */
-    protected function hasGlobalCap(string $cap, string $context): bool
-    {
-        if (! isset($this->caps[ $cap ][ $context ])) {
-            $this->caps[ $cap ][ $context ] = $this->capabilities->current_user_can($cap, $context);
-        }
-        return $this->caps[ $cap ][ $context ];
-    }
+	private EE_Capabilities $capabilities;
+
+	private array $caps = [];
+
+
+	/**
+	 * @param EE_Capabilities $capabilities
+	 */
+	public function __construct(EE_Capabilities $capabilities)
+	{
+		$this->capabilities = $capabilities;
+	}
+
+
+	/**
+	 * @param string     $cap
+	 * @param string     $context
+	 * @param int|string $ID
+	 * @return bool
+	 */
+	protected function hasEntityCap(string $cap, string $context, $ID): bool
+	{
+		if (! isset($this->caps[ $cap ][ $context ][ $ID ])) {
+			$this->caps[ $cap ][ $context ][ $ID ] = $this->capabilities->current_user_can($cap, $context, $ID);
+		}
+		return $this->caps[ $cap ][ $context ][ $ID ];
+	}
+
+
+	/**
+	 * @param string $cap
+	 * @param string $context
+	 * @return bool
+	 */
+	protected function hasGlobalCap(string $cap, string $context): bool
+	{
+		if (! isset($this->caps[ $cap ][ $context ])) {
+			$this->caps[ $cap ][ $context ] = $this->capabilities->current_user_can($cap, $context);
+		}
+		return $this->caps[ $cap ][ $context ];
+	}
 }

payment_methods/Invoice/templates/invoice_settings_header_gateway.template.php

Indentation +3 added lines, -3 removed lines

@@ -17,9 +17,9 @@
         <td>
             <h4 class="ee-status-outline ee-status-bg--info">
                 <?php esc_html_e(
-                    "The following settings affect the functioning of the Invoice gateway.",
-                    'event_espresso'
-                ); ?>
+					"The following settings affect the functioning of the Invoice gateway.",
+					'event_espresso'
+				); ?>
             </h4>
         </td>
     </tr>

payment_methods/Invoice/templates/invoice_settings_header_display.template.php

Indentation +3 added lines, -3 removed lines

@@ -17,9 +17,9 @@
         <td>
             <h4 class="ee-status-outline ee-status-bg--info">
                 <?php esc_html_e(
-                    "The following settings affect the content and/or appearance of the downloadable PDF invoice.",
-                    'event_espresso'
-                ); ?>
+					"The following settings affect the content and/or appearance of the downloadable PDF invoice.",
+					'event_espresso'
+				); ?>
             </h4>
         </td>
     </tr>

caffeinated/admin/extend/registration_form/Extend_Registration_Form_Questions_Admin_List_Table.class.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      */
     public function column_display_text(EE_Question $question, bool $prep_content = true): string
     {
-        $actions         = [];
+        $actions = [];
 
         if ($this->caps_handler->userCanEditQuestion($question)) {
             $actions['edit'] = $this->getActionLink(

Indentation +65 added lines, -65 removed lines

@@ -13,74 +13,74 @@
  */
 class Extend_Registration_Form_Questions_Admin_List_Table extends Registration_Form_Questions_Admin_List_Table
 {
-    /**
-     * @param EE_Question $question
-     * @param bool        $prep_content
-     * @return string
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function column_display_text(EE_Question $question, bool $prep_content = true): string
-    {
-        $actions         = [];
+	/**
+	 * @param EE_Question $question
+	 * @param bool        $prep_content
+	 * @return string
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function column_display_text(EE_Question $question, bool $prep_content = true): string
+	{
+		$actions         = [];
 
-        if ($this->caps_handler->userCanEditQuestion($question)) {
-            $actions['edit'] = $this->getActionLink(
-                $this->getActionUrl($question, self::ACTION_EDIT),
-                esc_html__('Edit', 'event_espresso'),
-                esc_attr__('Edit Question', 'event_espresso')
-            );
-        }
+		if ($this->caps_handler->userCanEditQuestion($question)) {
+			$actions['edit'] = $this->getActionLink(
+				$this->getActionUrl($question, self::ACTION_EDIT),
+				esc_html__('Edit', 'event_espresso'),
+				esc_attr__('Edit Question', 'event_espresso')
+			);
+		}
 
-        if (
-            $this->_view != 'trash'
-            && ! $question->is_system_question()
-            && $this->caps_handler->userCanTrashQuestion($question)
-        ) {
-            $actions['delete'] = $this->getActionLink(
-                $this->getActionUrl($question, self::ACTION_TRASH),
-                esc_html__('Trash', 'event_espresso'),
-                esc_attr__('Trash Question', 'event_espresso')
-            );
-        }
+		if (
+			$this->_view != 'trash'
+			&& ! $question->is_system_question()
+			&& $this->caps_handler->userCanTrashQuestion($question)
+		) {
+			$actions['delete'] = $this->getActionLink(
+				$this->getActionUrl($question, self::ACTION_TRASH),
+				esc_html__('Trash', 'event_espresso'),
+				esc_attr__('Trash Question', 'event_espresso')
+			);
+		}
 
-        if ($this->_view == 'trash') {
-            if ($this->caps_handler->userCanRestoreQuestion($question)) {
-                $actions['restore'] = $this->getActionLink(
-                    $this->getActionUrl($question, self::ACTION_RESTORE),
-                    esc_html__('Restore', 'event_espresso'),
-                    esc_attr__('Restore Question', 'event_espresso')
-                );
-            }
-            if (
-                $question->count_related('Answer') === 0
-                && $this->caps_handler->userCanDeleteQuestion($question)
-            ) {
-                $actions['delete'] = $this->getActionLink(
-                    $this->getActionUrl($question, self::ACTION_DELETE),
-                    esc_html__('Delete Permanently', 'event_espresso'),
-                    esc_attr__('Delete Question Permanently', 'event_espresso')
-                );
-            }
-        }
-        if ($this->caps_handler->userCanEditQuestion($question)) {
-            $actions['duplicate'] = $this->getActionLink(
-                $this->getActionUrl($question, self::ACTION_COPY),
-                esc_html__('Duplicate', 'event_espresso'),
-                esc_attr__('Duplicate Question', 'event_espresso')
-            );
-        }
+		if ($this->_view == 'trash') {
+			if ($this->caps_handler->userCanRestoreQuestion($question)) {
+				$actions['restore'] = $this->getActionLink(
+					$this->getActionUrl($question, self::ACTION_RESTORE),
+					esc_html__('Restore', 'event_espresso'),
+					esc_attr__('Restore Question', 'event_espresso')
+				);
+			}
+			if (
+				$question->count_related('Answer') === 0
+				&& $this->caps_handler->userCanDeleteQuestion($question)
+			) {
+				$actions['delete'] = $this->getActionLink(
+					$this->getActionUrl($question, self::ACTION_DELETE),
+					esc_html__('Delete Permanently', 'event_espresso'),
+					esc_attr__('Delete Question Permanently', 'event_espresso')
+				);
+			}
+		}
+		if ($this->caps_handler->userCanEditQuestion($question)) {
+			$actions['duplicate'] = $this->getActionLink(
+				$this->getActionUrl($question, self::ACTION_COPY),
+				esc_html__('Duplicate', 'event_espresso'),
+				esc_attr__('Duplicate Question', 'event_espresso')
+			);
+		}
 
-        $content = $this->caps_handler->userCanEditQuestion($question)
-            ? $this->getActionLink(
-                $this->getActionUrl($question, self::ACTION_EDIT),
-                $prep_content ? $question->display_text() : $question->admin_label(),
-                esc_attr__('Edit Question', 'event_espresso'),
-                'row-title'
-            )
-            : $question->display_text();
-        $content .= $this->row_actions($actions);
+		$content = $this->caps_handler->userCanEditQuestion($question)
+			? $this->getActionLink(
+				$this->getActionUrl($question, self::ACTION_EDIT),
+				$prep_content ? $question->display_text() : $question->admin_label(),
+				esc_attr__('Edit Question', 'event_espresso'),
+				'row-title'
+			)
+			: $question->display_text();
+		$content .= $this->row_actions($actions);
 
-        return $prep_content ? $this->columnContent('display_text', $content) : $content;
-    }
+		return $prep_content ? $this->columnContent('display_text', $content) : $content;
+	}
 }

caffeinated/admin/extend/registration_form/Registration_Form_Question_Groups_Admin_List_Table.class.php

Spacing +7 added lines, -7 removed lines

@@ -27,7 +27,7 @@ 
         $this->caps_handler = new RegFormListTableUserCapabilities(EE_Registry::instance()->CAP);
         parent::__construct($admin_page);
 
-        if (! defined('REG_ADMIN_URL')) {
+        if ( ! defined('REG_ADMIN_URL')) {
             define('REG_ADMIN_URL', EVENTS_ADMIN_URL);
         }
     }
@@ -121,7 +121,7 @@ 
             )
             : $notice;
         return $system_group || $has_questions_with_answers || ! $user_can_trash_it
-            ? '<span class="dashicons dashicons-lock ee-locked-entity ee-aria-tooltip" aria-label="' . $notice . '"></span>'
+            ? '<span class="dashicons dashicons-lock ee-locked-entity ee-aria-tooltip" aria-label="'.$notice.'"></span>'
               . sprintf(
                   '<input type="hidden" name="hdnchk[%1$d]" value="%1$d" />',
                   $item->ID()
@@ -136,9 +136,9 @@ 
     public function column_id(EE_Question_Group $item)
     {
         $content = '
-            <span class="ee-entity-id">' . $item->ID() . '</span>
+            <span class="ee-entity-id">' . $item->ID().'</span>
             <span class="show-on-mobile-view-only">
-                ' . $item->name() . '
+                ' . $item->name().'
             </span>';
         return $this->columnContent('id', $content, 'end');
     }
@@ -216,7 +216,7 @@ 
      */
     protected function getActionUrl(EE_Question_Group $question_group, $action): string
     {
-        if (! in_array($action, self::$actions)) {
+        if ( ! in_array($action, self::$actions)) {
             throw new DomainException(esc_html__('Invalid Action', 'event_espresso'));
         }
         return EE_Admin_Page::add_query_args_and_nonce(
@@ -244,12 +244,12 @@ 
 
     public function column_show_group_name(EE_Question_Group $question_group): string
     {
-        return $this->columnContent('show_group_name', $this->_yes_no[ $question_group->show_group_name() ]);
+        return $this->columnContent('show_group_name', $this->_yes_no[$question_group->show_group_name()]);
     }
 
 
     public function column_show_group_desc(EE_Question_Group $question_group): string
     {
-        return $this->columnContent('show_group_desc', $this->_yes_no[ $question_group->show_group_desc() ]);
+        return $this->columnContent('show_group_desc', $this->_yes_no[$question_group->show_group_desc()]);
     }
 }

Indentation +232 added lines, -232 removed lines

@@ -15,240 +15,240 @@
  */
 class Registration_Form_Question_Groups_Admin_List_Table extends EE_Admin_List_Table
 {
-    /**
-     * @var RegFormListTableUserCapabilities
-     */
-    protected $caps_handler;
-
-
-    public function __construct($admin_page)
-    {
-        $this->caps_handler = new RegFormListTableUserCapabilities(EE_Registry::instance()->CAP);
-        parent::__construct($admin_page);
-
-        if (! defined('REG_ADMIN_URL')) {
-            define('REG_ADMIN_URL', EVENTS_ADMIN_URL);
-        }
-    }
-
-
-    protected function _setup_data()
-    {
-        $this->_data = $this->_view != 'trash'
-            ? $this->_admin_page->get_question_groups($this->_per_page, $this->_current_page, false)
-            : $this->_admin_page->get_trashed_question_groups($this->_per_page, $this->_current_page, false);
-        $this->_all_data_count = $this->_view != 'trash'
-            ? $this->_admin_page->get_question_groups($this->_per_page, $this->_current_page, true)
-            : $this->_admin_page->get_trashed_question_groups($this->_per_page, $this->_current_page, true);
-    }
-
-
-    protected function _set_properties()
-    {
-        $this->_wp_list_args = array(
-            'singular' => esc_html__('question group', 'event_espresso'),
-            'plural'   => esc_html__('question groups', 'event_espresso'),
-            'ajax'     => true, // for now,
-            'screen'   => $this->_admin_page->get_current_screen()->id,
-        );
-
-        $this->_columns = array(
-            'cb'              => '<input type="checkbox" />',
-            'id'              => esc_html__('ID', 'event_espresso'),
-            'name'            => esc_html__('Group Name', 'event_espresso'),
-            'description'     => esc_html__('Description', 'event_espresso'),
-            'show_group_name' => esc_html__('Show Name', 'event_espresso'),
-            'show_group_desc' => esc_html__('Show Desc', 'event_espresso'),
-        );
-
-        $this->_sortable_columns = array(
-            'id'   => array('QSG_ID' => false),
-            'name' => array('QSG_name' => false),
-        );
-
-        $this->_hidden_columns = array(
-            'id',
-        );
-
-        $this->_ajax_sorting_callback = 'update_question_group_order';
-    }
-
-
-    // not needed
-    protected function _get_table_filters()
-    {
-        return [];
-    }
-
-
-    protected function _add_view_counts()
-    {
-        $this->_views['all']['count'] = $this->_admin_page->get_question_groups(
-            $this->_per_page,
-            $this->_current_page,
-            true
-        );
-        if ($this->caps_handler->userCanTrashQuestionGroups()) {
-            $this->_views['trash']['count'] = $this->_admin_page->get_trashed_question_groups(
-                $this->_per_page,
-                $this->_current_page,
-                true
-            );
-        }
-    }
-
-
-    /**
-     * @throws ReflectionException
-     * @throws EE_Error
-     */
-    public function column_cb($item)
-    {
-        $system_group = $item->get('QSG_system');
-        $user_can_trash_it = $this->caps_handler->userCanTrashQuestionGroup($item);
-        $has_questions_with_answers = ! $system_group && $this->_view == 'trash' && $item->has_questions_with_answers();
-        $notice = ! $user_can_trash_it
-            ? esc_html__('You do not have the required permissions to trash this question group', 'event_espresso')
-            : '';
-        $notice = $has_questions_with_answers
-            ? esc_html__('This question group is a system group and cannot be trashed', 'event_espresso')
-            : $notice;
-        $notice = $has_questions_with_answers
-            ? esc_html__(
-                'This question group has questions that have answers attached to it from registrations that have the question. It cannot be permanently deleted.',
-                'event_espresso'
-            )
-            : $notice;
-        return $system_group || $has_questions_with_answers || ! $user_can_trash_it
-            ? '<span class="dashicons dashicons-lock ee-locked-entity ee-aria-tooltip" aria-label="' . $notice . '"></span>'
-              . sprintf(
-                  '<input type="hidden" name="hdnchk[%1$d]" value="%1$d" />',
-                  $item->ID()
-              )
-            : sprintf(
-                '<input type="checkbox" id="QSG_ID[%1$d]" name="checkbox[%1$d]" value="%1$d" />',
-                $item->ID()
-            );
-    }
-
-
-    public function column_id(EE_Question_Group $item)
-    {
-        $content = '
+	/**
+	 * @var RegFormListTableUserCapabilities
+	 */
+	protected $caps_handler;
+
+
+	public function __construct($admin_page)
+	{
+		$this->caps_handler = new RegFormListTableUserCapabilities(EE_Registry::instance()->CAP);
+		parent::__construct($admin_page);
+
+		if (! defined('REG_ADMIN_URL')) {
+			define('REG_ADMIN_URL', EVENTS_ADMIN_URL);
+		}
+	}
+
+
+	protected function _setup_data()
+	{
+		$this->_data = $this->_view != 'trash'
+			? $this->_admin_page->get_question_groups($this->_per_page, $this->_current_page, false)
+			: $this->_admin_page->get_trashed_question_groups($this->_per_page, $this->_current_page, false);
+		$this->_all_data_count = $this->_view != 'trash'
+			? $this->_admin_page->get_question_groups($this->_per_page, $this->_current_page, true)
+			: $this->_admin_page->get_trashed_question_groups($this->_per_page, $this->_current_page, true);
+	}
+
+
+	protected function _set_properties()
+	{
+		$this->_wp_list_args = array(
+			'singular' => esc_html__('question group', 'event_espresso'),
+			'plural'   => esc_html__('question groups', 'event_espresso'),
+			'ajax'     => true, // for now,
+			'screen'   => $this->_admin_page->get_current_screen()->id,
+		);
+
+		$this->_columns = array(
+			'cb'              => '<input type="checkbox" />',
+			'id'              => esc_html__('ID', 'event_espresso'),
+			'name'            => esc_html__('Group Name', 'event_espresso'),
+			'description'     => esc_html__('Description', 'event_espresso'),
+			'show_group_name' => esc_html__('Show Name', 'event_espresso'),
+			'show_group_desc' => esc_html__('Show Desc', 'event_espresso'),
+		);
+
+		$this->_sortable_columns = array(
+			'id'   => array('QSG_ID' => false),
+			'name' => array('QSG_name' => false),
+		);
+
+		$this->_hidden_columns = array(
+			'id',
+		);
+
+		$this->_ajax_sorting_callback = 'update_question_group_order';
+	}
+
+
+	// not needed
+	protected function _get_table_filters()
+	{
+		return [];
+	}
+
+
+	protected function _add_view_counts()
+	{
+		$this->_views['all']['count'] = $this->_admin_page->get_question_groups(
+			$this->_per_page,
+			$this->_current_page,
+			true
+		);
+		if ($this->caps_handler->userCanTrashQuestionGroups()) {
+			$this->_views['trash']['count'] = $this->_admin_page->get_trashed_question_groups(
+				$this->_per_page,
+				$this->_current_page,
+				true
+			);
+		}
+	}
+
+
+	/**
+	 * @throws ReflectionException
+	 * @throws EE_Error
+	 */
+	public function column_cb($item)
+	{
+		$system_group = $item->get('QSG_system');
+		$user_can_trash_it = $this->caps_handler->userCanTrashQuestionGroup($item);
+		$has_questions_with_answers = ! $system_group && $this->_view == 'trash' && $item->has_questions_with_answers();
+		$notice = ! $user_can_trash_it
+			? esc_html__('You do not have the required permissions to trash this question group', 'event_espresso')
+			: '';
+		$notice = $has_questions_with_answers
+			? esc_html__('This question group is a system group and cannot be trashed', 'event_espresso')
+			: $notice;
+		$notice = $has_questions_with_answers
+			? esc_html__(
+				'This question group has questions that have answers attached to it from registrations that have the question. It cannot be permanently deleted.',
+				'event_espresso'
+			)
+			: $notice;
+		return $system_group || $has_questions_with_answers || ! $user_can_trash_it
+			? '<span class="dashicons dashicons-lock ee-locked-entity ee-aria-tooltip" aria-label="' . $notice . '"></span>'
+			  . sprintf(
+				  '<input type="hidden" name="hdnchk[%1$d]" value="%1$d" />',
+				  $item->ID()
+			  )
+			: sprintf(
+				'<input type="checkbox" id="QSG_ID[%1$d]" name="checkbox[%1$d]" value="%1$d" />',
+				$item->ID()
+			);
+	}
+
+
+	public function column_id(EE_Question_Group $item)
+	{
+		$content = '
             <span class="ee-entity-id">' . $item->ID() . '</span>
             <span class="show-on-mobile-view-only">
                 ' . $item->name() . '
             </span>';
-        return $this->columnContent('id', $content, 'end');
-    }
-
-
-    /**
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function column_name(EE_Question_Group $question_group, bool $prep_content = true): string
-    {
-        $actions = array();
-
-        if ($this->caps_handler->userCanEditQuestionGroup($question_group)) {
-            $actions['edit'] = $this->getActionLink(
-                $this->getActionUrl($question_group, self::ACTION_EDIT),
-                esc_html__('Edit', 'event_espresso'),
-                esc_attr__('Edit Question Group', 'event_espresso')
-            );
-        }
-        if (
-            $question_group->get('QSG_system') < 1
-            && $this->_view != 'trash'
-            && $this->caps_handler->userCanEditQuestionGroup($question_group)
-        ) {
-            $actions['delete'] = $this->getActionLink(
-                $this->getActionUrl($question_group, self::ACTION_TRASH),
-                esc_html__('Trash', 'event_espresso'),
-                esc_attr__('Trash Question Group', 'event_espresso')
-            );
-        }
-
-        if ($this->_view == 'trash') {
-            if ($this->caps_handler->userCanRestoreQuestionGroup($question_group)) {
-                $actions['restore'] = $this->getActionLink(
-                    $this->getActionUrl($question_group, self::ACTION_RESTORE),
-                    esc_html__('Restore', 'event_espresso'),
-                    esc_attr__('Restore Question Group', 'event_espresso')
-                );
-            }
-
-            if (
-                ! $question_group->has_questions_with_answers()
-                && $this->caps_handler->userCanDeleteQuestionGroup($question_group)
-            ) {
-                $actions['delete'] = $this->getActionLink(
-                    $this->getActionUrl($question_group, self::ACTION_DELETE),
-                    esc_html__('Delete Permanently', 'event_espresso'),
-                    esc_attr__('Delete Question Group Permanently', 'event_espresso')
-                );
-            }
-        }
-
-        $content = $this->caps_handler->userCanEditQuestionGroup($question_group)
-            ? $this->getActionLink(
-                $this->getActionUrl($question_group, self::ACTION_EDIT),
-                $question_group->name(),
-                esc_attr__('Edit Question Group', 'event_espresso'),
-                'row-title'
-            )
-            : $question_group->name();
-        $content .= $this->row_actions($actions);
-
-        return $prep_content ? $this->columnContent('name', $content) : $content;
-    }
-
-
-    /**
-     * @param EE_Question_Group $question_group
-     * @param                   $action
-     * @return string
-     * @throws EE_Error
-     * @throws ReflectionException
-     * @since 5.0.0.p
-     */
-    protected function getActionUrl(EE_Question_Group $question_group, $action): string
-    {
-        if (! in_array($action, self::$actions)) {
-            throw new DomainException(esc_html__('Invalid Action', 'event_espresso'));
-        }
-        return EE_Admin_Page::add_query_args_and_nonce(
-            [
-                'action'   => "{$action}_question_group",
-                'QSG_ID'   => $question_group->ID(),
-                'noheader' => $action !== self::ACTION_EDIT,
-            ],
-            EE_FORMS_ADMIN_URL
-        );
-    }
-
-
-    public function column_identifier(EE_Question_Group $question_group): string
-    {
-        return $this->columnContent('identifier', $question_group->identifier());
-    }
-
-
-    public function column_description(EE_Question_Group $question_group): string
-    {
-        return $this->columnContent('description', $question_group->desc());
-    }
-
-
-    public function column_show_group_name(EE_Question_Group $question_group): string
-    {
-        return $this->columnContent('show_group_name', $this->_yes_no[ $question_group->show_group_name() ]);
-    }
-
-
-    public function column_show_group_desc(EE_Question_Group $question_group): string
-    {
-        return $this->columnContent('show_group_desc', $this->_yes_no[ $question_group->show_group_desc() ]);
-    }
+		return $this->columnContent('id', $content, 'end');
+	}
+
+
+	/**
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function column_name(EE_Question_Group $question_group, bool $prep_content = true): string
+	{
+		$actions = array();
+
+		if ($this->caps_handler->userCanEditQuestionGroup($question_group)) {
+			$actions['edit'] = $this->getActionLink(
+				$this->getActionUrl($question_group, self::ACTION_EDIT),
+				esc_html__('Edit', 'event_espresso'),
+				esc_attr__('Edit Question Group', 'event_espresso')
+			);
+		}
+		if (
+			$question_group->get('QSG_system') < 1
+			&& $this->_view != 'trash'
+			&& $this->caps_handler->userCanEditQuestionGroup($question_group)
+		) {
+			$actions['delete'] = $this->getActionLink(
+				$this->getActionUrl($question_group, self::ACTION_TRASH),
+				esc_html__('Trash', 'event_espresso'),
+				esc_attr__('Trash Question Group', 'event_espresso')
+			);
+		}
+
+		if ($this->_view == 'trash') {
+			if ($this->caps_handler->userCanRestoreQuestionGroup($question_group)) {
+				$actions['restore'] = $this->getActionLink(
+					$this->getActionUrl($question_group, self::ACTION_RESTORE),
+					esc_html__('Restore', 'event_espresso'),
+					esc_attr__('Restore Question Group', 'event_espresso')
+				);
+			}
+
+			if (
+				! $question_group->has_questions_with_answers()
+				&& $this->caps_handler->userCanDeleteQuestionGroup($question_group)
+			) {
+				$actions['delete'] = $this->getActionLink(
+					$this->getActionUrl($question_group, self::ACTION_DELETE),
+					esc_html__('Delete Permanently', 'event_espresso'),
+					esc_attr__('Delete Question Group Permanently', 'event_espresso')
+				);
+			}
+		}
+
+		$content = $this->caps_handler->userCanEditQuestionGroup($question_group)
+			? $this->getActionLink(
+				$this->getActionUrl($question_group, self::ACTION_EDIT),
+				$question_group->name(),
+				esc_attr__('Edit Question Group', 'event_espresso'),
+				'row-title'
+			)
+			: $question_group->name();
+		$content .= $this->row_actions($actions);
+
+		return $prep_content ? $this->columnContent('name', $content) : $content;
+	}
+
+
+	/**
+	 * @param EE_Question_Group $question_group
+	 * @param                   $action
+	 * @return string
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 * @since 5.0.0.p
+	 */
+	protected function getActionUrl(EE_Question_Group $question_group, $action): string
+	{
+		if (! in_array($action, self::$actions)) {
+			throw new DomainException(esc_html__('Invalid Action', 'event_espresso'));
+		}
+		return EE_Admin_Page::add_query_args_and_nonce(
+			[
+				'action'   => "{$action}_question_group",
+				'QSG_ID'   => $question_group->ID(),
+				'noheader' => $action !== self::ACTION_EDIT,
+			],
+			EE_FORMS_ADMIN_URL
+		);
+	}
+
+
+	public function column_identifier(EE_Question_Group $question_group): string
+	{
+		return $this->columnContent('identifier', $question_group->identifier());
+	}
+
+
+	public function column_description(EE_Question_Group $question_group): string
+	{
+		return $this->columnContent('description', $question_group->desc());
+	}
+
+
+	public function column_show_group_name(EE_Question_Group $question_group): string
+	{
+		return $this->columnContent('show_group_name', $this->_yes_no[ $question_group->show_group_name() ]);
+	}
+
+
+	public function column_show_group_desc(EE_Question_Group $question_group): string
+	{
+		return $this->columnContent('show_group_desc', $this->_yes_no[ $question_group->show_group_desc() ]);
+	}
 }

core/admin/templates/admin_details_metabox_column_wrapper.template.php

Indentation +5 added lines, -5 removed lines

@@ -19,11 +19,11 @@
         </div> <!-- post-body-content -->
 
         <?php
-        // let's loop through the columns
-        for ($i = 1; $i <= $num_columns; $i++) {
-            $context = ($i === 1) ? 'normal' : 'side';
-            $context = ($i > 2) ? 'column' . $i : $context;
-            ?>
+		// let's loop through the columns
+		for ($i = 1; $i <= $num_columns; $i++) {
+			$context = ($i === 1) ? 'normal' : 'side';
+			$context = ($i > 2) ? 'column' . $i : $context;
+			?>
         <div id='postbox-container-<?php echo absint($i); ?>' class='postbox-container'>
             <?php do_meta_boxes($current_page, $context, null); ?>
         </div>

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
         // let's loop through the columns
         for ($i = 1; $i <= $num_columns; $i++) {
             $context = ($i === 1) ? 'normal' : 'side';
-            $context = ($i > 2) ? 'column' . $i : $context;
+            $context = ($i > 2) ? 'column'.$i : $context;
             ?>
         <div id='postbox-container-<?php echo absint($i); ?>' class='postbox-container'>
             <?php do_meta_boxes($current_page, $context, null); ?>

core/db_classes/EE_Checkin.class.php

Indentation +127 added lines, -127 removed lines

@@ -9,131 +9,131 @@
  */
 class EE_Checkin extends EE_Base_Class
 {
-    /**
-     * Registration can NOT be checked in or out
-     *
-     * @type int
-     */
-    const status_invalid = -1;
-
-    /**
-     * Registration has been checked out.
-     *
-     * @type int
-     */
-    const status_checked_out = 0;
-
-    /**
-     * Registration has been checked in.
-     *
-     * @type int
-     */
-    const status_checked_in = 1;
-
-    /**
-     * Registration has never been checked in.
-     *
-     * @type int
-     */
-    const status_checked_never = 2;
-
-
-    /**
-     *
-     * @param array|null  $props_n_values incoming values
-     * @param string|null $timezone       incoming timezone (if not set the timezone set for the website will be used.)
-     * @param array|null  $date_formats   incoming date_formats in an array
-     *                                    where the first value is the date_format
-     *                                    and the second value is the time format
-     * @return EE_Checkin
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public static function new_instance(
-        ?array $props_n_values = [],
-        ?string $timezone = '',
-        ?array $date_formats = []
-    ): EE_Checkin {
-        $has_object = parent::_check_for_object($props_n_values, __CLASS__, $timezone, $date_formats);
-        return $has_object ?: new self($props_n_values, false, $timezone, $date_formats);
-    }
-
-
-    /**
-     * @param array|null  $props_n_values incoming values from the database
-     * @param string|null $timezone       incoming timezone as set by the model.  If not set the timezone for
-     *                                    the website will be used.
-     * @return EE_Checkin
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public static function new_instance_from_db(?array $props_n_values = [], ?string $timezone = ''): EE_Checkin
-    {
-        return new self($props_n_values, true, $timezone);
-    }
-
-
-    public function ID()
-    {
-        return $this->get('CHK_ID');
-    }
-
-
-    /**
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function registration_id(): int
-    {
-        return (int) $this->get('REG_ID');
-    }
-
-
-    /**
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function datetime_id(): int
-    {
-        return (int) $this->get('DTT_ID');
-    }
-
-
-    /**
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function status(): int
-    {
-        return (int) $this->get('CHK_in');
-    }
-
-
-    /**
-     * @throws ReflectionException
-     * @throws EE_Error
-     */
-    public function timestamp()
-    {
-        return $this->get('CHK_timestamp');
-    }
-
-
-    /**
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function getCheckInText(): string
-    {
-        switch ($this->status()) {
-            case EE_Checkin::status_checked_in:
-                return esc_html__('Checked In', 'event_espresso');
-            case EE_Checkin::status_checked_out:
-                return esc_html__('Checked Out', 'event_espresso');
-            case EE_Checkin::status_checked_never:
-                return esc_html__('Never Checked In', 'event_espresso');
-            default:
-                return esc_html__('Can Not Check-in', 'event_espresso');
-        }
-    }
+	/**
+	 * Registration can NOT be checked in or out
+	 *
+	 * @type int
+	 */
+	const status_invalid = -1;
+
+	/**
+	 * Registration has been checked out.
+	 *
+	 * @type int
+	 */
+	const status_checked_out = 0;
+
+	/**
+	 * Registration has been checked in.
+	 *
+	 * @type int
+	 */
+	const status_checked_in = 1;
+
+	/**
+	 * Registration has never been checked in.
+	 *
+	 * @type int
+	 */
+	const status_checked_never = 2;
+
+
+	/**
+	 *
+	 * @param array|null  $props_n_values incoming values
+	 * @param string|null $timezone       incoming timezone (if not set the timezone set for the website will be used.)
+	 * @param array|null  $date_formats   incoming date_formats in an array
+	 *                                    where the first value is the date_format
+	 *                                    and the second value is the time format
+	 * @return EE_Checkin
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public static function new_instance(
+		?array $props_n_values = [],
+		?string $timezone = '',
+		?array $date_formats = []
+	): EE_Checkin {
+		$has_object = parent::_check_for_object($props_n_values, __CLASS__, $timezone, $date_formats);
+		return $has_object ?: new self($props_n_values, false, $timezone, $date_formats);
+	}
+
+
+	/**
+	 * @param array|null  $props_n_values incoming values from the database
+	 * @param string|null $timezone       incoming timezone as set by the model.  If not set the timezone for
+	 *                                    the website will be used.
+	 * @return EE_Checkin
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public static function new_instance_from_db(?array $props_n_values = [], ?string $timezone = ''): EE_Checkin
+	{
+		return new self($props_n_values, true, $timezone);
+	}
+
+
+	public function ID()
+	{
+		return $this->get('CHK_ID');
+	}
+
+
+	/**
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function registration_id(): int
+	{
+		return (int) $this->get('REG_ID');
+	}
+
+
+	/**
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function datetime_id(): int
+	{
+		return (int) $this->get('DTT_ID');
+	}
+
+
+	/**
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function status(): int
+	{
+		return (int) $this->get('CHK_in');
+	}
+
+
+	/**
+	 * @throws ReflectionException
+	 * @throws EE_Error
+	 */
+	public function timestamp()
+	{
+		return $this->get('CHK_timestamp');
+	}
+
+
+	/**
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function getCheckInText(): string
+	{
+		switch ($this->status()) {
+			case EE_Checkin::status_checked_in:
+				return esc_html__('Checked In', 'event_espresso');
+			case EE_Checkin::status_checked_out:
+				return esc_html__('Checked Out', 'event_espresso');
+			case EE_Checkin::status_checked_never:
+				return esc_html__('Never Checked In', 'event_espresso');
+			default:
+				return esc_html__('Can Not Check-in', 'event_espresso');
+		}
+	}
 }

core/helpers/EEH_Money.helper.php

Spacing +5 added lines, -5 removed lines

@@ -171,9 +171,9 @@ 
         $format          = 'f';
         $currency_config = EE_Currency_Config::getCurrencyConfig($CNT_ISO);
         // first get the decimal place and number of places
-        $format = "%'." . $currency_config->dec_plc . $format;
+        $format = "%'.".$currency_config->dec_plc.$format;
         // currency symbol on right side.
-        return $currency_config->sign_b4 ? $currency_config->sign . $format : $format . $currency_config->sign;
+        return $currency_config->sign_b4 ? $currency_config->sign.$format : $format.$currency_config->sign;
     }
 
 
@@ -194,10 +194,10 @@ 
         $currency_config            = EE_Currency_Config::getCurrencyConfig($CNT_ISO);
         $decimal_places_placeholder = str_pad('', $currency_config->dec_plc, '0');
         // first get the decimal place and number of places
-        $format = '#,##0.' . $decimal_places_placeholder;
+        $format = '#,##0.'.$decimal_places_placeholder;
         // currency symbol on right side.
-        $format          = $currency_config->sign_b4
-            ? $currency_config->sign . $format
+        $format = $currency_config->sign_b4
+            ? $currency_config->sign.$format
             : $format
               . $currency_config->sign;
         $formatterObject = [

Indentation +220 added lines, -220 removed lines

@@ -10,236 +10,236 @@
  */
 class EEH_Money extends EEH_Base
 {
-    /**
-     * This removes all localized money formatting from the incoming value
-     * Note: uses this site's currency settings for deciding what is considered a
-     * "thousands separator" (usually the character "," )
-     * and what is a "decimal mark" (usually the character ".")
-     *
-     * @param int|float|string $money_value
-     * @param string|null      $CNT_ISO
-     * @return float
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public static function strip_localized_money_formatting($money_value, ?string $CNT_ISO = ''): float
-    {
-        $currency_config = EE_Currency_Config::getCurrencyConfig($CNT_ISO);
-        $money_value     = str_replace(
-            [
-                $currency_config->thsnds,
-                $currency_config->dec_mrk,
-            ],
-            [
-                '', // remove thousands separator
-                '.', // convert decimal mark to what PHP expects
-            ],
-            $money_value
-        );
-        return (float) filter_var(
-            $money_value,
-            FILTER_SANITIZE_NUMBER_FLOAT,
-            FILTER_FLAG_ALLOW_FRACTION
-        );
-    }
+	/**
+	 * This removes all localized money formatting from the incoming value
+	 * Note: uses this site's currency settings for deciding what is considered a
+	 * "thousands separator" (usually the character "," )
+	 * and what is a "decimal mark" (usually the character ".")
+	 *
+	 * @param int|float|string $money_value
+	 * @param string|null      $CNT_ISO
+	 * @return float
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public static function strip_localized_money_formatting($money_value, ?string $CNT_ISO = ''): float
+	{
+		$currency_config = EE_Currency_Config::getCurrencyConfig($CNT_ISO);
+		$money_value     = str_replace(
+			[
+				$currency_config->thsnds,
+				$currency_config->dec_mrk,
+			],
+			[
+				'', // remove thousands separator
+				'.', // convert decimal mark to what PHP expects
+			],
+			$money_value
+		);
+		return (float) filter_var(
+			$money_value,
+			FILTER_SANITIZE_NUMBER_FLOAT,
+			FILTER_FLAG_ALLOW_FRACTION
+		);
+	}
 
 
-    /**
-     * This converts an incoming localized money value into a standard float item (to three decimal places)
-     * Only use this if you know the $money_value follows your currency configuration's
-     * settings. Note: this uses this site's currency settings for deciding what is considered a
-     * "thousands separator" (usually the character "," )
-     * and what is a "decimal mark" (usually the character ".")
-     *
-     * @param int|float|string $money_value
-     * @return float
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public static function convert_to_float_from_localized_money($money_value): float
-    {
-        // float it! and round to three decimal places
-        return round(EEH_Money::strip_localized_money_formatting($money_value), 3);
-    }
+	/**
+	 * This converts an incoming localized money value into a standard float item (to three decimal places)
+	 * Only use this if you know the $money_value follows your currency configuration's
+	 * settings. Note: this uses this site's currency settings for deciding what is considered a
+	 * "thousands separator" (usually the character "," )
+	 * and what is a "decimal mark" (usually the character ".")
+	 *
+	 * @param int|float|string $money_value
+	 * @return float
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public static function convert_to_float_from_localized_money($money_value): float
+	{
+		// float it! and round to three decimal places
+		return round(EEH_Money::strip_localized_money_formatting($money_value), 3);
+	}
 
 
-    /**
-     * For comparing floats. Default operator is '=', but see the $operator below for all options.
-     * This should be used to compare floats instead of normal '==' because floats
-     * are inherently imprecise, and so you can sometimes have two floats that appear to be identical
-     * but actually differ by 0.00000001.
-     *
-     * @see http://biostall.com/php-function-to-compare-floating-point-numbers
-     * @param int|float|string $float1
-     * @param int|float|string $float2
-     * @param string|null      $operator The operator. Valid options are =, <=, <, >=, >, <>, eq, lt, lte, gt, gte, ne
-     * @return bool whether the equation is true or false
-     * @throws EE_Error
-     */
-    public static function compare_floats($float1, $float2, ?string $operator = '='): bool
-    {
-        // Check numbers to 5 digits of precision
-        $epsilon = 0.00001;
-        $float1  = (float) $float1;
-        $float2  = (float) $float2;
-        switch ($operator) {
-            // equal
-            case '=':
-            case '==':
-            case '===':
-            case 'eq':
-                if (abs($float1 - $float2) < $epsilon) {
-                    return true;
-                }
-                break;
-            // less than
-            case '<':
-            case 'lt':
-                if (abs($float1 - $float2) < $epsilon) {
-                    return false;
-                }
-                if ($float1 < $float2) {
-                    return true;
-                }
-                break;
-            // less than or equal
-            case '<=':
-            case 'lte':
-                if (
-                    self::compare_floats($float1, $float2, '<')
-                    || self::compare_floats($float1, $float2)
-                ) {
-                    return true;
-                }
-                break;
-            // greater than
-            case '>':
-            case 'gt':
-                if (abs($float1 - $float2) < $epsilon) {
-                    return false;
-                }
-                if ($float1 > $float2) {
-                    return true;
-                }
-                break;
-            // greater than or equal
-            case '>=':
-            case 'gte':
-                if (
-                    self::compare_floats($float1, $float2, '>')
-                    || self::compare_floats($float1, $float2)
-                ) {
-                    return true;
-                }
-                break;
-            case '<>':
-            case '!=':
-            case '!==':
-            case 'ne':
-                if (abs($float1 - $float2) > $epsilon) {
-                    return true;
-                }
-                break;
-            default:
-                throw new EE_Error(
-                    sprintf(
-                        esc_html__(
-                            "Unknown operator %s in EEH_Money::compare_floats()",
-                            'event_espresso'
-                        ),
-                        $operator
-                    )
-                );
-        }
-        return false;
-    }
+	/**
+	 * For comparing floats. Default operator is '=', but see the $operator below for all options.
+	 * This should be used to compare floats instead of normal '==' because floats
+	 * are inherently imprecise, and so you can sometimes have two floats that appear to be identical
+	 * but actually differ by 0.00000001.
+	 *
+	 * @see http://biostall.com/php-function-to-compare-floating-point-numbers
+	 * @param int|float|string $float1
+	 * @param int|float|string $float2
+	 * @param string|null      $operator The operator. Valid options are =, <=, <, >=, >, <>, eq, lt, lte, gt, gte, ne
+	 * @return bool whether the equation is true or false
+	 * @throws EE_Error
+	 */
+	public static function compare_floats($float1, $float2, ?string $operator = '='): bool
+	{
+		// Check numbers to 5 digits of precision
+		$epsilon = 0.00001;
+		$float1  = (float) $float1;
+		$float2  = (float) $float2;
+		switch ($operator) {
+			// equal
+			case '=':
+			case '==':
+			case '===':
+			case 'eq':
+				if (abs($float1 - $float2) < $epsilon) {
+					return true;
+				}
+				break;
+			// less than
+			case '<':
+			case 'lt':
+				if (abs($float1 - $float2) < $epsilon) {
+					return false;
+				}
+				if ($float1 < $float2) {
+					return true;
+				}
+				break;
+			// less than or equal
+			case '<=':
+			case 'lte':
+				if (
+					self::compare_floats($float1, $float2, '<')
+					|| self::compare_floats($float1, $float2)
+				) {
+					return true;
+				}
+				break;
+			// greater than
+			case '>':
+			case 'gt':
+				if (abs($float1 - $float2) < $epsilon) {
+					return false;
+				}
+				if ($float1 > $float2) {
+					return true;
+				}
+				break;
+			// greater than or equal
+			case '>=':
+			case 'gte':
+				if (
+					self::compare_floats($float1, $float2, '>')
+					|| self::compare_floats($float1, $float2)
+				) {
+					return true;
+				}
+				break;
+			case '<>':
+			case '!=':
+			case '!==':
+			case 'ne':
+				if (abs($float1 - $float2) > $epsilon) {
+					return true;
+				}
+				break;
+			default:
+				throw new EE_Error(
+					sprintf(
+						esc_html__(
+							"Unknown operator %s in EEH_Money::compare_floats()",
+							'event_espresso'
+						),
+						$operator
+					)
+				);
+		}
+		return false;
+	}
 
 
-    /**
-     * This returns a localized format string suitable for jQplot.
-     *
-     * @param string|null $CNT_ISO If this is provided, then will attempt to get the currency settings for the country.
-     *                             Otherwise will use currency settings for current active country on site.
-     * @return string
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public static function get_format_for_jqplot(?string $CNT_ISO = ''): string
-    {
-        // default format
-        $format          = 'f';
-        $currency_config = EE_Currency_Config::getCurrencyConfig($CNT_ISO);
-        // first get the decimal place and number of places
-        $format = "%'." . $currency_config->dec_plc . $format;
-        // currency symbol on right side.
-        return $currency_config->sign_b4 ? $currency_config->sign . $format : $format . $currency_config->sign;
-    }
+	/**
+	 * This returns a localized format string suitable for jQplot.
+	 *
+	 * @param string|null $CNT_ISO If this is provided, then will attempt to get the currency settings for the country.
+	 *                             Otherwise will use currency settings for current active country on site.
+	 * @return string
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public static function get_format_for_jqplot(?string $CNT_ISO = ''): string
+	{
+		// default format
+		$format          = 'f';
+		$currency_config = EE_Currency_Config::getCurrencyConfig($CNT_ISO);
+		// first get the decimal place and number of places
+		$format = "%'." . $currency_config->dec_plc . $format;
+		// currency symbol on right side.
+		return $currency_config->sign_b4 ? $currency_config->sign . $format : $format . $currency_config->sign;
+	}
 
 
-    /**
-     * This returns a localized format string suitable for usage with the Google Charts API format param.
-     *
-     * @param string|null $CNT_ISO If this is provided, then will attempt to get the currency settings for the country.
-     *                             Otherwise will use currency settings for current active country on site.
-     *                             Note: GoogleCharts uses ICU pattern set
-     *                             (@return array
-     * @return array
-     * @throws EE_Error
-     * @throws ReflectionException
-     * @see http://icu-project.org/apiref/icu4c/classDecimalFormat.html#_details)
-     */
-    public static function get_format_for_google_charts(?string $CNT_ISO = ''): array
-    {
-        $currency_config            = EE_Currency_Config::getCurrencyConfig($CNT_ISO);
-        $decimal_places_placeholder = str_pad('', $currency_config->dec_plc, '0');
-        // first get the decimal place and number of places
-        $format = '#,##0.' . $decimal_places_placeholder;
-        // currency symbol on right side.
-        $format          = $currency_config->sign_b4
-            ? $currency_config->sign . $format
-            : $format
-              . $currency_config->sign;
-        $formatterObject = [
-            'decimalSymbol'  => $currency_config->dec_mrk,
-            'groupingSymbol' => $currency_config->thsnds,
-            'fractionDigits' => $currency_config->dec_plc,
-        ];
-        if ($currency_config->sign_b4) {
-            $formatterObject['prefix'] = $currency_config->sign;
-        } else {
-            $formatterObject['suffix'] = $currency_config->sign;
-        }
-        return [
-            'format'          => $format,
-            'formatterObject' => $formatterObject,
-        ];
-    }
+	/**
+	 * This returns a localized format string suitable for usage with the Google Charts API format param.
+	 *
+	 * @param string|null $CNT_ISO If this is provided, then will attempt to get the currency settings for the country.
+	 *                             Otherwise will use currency settings for current active country on site.
+	 *                             Note: GoogleCharts uses ICU pattern set
+	 *                             (@return array
+	 * @return array
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 * @see http://icu-project.org/apiref/icu4c/classDecimalFormat.html#_details)
+	 */
+	public static function get_format_for_google_charts(?string $CNT_ISO = ''): array
+	{
+		$currency_config            = EE_Currency_Config::getCurrencyConfig($CNT_ISO);
+		$decimal_places_placeholder = str_pad('', $currency_config->dec_plc, '0');
+		// first get the decimal place and number of places
+		$format = '#,##0.' . $decimal_places_placeholder;
+		// currency symbol on right side.
+		$format          = $currency_config->sign_b4
+			? $currency_config->sign . $format
+			: $format
+			  . $currency_config->sign;
+		$formatterObject = [
+			'decimalSymbol'  => $currency_config->dec_mrk,
+			'groupingSymbol' => $currency_config->thsnds,
+			'fractionDigits' => $currency_config->dec_plc,
+		];
+		if ($currency_config->sign_b4) {
+			$formatterObject['prefix'] = $currency_config->sign;
+		} else {
+			$formatterObject['suffix'] = $currency_config->sign;
+		}
+		return [
+			'format'          => $format,
+			'formatterObject' => $formatterObject,
+		];
+	}
 
 
-    /**
-     * @param string|null $CNT_ISO
-     * @return EE_Currency_Config
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public static function get_currency_config(?string $CNT_ISO = ''): EE_Currency_Config
-    {
-        return EE_Currency_Config::getCurrencyConfig($CNT_ISO);
-    }
+	/**
+	 * @param string|null $CNT_ISO
+	 * @return EE_Currency_Config
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public static function get_currency_config(?string $CNT_ISO = ''): EE_Currency_Config
+	{
+		return EE_Currency_Config::getCurrencyConfig($CNT_ISO);
+	}
 
 
-    /**
-     * @param string|null $CNT_ISO
-     * @param bool        $as_decimal if false [default] will return the number of decimal places ex: 1, 2, 3
-     *                                if true, will return the subunits as a decimal fraction ex: .1, .01, .001
-     * @return float
-     * @throws EE_Error
-     * @throws ReflectionException
-     * @since 5.0.0.p
-     */
-    public static function getCurrencySubUnits(?string $CNT_ISO = '', bool $as_decimal = false): float
-    {
-        $currency_config = EE_Currency_Config::getCurrencyConfig($CNT_ISO);
-        return $as_decimal ? pow(10, ($currency_config->dec_plc * -1)) : $currency_config->dec_plc;
-    }
+	/**
+	 * @param string|null $CNT_ISO
+	 * @param bool        $as_decimal if false [default] will return the number of decimal places ex: 1, 2, 3
+	 *                                if true, will return the subunits as a decimal fraction ex: .1, .01, .001
+	 * @return float
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 * @since 5.0.0.p
+	 */
+	public static function getCurrencySubUnits(?string $CNT_ISO = '', bool $as_decimal = false): float
+	{
+		$currency_config = EE_Currency_Config::getCurrencyConfig($CNT_ISO);
+		return $as_decimal ? pow(10, ($currency_config->dec_plc * -1)) : $currency_config->dec_plc;
+	}
 }

core/db_models/EEM_Price.model.php

Spacing +9 added lines, -9 removed lines

@@ -115,23 +115,23 @@ 
             'WP_User'    => new EE_Belongs_To_Relation(),
         ];
         // this model is generally available for reading
-        $this->_cap_restriction_generators[ EEM_Base::caps_read ] =
+        $this->_cap_restriction_generators[EEM_Base::caps_read] =
             new EE_Restriction_Generator_Default_Public(
                 'PRC_is_default',
                 'Ticket.Datetime.Event'
             );
         // account for default tickets in the caps
-        $this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
+        $this->_cap_restriction_generators[EEM_Base::caps_read_admin] =
             new EE_Restriction_Generator_Default_Protected(
                 'PRC_is_default',
                 'Ticket.Datetime.Event'
             );
-        $this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
+        $this->_cap_restriction_generators[EEM_Base::caps_edit] =
             new EE_Restriction_Generator_Default_Protected(
                 'PRC_is_default',
                 'Ticket.Datetime.Event'
             );
-        $this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
+        $this->_cap_restriction_generators[EEM_Base::caps_delete] =
             new EE_Restriction_Generator_Default_Protected(
                 'PRC_is_default',
                 'Ticket.Datetime.Event'
@@ -199,7 +199,7 @@ 
             'PRC_deleted'    => 0,
             'PRC_is_default' => 1,
         ];
-        if (! $include_taxes) {
+        if ( ! $include_taxes) {
             $_where['Price_Type.PBT_ID'] = ['!=', 4];
         }
         $_query_params = [
@@ -256,7 +256,7 @@ 
         );
         foreach ($all_taxes as $tax) {
             if ($tax instanceof EE_Price) {
-                $taxes[ $tax->order() ][ $tax->ID() ] = $tax;
+                $taxes[$tax->order()][$tax->ID()] = $tax;
             }
         }
         return $taxes;
@@ -281,7 +281,7 @@ 
             if ($default_prices) {
                 foreach ($default_prices as $price) {
                     if ($price instanceof EE_Price) {
-                        $array_of_price_objects[ $price->type() ][] = $price;
+                        $array_of_price_objects[$price->type()][] = $price;
                     }
                 }
                 return $array_of_price_objects;
@@ -298,10 +298,10 @@ 
             ]
         );
 
-        if (! empty($ticket_prices)) {
+        if ( ! empty($ticket_prices)) {
             foreach ($ticket_prices as $price) {
                 if ($price instanceof EE_Price) {
-                    $array_of_price_objects[ $price->type() ][] = $price;
+                    $array_of_price_objects[$price->type()][] = $price;
                 }
             }
             return $array_of_price_objects;

Indentation +349 added lines, -349 removed lines

@@ -12,380 +12,380 @@
  */
 class EEM_Price extends EEM_Soft_Delete_Base
 {
-    protected static ?EEM_Price $_instance = null;
+	protected static ?EEM_Price $_instance = null;
 
 
-    /**
-     * private constructor to prevent direct creation
-     *
-     * @Constructor
-     * @param string|null $timezone string representing the timezone we want to set for returned Date Time Strings
-     *                              (and any incoming timezone data that gets saved).
-     *                              Note this just sends the timezone info to the date time model field objects.
-     *                              Default is NULL
-     *                              (and will be assumed using the set timezone in the 'timezone_string' wp option)
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    protected function __construct(?string $timezone = '')
-    {
-        $this->singular_item = esc_html__('Price', 'event_espresso');
-        $this->plural_item   = esc_html__('Prices', 'event_espresso');
+	/**
+	 * private constructor to prevent direct creation
+	 *
+	 * @Constructor
+	 * @param string|null $timezone string representing the timezone we want to set for returned Date Time Strings
+	 *                              (and any incoming timezone data that gets saved).
+	 *                              Note this just sends the timezone info to the date time model field objects.
+	 *                              Default is NULL
+	 *                              (and will be assumed using the set timezone in the 'timezone_string' wp option)
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	protected function __construct(?string $timezone = '')
+	{
+		$this->singular_item = esc_html__('Price', 'event_espresso');
+		$this->plural_item   = esc_html__('Prices', 'event_espresso');
 
-        $this->_tables          = [
-            'Price' => new EE_Primary_Table('esp_price', 'PRC_ID'),
-        ];
-        $this->_fields          = [
-            'Price' => [
-                'PRC_ID'         => new EE_Primary_Key_Int_Field(
-                    'PRC_ID',
-                    'Price ID'
-                ),
-                'PRT_ID'         => new EE_Foreign_Key_Int_Field(
-                    'PRT_ID',
-                    esc_html__('Price type Id', 'event_espresso'),
-                    false,
-                    null,
-                    'Price_Type'
-                ),
-                'PRC_amount'     => new EE_Money_Field(
-                    'PRC_amount',
-                    esc_html__('Price Amount', 'event_espresso'),
-                    false,
-                    0
-                ),
-                'PRC_name'       => new EE_Plain_Text_Field(
-                    'PRC_name',
-                    esc_html__('Name of Price', 'event_espresso'),
-                    false,
-                    ''
-                ),
-                'PRC_desc'       => new EE_Post_Content_Field(
-                    'PRC_desc',
-                    esc_html__('Price Description', 'event_espresso'),
-                    false,
-                    ''
-                ),
-                'PRC_is_default' => new EE_Boolean_Field(
-                    'PRC_is_default',
-                    esc_html__('Flag indicating whether price is a default price', 'event_espresso'),
-                    false,
-                    false
-                ),
-                'PRC_overrides'  => new EE_Integer_Field(
-                    'PRC_overrides',
-                    esc_html__(
-                        'Price ID for a global Price that will be overridden by this Price  ( for replacing default prices )',
-                        'event_espresso'
-                    ),
-                    true,
-                    0
-                ),
-                'PRC_order'      => new EE_Integer_Field(
-                    'PRC_order',
-                    esc_html__(
-                        'Order of Application of Price (lower numbers apply first?)',
-                        'event_espresso'
-                    ),
-                    false,
-                    1
-                ),
-                'PRC_deleted'    => new EE_Trashed_Flag_Field(
-                    'PRC_deleted',
-                    esc_html__('Flag Indicating if this has been deleted or not', 'event_espresso'),
-                    false,
-                    false
-                ),
-                'PRC_parent'     => new EE_Integer_Field(
-                    'PRC_parent',
-                    esc_html__('Indicates what PRC_ID is the parent of this PRC_ID', 'event_espresso'),
-                    true,
-                    0
-                ),
-                'PRC_wp_user'    => new EE_WP_User_Field(
-                    'PRC_wp_user',
-                    esc_html__('Price Creator ID', 'event_espresso'),
-                    false
-                ),
-            ],
-        ];
-        $this->_model_relations = [
-            'Ticket'     => new EE_HABTM_Relation('Ticket_Price'),
-            'Price_Type' => new EE_Belongs_To_Relation(),
-            'WP_User'    => new EE_Belongs_To_Relation(),
-        ];
-        // this model is generally available for reading
-        $this->_cap_restriction_generators[ EEM_Base::caps_read ] =
-            new EE_Restriction_Generator_Default_Public(
-                'PRC_is_default',
-                'Ticket.Datetime.Event'
-            );
-        // account for default tickets in the caps
-        $this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
-            new EE_Restriction_Generator_Default_Protected(
-                'PRC_is_default',
-                'Ticket.Datetime.Event'
-            );
-        $this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
-            new EE_Restriction_Generator_Default_Protected(
-                'PRC_is_default',
-                'Ticket.Datetime.Event'
-            );
-        $this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
-            new EE_Restriction_Generator_Default_Protected(
-                'PRC_is_default',
-                'Ticket.Datetime.Event'
-            );
-        parent::__construct($timezone);
-    }
+		$this->_tables          = [
+			'Price' => new EE_Primary_Table('esp_price', 'PRC_ID'),
+		];
+		$this->_fields          = [
+			'Price' => [
+				'PRC_ID'         => new EE_Primary_Key_Int_Field(
+					'PRC_ID',
+					'Price ID'
+				),
+				'PRT_ID'         => new EE_Foreign_Key_Int_Field(
+					'PRT_ID',
+					esc_html__('Price type Id', 'event_espresso'),
+					false,
+					null,
+					'Price_Type'
+				),
+				'PRC_amount'     => new EE_Money_Field(
+					'PRC_amount',
+					esc_html__('Price Amount', 'event_espresso'),
+					false,
+					0
+				),
+				'PRC_name'       => new EE_Plain_Text_Field(
+					'PRC_name',
+					esc_html__('Name of Price', 'event_espresso'),
+					false,
+					''
+				),
+				'PRC_desc'       => new EE_Post_Content_Field(
+					'PRC_desc',
+					esc_html__('Price Description', 'event_espresso'),
+					false,
+					''
+				),
+				'PRC_is_default' => new EE_Boolean_Field(
+					'PRC_is_default',
+					esc_html__('Flag indicating whether price is a default price', 'event_espresso'),
+					false,
+					false
+				),
+				'PRC_overrides'  => new EE_Integer_Field(
+					'PRC_overrides',
+					esc_html__(
+						'Price ID for a global Price that will be overridden by this Price  ( for replacing default prices )',
+						'event_espresso'
+					),
+					true,
+					0
+				),
+				'PRC_order'      => new EE_Integer_Field(
+					'PRC_order',
+					esc_html__(
+						'Order of Application of Price (lower numbers apply first?)',
+						'event_espresso'
+					),
+					false,
+					1
+				),
+				'PRC_deleted'    => new EE_Trashed_Flag_Field(
+					'PRC_deleted',
+					esc_html__('Flag Indicating if this has been deleted or not', 'event_espresso'),
+					false,
+					false
+				),
+				'PRC_parent'     => new EE_Integer_Field(
+					'PRC_parent',
+					esc_html__('Indicates what PRC_ID is the parent of this PRC_ID', 'event_espresso'),
+					true,
+					0
+				),
+				'PRC_wp_user'    => new EE_WP_User_Field(
+					'PRC_wp_user',
+					esc_html__('Price Creator ID', 'event_espresso'),
+					false
+				),
+			],
+		];
+		$this->_model_relations = [
+			'Ticket'     => new EE_HABTM_Relation('Ticket_Price'),
+			'Price_Type' => new EE_Belongs_To_Relation(),
+			'WP_User'    => new EE_Belongs_To_Relation(),
+		];
+		// this model is generally available for reading
+		$this->_cap_restriction_generators[ EEM_Base::caps_read ] =
+			new EE_Restriction_Generator_Default_Public(
+				'PRC_is_default',
+				'Ticket.Datetime.Event'
+			);
+		// account for default tickets in the caps
+		$this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
+			new EE_Restriction_Generator_Default_Protected(
+				'PRC_is_default',
+				'Ticket.Datetime.Event'
+			);
+		$this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
+			new EE_Restriction_Generator_Default_Protected(
+				'PRC_is_default',
+				'Ticket.Datetime.Event'
+			);
+		$this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
+			new EE_Restriction_Generator_Default_Protected(
+				'PRC_is_default',
+				'Ticket.Datetime.Event'
+			);
+		parent::__construct($timezone);
+	}
 
 
-    /**
-     * instantiate a new price object with blank/empty properties
-     *
-     * @return EE_Price
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function get_new_price(): EE_Price
-    {
-        return $this->create_default_object();
-    }
+	/**
+	 * instantiate a new price object with blank/empty properties
+	 *
+	 * @return EE_Price
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function get_new_price(): EE_Price
+	{
+		return $this->create_default_object();
+	}
 
 
-    /**
-     * retrieve  ALL prices from db
-     *
-     * @return EE_Price[]
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function get_all_prices(): array
-    {
-        // retrieve all prices
-        return $this->get_all(['order_by' => ['PRC_amount' => 'ASC']]);
-    }
+	/**
+	 * retrieve  ALL prices from db
+	 *
+	 * @return EE_Price[]
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function get_all_prices(): array
+	{
+		// retrieve all prices
+		return $this->get_all(['order_by' => ['PRC_amount' => 'ASC']]);
+	}
 
 
-    /**
-     * retrieve all active prices for a particular event
-     *
-     * @param int $EVT_ID
-     * @return array on success
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function get_all_event_prices(int $EVT_ID = 0): array
-    {
-        return $this->get_all(
-            [
-                [
-                    'EVT_ID'            => $EVT_ID,
-                    'Price_Type.PBT_ID' => ['!=', EEM_Price_Type::base_type_tax],
-                ],
-                'order_by' => $this->_order_by_array_for_get_all_method(),
-            ]
-        );
-    }
+	/**
+	 * retrieve all active prices for a particular event
+	 *
+	 * @param int $EVT_ID
+	 * @return array on success
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function get_all_event_prices(int $EVT_ID = 0): array
+	{
+		return $this->get_all(
+			[
+				[
+					'EVT_ID'            => $EVT_ID,
+					'Price_Type.PBT_ID' => ['!=', EEM_Price_Type::base_type_tax],
+				],
+				'order_by' => $this->_order_by_array_for_get_all_method(),
+			]
+		);
+	}
 
 
-    /**
-     * retrieve all active global prices (that are not taxes (PBT_ID=4)) for a particular event
-     *
-     * @param bool $count return count
-     * @param bool $include_taxes
-     * @return int|EE_Price[]
-     * @throws EE_Error
-     * @throws ReflectionException
-     * @throws ReflectionException
-     */
-    public function get_all_default_prices(bool $count = false, bool $include_taxes = false)
-    {
-        $_where = [
-            'PRC_deleted'    => 0,
-            'PRC_is_default' => 1,
-        ];
-        if (! $include_taxes) {
-            $_where['Price_Type.PBT_ID'] = ['!=', 4];
-        }
-        $_query_params = [
-            $_where,
-            'order_by' => $this->_order_by_array_for_get_all_method(),
-        ];
-        return $count ? $this->count([$_where]) : $this->get_all($_query_params);
-    }
+	/**
+	 * retrieve all active global prices (that are not taxes (PBT_ID=4)) for a particular event
+	 *
+	 * @param bool $count return count
+	 * @param bool $include_taxes
+	 * @return int|EE_Price[]
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 * @throws ReflectionException
+	 */
+	public function get_all_default_prices(bool $count = false, bool $include_taxes = false)
+	{
+		$_where = [
+			'PRC_deleted'    => 0,
+			'PRC_is_default' => 1,
+		];
+		if (! $include_taxes) {
+			$_where['Price_Type.PBT_ID'] = ['!=', 4];
+		}
+		$_query_params = [
+			$_where,
+			'order_by' => $this->_order_by_array_for_get_all_method(),
+		];
+		return $count ? $this->count([$_where]) : $this->get_all($_query_params);
+	}
 
 
-    /**
-     * retrieve all active global prices that are taxes
-     *
-     * @return EE_Price[]
-     * @throws EE_Error
-     * @throws ReflectionException
-     * @since   5.0.0.p
-     */
-    public function getAllDefaultTaxes(): array
-    {
-        return $this->get_all(
-            [
-                [
-                    'PRC_deleted'       => 0,
-                    'PRC_is_default'    => 1,
-                    'Price_Type.PBT_ID' => EEM_Price_Type::base_type_tax,
-                ],
-                'order_by' => [
-                    'Price_Type.PRT_order' => 'ASC',
-                    'PRC_order'            => 'ASC',
-                ],
-            ]
-        );
-    }
+	/**
+	 * retrieve all active global prices that are taxes
+	 *
+	 * @return EE_Price[]
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 * @since   5.0.0.p
+	 */
+	public function getAllDefaultTaxes(): array
+	{
+		return $this->get_all(
+			[
+				[
+					'PRC_deleted'       => 0,
+					'PRC_is_default'    => 1,
+					'Price_Type.PBT_ID' => EEM_Price_Type::base_type_tax,
+				],
+				'order_by' => [
+					'Price_Type.PRT_order' => 'ASC',
+					'PRC_order'            => 'ASC',
+				],
+			]
+		);
+	}
 
 
-    /**
-     * retrieve all prices that are taxes
-     *
-     * @return EE_Price[]
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws ReflectionException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    public function get_all_prices_that_are_taxes(): array
-    {
-        $taxes     = [];
-        $all_taxes = $this->get_all(
-            [
-                ['Price_Type.PBT_ID' => EEM_Price_Type::base_type_tax, 'PRC_is_default' => 1],
-                'order_by' => ['Price_Type.PRT_order' => 'ASC', 'PRC_order' => 'ASC'],
-            ]
-        );
-        foreach ($all_taxes as $tax) {
-            if ($tax instanceof EE_Price) {
-                $taxes[ $tax->order() ][ $tax->ID() ] = $tax;
-            }
-        }
-        return $taxes;
-    }
+	/**
+	 * retrieve all prices that are taxes
+	 *
+	 * @return EE_Price[]
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws ReflectionException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	public function get_all_prices_that_are_taxes(): array
+	{
+		$taxes     = [];
+		$all_taxes = $this->get_all(
+			[
+				['Price_Type.PBT_ID' => EEM_Price_Type::base_type_tax, 'PRC_is_default' => 1],
+				'order_by' => ['Price_Type.PRT_order' => 'ASC', 'PRC_order' => 'ASC'],
+			]
+		);
+		foreach ($all_taxes as $tax) {
+			if ($tax instanceof EE_Price) {
+				$taxes[ $tax->order() ][ $tax->ID() ] = $tax;
+			}
+		}
+		return $taxes;
+	}
 
 
-    /**
-     * retrieve all prices for an ticket plus default global prices, but not taxes
-     *
-     * @param int $TKT_ID the id of the event.  If not included then we assume that this is a new ticket.
-     * @return EE_Price[]
-     * @throws EE_Error
-     * @throws ReflectionException
-     * @throws ReflectionException
-     * @throws ReflectionException
-     */
-    public function get_all_ticket_prices_for_admin(int $TKT_ID = 0): array
-    {
-        $array_of_price_objects = [];
-        if (empty($TKT_ID)) {
-            // if there is no tkt, get prices with no tkt ID, are global, are not a tax, and are active
-            // return that list
-            $default_prices = $this->get_all_default_prices();
+	/**
+	 * retrieve all prices for an ticket plus default global prices, but not taxes
+	 *
+	 * @param int $TKT_ID the id of the event.  If not included then we assume that this is a new ticket.
+	 * @return EE_Price[]
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 * @throws ReflectionException
+	 * @throws ReflectionException
+	 */
+	public function get_all_ticket_prices_for_admin(int $TKT_ID = 0): array
+	{
+		$array_of_price_objects = [];
+		if (empty($TKT_ID)) {
+			// if there is no tkt, get prices with no tkt ID, are global, are not a tax, and are active
+			// return that list
+			$default_prices = $this->get_all_default_prices();
 
-            if ($default_prices) {
-                foreach ($default_prices as $price) {
-                    if ($price instanceof EE_Price) {
-                        $array_of_price_objects[ $price->type() ][] = $price;
-                    }
-                }
-                return $array_of_price_objects;
-            }
-            return [];
-        }
-        $ticket_prices = $this->get_all(
-            [
-                [
-                    'TKT_ID'      => $TKT_ID,
-                    'PRC_deleted' => 0,
-                ],
-                'order_by' => ['PRC_order' => 'ASC'],
-            ]
-        );
+			if ($default_prices) {
+				foreach ($default_prices as $price) {
+					if ($price instanceof EE_Price) {
+						$array_of_price_objects[ $price->type() ][] = $price;
+					}
+				}
+				return $array_of_price_objects;
+			}
+			return [];
+		}
+		$ticket_prices = $this->get_all(
+			[
+				[
+					'TKT_ID'      => $TKT_ID,
+					'PRC_deleted' => 0,
+				],
+				'order_by' => ['PRC_order' => 'ASC'],
+			]
+		);
 
-        if (! empty($ticket_prices)) {
-            foreach ($ticket_prices as $price) {
-                if ($price instanceof EE_Price) {
-                    $array_of_price_objects[ $price->type() ][] = $price;
-                }
-            }
-            return $array_of_price_objects;
-        }
-        return [];
-    }
+		if (! empty($ticket_prices)) {
+			foreach ($ticket_prices as $price) {
+				if ($price instanceof EE_Price) {
+					$array_of_price_objects[ $price->type() ][] = $price;
+				}
+			}
+			return $array_of_price_objects;
+		}
+		return [];
+	}
 
 
-    /**
-     * @param EE_Price $price_a
-     * @param EE_Price $price_b
-     * @return int
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function _sort_event_prices_by_type(EE_Price $price_a, EE_Price $price_b): int
-    {
-        if ($price_a->type_obj()->order() === $price_b->type_obj()->order()) {
-            return $this->_sort_event_prices_by_order($price_a, $price_b);
-        }
-        return $price_a->type_obj()->order() < $price_b->type_obj()->order() ? -1 : 1;
-    }
+	/**
+	 * @param EE_Price $price_a
+	 * @param EE_Price $price_b
+	 * @return int
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function _sort_event_prices_by_type(EE_Price $price_a, EE_Price $price_b): int
+	{
+		if ($price_a->type_obj()->order() === $price_b->type_obj()->order()) {
+			return $this->_sort_event_prices_by_order($price_a, $price_b);
+		}
+		return $price_a->type_obj()->order() < $price_b->type_obj()->order() ? -1 : 1;
+	}
 
 
-    /**
-     * @param EE_Price $price_a
-     * @param EE_Price $price_b
-     * @return int
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function _sort_event_prices_by_order(EE_Price $price_a, EE_Price $price_b): int
-    {
-        if ($price_a->order() === $price_b->order()) {
-            return 0;
-        }
-        return $price_a->order() < $price_b->order() ? -1 : 1;
-    }
+	/**
+	 * @param EE_Price $price_a
+	 * @param EE_Price $price_b
+	 * @return int
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function _sort_event_prices_by_order(EE_Price $price_a, EE_Price $price_b): int
+	{
+		if ($price_a->order() === $price_b->order()) {
+			return 0;
+		}
+		return $price_a->order() < $price_b->order() ? -1 : 1;
+	}
 
 
-    /**
-     * get all prices of a specific type
-     *
-     * @param int $type - PRT_ID
-     * @return EE_Price[]
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function get_all_prices_that_are_type(int $type = 0): array
-    {
-        return $this->get_all(
-            [
-                [
-                    'PRT_ID' => $type,
-                ],
-                'order_by' => $this->_order_by_array_for_get_all_method(),
-            ]
-        );
-    }
+	/**
+	 * get all prices of a specific type
+	 *
+	 * @param int $type - PRT_ID
+	 * @return EE_Price[]
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function get_all_prices_that_are_type(int $type = 0): array
+	{
+		return $this->get_all(
+			[
+				[
+					'PRT_ID' => $type,
+				],
+				'order_by' => $this->_order_by_array_for_get_all_method(),
+			]
+		);
+	}
 
 
-    /**
-     * Returns an array of the normal 'order_by' query parameter provided to the get_all query.
-     * Of course you don't have to use it, but this is the order we usually want to sort prices by
-     *
-     * @return array which can be used like so: $this->get_all(array(array(...where
-     *               stuff...),'order_by'=>$this->_order_by_array_for_get_all_method()));
-     */
-    public function _order_by_array_for_get_all_method(): array
-    {
-        return [
-            'PRC_order'            => 'ASC',
-            'Price_Type.PRT_order' => 'ASC',
-            'PRC_ID'               => 'ASC',
-        ];
-    }
+	/**
+	 * Returns an array of the normal 'order_by' query parameter provided to the get_all query.
+	 * Of course you don't have to use it, but this is the order we usually want to sort prices by
+	 *
+	 * @return array which can be used like so: $this->get_all(array(array(...where
+	 *               stuff...),'order_by'=>$this->_order_by_array_for_get_all_method()));
+	 */
+	public function _order_by_array_for_get_all_method(): array
+	{
+		return [
+			'PRC_order'            => 'ASC',
+			'Price_Type.PRT_order' => 'ASC',
+			'PRC_ID'               => 'ASC',
+		];
+	}
 }