eventespresso / event-espresso-core

admin_pages/maintenance/templates/ee_system_stati_page.template.php

Spacing +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
                                     $keys_only = empty(
                                         array_filter(
                                             array_values($data_value),
-                                            function ($v) {
+                                            function($v) {
                                                 return $v !== null && (is_array($v) || trim($v) !== '');
                                             }
                                         )

Indentation +61 added lines, -61 removed lines

@@ -19,38 +19,38 @@ 
  */
 function ee_recurse_into_array_for_display($data, $depth = 0, $td = true)
 {
-    if (is_object($data) || $data instanceof __PHP_Incomplete_Class) {
-        // is_object($incomplete_class) actually returns false, hence why we check for it
-        $data = json_decode(json_encode($data), true);
-    }
-    if (empty($data)) {
-        return;
-    }
-    if (is_array($data)) {
-        $depth++;
-        ksort($data, SORT_NATURAL | SORT_FLAG_CASE);
-        if (EEH_Array::is_associative_array($data)) { ?>
+	if (is_object($data) || $data instanceof __PHP_Incomplete_Class) {
+		// is_object($incomplete_class) actually returns false, hence why we check for it
+		$data = json_decode(json_encode($data), true);
+	}
+	if (empty($data)) {
+		return;
+	}
+	if (is_array($data)) {
+		$depth++;
+		ksort($data, SORT_NATURAL | SORT_FLAG_CASE);
+		if (EEH_Array::is_associative_array($data)) { ?>
             <table class='ee-system-stati ee-system-stati-<?php echo absint($depth); ?>'>
                 <tbody>
                     <?php foreach ($data as $data_key => $data_value) {
-                        // if the value is a single element array with no key,
-                        // and the value is a primitive (not an array, object, etc.)
-                        if (
-                            is_array($data_value)
-                            && count($data_value) === 1
-                            && empty(key($data_value))
-                            && is_scalar(reset($data_value))
-                        ) {
-                            ?>
+						// if the value is a single element array with no key,
+						// and the value is a primitive (not an array, object, etc.)
+						if (
+							is_array($data_value)
+							&& count($data_value) === 1
+							&& empty(key($data_value))
+							&& is_scalar(reset($data_value))
+						) {
+							?>
                             <tr>
                                 <td class="ee-system-stati-value" colspan="2">
                                     <?php echo esc_html(reset($data_value)); ?>
                                 </td>
                             </tr>
                             <?php
-                            continue;
-                        }
-                        ?>
+							continue;
+						}
+						?>
                         <tr>
                             <td class='ee-system-stati-key'>
                                 <span class="ee-system-stati-label">
@@ -65,31 +65,31 @@ 
                             <?php if (is_scalar($data_value)) { ?>
                                 <?php ee_recurse_into_array_for_display($data_value, $depth); ?>
                             <?php } else {
-                                if (is_array($data_value) && count($data_value) === 1) {
-                                    // verify that values have been set
-                                    $keys_only = empty(
-                                        array_filter(
-                                            array_values($data_value),
-                                            function ($v) {
-                                                return $v !== null && (is_array($v) || trim($v) !== '');
-                                            }
-                                        )
-                                    );
-                                    // if the array only consists of keys (no values)
-                                    if ($keys_only) {
-                                        // then use keys for values
-                                        $data_value = array_keys($data_value);
-                                        // but if there is only one value now
-                                        if (count($data_value) === 1) {
-                                            // then pass that single value back into this function
-                                            // this prevents extra empty layers of nothing being added to the output
-                                            ee_recurse_into_array_for_display(reset($data_value), $depth);
-                                            // need to finish the row off though
-                                            echo '</tr>';
-                                            continue;
-                                        }
-                                    }
-                                } ?>
+								if (is_array($data_value) && count($data_value) === 1) {
+									// verify that values have been set
+									$keys_only = empty(
+										array_filter(
+											array_values($data_value),
+											function ($v) {
+												return $v !== null && (is_array($v) || trim($v) !== '');
+											}
+										)
+									);
+									// if the array only consists of keys (no values)
+									if ($keys_only) {
+										// then use keys for values
+										$data_value = array_keys($data_value);
+										// but if there is only one value now
+										if (count($data_value) === 1) {
+											// then pass that single value back into this function
+											// this prevents extra empty layers of nothing being added to the output
+											ee_recurse_into_array_for_display(reset($data_value), $depth);
+											// need to finish the row off though
+											echo '</tr>';
+											continue;
+										}
+									}
+								} ?>
                                 <td class="ee-system-stati-sub-values">
                                     <?php ee_recurse_into_array_for_display($data_value, $depth); ?>
                                 </td>
@@ -107,19 +107,19 @@ 
                 <?php } ?>
             </ul>
         <?php } else {
-            // there's no key (label) and there's only one value,
-            // so let's just pass that value back into this function,
-            // but reset the depth back to where it was.
-            // this prevents extra empty layers of nothing being added to the output
-            $depth--;
-            ee_recurse_into_array_for_display(reset($data), $depth, false);
-        }
-    } else {
-        echo wp_kses($td ? '<td class="ee-system-stati-value">' : '', AllowedTags::getAllowedTags());
-        // simple value
-        echo wp_kses($data, AllowedTags::getAllowedTags());
-        echo wp_kses($td ? '</td>' : '', AllowedTags::getAllowedTags());
-    }
+			// there's no key (label) and there's only one value,
+			// so let's just pass that value back into this function,
+			// but reset the depth back to where it was.
+			// this prevents extra empty layers of nothing being added to the output
+			$depth--;
+			ee_recurse_into_array_for_display(reset($data), $depth, false);
+		}
+	} else {
+		echo wp_kses($td ? '<td class="ee-system-stati-value">' : '', AllowedTags::getAllowedTags());
+		// simple value
+		echo wp_kses($data, AllowedTags::getAllowedTags());
+		echo wp_kses($td ? '</td>' : '', AllowedTags::getAllowedTags());
+	}
 }
 
 ?>

core/domain/services/graphql/mutators/BulkEntityDelete.php

Indentation +66 added lines, -66 removed lines

@@ -17,76 +17,76 @@
 
 class BulkEntityDelete extends EntityMutator
 {
-    /**
-     * Defines the mutation data modification closure.
-     *
-     * @return callable
-     */
-    public static function mutateAndGetPayload()
-    {
-        /**
-         * Updates an entity.
-         *
-         * @param array       $input   The input for the mutation
-         * @param AppContext  $context The AppContext passed down to all resolvers
-         * @param ResolveInfo $info    The ResolveInfo passed down to all resolvers
-         * @return array
-         * @throws UserError
-         * @throws ReflectionException
-         * @throws InvalidArgumentException
-         * @throws InvalidInterfaceException
-         * @throws InvalidDataTypeException
-         * @throws EE_Error
-         */
-        return static function (array $input, AppContext $context, ResolveInfo $info): array {
-            /**
-             * Stop now if a user isn't allowed to delete.
-             */
-            if (! current_user_can('ee_delete_events')) {
-                throw new UserError(
-                    esc_html__('Sorry, you do not have the required permissions to delete entities', 'event_espresso')
-                );
-            }
+	/**
+	 * Defines the mutation data modification closure.
+	 *
+	 * @return callable
+	 */
+	public static function mutateAndGetPayload()
+	{
+		/**
+		 * Updates an entity.
+		 *
+		 * @param array       $input   The input for the mutation
+		 * @param AppContext  $context The AppContext passed down to all resolvers
+		 * @param ResolveInfo $info    The ResolveInfo passed down to all resolvers
+		 * @return array
+		 * @throws UserError
+		 * @throws ReflectionException
+		 * @throws InvalidArgumentException
+		 * @throws InvalidInterfaceException
+		 * @throws InvalidDataTypeException
+		 * @throws EE_Error
+		 */
+		return static function (array $input, AppContext $context, ResolveInfo $info): array {
+			/**
+			 * Stop now if a user isn't allowed to delete.
+			 */
+			if (! current_user_can('ee_delete_events')) {
+				throw new UserError(
+					esc_html__('Sorry, you do not have the required permissions to delete entities', 'event_espresso')
+				);
+			}
 
-            $details = EntityReorder::prepareEntityDetailsFromInput($input);
+			$details = EntityReorder::prepareEntityDetailsFromInput($input);
 
-            $deletePermanently = ! empty($input['deletePermanently']);
+			$deletePermanently = ! empty($input['deletePermanently']);
 
-            $deletionMethod = __NAMESPACE__;
-            // if it's for datetimes.
-            if ($details['entityType'] === EEM_Datetime::instance()->item_name()) {
-                $deletionMethod .= '\DatetimeDelete::' . ($deletePermanently ? 'deleteDatetimeAndRelations' : 'trashDatetimeAndRelations');
-            } elseif ($details['entityType'] === EEM_Ticket::instance()->item_name()) {
-                $deletionMethod .= '\TicketDelete::' . ($deletePermanently ? 'deleteTicketAndRelations' : 'trashTicket');
-            } elseif ($details['entityType'] === EEM_Price::instance()->item_name()) {
-                $deletionMethod .= '\PriceDelete::deletePriceAndRelations';
-            } else {
-                throw new UserError(
-                    esc_html__(
-                        'A valid data model could not be obtained. Did you supply a valid entity type?',
-                        'event_espresso'
-                    )
-                );
-            }
+			$deletionMethod = __NAMESPACE__;
+			// if it's for datetimes.
+			if ($details['entityType'] === EEM_Datetime::instance()->item_name()) {
+				$deletionMethod .= '\DatetimeDelete::' . ($deletePermanently ? 'deleteDatetimeAndRelations' : 'trashDatetimeAndRelations');
+			} elseif ($details['entityType'] === EEM_Ticket::instance()->item_name()) {
+				$deletionMethod .= '\TicketDelete::' . ($deletePermanently ? 'deleteTicketAndRelations' : 'trashTicket');
+			} elseif ($details['entityType'] === EEM_Price::instance()->item_name()) {
+				$deletionMethod .= '\PriceDelete::deletePriceAndRelations';
+			} else {
+				throw new UserError(
+					esc_html__(
+						'A valid data model could not be obtained. Did you supply a valid entity type?',
+						'event_espresso'
+					)
+				);
+			}
 
-            $deleted = [];
-            $failed  = [];
+			$deleted = [];
+			$failed  = [];
 
-            foreach ($details['entity_db_ids'] as $key => $entityDbid) {
-                $guid = $details['entityGuids'][ $key ];
-                $entity = $details['entities'][ $entityDbid ];
-                try {
-                    $result = $deletionMethod($entity);
-                    EntityMutator::validateResults($result);
-                    // we are here it means the deletion was successful.
-                    $deleted[] = $guid;
-                } catch (Exception $e) {
-                    // sorry mate, couldn't help you :(
-                    $failed[] = $guid;
-                }
-            }
+			foreach ($details['entity_db_ids'] as $key => $entityDbid) {
+				$guid = $details['entityGuids'][ $key ];
+				$entity = $details['entities'][ $entityDbid ];
+				try {
+					$result = $deletionMethod($entity);
+					EntityMutator::validateResults($result);
+					// we are here it means the deletion was successful.
+					$deleted[] = $guid;
+				} catch (Exception $e) {
+					// sorry mate, couldn't help you :(
+					$failed[] = $guid;
+				}
+			}
 
-            return compact('deleted', 'failed');
-        };
-    }
+			return compact('deleted', 'failed');
+		};
+	}
 }

Spacing +6 added lines, -6 removed lines

@@ -38,11 +38,11 @@ 
          * @throws InvalidDataTypeException
          * @throws EE_Error
          */
-        return static function (array $input, AppContext $context, ResolveInfo $info): array {
+        return static function(array $input, AppContext $context, ResolveInfo $info): array {
             /**
              * Stop now if a user isn't allowed to delete.
              */
-            if (! current_user_can('ee_delete_events')) {
+            if ( ! current_user_can('ee_delete_events')) {
                 throw new UserError(
                     esc_html__('Sorry, you do not have the required permissions to delete entities', 'event_espresso')
                 );
@@ -55,9 +55,9 @@ 
             $deletionMethod = __NAMESPACE__;
             // if it's for datetimes.
             if ($details['entityType'] === EEM_Datetime::instance()->item_name()) {
-                $deletionMethod .= '\DatetimeDelete::' . ($deletePermanently ? 'deleteDatetimeAndRelations' : 'trashDatetimeAndRelations');
+                $deletionMethod .= '\DatetimeDelete::'.($deletePermanently ? 'deleteDatetimeAndRelations' : 'trashDatetimeAndRelations');
             } elseif ($details['entityType'] === EEM_Ticket::instance()->item_name()) {
-                $deletionMethod .= '\TicketDelete::' . ($deletePermanently ? 'deleteTicketAndRelations' : 'trashTicket');
+                $deletionMethod .= '\TicketDelete::'.($deletePermanently ? 'deleteTicketAndRelations' : 'trashTicket');
             } elseif ($details['entityType'] === EEM_Price::instance()->item_name()) {
                 $deletionMethod .= '\PriceDelete::deletePriceAndRelations';
             } else {
@@ -73,8 +73,8 @@ 
             $failed  = [];
 
             foreach ($details['entity_db_ids'] as $key => $entityDbid) {
-                $guid = $details['entityGuids'][ $key ];
-                $entity = $details['entities'][ $entityDbid ];
+                $guid = $details['entityGuids'][$key];
+                $entity = $details['entities'][$entityDbid];
                 try {
                     $result = $deletionMethod($entity);
                     EntityMutator::validateResults($result);

core/services/editor/BlockRegistrationManager.php

Spacing +8 added lines, -8 removed lines

@@ -106,7 +106,7 @@ 
     {
         // $block_editor_context can be either an object or a string
         // so checking it here, thus no type hinting in function params
-        if (! $block_editor_context instanceof WP_Block_Editor_Context) {
+        if ( ! $block_editor_context instanceof WP_Block_Editor_Context) {
             return $categories;
         }
         return array_merge(
@@ -167,7 +167,7 @@ 
             foreach ($this->blocks as $block) {
                 $block->initialize();
                 $this->trackAssetManagersForBlocks($block);
-                if (! $this->block_asset_manager_collection->has($block->assetManager())) {
+                if ( ! $this->block_asset_manager_collection->has($block->assetManager())) {
                     $this->block_asset_manager_collection->add($block->assetManager());
                     $block->assetManager()->setAssetHandles();
                 }
@@ -189,7 +189,7 @@ 
         $supported_routes = $block->supportedRoutes();
         foreach ($supported_routes as $supported_route) {
             if ($this->route_manager->routeMatchesCurrentRequest($supported_route)) {
-                $this->block_asset_managers[ $block->blockType() ] = $block->assetManager()->assetNamespace();
+                $this->block_asset_managers[$block->blockType()] = $block->assetManager()->assetNamespace();
             }
         }
     }
@@ -205,10 +205,10 @@ 
      */
     public function matchesRoute(BlockInterface $block)
     {
-        if (isset($this->block_asset_managers[ $block->blockType() ])) {
+        if (isset($this->block_asset_managers[$block->blockType()])) {
             return true;
         }
-        unset($this->block_asset_managers[ $block->blockType() ]);
+        unset($this->block_asset_managers[$block->blockType()]);
         return false;
     }
 
@@ -224,12 +224,12 @@ 
         try {
             // cycle thru block loader folders
             foreach ($this->blocks as $block) {
-                if (! $this->matchesRoute($block)) {
+                if ( ! $this->matchesRoute($block)) {
                     continue;
                 }
                 // perform any setup required for the block
                 $block_type = $block->registerBlock();
-                if (! $block_type instanceof WP_Block_Type) {
+                if ( ! $block_type instanceof WP_Block_Type) {
                     throw new InvalidEntityException($block_type, 'WP_Block_Type');
                 }
                 do_action(
@@ -248,7 +248,7 @@ 
         $assets = array_flip($this->block_asset_managers);
         foreach ($this->block_asset_manager_collection as $asset_manager) {
             // if there are no longer any blocks that require these assets,
-            if (! isset($assets[ $asset_manager->assetNamespace() ])) {
+            if ( ! isset($assets[$asset_manager->assetNamespace()])) {
                 // then unset asset enqueueing and bail
                 remove_action('wp_enqueue_scripts', array($asset_manager, 'addManifestFile'), 0);
                 remove_action('admin_enqueue_scripts', array($asset_manager, 'addManifestFile'), 0);

Indentation +203 added lines, -203 removed lines

@@ -32,228 +32,228 @@
  */
 class BlockRegistrationManager extends BlockManager
 {
-    /**
-     * @var BlockAssetManagerCollection $block_asset_manager_collection
-     */
-    protected $block_asset_manager_collection;
+	/**
+	 * @var BlockAssetManagerCollection $block_asset_manager_collection
+	 */
+	protected $block_asset_manager_collection;
 
-    /**
-     * @var RouteMatchSpecificationManager $route_manager
-     */
-    protected $route_manager;
+	/**
+	 * @var RouteMatchSpecificationManager $route_manager
+	 */
+	protected $route_manager;
 
-    /**
-     * array for tracking asset managers required by blocks for the current route
-     *
-     * @var array $block_asset_managers
-     */
-    protected $block_asset_managers = array();
+	/**
+	 * array for tracking asset managers required by blocks for the current route
+	 *
+	 * @var array $block_asset_managers
+	 */
+	protected $block_asset_managers = array();
 
 
-    /**
-     * BlockRegistrationManager constructor.
-     *
-     * @param BlockAssetManagerCollection    $block_asset_manager_collection
-     * @param BlockCollection                $blocks
-     * @param RouteMatchSpecificationManager $route_manager
-     * @param RequestInterface               $request
-     */
-    public function __construct(
-        BlockAssetManagerCollection $block_asset_manager_collection,
-        BlockCollection $blocks,
-        RouteMatchSpecificationManager $route_manager,
-        RequestInterface $request
-    ) {
-        $this->block_asset_manager_collection = $block_asset_manager_collection;
-        $this->route_manager = $route_manager;
-        parent::__construct($blocks, $request);
-    }
+	/**
+	 * BlockRegistrationManager constructor.
+	 *
+	 * @param BlockAssetManagerCollection    $block_asset_manager_collection
+	 * @param BlockCollection                $blocks
+	 * @param RouteMatchSpecificationManager $route_manager
+	 * @param RequestInterface               $request
+	 */
+	public function __construct(
+		BlockAssetManagerCollection $block_asset_manager_collection,
+		BlockCollection $blocks,
+		RouteMatchSpecificationManager $route_manager,
+		RequestInterface $request
+	) {
+		$this->block_asset_manager_collection = $block_asset_manager_collection;
+		$this->route_manager = $route_manager;
+		parent::__construct($blocks, $request);
+	}
 
 
-    /**
-     *  Returns the name of a hook point to be used to call initialize()
-     *
-     * @return string
-     */
-    public function initHook()
-    {
-        return 'AHEE__EE_System__initialize';
-    }
+	/**
+	 *  Returns the name of a hook point to be used to call initialize()
+	 *
+	 * @return string
+	 */
+	public function initHook()
+	{
+		return 'AHEE__EE_System__initialize';
+	}
 
 
-    /**
-     * Perform any early setup required for block editors to functions
-     *
-     * @return void
-     * @throws Exception
-     */
-    public function initialize()
-    {
-        $this->initializeBlocks();
-        add_action('AHEE__EE_System__initialize_last', array($this, 'registerBlocks'));
-        add_action('wp_loaded', array($this, 'unloadAssets'));
-        add_filter('block_categories_all', array($this, 'addEspressoBlockCategories'), 10, 2);
-    }
+	/**
+	 * Perform any early setup required for block editors to functions
+	 *
+	 * @return void
+	 * @throws Exception
+	 */
+	public function initialize()
+	{
+		$this->initializeBlocks();
+		add_action('AHEE__EE_System__initialize_last', array($this, 'registerBlocks'));
+		add_action('wp_loaded', array($this, 'unloadAssets'));
+		add_filter('block_categories_all', array($this, 'addEspressoBlockCategories'), 10, 2);
+	}
 
 
-    /**
-     * @param array $categories
-     * @since 4.9.71.p
-     * @return array
-     */
-    public function addEspressoBlockCategories(array $categories, $block_editor_context): array
-    {
-        // $block_editor_context can be either an object or a string
-        // so checking it here, thus no type hinting in function params
-        if (! $block_editor_context instanceof WP_Block_Editor_Context) {
-            return $categories;
-        }
-        return array_merge(
-            $categories,
-            [
-                [
-                    'slug'  => 'event-espresso',
-                    'title' => esc_html__('Event Espresso', 'event_espresso'),
-                ]
-            ]
-        );
-    }
+	/**
+	 * @param array $categories
+	 * @since 4.9.71.p
+	 * @return array
+	 */
+	public function addEspressoBlockCategories(array $categories, $block_editor_context): array
+	{
+		// $block_editor_context can be either an object or a string
+		// so checking it here, thus no type hinting in function params
+		if (! $block_editor_context instanceof WP_Block_Editor_Context) {
+			return $categories;
+		}
+		return array_merge(
+			$categories,
+			[
+				[
+					'slug'  => 'event-espresso',
+					'title' => esc_html__('Event Espresso', 'event_espresso'),
+				]
+			]
+		);
+	}
 
 
-    /**
-     * @return CollectionInterface|BlockInterface[]
-     * @throws CollectionLoaderException
-     * @throws CollectionDetailsException
-     */
-    protected function populateBlockCollection()
-    {
-        $loader = new CollectionLoader(
-            new CollectionDetails(
-                // collection name
-                'editor_blocks',
-                // collection interface
-                'EventEspresso\core\domain\entities\editor\BlockInterface',
-                // FQCNs for classes to add (all classes within each namespace will be loaded)
-                apply_filters(
-                    'FHEE__EventEspresso_core_services_editor_BlockManager__populateBlockCollection__collection_FQCNs',
-                    array('EventEspresso\core\domain\entities\editor\blocks')
-                ),
-                // file paths to classes to add
-                array(),
-                // file mask to use if parsing folder for files to add
-                '',
-                // what to use as identifier for collection entities
-                // using CLASS NAME prevents duplicates (works like a singleton)
-                CollectionDetails::ID_CLASS_NAME
-            ),
-            $this->blocks
-        );
-        return $loader->getCollection();
-    }
+	/**
+	 * @return CollectionInterface|BlockInterface[]
+	 * @throws CollectionLoaderException
+	 * @throws CollectionDetailsException
+	 */
+	protected function populateBlockCollection()
+	{
+		$loader = new CollectionLoader(
+			new CollectionDetails(
+				// collection name
+				'editor_blocks',
+				// collection interface
+				'EventEspresso\core\domain\entities\editor\BlockInterface',
+				// FQCNs for classes to add (all classes within each namespace will be loaded)
+				apply_filters(
+					'FHEE__EventEspresso_core_services_editor_BlockManager__populateBlockCollection__collection_FQCNs',
+					array('EventEspresso\core\domain\entities\editor\blocks')
+				),
+				// file paths to classes to add
+				array(),
+				// file mask to use if parsing folder for files to add
+				'',
+				// what to use as identifier for collection entities
+				// using CLASS NAME prevents duplicates (works like a singleton)
+				CollectionDetails::ID_CLASS_NAME
+			),
+			$this->blocks
+		);
+		return $loader->getCollection();
+	}
 
 
-    /**
-     * populates the BlockCollection and calls initialize() on all installed blocks
-     *
-     * @return void
-     * @throws Exception
-     */
-    public function initializeBlocks()
-    {
-        try {
-            $this->populateBlockCollection();
-            // cycle thru block loaders and initialize each loader
-            foreach ($this->blocks as $block) {
-                $block->initialize();
-                $this->trackAssetManagersForBlocks($block);
-                if (! $this->block_asset_manager_collection->has($block->assetManager())) {
-                    $this->block_asset_manager_collection->add($block->assetManager());
-                    $block->assetManager()->setAssetHandles();
-                }
-            }
-        } catch (Exception $exception) {
-            new ExceptionStackTraceDisplay($exception);
-        }
-    }
+	/**
+	 * populates the BlockCollection and calls initialize() on all installed blocks
+	 *
+	 * @return void
+	 * @throws Exception
+	 */
+	public function initializeBlocks()
+	{
+		try {
+			$this->populateBlockCollection();
+			// cycle thru block loaders and initialize each loader
+			foreach ($this->blocks as $block) {
+				$block->initialize();
+				$this->trackAssetManagersForBlocks($block);
+				if (! $this->block_asset_manager_collection->has($block->assetManager())) {
+					$this->block_asset_manager_collection->add($block->assetManager());
+					$block->assetManager()->setAssetHandles();
+				}
+			}
+		} catch (Exception $exception) {
+			new ExceptionStackTraceDisplay($exception);
+		}
+	}
 
 
-    /**
-     * track blocks with routes that match the current request
-     *
-     * @param BlockInterface $block
-     * @throws InvalidClassException
-     */
-    private function trackAssetManagersForBlocks(BlockInterface $block)
-    {
-        $supported_routes = $block->supportedRoutes();
-        foreach ($supported_routes as $supported_route) {
-            if ($this->route_manager->routeMatchesCurrentRequest($supported_route)) {
-                $this->block_asset_managers[ $block->blockType() ] = $block->assetManager()->assetNamespace();
-            }
-        }
-    }
+	/**
+	 * track blocks with routes that match the current request
+	 *
+	 * @param BlockInterface $block
+	 * @throws InvalidClassException
+	 */
+	private function trackAssetManagersForBlocks(BlockInterface $block)
+	{
+		$supported_routes = $block->supportedRoutes();
+		foreach ($supported_routes as $supported_route) {
+			if ($this->route_manager->routeMatchesCurrentRequest($supported_route)) {
+				$this->block_asset_managers[ $block->blockType() ] = $block->assetManager()->assetNamespace();
+			}
+		}
+	}
 
 
-    /**
-     * returns true if the block should be registered for the current request
-     * else removes block from block_routes array and returns false
-     *
-     * @param BlockInterface $block
-     * @return boolean
-     * @throws InvalidClassException
-     */
-    public function matchesRoute(BlockInterface $block)
-    {
-        if (isset($this->block_asset_managers[ $block->blockType() ])) {
-            return true;
-        }
-        unset($this->block_asset_managers[ $block->blockType() ]);
-        return false;
-    }
+	/**
+	 * returns true if the block should be registered for the current request
+	 * else removes block from block_routes array and returns false
+	 *
+	 * @param BlockInterface $block
+	 * @return boolean
+	 * @throws InvalidClassException
+	 */
+	public function matchesRoute(BlockInterface $block)
+	{
+		if (isset($this->block_asset_managers[ $block->blockType() ])) {
+			return true;
+		}
+		unset($this->block_asset_managers[ $block->blockType() ]);
+		return false;
+	}
 
 
-    /**
-     * calls registerBlock() and load assets for all installed blocks
-     *
-     * @return void
-     * @throws Exception
-     */
-    public function registerBlocks()
-    {
-        try {
-            // cycle thru block loader folders
-            foreach ($this->blocks as $block) {
-                if (! $this->matchesRoute($block)) {
-                    continue;
-                }
-                // perform any setup required for the block
-                $block_type = $block->registerBlock();
-                if (! $block_type instanceof WP_Block_Type) {
-                    throw new InvalidEntityException($block_type, 'WP_Block_Type');
-                }
-                do_action(
-                    'FHEE__EventEspresso_core_services_editor_BlockManager__registerBlocks__block_type_registered',
-                    $block,
-                    $block_type
-                );
-            }
-        } catch (Exception $exception) {
-            new ExceptionStackTraceDisplay($exception);
-        }
-    }
+	/**
+	 * calls registerBlock() and load assets for all installed blocks
+	 *
+	 * @return void
+	 * @throws Exception
+	 */
+	public function registerBlocks()
+	{
+		try {
+			// cycle thru block loader folders
+			foreach ($this->blocks as $block) {
+				if (! $this->matchesRoute($block)) {
+					continue;
+				}
+				// perform any setup required for the block
+				$block_type = $block->registerBlock();
+				if (! $block_type instanceof WP_Block_Type) {
+					throw new InvalidEntityException($block_type, 'WP_Block_Type');
+				}
+				do_action(
+					'FHEE__EventEspresso_core_services_editor_BlockManager__registerBlocks__block_type_registered',
+					$block,
+					$block_type
+				);
+			}
+		} catch (Exception $exception) {
+			new ExceptionStackTraceDisplay($exception);
+		}
+	}
 
-    public function unloadAssets()
-    {
-        $assets = array_flip($this->block_asset_managers);
-        foreach ($this->block_asset_manager_collection as $asset_manager) {
-            // if there are no longer any blocks that require these assets,
-            if (! isset($assets[ $asset_manager->assetNamespace() ])) {
-                // then unset asset enqueueing and bail
-                remove_action('wp_enqueue_scripts', array($asset_manager, 'addManifestFile'), 0);
-                remove_action('admin_enqueue_scripts', array($asset_manager, 'addManifestFile'), 0);
-                remove_action('wp_enqueue_scripts', array($asset_manager, 'addAssets'), 2);
-                remove_action('admin_enqueue_scripts', array($asset_manager, 'addAssets'), 2);
-            }
-        }
-    }
+	public function unloadAssets()
+	{
+		$assets = array_flip($this->block_asset_managers);
+		foreach ($this->block_asset_manager_collection as $asset_manager) {
+			// if there are no longer any blocks that require these assets,
+			if (! isset($assets[ $asset_manager->assetNamespace() ])) {
+				// then unset asset enqueueing and bail
+				remove_action('wp_enqueue_scripts', array($asset_manager, 'addManifestFile'), 0);
+				remove_action('admin_enqueue_scripts', array($asset_manager, 'addManifestFile'), 0);
+				remove_action('wp_enqueue_scripts', array($asset_manager, 'addAssets'), 2);
+				remove_action('admin_enqueue_scripts', array($asset_manager, 'addAssets'), 2);
+			}
+		}
+	}
 }

caffeinated/admin/extend/events/templates/default_tickets_moved_notice.template.php

Indentation +21 added lines, -21 removed lines

@@ -2,12 +2,12 @@ 
     <h2><?php esc_html_e('The Default Tickets Manager Has Moved', 'event_espresso'); ?></h2>
     <p>
         <?php printf(
-            esc_html__(
-                'The new Advanced Event Editor introduces a better way to manage your events\' default tickets...%1$sright from within the Event Editor.',
-                'event_espresso'
-            ),
-            '<br />'
-        ); ?>
+			esc_html__(
+				'The new Advanced Event Editor introduces a better way to manage your events\' default tickets...%1$sright from within the Event Editor.',
+				'event_espresso'
+			),
+			'<br />'
+		); ?>
     </p>
     <h5>
         <?php esc_html_e('Follow these simple steps:', 'event_espresso'); ?>
@@ -19,26 +19,26 @@ 
         <li>
             <p>
                 <?php esc_html_e(
-                    'Edit an existing event or create a new event by clicking the "Add New Event" button.',
-                    'event_espresso'
-                ); ?>
+					'Edit an existing event or create a new event by clicking the "Add New Event" button.',
+					'event_espresso'
+				); ?>
             </p>
         </li>
         <li>
             <p>
                 <?php esc_html_e(
-                    'Scroll down to the bottom of the "Available Tickets" list.',
-                    'event_espresso'
-                ); ?>
+					'Scroll down to the bottom of the "Available Tickets" list.',
+					'event_espresso'
+				); ?>
             </p>
         </li>
         <li>
             <div>
                 <p>
                     <?php esc_html_e(
-                        'Click on the menu button to the right of the "Add New Ticket" button.',
-                        'event_espresso'
-                    ); ?>
+						'Click on the menu button to the right of the "Add New Ticket" button.',
+						'event_espresso'
+					); ?>
                 </p>
                 <img alt='event editor tickets list menu button'
                      class='ee-new-edtr-notice__img'
@@ -50,9 +50,9 @@ 
             <div>
             <p>
                 <?php esc_html_e(
-                    'Click on the "Default Tickets" option to open the "Default Tickets Manager".',
-                    'event_espresso'
-                ); ?>
+					'Click on the "Default Tickets" option to open the "Default Tickets Manager".',
+					'event_espresso'
+				); ?>
             </p>
             <img alt='event editor default tickets menu option'
                  class='ee-new-edtr-notice__img'
@@ -64,9 +64,9 @@ 
             <div>
             <p>
                 <?php esc_html_e(
-                    'Use existing tickets as templates or create new default tickets from scratch',
-                    'event_espresso'
-                ); ?>
+					'Use existing tickets as templates or create new default tickets from scratch',
+					'event_espresso'
+				); ?>
             </p>
             <img alt='the default tickets editor'
                  class="ee-new-edtr-notice__img ee-new-edtr-notice__img--big"

Spacing +3 added lines, -3 removed lines

@@ -42,7 +42,7 @@ 
                 </p>
                 <img alt='event editor tickets list menu button'
                      class='ee-new-edtr-notice__img'
-                     src='<?php echo esc_url_raw(EE_IMAGES_URL . 'default-tickets-1.png'); ?>'
+                     src='<?php echo esc_url_raw(EE_IMAGES_URL.'default-tickets-1.png'); ?>'
                 />
             </div>
         </li>
@@ -56,7 +56,7 @@ 
             </p>
             <img alt='event editor default tickets menu option'
                  class='ee-new-edtr-notice__img'
-                 src='<?php echo esc_url_raw(EE_IMAGES_URL . 'default-tickets-2.png'); ?>'
+                 src='<?php echo esc_url_raw(EE_IMAGES_URL.'default-tickets-2.png'); ?>'
             />
             </div>
         </li>
@@ -70,7 +70,7 @@ 
             </p>
             <img alt='the default tickets editor'
                  class="ee-new-edtr-notice__img ee-new-edtr-notice__img--big"
-                 src='<?php echo esc_url_raw(EE_IMAGES_URL . 'default-tickets-3.png'); ?>'
+                 src='<?php echo esc_url_raw(EE_IMAGES_URL.'default-tickets-3.png'); ?>'
             />
             </div>
         </li>

core/domain/services/capabilities/FeatureFlags.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
      */
     public function allowed(string $feature): bool
     {
-        $flag = $this->feature_flags[ $feature ] ?? false;
+        $flag = $this->feature_flags[$feature] ?? false;
         try {
             return $flag instanceof CapCheck
                 ? $this->capabilities_checker->processCapCheck($flag)

Indentation +68 added lines, -68 removed lines

@@ -15,80 +15,80 @@
  */
 class FeatureFlags
 {
-    /**
-     * @var CapabilitiesChecker $capabilities_checker
-     */
-    private $capabilities_checker;
+	/**
+	 * @var CapabilitiesChecker $capabilities_checker
+	 */
+	private $capabilities_checker;
 
-    /**
-     * array of key value pairs where the key is the feature flag in question
-     * and the value is either a boolean or a CapCheck object defining the required permissions
-     * example:
-     *       [
-     *          'use_bulk_edit' => true,
-     *          'use_death_ray' => new CapCheck( 'ee-death-ray-cap', 'context-desc' )
-     *      ]
-     * array is filterable via FHEE__EventEspresso_core_domain_services_capabilities_FeatureFlags
-     *
-     * @var boolean[]|CapCheck[]
-     */
-    private $feature_flags;
+	/**
+	 * array of key value pairs where the key is the feature flag in question
+	 * and the value is either a boolean or a CapCheck object defining the required permissions
+	 * example:
+	 *       [
+	 *          'use_bulk_edit' => true,
+	 *          'use_death_ray' => new CapCheck( 'ee-death-ray-cap', 'context-desc' )
+	 *      ]
+	 * array is filterable via FHEE__EventEspresso_core_domain_services_capabilities_FeatureFlags
+	 *
+	 * @var boolean[]|CapCheck[]
+	 */
+	private $feature_flags;
 
-    /**
-     * @var Domain
-     */
-    protected $domain;
+	/**
+	 * @var Domain
+	 */
+	protected $domain;
 
 
-    /**
-     * FeatureFlags constructor.
-     *
-     * @param CapabilitiesChecker $capabilities_checker
-     * @param Domain              $domain
-     */
-    public function __construct(CapabilitiesChecker $capabilities_checker, Domain $domain)
-    {
-        $this->capabilities_checker = $capabilities_checker;
-        $this->domain = $domain;
-        $this->feature_flags = apply_filters(
-            'FHEE__EventEspresso_core_domain_services_capabilities_FeatureFlags',
-            [
-                'ee_advanced_event_editor'   => $this->domain->isCaffeinated() && ! $this->domain->isMultiSite(),
-                'ee_event_editor_bulk_edit'  => $this->domain->isCaffeinated() && ! $this->domain->isMultiSite(),
-                'use_default_ticket_manager' => true,
-                'use_event_description_rte'  => false,
-                'use_experimental_rte'       => false,
-                'use_reg_form_builder'       => false,
-                'use_reg_options_meta_box'   => false,
-            ]
-        );
-    }
+	/**
+	 * FeatureFlags constructor.
+	 *
+	 * @param CapabilitiesChecker $capabilities_checker
+	 * @param Domain              $domain
+	 */
+	public function __construct(CapabilitiesChecker $capabilities_checker, Domain $domain)
+	{
+		$this->capabilities_checker = $capabilities_checker;
+		$this->domain = $domain;
+		$this->feature_flags = apply_filters(
+			'FHEE__EventEspresso_core_domain_services_capabilities_FeatureFlags',
+			[
+				'ee_advanced_event_editor'   => $this->domain->isCaffeinated() && ! $this->domain->isMultiSite(),
+				'ee_event_editor_bulk_edit'  => $this->domain->isCaffeinated() && ! $this->domain->isMultiSite(),
+				'use_default_ticket_manager' => true,
+				'use_event_description_rte'  => false,
+				'use_experimental_rte'       => false,
+				'use_reg_form_builder'       => false,
+				'use_reg_options_meta_box'   => false,
+			]
+		);
+	}
 
 
-    /**
-     * @param string $feature
-     * @return bool
-     */
-    public function allowed(string $feature): bool
-    {
-        $flag = $this->feature_flags[ $feature ] ?? false;
-        try {
-            return $flag instanceof CapCheck
-                ? $this->capabilities_checker->processCapCheck($flag)
-                : filter_var($flag, FILTER_VALIDATE_BOOLEAN);
-        } catch (InsufficientPermissionsException $e) {
-            // eat the exception
-        }
-        return false;
-    }
+	/**
+	 * @param string $feature
+	 * @return bool
+	 */
+	public function allowed(string $feature): bool
+	{
+		$flag = $this->feature_flags[ $feature ] ?? false;
+		try {
+			return $flag instanceof CapCheck
+				? $this->capabilities_checker->processCapCheck($flag)
+				: filter_var($flag, FILTER_VALIDATE_BOOLEAN);
+		} catch (InsufficientPermissionsException $e) {
+			// eat the exception
+		}
+		return false;
+	}
 
 
-    /**
-     * @return array
-     */
-    public function getAllowedFeatures(): array
-    {
-        $allowed = array_filter($this->feature_flags, [$this, 'allowed'], ARRAY_FILTER_USE_KEY);
-        return array_keys($allowed);
-    }
+	/**
+	 * @return array
+	 */
+	public function getAllowedFeatures(): array
+	{
+		$allowed = array_filter($this->feature_flags, [$this, 'allowed'], ARRAY_FILTER_USE_KEY);
+		return array_keys($allowed);
+	}
 }

core/libraries/shortcodes/EE_Line_Item_List_Shortcodes.lib.php

Spacing +10 added lines, -10 removed lines

@@ -71,7 +71,7 @@ 
     {
         $this->_validate_list_requirements();
 
-        if (! $this->_data['data'] instanceof EE_Ticket) {
+        if ( ! $this->_data['data'] instanceof EE_Ticket) {
             return '';
         }
 
@@ -82,10 +82,10 @@ 
         $addressee_obj = $this->_extra_data['data'];
 
         // made it here so we have an EE_Ticket, so we should have what we need.
-        $ticket_line_item = isset($addressee_obj->tickets[ $ticket->ID() ]['line_item'])
-            ? $addressee_obj->tickets[ $ticket->ID() ]['line_item'] : null;
-        $sub_line_items = isset($addressee_obj->tickets[ $ticket->ID() ]['sub_line_items'])
-            ? $addressee_obj->tickets[ $ticket->ID() ]['sub_line_items'] : array();
+        $ticket_line_item = isset($addressee_obj->tickets[$ticket->ID()]['line_item'])
+            ? $addressee_obj->tickets[$ticket->ID()]['line_item'] : null;
+        $sub_line_items = isset($addressee_obj->tickets[$ticket->ID()]['sub_line_items'])
+            ? $addressee_obj->tickets[$ticket->ID()]['sub_line_items'] : array();
 
         $template = count($sub_line_items) < 2 ? $templates['ticket_line_item_no_pms']
             : $templates['ticket_line_item_pms'];
@@ -120,7 +120,7 @@ 
         }
         $this->_validate_list_requirements();
 
-        if (! $this->_data['data'] instanceof EE_Messages_Addressee) {
+        if ( ! $this->_data['data'] instanceof EE_Messages_Addressee) {
             return '';
         }
 
@@ -154,7 +154,7 @@ 
 
         $this->_validate_list_requirements();
 
-        if (! $this->_data['data'] instanceof EE_Messages_Addressee) {
+        if ( ! $this->_data['data'] instanceof EE_Messages_Addressee) {
             return '';
         }
 
@@ -190,7 +190,7 @@ 
     {
         $this->_validate_list_requirements();
 
-        if (! $this->_data['data'] instanceof EE_Line_Item) {
+        if ( ! $this->_data['data'] instanceof EE_Line_Item) {
             return '';
         }
 
@@ -203,8 +203,8 @@ 
 
         $main_line_item_id = $main_line_item->ID();
 
-        $price_mod_line_items = ! empty($addressee_obj->line_items_with_children[ $main_line_item_id ]['children'])
-            ? $addressee_obj->line_items_with_children[ $main_line_item_id ]['children'] : array();
+        $price_mod_line_items = ! empty($addressee_obj->line_items_with_children[$main_line_item_id]['children'])
+            ? $addressee_obj->line_items_with_children[$main_line_item_id]['children'] : array();
 
         $line_item_list = '';
 

Indentation +197 added lines, -197 removed lines

@@ -20,201 +20,201 @@
  */
 class EE_Line_Item_List_Shortcodes extends EE_Shortcodes
 {
-    protected function _init_props()
-    {
-        $this->label = esc_html__('Line Item List Shortcodes', 'event_espresso');
-        $this->description = esc_html__('All shortcodes specific to line item lists', 'event_espresso');
-        $this->_shortcodes = array(
-            '[TICKET_LINE_ITEM_LIST]'         => esc_html__('Outputs a list of ticket line items.', 'event_espresso'),
-            '[TAX_LINE_ITEM_LIST]'            => esc_html__('Outputs a list of tax line items.', 'event_espresso'),
-            '[ADDITIONAL_LINE_ITEM_LIST]'     => esc_html__(
-                'Outputs a list of additional line items (other charges or discounts)',
-                'event_espresso'
-            ),
-            '[PRICE_MODIFIER_LINE_ITEM_LIST]' => esc_html__('Outputs a list of price modifier line items', 'event_espresso'),
-        );
-    }
-
-
-    protected function _parser($shortcode)
-    {
-
-        switch ($shortcode) {
-            case '[TICKET_LINE_ITEM_LIST]':
-                return $this->_get_ticket_line_item_list();
-
-            case '[TAX_LINE_ITEM_LIST]':
-                return $this->_get_tax_line_item_list();
-
-            case '[PRICE_MODIFIER_LINE_ITEM_LIST]':
-                return $this->_get_price_mod_line_item_list();
-
-            case '[ADDITIONAL_LINE_ITEM_LIST]':
-                return $this->_get_additional_line_item_list();
-
-            default:
-                return '';
-        }
-    }
-
-
-    /**
-     * verify incoming data contains what is needed for retrieving and parsing each ticket line item for an event.
-     *
-     * @since 4.5.0
-     *
-     * @return string parsed ticket line item list.
-     */
-    private function _get_ticket_line_item_list()
-    {
-        $this->_validate_list_requirements();
-
-        if (! $this->_data['data'] instanceof EE_Ticket) {
-            return '';
-        }
-
-        $valid_shortcodes = array('line_item', 'line_item_list', 'ticket');
-
-        $ticket = $this->_data['data'];
-        $templates = $this->_extra_data['template'];
-        $addressee_obj = $this->_extra_data['data'];
-
-        // made it here so we have an EE_Ticket, so we should have what we need.
-        $ticket_line_item = isset($addressee_obj->tickets[ $ticket->ID() ]['line_item'])
-            ? $addressee_obj->tickets[ $ticket->ID() ]['line_item'] : null;
-        $sub_line_items = isset($addressee_obj->tickets[ $ticket->ID() ]['sub_line_items'])
-            ? $addressee_obj->tickets[ $ticket->ID() ]['sub_line_items'] : array();
-
-        $template = count($sub_line_items) < 2 ? $templates['ticket_line_item_no_pms']
-            : $templates['ticket_line_item_pms'];
-
-        if (empty($ticket_line_item) || empty($sub_line_items)) {
-            return '';
-        }
-
-        // now we just return the appropriate template parsed for each ticket.
-        return $this->_shortcode_helper->parse_line_item_list_template(
-            $template,
-            $ticket_line_item,
-            $valid_shortcodes,
-            $this->_extra_data
-        );
-    }
-
-
-    /**
-     * Verify incoming data contains what is needed for retrieving and parsing each tax line item for a transaction.
-     *
-     * @return string  parsed tax line item list.
-     * @throws EE_Error
-     * @since 4.5.0
-     */
-    private function _get_tax_line_item_list()
-    {
-        /** @var EE_Admin_Config $admin_config */
-        $admin_config = LoaderFactory::getShared(EE_Admin_Config::class);
-        if ($admin_config->useAdvancedEditor()) {
-            return '';
-        }
-        $this->_validate_list_requirements();
-
-        if (! $this->_data['data'] instanceof EE_Messages_Addressee) {
-            return '';
-        }
-
-        // made it here so we're good to go.
-        $valid_shortcodes = array('line_item');
-        $templates = $this->_data['template'];
-
-        $tax_line_items = $this->_data['data']->tax_line_items;
-        $line_item_list = '';
-        foreach ($tax_line_items as $line_item) {
-            $line_item_list .= $this->_shortcode_helper->parse_line_item_list_template(
-                $templates['tax_line_item_list'],
-                $line_item,
-                $valid_shortcodes,
-                $this->_extra_data
-            );
-        }
-
-        return $line_item_list;
-    }
-
-    /**
-     * Verify incoming data contains what is needed for retrieving and parsing each other line item for a transaction.
-     *
-     * @since 4.5.0
-     *
-     * @return string  parsed other line item list.
-     */
-    private function _get_additional_line_item_list()
-    {
-
-        $this->_validate_list_requirements();
-
-        if (! $this->_data['data'] instanceof EE_Messages_Addressee) {
-            return '';
-        }
-
-        // made it here so we're good to go.
-        $valid_shortcodes = array('line_item');
-        $templates = $this->_data['template'];
-
-        $additional_line_items = $this->_data['data']->additional_line_items;
-        $line_item_list = '';
-        foreach ($additional_line_items as $line_item) {
-            $line_item_list .= $this->_shortcode_helper->parse_line_item_list_template(
-                $templates['additional_line_item_list'],
-                $line_item,
-                $valid_shortcodes,
-                $this->_extra_data
-            );
-        }
-
-
-        return $line_item_list;
-    }
-
-
-    /**
-     * Verify incoming data contains what is needed for retrieving and parsing each price modifier line item for a
-     * parent ticket line item.
-     *
-     * @since 4.5.0
-     *
-     * @return string parsed price modifier line item list.
-     */
-    private function _get_price_mod_line_item_list()
-    {
-        $this->_validate_list_requirements();
-
-        if (! $this->_data['data'] instanceof EE_Line_Item) {
-            return '';
-        }
-
-        // made it here so we're good to go.
-        $main_line_item = $this->_data['data'];
-        $templates = $this->_extra_data['template'];
-        $addressee_obj = $this->_extra_data['data'];
-
-        $valid_shortcodes = array('line_item');
-
-        $main_line_item_id = $main_line_item->ID();
-
-        $price_mod_line_items = ! empty($addressee_obj->line_items_with_children[ $main_line_item_id ]['children'])
-            ? $addressee_obj->line_items_with_children[ $main_line_item_id ]['children'] : array();
-
-        $line_item_list = '';
-
-        foreach ($price_mod_line_items as $line_item) {
-            $line_item_list .= $this->_shortcode_helper->parse_line_item_list_template(
-                $templates['price_modifier_line_item_list'],
-                $line_item,
-                $valid_shortcodes,
-                $this->_extra_data
-            );
-        }
-
-        return $line_item_list;
-    }
+	protected function _init_props()
+	{
+		$this->label = esc_html__('Line Item List Shortcodes', 'event_espresso');
+		$this->description = esc_html__('All shortcodes specific to line item lists', 'event_espresso');
+		$this->_shortcodes = array(
+			'[TICKET_LINE_ITEM_LIST]'         => esc_html__('Outputs a list of ticket line items.', 'event_espresso'),
+			'[TAX_LINE_ITEM_LIST]'            => esc_html__('Outputs a list of tax line items.', 'event_espresso'),
+			'[ADDITIONAL_LINE_ITEM_LIST]'     => esc_html__(
+				'Outputs a list of additional line items (other charges or discounts)',
+				'event_espresso'
+			),
+			'[PRICE_MODIFIER_LINE_ITEM_LIST]' => esc_html__('Outputs a list of price modifier line items', 'event_espresso'),
+		);
+	}
+
+
+	protected function _parser($shortcode)
+	{
+
+		switch ($shortcode) {
+			case '[TICKET_LINE_ITEM_LIST]':
+				return $this->_get_ticket_line_item_list();
+
+			case '[TAX_LINE_ITEM_LIST]':
+				return $this->_get_tax_line_item_list();
+
+			case '[PRICE_MODIFIER_LINE_ITEM_LIST]':
+				return $this->_get_price_mod_line_item_list();
+
+			case '[ADDITIONAL_LINE_ITEM_LIST]':
+				return $this->_get_additional_line_item_list();
+
+			default:
+				return '';
+		}
+	}
+
+
+	/**
+	 * verify incoming data contains what is needed for retrieving and parsing each ticket line item for an event.
+	 *
+	 * @since 4.5.0
+	 *
+	 * @return string parsed ticket line item list.
+	 */
+	private function _get_ticket_line_item_list()
+	{
+		$this->_validate_list_requirements();
+
+		if (! $this->_data['data'] instanceof EE_Ticket) {
+			return '';
+		}
+
+		$valid_shortcodes = array('line_item', 'line_item_list', 'ticket');
+
+		$ticket = $this->_data['data'];
+		$templates = $this->_extra_data['template'];
+		$addressee_obj = $this->_extra_data['data'];
+
+		// made it here so we have an EE_Ticket, so we should have what we need.
+		$ticket_line_item = isset($addressee_obj->tickets[ $ticket->ID() ]['line_item'])
+			? $addressee_obj->tickets[ $ticket->ID() ]['line_item'] : null;
+		$sub_line_items = isset($addressee_obj->tickets[ $ticket->ID() ]['sub_line_items'])
+			? $addressee_obj->tickets[ $ticket->ID() ]['sub_line_items'] : array();
+
+		$template = count($sub_line_items) < 2 ? $templates['ticket_line_item_no_pms']
+			: $templates['ticket_line_item_pms'];
+
+		if (empty($ticket_line_item) || empty($sub_line_items)) {
+			return '';
+		}
+
+		// now we just return the appropriate template parsed for each ticket.
+		return $this->_shortcode_helper->parse_line_item_list_template(
+			$template,
+			$ticket_line_item,
+			$valid_shortcodes,
+			$this->_extra_data
+		);
+	}
+
+
+	/**
+	 * Verify incoming data contains what is needed for retrieving and parsing each tax line item for a transaction.
+	 *
+	 * @return string  parsed tax line item list.
+	 * @throws EE_Error
+	 * @since 4.5.0
+	 */
+	private function _get_tax_line_item_list()
+	{
+		/** @var EE_Admin_Config $admin_config */
+		$admin_config = LoaderFactory::getShared(EE_Admin_Config::class);
+		if ($admin_config->useAdvancedEditor()) {
+			return '';
+		}
+		$this->_validate_list_requirements();
+
+		if (! $this->_data['data'] instanceof EE_Messages_Addressee) {
+			return '';
+		}
+
+		// made it here so we're good to go.
+		$valid_shortcodes = array('line_item');
+		$templates = $this->_data['template'];
+
+		$tax_line_items = $this->_data['data']->tax_line_items;
+		$line_item_list = '';
+		foreach ($tax_line_items as $line_item) {
+			$line_item_list .= $this->_shortcode_helper->parse_line_item_list_template(
+				$templates['tax_line_item_list'],
+				$line_item,
+				$valid_shortcodes,
+				$this->_extra_data
+			);
+		}
+
+		return $line_item_list;
+	}
+
+	/**
+	 * Verify incoming data contains what is needed for retrieving and parsing each other line item for a transaction.
+	 *
+	 * @since 4.5.0
+	 *
+	 * @return string  parsed other line item list.
+	 */
+	private function _get_additional_line_item_list()
+	{
+
+		$this->_validate_list_requirements();
+
+		if (! $this->_data['data'] instanceof EE_Messages_Addressee) {
+			return '';
+		}
+
+		// made it here so we're good to go.
+		$valid_shortcodes = array('line_item');
+		$templates = $this->_data['template'];
+
+		$additional_line_items = $this->_data['data']->additional_line_items;
+		$line_item_list = '';
+		foreach ($additional_line_items as $line_item) {
+			$line_item_list .= $this->_shortcode_helper->parse_line_item_list_template(
+				$templates['additional_line_item_list'],
+				$line_item,
+				$valid_shortcodes,
+				$this->_extra_data
+			);
+		}
+
+
+		return $line_item_list;
+	}
+
+
+	/**
+	 * Verify incoming data contains what is needed for retrieving and parsing each price modifier line item for a
+	 * parent ticket line item.
+	 *
+	 * @since 4.5.0
+	 *
+	 * @return string parsed price modifier line item list.
+	 */
+	private function _get_price_mod_line_item_list()
+	{
+		$this->_validate_list_requirements();
+
+		if (! $this->_data['data'] instanceof EE_Line_Item) {
+			return '';
+		}
+
+		// made it here so we're good to go.
+		$main_line_item = $this->_data['data'];
+		$templates = $this->_extra_data['template'];
+		$addressee_obj = $this->_extra_data['data'];
+
+		$valid_shortcodes = array('line_item');
+
+		$main_line_item_id = $main_line_item->ID();
+
+		$price_mod_line_items = ! empty($addressee_obj->line_items_with_children[ $main_line_item_id ]['children'])
+			? $addressee_obj->line_items_with_children[ $main_line_item_id ]['children'] : array();
+
+		$line_item_list = '';
+
+		foreach ($price_mod_line_items as $line_item) {
+			$line_item_list .= $this->_shortcode_helper->parse_line_item_list_template(
+				$templates['price_modifier_line_item_list'],
+				$line_item,
+				$valid_shortcodes,
+				$this->_extra_data
+			);
+		}
+
+		return $line_item_list;
+	}
 }

core/db_classes/EE_Taxes.class.php

Spacing +3 added lines, -3 removed lines

@@ -42,7 +42,7 @@ 
         $tax = 0;
         $total_tax = 0;
         // This first checks to see if the given ticket is taxable.
-        if (! $ticket->taxable()) {
+        if ( ! $ticket->taxable()) {
             return $tax;
         }
         // get subtotal (notice we're only retrieving a subtotal if there isn't one given)
@@ -85,7 +85,7 @@ 
     public static function get_subtotal_for_admin(EE_Ticket $ticket)
     {
         $TKT_ID = $ticket->ID();
-        return EE_Taxes::$_subtotal[ $TKT_ID ] ?? EE_Taxes::_get_subtotal_for_admin($ticket);
+        return EE_Taxes::$_subtotal[$TKT_ID] ?? EE_Taxes::_get_subtotal_for_admin($ticket);
     }
 
 
@@ -133,7 +133,7 @@ 
             }
         }
         $TKT_ID = $ticket->ID();
-        EE_Taxes::$_subtotal[ $TKT_ID ] = $subtotal;
+        EE_Taxes::$_subtotal[$TKT_ID] = $subtotal;
         return $subtotal;
     }
 

Indentation +132 added lines, -132 removed lines

@@ -11,147 +11,147 @@
  */
 class EE_Taxes extends EE_Base
 {
-    /**
-     * This is used for when EE_Taxes is used statically by the admin
-     *
-     * @var array
-     */
-    private static $_subtotal = [];
+	/**
+	 * This is used for when EE_Taxes is used statically by the admin
+	 *
+	 * @var array
+	 */
+	private static $_subtotal = [];
 
-    /**
-     * This holds an array of EE_Price objects that are of PRT_ID == 4 (tax price types)
-     *
-     * @var EE_Price[]
-     */
-    private static $_default_taxes = [];
+	/**
+	 * This holds an array of EE_Price objects that are of PRT_ID == 4 (tax price types)
+	 *
+	 * @var EE_Price[]
+	 */
+	private static $_default_taxes = [];
 
 
-    /**
-     * This method simply calculates the total taxes for a given ticket (by pulling the prices attached to the ticket
-     * and applying default taxes to it). Note: this is just an intermediary helper method added to facilitate quick
-     * calc of taxes for tickets listed in the event editor.
-     *
-     * @param EE_Ticket $ticket incoming EE_Ticket
-     * @return float             total taxes to apply to ticket.
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public static function get_total_taxes_for_admin(EE_Ticket $ticket)
-    {
-        $tax = 0;
-        $total_tax = 0;
-        // This first checks to see if the given ticket is taxable.
-        if (! $ticket->taxable()) {
-            return $tax;
-        }
-        // get subtotal (notice we're only retrieving a subtotal if there isn't one given)
-        $subtotal = EE_Taxes::get_subtotal_for_admin($ticket);
-        // get taxes
-        $taxes = EE_Taxes::get_taxes_for_admin();
-        // apply taxes to subtotal
-        foreach ($taxes as $tax) {
-            // assuming taxes are not cumulative
-            $total_tax += $subtotal * $tax->amount() / 100;
-        }
-        return $total_tax;
-    }
+	/**
+	 * This method simply calculates the total taxes for a given ticket (by pulling the prices attached to the ticket
+	 * and applying default taxes to it). Note: this is just an intermediary helper method added to facilitate quick
+	 * calc of taxes for tickets listed in the event editor.
+	 *
+	 * @param EE_Ticket $ticket incoming EE_Ticket
+	 * @return float             total taxes to apply to ticket.
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public static function get_total_taxes_for_admin(EE_Ticket $ticket)
+	{
+		$tax = 0;
+		$total_tax = 0;
+		// This first checks to see if the given ticket is taxable.
+		if (! $ticket->taxable()) {
+			return $tax;
+		}
+		// get subtotal (notice we're only retrieving a subtotal if there isn't one given)
+		$subtotal = EE_Taxes::get_subtotal_for_admin($ticket);
+		// get taxes
+		$taxes = EE_Taxes::get_taxes_for_admin();
+		// apply taxes to subtotal
+		foreach ($taxes as $tax) {
+			// assuming taxes are not cumulative
+			$total_tax += $subtotal * $tax->amount() / 100;
+		}
+		return $total_tax;
+	}
 
 
-    /**
-     * Gets the total percentage of tax that should be applied to taxable line items
-     *
-     * @return float the percentage of tax that should be added to taxable items
-     * @throws EE_Error
-     * @throws ReflectionException
-     * eg 20 for %20 tax (NOT 0.20, which
-     */
-    public static function get_total_taxes_percentage()
-    {
-        $total_tax_percent = 0;
-        foreach (EE_Taxes::get_taxes_for_admin() as $tax_price) {
-            $total_tax_percent += $tax_price->get('PRC_amount');
-        }
-        return $total_tax_percent;
-    }
+	/**
+	 * Gets the total percentage of tax that should be applied to taxable line items
+	 *
+	 * @return float the percentage of tax that should be added to taxable items
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 * eg 20 for %20 tax (NOT 0.20, which
+	 */
+	public static function get_total_taxes_percentage()
+	{
+		$total_tax_percent = 0;
+		foreach (EE_Taxes::get_taxes_for_admin() as $tax_price) {
+			$total_tax_percent += $tax_price->get('PRC_amount');
+		}
+		return $total_tax_percent;
+	}
 
 
-    /**
-     * @param EE_Ticket $ticket
-     * @return float
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public static function get_subtotal_for_admin(EE_Ticket $ticket)
-    {
-        $TKT_ID = $ticket->ID();
-        return EE_Taxes::$_subtotal[ $TKT_ID ] ?? EE_Taxes::_get_subtotal_for_admin($ticket);
-    }
+	/**
+	 * @param EE_Ticket $ticket
+	 * @return float
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public static function get_subtotal_for_admin(EE_Ticket $ticket)
+	{
+		$TKT_ID = $ticket->ID();
+		return EE_Taxes::$_subtotal[ $TKT_ID ] ?? EE_Taxes::_get_subtotal_for_admin($ticket);
+	}
 
 
-    /**
-     * simply take an incoming ticket and calculate the subtotal for the ticket
-     *
-     * @param EE_Ticket $ticket
-     * @return float     subtotal calculated from all EE_Price[] on Ticket.
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    private static function _get_subtotal_for_admin(EE_Ticket $ticket)
-    {
-        $subtotal = 0;
-        // get all prices
-        $prices = $ticket->get_many_related(
-            'Price',
-            [
-                0 => [
-                    'Price_Type.PBT_ID' => ['!=', EEM_Price_Type::base_type_tax],
-                ],
-                'default_where_conditions' => 'none',
-                'order_by'                 => ['PRC_order' => 'ASC'],
-            ]
-        );
-        // let's loop through them (base price is always the first item)
-        foreach ($prices as $price) {
-            if ($price instanceof EE_Price) {
-                $price_type = $price->type_obj();
-                if ($price_type instanceof EE_Price_Type) {
-                    switch ($price->type_obj()->base_type()) {
-                        case 1: // base price
-                        case 3: // surcharges
-                            $subtotal += $price->is_percent()
-                                ? $subtotal * $price->get('PRC_amount') / 100
-                                : $price->get('PRC_amount');
-                            break;
-                        case 2: // discounts
-                            $subtotal -= $price->is_percent()
-                                ? $subtotal * $price->get('PRC_amount') / 100
-                                : $price->get('PRC_amount');
-                            break;
-                    }
-                }
-            }
-        }
-        $TKT_ID = $ticket->ID();
-        EE_Taxes::$_subtotal[ $TKT_ID ] = $subtotal;
-        return $subtotal;
-    }
+	/**
+	 * simply take an incoming ticket and calculate the subtotal for the ticket
+	 *
+	 * @param EE_Ticket $ticket
+	 * @return float     subtotal calculated from all EE_Price[] on Ticket.
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	private static function _get_subtotal_for_admin(EE_Ticket $ticket)
+	{
+		$subtotal = 0;
+		// get all prices
+		$prices = $ticket->get_many_related(
+			'Price',
+			[
+				0 => [
+					'Price_Type.PBT_ID' => ['!=', EEM_Price_Type::base_type_tax],
+				],
+				'default_where_conditions' => 'none',
+				'order_by'                 => ['PRC_order' => 'ASC'],
+			]
+		);
+		// let's loop through them (base price is always the first item)
+		foreach ($prices as $price) {
+			if ($price instanceof EE_Price) {
+				$price_type = $price->type_obj();
+				if ($price_type instanceof EE_Price_Type) {
+					switch ($price->type_obj()->base_type()) {
+						case 1: // base price
+						case 3: // surcharges
+							$subtotal += $price->is_percent()
+								? $subtotal * $price->get('PRC_amount') / 100
+								: $price->get('PRC_amount');
+							break;
+						case 2: // discounts
+							$subtotal -= $price->is_percent()
+								? $subtotal * $price->get('PRC_amount') / 100
+								: $price->get('PRC_amount');
+							break;
+					}
+				}
+			}
+		}
+		$TKT_ID = $ticket->ID();
+		EE_Taxes::$_subtotal[ $TKT_ID ] = $subtotal;
+		return $subtotal;
+	}
 
 
-    /**
-     * get all default prices that are a Tax price type (PRT_ID = 4) and return
-     *
-     * @return EE_Price[] EE_Price objects that have PRT_ID == 4
-     * @throws EE_Error
-     */
-    public static function get_taxes_for_admin(): array
-    {
-        if (empty(EE_Taxes::$_default_taxes)) {
-            /** @var EEM_Price $price_model */
-            $price_model = LoaderFactory::getLoader()->getShared('EEM_Price');
-            EE_Taxes::$_default_taxes = $price_model->get_all(
-                [['PRC_is_default' => 1, 'Price_Type.PBT_ID' => 4]]
-            );
-        }
-        return EE_Taxes::$_default_taxes;
-    }
+	/**
+	 * get all default prices that are a Tax price type (PRT_ID = 4) and return
+	 *
+	 * @return EE_Price[] EE_Price objects that have PRT_ID == 4
+	 * @throws EE_Error
+	 */
+	public static function get_taxes_for_admin(): array
+	{
+		if (empty(EE_Taxes::$_default_taxes)) {
+			/** @var EEM_Price $price_model */
+			$price_model = LoaderFactory::getLoader()->getShared('EEM_Price');
+			EE_Taxes::$_default_taxes = $price_model->get_all(
+				[['PRC_is_default' => 1, 'Price_Type.PBT_ID' => 4]]
+			);
+		}
+		return EE_Taxes::$_default_taxes;
+	}
 }

core/interfaces/line_items/EE_Line_Item_Filter.interface.php

Indentation +7 added lines, -7 removed lines

@@ -15,11 +15,11 @@
  */
 interface EE_Line_Item_Filter
 {
-    /**
-     * process
-     *
-     * @param EE_Line_Item $line_item
-     * @return EE_Line_Item
-     */
-    public function process(EE_Line_Item $line_item): ?EE_Line_Item;
+	/**
+	 * process
+	 *
+	 * @param EE_Line_Item $line_item
+	 * @return EE_Line_Item
+	 */
+	public function process(EE_Line_Item $line_item): ?EE_Line_Item;
 }

core/domain/entities/tickets/TicketPriceModifiers.php

Indentation +230 added lines, -230 removed lines

@@ -20,234 +20,234 @@
  */
 class TicketPriceModifiers
 {
-    /**
-     * @var EE_Ticket
-     */
-    private $ticket;
-
-
-    /**
-     * @var EE_Price[]
-     */
-    private $ticket_prices;
-
-
-    /**
-     * @param EE_Ticket $ticket
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function __construct(EE_Ticket $ticket)
-    {
-        $this->ticket = $ticket;
-        // run a query to retrieve ALL of this ticket's related prices before doing anything else
-        $this->ticket_prices = $this->ticket->prices();
-    }
-
-
-    /**
-     * @param callable  $filter
-     * @return array
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    private function filterModifiersForTicket(callable $filter): array
-    {
-        $ticket_modifiers = $this->ticket->get_all_from_cache('Price');
-        if (empty($ticket_modifiers)) {
-            $ticket_modifiers = $this->ticket_prices;
-        }
-        return array_filter($ticket_modifiers, $filter);
-    }
-
-
-    /**
-     * retrieve all price modifiers for the this ticket
-     *
-     * @return array
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function getAllModifiersForTicket(): array
-    {
-        return $this->filterModifiersForTicket([$this, 'allModifiersFilter']);
-    }
-
-
-    /**
-     * pass to filterModifiersForTicket() with a ticket to retrieve all price modifiers for the this ticket
-     *
-     * @param EE_Price $price_modifier
-     * @return bool
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function allModifiersFilter(EE_Price $price_modifier): bool
-    {
-        return ! $price_modifier->is_base_price();
-    }
-
-
-    /**
-     * retrieve all discounts for the this ticket
-     *
-     * @return array
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function getAllDiscountModifiersForTicket(): array
-    {
-        return $this->filterModifiersForTicket([$this, 'discountModifierFilter']);
-    }
-
-
-    /**
-     * pass to filterModifiersForTicket() with a ticket to retrieve all discounts for the this ticket
-     *
-     * @param EE_Price $price_modifier
-     * @return bool
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function discountModifierFilter(EE_Price $price_modifier): bool
-    {
-        return $price_modifier->is_discount();
-    }
-
-
-    /**
-     * retrieve all surcharges for the this ticket
-     *
-     * @return array
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function getAllSurchargeModifiersForTicket(): array
-    {
-        return $this->filterModifiersForTicket([$this, 'surchargeModifierFilter']);
-    }
-
-
-    /**
-     * pass to filterModifiersForTicket() with a ticket to retrieve all surcharges for the this ticket
-     *
-     * @param EE_Price $price_modifier
-     * @return bool
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function surchargeModifierFilter(EE_Price $price_modifier): bool
-    {
-        return $price_modifier->is_surcharge();
-    }
-
-
-    /**
-     * retrieve all discounts AND surcharges for the this ticket
-     *
-     * @return array
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function getAllDiscountAndSurchargeModifiersForTicket(): array
-    {
-        return $this->filterModifiersForTicket([$this, 'discountAndSurchargeModifierFilter']);
-    }
-
-
-    /**
-     * pass to filterModifiersForTicket() with a ticket to retrieve all discounts AND surcharges for the this ticket
-     *
-     * @param EE_Price $price_modifier
-     * @return bool
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function discountAndSurchargeModifierFilter(EE_Price $price_modifier): bool
-    {
-        return $price_modifier->is_discount() || $price_modifier->is_surcharge();
-    }
-
-
-    /**
-     * retrieve all taxes for the this ticket
-     *
-     * @return array
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function getAllTaxesForTicket(): array
-    {
-        return $this->filterModifiersForTicket([$this, 'taxModifierFilter']);
-    }
-
-
-    /**
-     * pass to filterModifiersForTicket() with a ticket to retrieve all taxes for the this ticket
-     *
-     * @param EE_Price $price_modifier
-     * @return bool
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function taxModifierFilter(EE_Price $price_modifier): bool
-    {
-        return $price_modifier->is_tax();
-    }
-
-
-    /**
-     * retrieve ALL PRICES that are NOT taxes for the this ticket
-     *
-     * @return array
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function getAllNonTaxPricesForTicket(): array
-    {
-        return $this->filterModifiersForTicket([$this, 'nonTaxPriceFilter']);
-    }
-
-
-    /**
-     * pass to filterModifiersForTicket() with a ticket
-     * to retrieve ALL PRICES that are NOT taxes for the this ticket
-     *
-     * @param EE_Price $price_modifier
-     * @return bool
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function nonTaxPriceFilter(EE_Price $price_modifier): bool
-    {
-        return ! $price_modifier->is_base_price() && ! $price_modifier->is_tax();
-    }
-
-
-    /**
-     * retrieve the base price for the this ticket
-     *
-     * @return array
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function getBasePrice(): array
-    {
-        return $this->filterModifiersForTicket([$this, 'basePriceFilter']);
-    }
-
-
-    /**
-     * pass to filterModifiersForTicket()
-     * to retrieve the base price for the this ticket
-     *
-     * @param EE_Price $price_modifier
-     * @return bool
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function basePriceFilter(EE_Price $price_modifier): bool
-    {
-        return $price_modifier->is_base_price();
-    }
+	/**
+	 * @var EE_Ticket
+	 */
+	private $ticket;
+
+
+	/**
+	 * @var EE_Price[]
+	 */
+	private $ticket_prices;
+
+
+	/**
+	 * @param EE_Ticket $ticket
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function __construct(EE_Ticket $ticket)
+	{
+		$this->ticket = $ticket;
+		// run a query to retrieve ALL of this ticket's related prices before doing anything else
+		$this->ticket_prices = $this->ticket->prices();
+	}
+
+
+	/**
+	 * @param callable  $filter
+	 * @return array
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	private function filterModifiersForTicket(callable $filter): array
+	{
+		$ticket_modifiers = $this->ticket->get_all_from_cache('Price');
+		if (empty($ticket_modifiers)) {
+			$ticket_modifiers = $this->ticket_prices;
+		}
+		return array_filter($ticket_modifiers, $filter);
+	}
+
+
+	/**
+	 * retrieve all price modifiers for the this ticket
+	 *
+	 * @return array
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function getAllModifiersForTicket(): array
+	{
+		return $this->filterModifiersForTicket([$this, 'allModifiersFilter']);
+	}
+
+
+	/**
+	 * pass to filterModifiersForTicket() with a ticket to retrieve all price modifiers for the this ticket
+	 *
+	 * @param EE_Price $price_modifier
+	 * @return bool
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function allModifiersFilter(EE_Price $price_modifier): bool
+	{
+		return ! $price_modifier->is_base_price();
+	}
+
+
+	/**
+	 * retrieve all discounts for the this ticket
+	 *
+	 * @return array
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function getAllDiscountModifiersForTicket(): array
+	{
+		return $this->filterModifiersForTicket([$this, 'discountModifierFilter']);
+	}
+
+
+	/**
+	 * pass to filterModifiersForTicket() with a ticket to retrieve all discounts for the this ticket
+	 *
+	 * @param EE_Price $price_modifier
+	 * @return bool
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function discountModifierFilter(EE_Price $price_modifier): bool
+	{
+		return $price_modifier->is_discount();
+	}
+
+
+	/**
+	 * retrieve all surcharges for the this ticket
+	 *
+	 * @return array
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function getAllSurchargeModifiersForTicket(): array
+	{
+		return $this->filterModifiersForTicket([$this, 'surchargeModifierFilter']);
+	}
+
+
+	/**
+	 * pass to filterModifiersForTicket() with a ticket to retrieve all surcharges for the this ticket
+	 *
+	 * @param EE_Price $price_modifier
+	 * @return bool
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function surchargeModifierFilter(EE_Price $price_modifier): bool
+	{
+		return $price_modifier->is_surcharge();
+	}
+
+
+	/**
+	 * retrieve all discounts AND surcharges for the this ticket
+	 *
+	 * @return array
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function getAllDiscountAndSurchargeModifiersForTicket(): array
+	{
+		return $this->filterModifiersForTicket([$this, 'discountAndSurchargeModifierFilter']);
+	}
+
+
+	/**
+	 * pass to filterModifiersForTicket() with a ticket to retrieve all discounts AND surcharges for the this ticket
+	 *
+	 * @param EE_Price $price_modifier
+	 * @return bool
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function discountAndSurchargeModifierFilter(EE_Price $price_modifier): bool
+	{
+		return $price_modifier->is_discount() || $price_modifier->is_surcharge();
+	}
+
+
+	/**
+	 * retrieve all taxes for the this ticket
+	 *
+	 * @return array
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function getAllTaxesForTicket(): array
+	{
+		return $this->filterModifiersForTicket([$this, 'taxModifierFilter']);
+	}
+
+
+	/**
+	 * pass to filterModifiersForTicket() with a ticket to retrieve all taxes for the this ticket
+	 *
+	 * @param EE_Price $price_modifier
+	 * @return bool
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function taxModifierFilter(EE_Price $price_modifier): bool
+	{
+		return $price_modifier->is_tax();
+	}
+
+
+	/**
+	 * retrieve ALL PRICES that are NOT taxes for the this ticket
+	 *
+	 * @return array
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function getAllNonTaxPricesForTicket(): array
+	{
+		return $this->filterModifiersForTicket([$this, 'nonTaxPriceFilter']);
+	}
+
+
+	/**
+	 * pass to filterModifiersForTicket() with a ticket
+	 * to retrieve ALL PRICES that are NOT taxes for the this ticket
+	 *
+	 * @param EE_Price $price_modifier
+	 * @return bool
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function nonTaxPriceFilter(EE_Price $price_modifier): bool
+	{
+		return ! $price_modifier->is_base_price() && ! $price_modifier->is_tax();
+	}
+
+
+	/**
+	 * retrieve the base price for the this ticket
+	 *
+	 * @return array
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function getBasePrice(): array
+	{
+		return $this->filterModifiersForTicket([$this, 'basePriceFilter']);
+	}
+
+
+	/**
+	 * pass to filterModifiersForTicket()
+	 * to retrieve the base price for the this ticket
+	 *
+	 * @param EE_Price $price_modifier
+	 * @return bool
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function basePriceFilter(EE_Price $price_modifier): bool
+	{
+		return $price_modifier->is_base_price();
+	}
 }