eventespresso / event-espresso-core

core/domain/services/graphql/data/mutations/PriceMutation.php

Spacing +6 added lines, -6 removed lines

@@ -44,27 +44,27 @@
             $args['PRC_name'] = sanitize_text_field($input['name']);
         }
 
-        if (! empty($input['order'])) {
+        if ( ! empty($input['order'])) {
             $args['PRC_order'] = (int) $input['order'];
         }
 
-        if (! empty($input['overrides'])) {
+        if ( ! empty($input['overrides'])) {
             $args['PRC_overrides'] = (int) $input['overrides'];
         }
 
-        if (! empty($input['parent'])) {
+        if ( ! empty($input['parent'])) {
             $parts = Relay::fromGlobalId(sanitize_text_field($input['parent']));
             $args['PRC_parent'] = ! empty($parts['id']) ? absint($parts['id']) : 0;
         }
 
-        if (! empty($input['priceType'])) {
+        if ( ! empty($input['priceType'])) {
             $parts = Relay::fromGlobalId(sanitize_text_field($input['priceType']));
             $args['PRT_ID'] = ! empty($parts['id']) ? absint($parts['id']) : 0;
         }
 
-        if (! empty($input['wpUser'])) {
+        if ( ! empty($input['wpUser'])) {
             $parts = Relay::fromGlobalId(sanitize_text_field($input['wpUser']));
-            $args['PRC_wp_user'] = (! empty($parts['id']) && is_int($parts['id'])) ? $parts['id'] : null;
+            $args['PRC_wp_user'] = ( ! empty($parts['id']) && is_int($parts['id'])) ? $parts['id'] : null;
         }
 
         return apply_filters(

Indentation +49 added lines, -49 removed lines

@@ -12,64 +12,64 @@
  */
 class PriceMutation
 {
-    /**
-     * Maps the GraphQL input to a format that the model functions can use
-     *
-     * @param array $input Data coming from the GraphQL mutation query input
-     * @return array
-     */
-    public static function prepareFields(array $input): array
-    {
-        $args = [];
+	/**
+	 * Maps the GraphQL input to a format that the model functions can use
+	 *
+	 * @param array $input Data coming from the GraphQL mutation query input
+	 * @return array
+	 */
+	public static function prepareFields(array $input): array
+	{
+		$args = [];
 
-        // amount can be 0
-        if (array_key_exists('amount', $input)) {
-            $args['PRC_amount'] = (float) $input['amount'];
-        }
+		// amount can be 0
+		if (array_key_exists('amount', $input)) {
+			$args['PRC_amount'] = (float) $input['amount'];
+		}
 
-        if (isset($input['description'])) {
-            $args['PRC_desc'] = wp_kses_post($input['description']);
-        }
+		if (isset($input['description'])) {
+			$args['PRC_desc'] = wp_kses_post($input['description']);
+		}
 
-        if (array_key_exists('isDefault', $input)) {
-            $args['PRC_is_default'] = (bool) $input['isDefault'];
-        }
+		if (array_key_exists('isDefault', $input)) {
+			$args['PRC_is_default'] = (bool) $input['isDefault'];
+		}
 
-        if (array_key_exists('isTrashed', $input)) {
-            $args['PRC_deleted'] = (bool) $input['isTrashed'];
-        }
+		if (array_key_exists('isTrashed', $input)) {
+			$args['PRC_deleted'] = (bool) $input['isTrashed'];
+		}
 
-        if (isset($input['name'])) {
-            $args['PRC_name'] = sanitize_text_field($input['name']);
-        }
+		if (isset($input['name'])) {
+			$args['PRC_name'] = sanitize_text_field($input['name']);
+		}
 
-        if (! empty($input['order'])) {
-            $args['PRC_order'] = (int) $input['order'];
-        }
+		if (! empty($input['order'])) {
+			$args['PRC_order'] = (int) $input['order'];
+		}
 
-        if (! empty($input['overrides'])) {
-            $args['PRC_overrides'] = (int) $input['overrides'];
-        }
+		if (! empty($input['overrides'])) {
+			$args['PRC_overrides'] = (int) $input['overrides'];
+		}
 
-        if (! empty($input['parent'])) {
-            $parts = Relay::fromGlobalId(sanitize_text_field($input['parent']));
-            $args['PRC_parent'] = ! empty($parts['id']) ? absint($parts['id']) : 0;
-        }
+		if (! empty($input['parent'])) {
+			$parts = Relay::fromGlobalId(sanitize_text_field($input['parent']));
+			$args['PRC_parent'] = ! empty($parts['id']) ? absint($parts['id']) : 0;
+		}
 
-        if (! empty($input['priceType'])) {
-            $parts = Relay::fromGlobalId(sanitize_text_field($input['priceType']));
-            $args['PRT_ID'] = ! empty($parts['id']) ? absint($parts['id']) : 0;
-        }
+		if (! empty($input['priceType'])) {
+			$parts = Relay::fromGlobalId(sanitize_text_field($input['priceType']));
+			$args['PRT_ID'] = ! empty($parts['id']) ? absint($parts['id']) : 0;
+		}
 
-        if (! empty($input['wpUser'])) {
-            $parts = Relay::fromGlobalId(sanitize_text_field($input['wpUser']));
-            $args['PRC_wp_user'] = (! empty($parts['id']) && is_int($parts['id'])) ? $parts['id'] : null;
-        }
+		if (! empty($input['wpUser'])) {
+			$parts = Relay::fromGlobalId(sanitize_text_field($input['wpUser']));
+			$args['PRC_wp_user'] = (! empty($parts['id']) && is_int($parts['id'])) ? $parts['id'] : null;
+		}
 
-        return apply_filters(
-            'FHEE__EventEspresso_core_domain_services_graphql_data_mutations__price_args',
-            $args,
-            $input
-        );
-    }
+		return apply_filters(
+			'FHEE__EventEspresso_core_domain_services_graphql_data_mutations__price_args',
+			$args,
+			$input
+		);
+	}
 }

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

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         // $errors = [];
 
         try {
-            if (empty($input['uniqueInputs']) || !is_array($input['uniqueInputs'])) {
+            if (empty($input['uniqueInputs']) || ! is_array($input['uniqueInputs'])) {
                 throw new OutOfBoundsException(
                     esc_html__('A valid input was not provided.', 'event_espresso')
                 );

Indentation +59 added lines, -59 removed lines

@@ -10,69 +10,69 @@
 class BulkEntityMutator extends EntityMutator
 {
 
-    /**
-     * @var callable $entity_mutator .
-     */
-    protected $entity_mutator;
+	/**
+	 * @var callable $entity_mutator .
+	 */
+	protected $entity_mutator;
 
-    /**
-     * BulkEntityMutator constructor.
-     *
-     * @param callable $entity_mutator The mutator for an entity.
-     */
-    public function __construct(callable $entity_mutator)
-    {
-        $this->entity_mutator = $entity_mutator;
-    }
-    /**
-     * Bulk updates entities.
-     *
-     * @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
-     */
-    public function updateEntities(array $input, AppContext $context, ResolveInfo $info): array
-    {
+	/**
+	 * BulkEntityMutator constructor.
+	 *
+	 * @param callable $entity_mutator The mutator for an entity.
+	 */
+	public function __construct(callable $entity_mutator)
+	{
+		$this->entity_mutator = $entity_mutator;
+	}
+	/**
+	 * Bulk updates entities.
+	 *
+	 * @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
+	 */
+	public function updateEntities(array $input, AppContext $context, ResolveInfo $info): array
+	{
 
-        $updated = [];
-        $failed = [];
-        // TODO Add meaningful error messages for every failure.
-        // $errors = [];
+		$updated = [];
+		$failed = [];
+		// TODO Add meaningful error messages for every failure.
+		// $errors = [];
 
-        try {
-            if (empty($input['uniqueInputs']) || !is_array($input['uniqueInputs'])) {
-                throw new OutOfBoundsException(
-                    esc_html__('A valid input was not provided.', 'event_espresso')
-                );
-            }
+		try {
+			if (empty($input['uniqueInputs']) || !is_array($input['uniqueInputs'])) {
+				throw new OutOfBoundsException(
+					esc_html__('A valid input was not provided.', 'event_espresso')
+				);
+			}
 
-            $sharedInput = ! empty($input['sharedInput']) ? $input['sharedInput'] : [];
+			$sharedInput = ! empty($input['sharedInput']) ? $input['sharedInput'] : [];
 
-            foreach ($input['uniqueInputs'] as $uniqueInput) {
-                try {
-                    // values in $uniqueInput will override those in $sharedInput
-                    $finalInput = array_merge($sharedInput, $uniqueInput);
-                    // mutate the individual entity.
-                    $mutator = $this->entity_mutator;
-                    $mutator($finalInput, $context, $info);
-                    // we are here it means the update was successful.
-                    $updated[] = $uniqueInput['id'];
-                } catch (Exception $e) {
-                    // sorry mate, couldn't help you :(
-                    $failed[] = $uniqueInput['id'];
-                }
-            }
-        } catch (Exception $exception) {
-            EntityMutator::handleExceptions(
-                $exception,
-                esc_html__(
-                    'Could not perform the update because of the following error(s)',
-                    'event_espresso'
-                )
-            );
-        }
+			foreach ($input['uniqueInputs'] as $uniqueInput) {
+				try {
+					// values in $uniqueInput will override those in $sharedInput
+					$finalInput = array_merge($sharedInput, $uniqueInput);
+					// mutate the individual entity.
+					$mutator = $this->entity_mutator;
+					$mutator($finalInput, $context, $info);
+					// we are here it means the update was successful.
+					$updated[] = $uniqueInput['id'];
+				} catch (Exception $e) {
+					// sorry mate, couldn't help you :(
+					$failed[] = $uniqueInput['id'];
+				}
+			}
+		} catch (Exception $exception) {
+			EntityMutator::handleExceptions(
+				$exception,
+				esc_html__(
+					'Could not perform the update because of the following error(s)',
+					'event_espresso'
+				)
+			);
+		}
 
-        return compact('updated', 'failed');
-    }
+		return compact('updated', 'failed');
+	}
 }

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

Indentation +13 added lines, -13 removed lines

@@ -8,17 +8,17 @@
 class TicketBulkUpdate extends EntityMutator
 {
 
-    /**
-     * Defines the mutation data modification closure.
-     *
-     * @param EEM_Ticket $model
-     * @param Ticket     $type
-     * @return callable
-     */
-    public static function mutateAndGetPayload(EEM_Ticket $model, Ticket $type)
-    {
-        $entityMutator = TicketUpdate::mutateAndGetPayload($model, $type);
-        $bulkMutator = new BulkEntityMutator($entityMutator);
-        return array($bulkMutator, 'updateEntities');
-    }
+	/**
+	 * Defines the mutation data modification closure.
+	 *
+	 * @param EEM_Ticket $model
+	 * @param Ticket     $type
+	 * @return callable
+	 */
+	public static function mutateAndGetPayload(EEM_Ticket $model, Ticket $type)
+	{
+		$entityMutator = TicketUpdate::mutateAndGetPayload($model, $type);
+		$bulkMutator = new BulkEntityMutator($entityMutator);
+		return array($bulkMutator, 'updateEntities');
+	}
 }

core/services/assets/AssetManagerInterface.php

Indentation +61 added lines, -61 removed lines

@@ -18,75 +18,75 @@
  */
 interface AssetManagerInterface
 {
-    /**
-     * @since 4.9.71.p
-     * @return string
-     */
-    public function assetNamespace();
+	/**
+	 * @since 4.9.71.p
+	 * @return string
+	 */
+	public function assetNamespace();
 
-    /**
-     * @since 4.9.62.p
-     */
-    public function addAssets();
+	/**
+	 * @since 4.9.62.p
+	 */
+	public function addAssets();
 
 
-    /**
-     * @param string $handle
-     * @param string $source
-     * @param array  $dependencies
-     * @param bool   $load_in_footer
-     * @return JavascriptAsset
-     * @throws DuplicateCollectionIdentifierException
-     * @throws InvalidDataTypeException
-     * @throws InvalidEntityException
-     * @since 4.9.62.p
-     */
-    public function addJavascript(
-        $handle,
-        $source,
-        array $dependencies = array(),
-        $load_in_footer = true
-    );
+	/**
+	 * @param string $handle
+	 * @param string $source
+	 * @param array  $dependencies
+	 * @param bool   $load_in_footer
+	 * @return JavascriptAsset
+	 * @throws DuplicateCollectionIdentifierException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidEntityException
+	 * @since 4.9.62.p
+	 */
+	public function addJavascript(
+		$handle,
+		$source,
+		array $dependencies = array(),
+		$load_in_footer = true
+	);
 
 
-    /**
-     * @since 4.9.71.p
-     * @param string $handle
-     * @param array  $dependencies
-     * @param bool   $load_in_footer
-     * @return JavascriptAsset
-     */
-    public function addVendorJavascript(
-        $handle,
-        array $dependencies = array(),
-        $load_in_footer = true
-    );
+	/**
+	 * @since 4.9.71.p
+	 * @param string $handle
+	 * @param array  $dependencies
+	 * @param bool   $load_in_footer
+	 * @return JavascriptAsset
+	 */
+	public function addVendorJavascript(
+		$handle,
+		array $dependencies = array(),
+		$load_in_footer = true
+	);
 
 
 
-    /**
-     * @param string $handle
-     * @param string $source
-     * @param array  $dependencies
-     * @param string $media
-     * @return StylesheetAsset
-     * @throws DuplicateCollectionIdentifierException
-     * @throws InvalidDataTypeException
-     * @throws InvalidEntityException
-     * @since 4.9.62.p
-     */
-    public function addStylesheet(
-        $handle,
-        $source,
-        array $dependencies = array(),
-        $media = 'all'
-    );
+	/**
+	 * @param string $handle
+	 * @param string $source
+	 * @param array  $dependencies
+	 * @param string $media
+	 * @return StylesheetAsset
+	 * @throws DuplicateCollectionIdentifierException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidEntityException
+	 * @since 4.9.62.p
+	 */
+	public function addStylesheet(
+		$handle,
+		$source,
+		array $dependencies = array(),
+		$media = 'all'
+	);
 
 
-    /**
-     * @param string $handle
-     * @return bool
-     * @since 4.9.62.p
-     */
-    public function enqueueAsset($handle);
+	/**
+	 * @param string $handle
+	 * @return bool
+	 * @since 4.9.62.p
+	 */
+	public function enqueueAsset($handle);
 }

core/domain/values/assets/ManifestFile.php

Indentation +28 added lines, -28 removed lines

@@ -16,32 +16,32 @@
 class ManifestFile extends Asset
 {
 
-    /**
-     * Asset constructor.
-     *
-     * @param DomainInterface $domain
-     * @throws InvalidDataTypeException
-     */
-    public function __construct(DomainInterface $domain)
-    {
-        parent::__construct(Asset::TYPE_MANIFEST, $domain->assetNamespace(), $domain);
-    }
-
-
-    /**
-     * @return string
-     */
-    public function urlBase()
-    {
-        return $this->domain->distributionAssetsUrl();
-    }
-
-
-    /**
-     * @return string
-     */
-    public function filepath()
-    {
-        return $this->domain->distributionAssetsPath();
-    }
+	/**
+	 * Asset constructor.
+	 *
+	 * @param DomainInterface $domain
+	 * @throws InvalidDataTypeException
+	 */
+	public function __construct(DomainInterface $domain)
+	{
+		parent::__construct(Asset::TYPE_MANIFEST, $domain->assetNamespace(), $domain);
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function urlBase()
+	{
+		return $this->domain->distributionAssetsUrl();
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function filepath()
+	{
+		return $this->domain->distributionAssetsPath();
+	}
 }

core/services/factory/Factory.php

Indentation +13 added lines, -13 removed lines

@@ -7,19 +7,19 @@
 abstract class Factory implements FactoryInterface
 {
 
-    /**
-     * @var LoaderInterface $loader
-     */
-    protected $loader;
+	/**
+	 * @var LoaderInterface $loader
+	 */
+	protected $loader;
 
 
-    /**
-     * RouteFactory constructor.
-     *
-     * @param LoaderInterface $loader
-     */
-    public function __construct(LoaderInterface $loader)
-    {
-        $this->loader = $loader;
-    }
+	/**
+	 * RouteFactory constructor.
+	 *
+	 * @param LoaderInterface $loader
+	 */
+	public function __construct(LoaderInterface $loader)
+	{
+		$this->loader = $loader;
+	}
 }

core/services/loaders/ObjectIdentifier.php

Spacing +3 added lines, -3 removed lines

@@ -70,7 +70,7 @@ 
     {
         $fqcn = str_replace('\\', '_', $fqcn);
         return $fqcn === $object_identifier
-               || strpos($object_identifier, $fqcn . ObjectIdentifier::DELIMITER) === 0;
+               || strpos($object_identifier, $fqcn.ObjectIdentifier::DELIMITER) === 0;
     }
 
 
@@ -90,8 +90,8 @@ 
         )
             ? $this->getIdentifierForArguments($arguments)
             : '';
-        if (! empty($identifier)) {
-            $fqcn .= ObjectIdentifier::DELIMITER . md5($identifier);
+        if ( ! empty($identifier)) {
+            $fqcn .= ObjectIdentifier::DELIMITER.md5($identifier);
         }
         $fqcn = str_replace('\\', '_', $fqcn);
         return $fqcn;

Indentation +104 added lines, -104 removed lines

@@ -15,119 +15,119 @@
  */
 class ObjectIdentifier
 {
-    /**
-     * used to separate the FQCN from the class's arguments identifier
-     */
-    const DELIMITER = '____';
+	/**
+	 * used to separate the FQCN from the class's arguments identifier
+	 */
+	const DELIMITER = '____';
 
-    /**
-     * @var ClassInterfaceCache $class_cache
-     */
-    private $class_cache;
+	/**
+	 * @var ClassInterfaceCache $class_cache
+	 */
+	private $class_cache;
 
 
-    /**
-     * ObjectIdentifier constructor.
-     *
-     * @param ClassInterfaceCache $class_cache
-     */
-    public function __construct(ClassInterfaceCache $class_cache)
-    {
-        $this->class_cache = $class_cache;
-    }
+	/**
+	 * ObjectIdentifier constructor.
+	 *
+	 * @param ClassInterfaceCache $class_cache
+	 */
+	public function __construct(ClassInterfaceCache $class_cache)
+	{
+		$this->class_cache = $class_cache;
+	}
 
 
-    /**
-     * Returns true if the supplied $object_identifier contains
-     * the delimiter used to separate an fqcn from the arguments hash
-     *
-     * @param string $object_identifier
-     * @return bool
-     */
-    public function hasArguments($object_identifier)
-    {
-        // type casting to bool instead of using strpos() !== false
-        // because an object identifier should never begin with the delimiter
-        // therefore the delimiter should NOT be found at position 0
-        return (bool) strpos($object_identifier, ObjectIdentifier::DELIMITER);
-    }
+	/**
+	 * Returns true if the supplied $object_identifier contains
+	 * the delimiter used to separate an fqcn from the arguments hash
+	 *
+	 * @param string $object_identifier
+	 * @return bool
+	 */
+	public function hasArguments($object_identifier)
+	{
+		// type casting to bool instead of using strpos() !== false
+		// because an object identifier should never begin with the delimiter
+		// therefore the delimiter should NOT be found at position 0
+		return (bool) strpos($object_identifier, ObjectIdentifier::DELIMITER);
+	}
 
 
-    /**
-     * Returns true if the supplied FQCN equals the supplied $object_identifier
-     * OR the supplied FQCN matches the FQCN portion of the supplied $object_identifier
-     * AND that $object_identifier is for an object with arguments.
-     * This allows a request for an object using a FQCN to match
-     * a previously instantiated object with arguments
-     * without having to know those arguments.
-     *
-     * @param string $fqcn
-     * @param string $object_identifier
-     * @return bool
-     */
-    public function fqcnMatchesObjectIdentifier($fqcn, $object_identifier)
-    {
-        $fqcn = str_replace('\\', '_', $fqcn);
-        return $fqcn === $object_identifier
-               || strpos($object_identifier, $fqcn . ObjectIdentifier::DELIMITER) === 0;
-    }
+	/**
+	 * Returns true if the supplied FQCN equals the supplied $object_identifier
+	 * OR the supplied FQCN matches the FQCN portion of the supplied $object_identifier
+	 * AND that $object_identifier is for an object with arguments.
+	 * This allows a request for an object using a FQCN to match
+	 * a previously instantiated object with arguments
+	 * without having to know those arguments.
+	 *
+	 * @param string $fqcn
+	 * @param string $object_identifier
+	 * @return bool
+	 */
+	public function fqcnMatchesObjectIdentifier($fqcn, $object_identifier)
+	{
+		$fqcn = str_replace('\\', '_', $fqcn);
+		return $fqcn === $object_identifier
+			   || strpos($object_identifier, $fqcn . ObjectIdentifier::DELIMITER) === 0;
+	}
 
 
-    /**
-     * build a string representation of an object's FQCN and arguments
-     *
-     * @param string $fqcn
-     * @param array  $arguments
-     * @return string
-     */
-    public function getIdentifier($fqcn, array $arguments = array())
-    {
-        // only build identifier from arguments if class is not ReservedInstanceInterface
-        $identifier = ! $this->class_cache->hasInterface(
-            $fqcn,
-            'EventEspresso\core\interfaces\ReservedInstanceInterface'
-        )
-            ? $this->getIdentifierForArguments($arguments)
-            : '';
-        if (! empty($identifier)) {
-            $fqcn .= ObjectIdentifier::DELIMITER . md5($identifier);
-        }
-        $fqcn = str_replace('\\', '_', $fqcn);
-        return $fqcn;
-    }
+	/**
+	 * build a string representation of an object's FQCN and arguments
+	 *
+	 * @param string $fqcn
+	 * @param array  $arguments
+	 * @return string
+	 */
+	public function getIdentifier($fqcn, array $arguments = array())
+	{
+		// only build identifier from arguments if class is not ReservedInstanceInterface
+		$identifier = ! $this->class_cache->hasInterface(
+			$fqcn,
+			'EventEspresso\core\interfaces\ReservedInstanceInterface'
+		)
+			? $this->getIdentifierForArguments($arguments)
+			: '';
+		if (! empty($identifier)) {
+			$fqcn .= ObjectIdentifier::DELIMITER . md5($identifier);
+		}
+		$fqcn = str_replace('\\', '_', $fqcn);
+		return $fqcn;
+	}
 
 
-    /**
-     * build a string representation of a object's arguments
-     * (mostly because Closures can't be serialized)
-     *
-     * @param array $arguments
-     * @return string
-     */
-    protected function getIdentifierForArguments(array $arguments)
-    {
-        if (empty($arguments)) {
-            return '';
-        }
-        $identifier = '';
-        foreach ($arguments as $key => $argument) {
-            // don't include arguments used to assist with loading legacy classes
-            if (is_string($key) && strpos($key, 'EE_Registry::create') === 0) {
-                continue;
-            }
-            switch (true) {
-                case is_object($argument):
-                case $argument instanceof Closure:
-                    $identifier .= spl_object_hash($argument);
-                    break;
-                case is_array($argument):
-                    $identifier .= $this->getIdentifierForArguments($argument);
-                    break;
-                default:
-                    $identifier .= $argument;
-                    break;
-            }
-        }
-        return $identifier;
-    }
+	/**
+	 * build a string representation of a object's arguments
+	 * (mostly because Closures can't be serialized)
+	 *
+	 * @param array $arguments
+	 * @return string
+	 */
+	protected function getIdentifierForArguments(array $arguments)
+	{
+		if (empty($arguments)) {
+			return '';
+		}
+		$identifier = '';
+		foreach ($arguments as $key => $argument) {
+			// don't include arguments used to assist with loading legacy classes
+			if (is_string($key) && strpos($key, 'EE_Registry::create') === 0) {
+				continue;
+			}
+			switch (true) {
+				case is_object($argument):
+				case $argument instanceof Closure:
+					$identifier .= spl_object_hash($argument);
+					break;
+				case is_array($argument):
+					$identifier .= $this->getIdentifierForArguments($argument);
+					break;
+				default:
+					$identifier .= $argument;
+					break;
+			}
+		}
+		return $identifier;
+	}
 }

core/services/assets/AssetManifestFactory.php

Spacing +5 added lines, -5 removed lines

@@ -53,7 +53,7 @@ 
      */
     public function create($fqcn, array $arguments = [])
     {
-        if (! isset($arguments[0]) || ! $arguments[0] instanceof DomainInterface) {
+        if ( ! isset($arguments[0]) || ! $arguments[0] instanceof DomainInterface) {
             throw new InvalidArgumentException(
                 esc_html__(
                     'In order to generate an AssetManifest class you need to supply an array where the first argument is an instance of DomainInterface.',
@@ -73,9 +73,9 @@ 
     private function getAssetManifestForDomain($manifest_fqcn, DomainInterface $domain)
     {
         $domain_fqcn = get_class($domain);
-        if (! isset(AssetManifestFactory::$manifests[ $domain_fqcn ])) {
+        if ( ! isset(AssetManifestFactory::$manifests[$domain_fqcn])) {
             $asset_manifest = new $manifest_fqcn($domain);
-            if (! $asset_manifest instanceof AssetManifestInterface || ! $asset_manifest instanceof $manifest_fqcn) {
+            if ( ! $asset_manifest instanceof AssetManifestInterface || ! $asset_manifest instanceof $manifest_fqcn) {
                 throw new DomainException(
                     sprintf(
                         esc_html__(
@@ -88,8 +88,8 @@ 
             }
             // we still need to share this with the core loader to facilitate automatic dependency injection
             $this->loader->share(AssetManifest::class, $asset_manifest, [$domain]);
-            AssetManifestFactory::$manifests[ $domain_fqcn ] = $asset_manifest;
+            AssetManifestFactory::$manifests[$domain_fqcn] = $asset_manifest;
         }
-        return AssetManifestFactory::$manifests[ $domain_fqcn ];
+        return AssetManifestFactory::$manifests[$domain_fqcn];
     }
 }

Indentation +73 added lines, -73 removed lines

@@ -16,86 +16,86 @@
  */
 class AssetManifestFactory implements FactoryInterface
 {
-    /**
-     * @var AssetManifestInterface[]
-     */
-    private static $manifests = [];
+	/**
+	 * @var AssetManifestInterface[]
+	 */
+	private static $manifests = [];
 
-    /**
-     * @var LoaderInterface $loader
-     */
-    protected $loader;
+	/**
+	 * @var LoaderInterface $loader
+	 */
+	protected $loader;
 
 
-    /**
-     * AssetManifestFactory constructor.
-     *
-     * @param LoaderInterface $loader
-     */
-    public function __construct(LoaderInterface $loader)
-    {
-        $this->loader = $loader;
-    }
+	/**
+	 * AssetManifestFactory constructor.
+	 *
+	 * @param LoaderInterface $loader
+	 */
+	public function __construct(LoaderInterface $loader)
+	{
+		$this->loader = $loader;
+	}
 
 
-    /**
-     * returns the applicable AssetManifest for the provided Domain
-     *
-     * @param DomainInterface $domain
-     * @return AssetManifestInterface
-     */
-    public function createFromDomainObject(DomainInterface $domain)
-    {
-        return $this->getAssetManifestForDomain(AssetManifest::class, $domain);
-    }
+	/**
+	 * returns the applicable AssetManifest for the provided Domain
+	 *
+	 * @param DomainInterface $domain
+	 * @return AssetManifestInterface
+	 */
+	public function createFromDomainObject(DomainInterface $domain)
+	{
+		return $this->getAssetManifestForDomain(AssetManifest::class, $domain);
+	}
 
 
-    /**
-     * for creating an atypical AssetManifest for the Domain provided in the $arguments array
-     *
-     * @param string $fqcn      Fully Qualified Class Name
-     * @param array  $arguments [optional] array of data required for construction
-     * @return AssetManifestInterface
-     */
-    public function create($fqcn, array $arguments = [])
-    {
-        if (! isset($arguments[0]) || ! $arguments[0] instanceof DomainInterface) {
-            throw new InvalidArgumentException(
-                esc_html__(
-                    'In order to generate an AssetManifest class you need to supply an array where the first argument is an instance of DomainInterface.',
-                    'event_espresso'
-                )
-            );
-        }
-        return $this->getAssetManifestForDomain($fqcn, $arguments[0]);
-    }
+	/**
+	 * for creating an atypical AssetManifest for the Domain provided in the $arguments array
+	 *
+	 * @param string $fqcn      Fully Qualified Class Name
+	 * @param array  $arguments [optional] array of data required for construction
+	 * @return AssetManifestInterface
+	 */
+	public function create($fqcn, array $arguments = [])
+	{
+		if (! isset($arguments[0]) || ! $arguments[0] instanceof DomainInterface) {
+			throw new InvalidArgumentException(
+				esc_html__(
+					'In order to generate an AssetManifest class you need to supply an array where the first argument is an instance of DomainInterface.',
+					'event_espresso'
+				)
+			);
+		}
+		return $this->getAssetManifestForDomain($fqcn, $arguments[0]);
+	}
 
 
-    /**
-     * @param string          $manifest_fqcn
-     * @param DomainInterface $domain
-     * @return AssetManifestInterface
-     */
-    private function getAssetManifestForDomain($manifest_fqcn, DomainInterface $domain)
-    {
-        $domain_fqcn = get_class($domain);
-        if (! isset(AssetManifestFactory::$manifests[ $domain_fqcn ])) {
-            $asset_manifest = new $manifest_fqcn($domain);
-            if (! $asset_manifest instanceof AssetManifestInterface || ! $asset_manifest instanceof $manifest_fqcn) {
-                throw new DomainException(
-                    sprintf(
-                        esc_html__(
-                            'The requested AssetManifest class "%1$s" could not be loaded.',
-                            'event_espresso'
-                        ),
-                        $manifest_fqcn
-                    )
-                );
-            }
-            // we still need to share this with the core loader to facilitate automatic dependency injection
-            $this->loader->share(AssetManifest::class, $asset_manifest, [$domain]);
-            AssetManifestFactory::$manifests[ $domain_fqcn ] = $asset_manifest;
-        }
-        return AssetManifestFactory::$manifests[ $domain_fqcn ];
-    }
+	/**
+	 * @param string          $manifest_fqcn
+	 * @param DomainInterface $domain
+	 * @return AssetManifestInterface
+	 */
+	private function getAssetManifestForDomain($manifest_fqcn, DomainInterface $domain)
+	{
+		$domain_fqcn = get_class($domain);
+		if (! isset(AssetManifestFactory::$manifests[ $domain_fqcn ])) {
+			$asset_manifest = new $manifest_fqcn($domain);
+			if (! $asset_manifest instanceof AssetManifestInterface || ! $asset_manifest instanceof $manifest_fqcn) {
+				throw new DomainException(
+					sprintf(
+						esc_html__(
+							'The requested AssetManifest class "%1$s" could not be loaded.',
+							'event_espresso'
+						),
+						$manifest_fqcn
+					)
+				);
+			}
+			// we still need to share this with the core loader to facilitate automatic dependency injection
+			$this->loader->share(AssetManifest::class, $asset_manifest, [$domain]);
+			AssetManifestFactory::$manifests[ $domain_fqcn ] = $asset_manifest;
+		}
+		return AssetManifestFactory::$manifests[ $domain_fqcn ];
+	}
 }

core/services/assets/Barista.php

Indentation +79 added lines, -79 removed lines

@@ -13,94 +13,94 @@
 class Barista implements BaristaInterface
 {
 
-    const DEPENDENCY_LIST_REGISTERED = 'registered';
+	const DEPENDENCY_LIST_REGISTERED = 'registered';
 
 
-    /**
-     * @var AssetManifestInterface
-     */
-    private $asset_manifest;
+	/**
+	 * @var AssetManifestInterface
+	 */
+	private $asset_manifest;
 
-    /**
-     * @var bool
-     */
-    protected $initialized = false;
+	/**
+	 * @var bool
+	 */
+	protected $initialized = false;
 
 
-    /**
-     * Barista constructor.
-     *
-     * @param AssetManifestInterface $asset_manifest
-     */
-    public function __construct(AssetManifestInterface $asset_manifest)
-    {
-        $this->asset_manifest = $asset_manifest;
-    }
+	/**
+	 * Barista constructor.
+	 *
+	 * @param AssetManifestInterface $asset_manifest
+	 */
+	public function __construct(AssetManifestInterface $asset_manifest)
+	{
+		$this->asset_manifest = $asset_manifest;
+	}
 
 
-    /**
-     * @return void
-     */
-    public function initialize()
-    {
-        if (! $this->initialized) {
-            $this->asset_manifest->initialize();
-            add_action('wp_enqueue_scripts', [$this, 'registerScripts'], 0);
-            add_action('admin_enqueue_scripts', [$this, 'registerScripts'], 0);
-            add_action('wp_enqueue_scripts', [$this, 'registerStyles'], 0);
-            add_action('admin_enqueue_scripts', [$this, 'registerStyles'], 0);
-            $this->initialized = true;
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function initialize()
+	{
+		if (! $this->initialized) {
+			$this->asset_manifest->initialize();
+			add_action('wp_enqueue_scripts', [$this, 'registerScripts'], 0);
+			add_action('admin_enqueue_scripts', [$this, 'registerScripts'], 0);
+			add_action('wp_enqueue_scripts', [$this, 'registerStyles'], 0);
+			add_action('admin_enqueue_scripts', [$this, 'registerStyles'], 0);
+			$this->initialized = true;
+		}
+	}
 
 
-    /**
-     * Registers all the WordPress packages scripts that are in the standardized
-     * `build/` location.
-     *
-     * @return void
-     * @throws Exception
-     */
-    public function registerScripts()
-    {
-        try {
-            $entry_points = $this->asset_manifest->getEntryPoints();
-            foreach ($entry_points as $entry_point) {
-                if ($this->asset_manifest->hasAsset($entry_point)) {
-                    $handle = $this->asset_manifest->getAssetHandle($entry_point);
-                    $source = $this->asset_manifest->getAssetUrl($entry_point);
-                    $dependencies = $this->asset_manifest->getAssetDependencies($entry_point);
-                    $version = $this->asset_manifest->getAssetVersion($entry_point);
-                    wp_register_script($handle, $source, $dependencies, $version, true);
-                }
-            }
-        } catch (Exception $exception) {
-            wp_die(new ExceptionStackTraceDisplay($exception));
-        }
-    }
+	/**
+	 * Registers all the WordPress packages scripts that are in the standardized
+	 * `build/` location.
+	 *
+	 * @return void
+	 * @throws Exception
+	 */
+	public function registerScripts()
+	{
+		try {
+			$entry_points = $this->asset_manifest->getEntryPoints();
+			foreach ($entry_points as $entry_point) {
+				if ($this->asset_manifest->hasAsset($entry_point)) {
+					$handle = $this->asset_manifest->getAssetHandle($entry_point);
+					$source = $this->asset_manifest->getAssetUrl($entry_point);
+					$dependencies = $this->asset_manifest->getAssetDependencies($entry_point);
+					$version = $this->asset_manifest->getAssetVersion($entry_point);
+					wp_register_script($handle, $source, $dependencies, $version, true);
+				}
+			}
+		} catch (Exception $exception) {
+			wp_die(new ExceptionStackTraceDisplay($exception));
+		}
+	}
 
 
-    /**
-     * Registers all the packages and domain styles that are in the build folder.
-     *
-     * @return void
-     * @throws Exception
-     */
-    public function registerStyles()
-    {
-        try {
-            $entry_points = $this->asset_manifest->getEntryPoints();
-            foreach ($entry_points as $entry_point) {
-                if ($this->asset_manifest->hasAsset($entry_point, AssetManifest::ASSET_EXT_CSS)) {
-                    $handle = $this->asset_manifest->getAssetHandle($entry_point);
-                    $source = $this->asset_manifest->getAssetUrl($entry_point, AssetManifest::ASSET_EXT_CSS);
-                    $dependencies = $this->asset_manifest->getAssetDependencies($entry_point, AssetManifest::ASSET_EXT_CSS);
-                    $version = $this->asset_manifest->getAssetVersion($entry_point, AssetManifest::ASSET_EXT_CSS);
-                    wp_register_style($handle, $source, $dependencies, $version, 'all');
-                }
-            }
-        } catch (Exception $exception) {
-            wp_die(new ExceptionStackTraceDisplay($exception));
-        }
-    }
+	/**
+	 * Registers all the packages and domain styles that are in the build folder.
+	 *
+	 * @return void
+	 * @throws Exception
+	 */
+	public function registerStyles()
+	{
+		try {
+			$entry_points = $this->asset_manifest->getEntryPoints();
+			foreach ($entry_points as $entry_point) {
+				if ($this->asset_manifest->hasAsset($entry_point, AssetManifest::ASSET_EXT_CSS)) {
+					$handle = $this->asset_manifest->getAssetHandle($entry_point);
+					$source = $this->asset_manifest->getAssetUrl($entry_point, AssetManifest::ASSET_EXT_CSS);
+					$dependencies = $this->asset_manifest->getAssetDependencies($entry_point, AssetManifest::ASSET_EXT_CSS);
+					$version = $this->asset_manifest->getAssetVersion($entry_point, AssetManifest::ASSET_EXT_CSS);
+					wp_register_style($handle, $source, $dependencies, $version, 'all');
+				}
+			}
+		} catch (Exception $exception) {
+			wp_die(new ExceptionStackTraceDisplay($exception));
+		}
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      */
     public function initialize()
     {
-        if (! $this->initialized) {
+        if ( ! $this->initialized) {
             $this->asset_manifest->initialize();
             add_action('wp_enqueue_scripts', [$this, 'registerScripts'], 0);
             add_action('admin_enqueue_scripts', [$this, 'registerScripts'], 0);