eventespresso / event-espresso-core

core/services/address/formatters/InlineAddressFormatter.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
             )
         );
         // if the incoming country has a set format, use that, else use the default
-        $formatted_address = $address_formats[ $CNT_ISO ] ?? $address_formats['ZZZ'];
+        $formatted_address = $address_formats[$CNT_ISO] ?? $address_formats['ZZZ'];
         return $this->parse_formatted_address(
             $address,
             $address2,

Indentation +41 added lines, -41 removed lines

@@ -13,45 +13,45 @@
  */
 class InlineAddressFormatter extends AddressFormatter implements AddressFormatterInterface
 {
-    /**
-     * @param string $address
-     * @param string $address2
-     * @param string $city
-     * @param string $state
-     * @param string $zip
-     * @param string $country
-     * @param string $CNT_ISO
-     * @return string|null
-     */
-    public function format(
-        string $address,
-        string $address2,
-        string $city,
-        string $state,
-        string $zip,
-        string $country,
-        string $CNT_ISO
-    ): ?string {
-        $address_formats = apply_filters(
-            'FHEE__EE_Inline_Address_Formatter__address_formats',
-            array(
-                'CA'  => "{address}%{address2}%{city}%{state}%{country}%{zip}",
-                'GB'  => "{address}%{address2}%{city}%{state}%{zip}%{country}",
-                'US'  => "{address}%{address2}%{city}%{state}%{zip}%{country}",
-                'ZZZ' => "{address}%{address2}%{city}%{state}%{zip}%{country}",
-            )
-        );
-        // if the incoming country has a set format, use that, else use the default
-        $formatted_address = $address_formats[ $CNT_ISO ] ?? $address_formats['ZZZ'];
-        return $this->parse_formatted_address(
-            $address,
-            $address2,
-            $city,
-            $state,
-            $zip,
-            $country,
-            $formatted_address,
-            ', '
-        );
-    }
+	/**
+	 * @param string $address
+	 * @param string $address2
+	 * @param string $city
+	 * @param string $state
+	 * @param string $zip
+	 * @param string $country
+	 * @param string $CNT_ISO
+	 * @return string|null
+	 */
+	public function format(
+		string $address,
+		string $address2,
+		string $city,
+		string $state,
+		string $zip,
+		string $country,
+		string $CNT_ISO
+	): ?string {
+		$address_formats = apply_filters(
+			'FHEE__EE_Inline_Address_Formatter__address_formats',
+			array(
+				'CA'  => "{address}%{address2}%{city}%{state}%{country}%{zip}",
+				'GB'  => "{address}%{address2}%{city}%{state}%{zip}%{country}",
+				'US'  => "{address}%{address2}%{city}%{state}%{zip}%{country}",
+				'ZZZ' => "{address}%{address2}%{city}%{state}%{zip}%{country}",
+			)
+		);
+		// if the incoming country has a set format, use that, else use the default
+		$formatted_address = $address_formats[ $CNT_ISO ] ?? $address_formats['ZZZ'];
+		return $this->parse_formatted_address(
+			$address,
+			$address2,
+			$city,
+			$state,
+			$zip,
+			$country,
+			$formatted_address,
+			', '
+		);
+	}
 }

core/services/address/formatters/MultiLineAddressFormatter.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
             )
         );
         // if the incoming country has a set format, use that, else use the default
-        $formatted_address = $address_formats[ $CNT_ISO ] ?? $address_formats['ZZ'];
+        $formatted_address = $address_formats[$CNT_ISO] ?? $address_formats['ZZ'];
         return $this->parse_formatted_address(
             $address,
             $address2,

Indentation +41 added lines, -41 removed lines

@@ -13,45 +13,45 @@
  */
 class MultiLineAddressFormatter extends AddressFormatter implements AddressFormatterInterface
 {
-    /**
-     * @param string $address
-     * @param string $address2
-     * @param string $city
-     * @param string $state
-     * @param string $zip
-     * @param string $country
-     * @param string $CNT_ISO
-     * @return string
-     */
-    public function format(
-        string $address,
-        string $address2,
-        string $city,
-        string $state,
-        string $zip,
-        string $country,
-        string $CNT_ISO
-    ): ?string {
-        $address_formats = apply_filters(
-            'FHEE__EE_MultiLine_Address_Formatter__address_formats',
-            array(
-                'CA' => "{address}%{address2}%{city}%{state}%{country}%{zip}",
-                'GB' => "{address}%{address2}%{city}%{state}%{zip}%{country}",
-                'US' => "{address}%{address2}%{city}%{state}%{zip}%{country}",
-                'ZZ' => "{address}%{address2}%{city}%{state}%{zip}%{country}",
-            )
-        );
-        // if the incoming country has a set format, use that, else use the default
-        $formatted_address = $address_formats[ $CNT_ISO ] ?? $address_formats['ZZ'];
-        return $this->parse_formatted_address(
-            $address,
-            $address2,
-            $city,
-            $state,
-            $zip,
-            $country,
-            $formatted_address,
-            '<br />'
-        );
-    }
+	/**
+	 * @param string $address
+	 * @param string $address2
+	 * @param string $city
+	 * @param string $state
+	 * @param string $zip
+	 * @param string $country
+	 * @param string $CNT_ISO
+	 * @return string
+	 */
+	public function format(
+		string $address,
+		string $address2,
+		string $city,
+		string $state,
+		string $zip,
+		string $country,
+		string $CNT_ISO
+	): ?string {
+		$address_formats = apply_filters(
+			'FHEE__EE_MultiLine_Address_Formatter__address_formats',
+			array(
+				'CA' => "{address}%{address2}%{city}%{state}%{country}%{zip}",
+				'GB' => "{address}%{address2}%{city}%{state}%{zip}%{country}",
+				'US' => "{address}%{address2}%{city}%{state}%{zip}%{country}",
+				'ZZ' => "{address}%{address2}%{city}%{state}%{zip}%{country}",
+			)
+		);
+		// if the incoming country has a set format, use that, else use the default
+		$formatted_address = $address_formats[ $CNT_ISO ] ?? $address_formats['ZZ'];
+		return $this->parse_formatted_address(
+			$address,
+			$address2,
+			$city,
+			$state,
+			$zip,
+			$country,
+			$formatted_address,
+			'<br />'
+		);
+	}
 }

core/services/address/formatters/AddressFormatter.php

Indentation +34 added lines, -34 removed lines

@@ -12,38 +12,38 @@
  */
 class AddressFormatter
 {
-    // phpcs:disable PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    /**
-     * @param string $address
-     * @param string $address2
-     * @param string $city
-     * @param string $state
-     * @param string $zip
-     * @param string $country
-     * @param string $formatted_address
-     * @param string $sub
-     * @return string
-     */
-    protected function parse_formatted_address(
-        string $address,
-        string $address2,
-        string $city,
-        string $state,
-        string $zip,
-        string $country,
-        string $formatted_address,
-        string $sub
-    ): string {
-        // swap address part placeholders for the real text
-        $formatted_address = str_replace(
-            // find
-            array('{address}', '{address2}', '{city}', '{state}', '{zip}', '{country}'),
-            // replace
-            array($address, $address2, $city, $state, $zip, $country),
-            // string
-            $formatted_address
-        );
-        // remove placeholder from start and end, reduce repeating placeholders to singles, then replace with HTML line breaks
-        return preg_replace('/%+/', $sub, trim($formatted_address, '%'));
-    }
+	// phpcs:disable PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	/**
+	 * @param string $address
+	 * @param string $address2
+	 * @param string $city
+	 * @param string $state
+	 * @param string $zip
+	 * @param string $country
+	 * @param string $formatted_address
+	 * @param string $sub
+	 * @return string
+	 */
+	protected function parse_formatted_address(
+		string $address,
+		string $address2,
+		string $city,
+		string $state,
+		string $zip,
+		string $country,
+		string $formatted_address,
+		string $sub
+	): string {
+		// swap address part placeholders for the real text
+		$formatted_address = str_replace(
+			// find
+			array('{address}', '{address2}', '{city}', '{state}', '{zip}', '{country}'),
+			// replace
+			array($address, $address2, $city, $state, $zip, $country),
+			// string
+			$formatted_address
+		);
+		// remove placeholder from start and end, reduce repeating placeholders to singles, then replace with HTML line breaks
+		return preg_replace('/%+/', $sub, trim($formatted_address, '%'));
+	}
 }

core/services/graphql/fields/GraphQLField.php

Indentation +257 added lines, -257 removed lines

@@ -19,261 +19,261 @@
  */
 class GraphQLField implements GraphQLFieldInterface
 {
-    /**
-     * @var string $name
-     */
-    protected $name;
-
-    /**
-     * @var string|string[] $type
-     */
-    protected $type;
-
-    /**
-     * @var string|null $key
-     */
-    protected $key;
-
-    /**
-     * @var string $description
-     */
-    protected $description;
-
-    /**
-     * @var callable $formatter
-     */
-    protected $formatter;
-
-    /**
-     * @var callable $resolve
-     */
-    protected $resolver;
-
-    /**
-     * @var array $caps
-     */
-    protected $caps;
-
-    /**
-     * @var array $args
-     */
-    protected $args;
-
-    /**
-     * @var bool $use_for_input
-     */
-    protected $use_for_input = true;
-
-    /**
-     * @var bool $use_for_output
-     */
-    protected $use_for_output = true;
-
-
-    /**
-     * @param string          $name
-     * @param string|string[] $type
-     * @param string|null     $key
-     * @param string          $description
-     * @param callable|null   $formatter
-     * @param callable|null   $resolver
-     * @param array           $caps
-     * @param array           $args
-     */
-    public function __construct(
-        string $name,
-        $type,
-        string $key = null,
-        string $description = '',
-        callable $formatter = null,
-        callable $resolver = null,
-        array $caps = [],
-        array $args = []
-    ) {
-        $this->name = $name;
-        $this->type = $type;
-        $this->key = $key;
-        $this->description = $description;
-        $this->formatter = $formatter;
-        $this->resolver = $resolver;
-        $this->caps = $caps;
-        $this->args = $args;
-    }
-
-
-    /**
-     * @return array
-     */
-    public function caps(): array
-    {
-        return $this->caps;
-    }
-
-
-    /**
-     * @return string
-     */
-    public function description(): string
-    {
-        return $this->description;
-    }
-
-
-    /**
-     * @return string
-     */
-    public function key(): ?string
-    {
-        return $this->key;
-    }
-
-
-    /**
-     * @return string
-     */
-    public function name(): string
-    {
-        return $this->name;
-    }
-
-
-    /**
-     * @return string|string[]
-     */
-    public function type()
-    {
-        return $this->type;
-    }
-
-
-    /**
-     * Convert the field to array to be
-     * able to pass as config to WP GraphQL
-     *
-     * @return array
-     */
-    public function toArray(): array
-    {
-        return get_object_vars($this);
-    }
-
-
-    /**
-     * Sets the value for use_for_input.
-     *
-     * @param bool $use_for_input
-     */
-    protected function setUseForInput(bool $use_for_input)
-    {
-        $this->use_for_input = filter_var($use_for_input, FILTER_VALIDATE_BOOLEAN);
-    }
-
-
-    /**
-     * Whether the field should be used for
-     * mutation inputs.
-     *
-     * @return bool
-     */
-    public function useForInput(): bool
-    {
-        return $this->use_for_input;
-    }
-
-
-    /**
-     * Whether the field should be used for
-     * query outputs.
-     *
-     * @return bool
-     */
-    public function useForOutput(): bool
-    {
-        return $this->use_for_output;
-    }
-
-
-    /**
-     * Sets the value for use_for_output
-     *
-     * @param bool $use_for_output
-     */
-    protected function setUseForOutput(bool $use_for_output)
-    {
-        $this->use_for_output = filter_var($use_for_output, FILTER_VALIDATE_BOOLEAN);
-    }
-
-
-    /**
-     * Whether the field should resolve
-     * based on the user caps etc.
-     *
-     * @return boolean
-     */
-    public function shouldResolve(): bool
-    {
-        foreach ($this->caps as $cap) {
-            if (! current_user_can($cap)) {
-                return false;
-            }
-        }
-        return true;
-    }
-
-
-    /**
-     * Whether the field has an explicit resolver set.
-     *
-     * @return boolean
-     */
-    public function hasInternalResolver(): bool
-    {
-        return is_callable($this->resolver);
-    }
-
-
-    /**
-     * Whether the field has an explicit resolver set.
-     *
-     * @param mixed       $source  The source that's passed down the GraphQL queries
-     * @param array       $args    The inputArgs on the field
-     * @param AppContext  $context The AppContext passed down the GraphQL tree
-     * @param ResolveInfo $info    The ResolveInfo passed down the GraphQL tree
-     * @return mixed
-     * @throws LogicException
-     */
-    public function resolve($source, array $args, AppContext $context, ResolveInfo $info)
-    {
-        if (! $this->hasInternalResolver()) {
-            throw new LogicException('GraphQLField has no internal resolver.');
-        }
-        // dynamic methods using $this don't play nice
-        // so capture resolver to a single var first
-        $resolver = $this->resolver;
-        return $resolver($source, $args, $context, $info);
-    }
-
-
-    /**
-     * Checks if the format callback is set.
-     * If yes, then uses it to format the value.
-     *
-     * @param mixed         $value
-     * @param EE_Base_Class $source
-     * @return mixed The formatted value.
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     * @throws InvalidArgumentException
-     * @throws ReflectionException
-     */
-    public function mayBeFormatValue($value, EE_Base_Class $source)
-    {
-        if (is_callable($this->formatter)) {
-            // dynamic methods using $this don't play nice
-            // so capture formatter to a single var first
-            $formatter = $this->formatter;
-            return $formatter($value, $source);
-        }
-        return $value;
-    }
+	/**
+	 * @var string $name
+	 */
+	protected $name;
+
+	/**
+	 * @var string|string[] $type
+	 */
+	protected $type;
+
+	/**
+	 * @var string|null $key
+	 */
+	protected $key;
+
+	/**
+	 * @var string $description
+	 */
+	protected $description;
+
+	/**
+	 * @var callable $formatter
+	 */
+	protected $formatter;
+
+	/**
+	 * @var callable $resolve
+	 */
+	protected $resolver;
+
+	/**
+	 * @var array $caps
+	 */
+	protected $caps;
+
+	/**
+	 * @var array $args
+	 */
+	protected $args;
+
+	/**
+	 * @var bool $use_for_input
+	 */
+	protected $use_for_input = true;
+
+	/**
+	 * @var bool $use_for_output
+	 */
+	protected $use_for_output = true;
+
+
+	/**
+	 * @param string          $name
+	 * @param string|string[] $type
+	 * @param string|null     $key
+	 * @param string          $description
+	 * @param callable|null   $formatter
+	 * @param callable|null   $resolver
+	 * @param array           $caps
+	 * @param array           $args
+	 */
+	public function __construct(
+		string $name,
+		$type,
+		string $key = null,
+		string $description = '',
+		callable $formatter = null,
+		callable $resolver = null,
+		array $caps = [],
+		array $args = []
+	) {
+		$this->name = $name;
+		$this->type = $type;
+		$this->key = $key;
+		$this->description = $description;
+		$this->formatter = $formatter;
+		$this->resolver = $resolver;
+		$this->caps = $caps;
+		$this->args = $args;
+	}
+
+
+	/**
+	 * @return array
+	 */
+	public function caps(): array
+	{
+		return $this->caps;
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function description(): string
+	{
+		return $this->description;
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function key(): ?string
+	{
+		return $this->key;
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function name(): string
+	{
+		return $this->name;
+	}
+
+
+	/**
+	 * @return string|string[]
+	 */
+	public function type()
+	{
+		return $this->type;
+	}
+
+
+	/**
+	 * Convert the field to array to be
+	 * able to pass as config to WP GraphQL
+	 *
+	 * @return array
+	 */
+	public function toArray(): array
+	{
+		return get_object_vars($this);
+	}
+
+
+	/**
+	 * Sets the value for use_for_input.
+	 *
+	 * @param bool $use_for_input
+	 */
+	protected function setUseForInput(bool $use_for_input)
+	{
+		$this->use_for_input = filter_var($use_for_input, FILTER_VALIDATE_BOOLEAN);
+	}
+
+
+	/**
+	 * Whether the field should be used for
+	 * mutation inputs.
+	 *
+	 * @return bool
+	 */
+	public function useForInput(): bool
+	{
+		return $this->use_for_input;
+	}
+
+
+	/**
+	 * Whether the field should be used for
+	 * query outputs.
+	 *
+	 * @return bool
+	 */
+	public function useForOutput(): bool
+	{
+		return $this->use_for_output;
+	}
+
+
+	/**
+	 * Sets the value for use_for_output
+	 *
+	 * @param bool $use_for_output
+	 */
+	protected function setUseForOutput(bool $use_for_output)
+	{
+		$this->use_for_output = filter_var($use_for_output, FILTER_VALIDATE_BOOLEAN);
+	}
+
+
+	/**
+	 * Whether the field should resolve
+	 * based on the user caps etc.
+	 *
+	 * @return boolean
+	 */
+	public function shouldResolve(): bool
+	{
+		foreach ($this->caps as $cap) {
+			if (! current_user_can($cap)) {
+				return false;
+			}
+		}
+		return true;
+	}
+
+
+	/**
+	 * Whether the field has an explicit resolver set.
+	 *
+	 * @return boolean
+	 */
+	public function hasInternalResolver(): bool
+	{
+		return is_callable($this->resolver);
+	}
+
+
+	/**
+	 * Whether the field has an explicit resolver set.
+	 *
+	 * @param mixed       $source  The source that's passed down the GraphQL queries
+	 * @param array       $args    The inputArgs on the field
+	 * @param AppContext  $context The AppContext passed down the GraphQL tree
+	 * @param ResolveInfo $info    The ResolveInfo passed down the GraphQL tree
+	 * @return mixed
+	 * @throws LogicException
+	 */
+	public function resolve($source, array $args, AppContext $context, ResolveInfo $info)
+	{
+		if (! $this->hasInternalResolver()) {
+			throw new LogicException('GraphQLField has no internal resolver.');
+		}
+		// dynamic methods using $this don't play nice
+		// so capture resolver to a single var first
+		$resolver = $this->resolver;
+		return $resolver($source, $args, $context, $info);
+	}
+
+
+	/**
+	 * Checks if the format callback is set.
+	 * If yes, then uses it to format the value.
+	 *
+	 * @param mixed         $value
+	 * @param EE_Base_Class $source
+	 * @return mixed The formatted value.
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 * @throws InvalidArgumentException
+	 * @throws ReflectionException
+	 */
+	public function mayBeFormatValue($value, EE_Base_Class $source)
+	{
+		if (is_callable($this->formatter)) {
+			// dynamic methods using $this don't play nice
+			// so capture formatter to a single var first
+			$formatter = $this->formatter;
+			return $formatter($value, $source);
+		}
+		return $value;
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -213,7 +213,7 @@ 
     public function shouldResolve(): bool
     {
         foreach ($this->caps as $cap) {
-            if (! current_user_can($cap)) {
+            if ( ! current_user_can($cap)) {
                 return false;
             }
         }
@@ -244,7 +244,7 @@ 
      */
     public function resolve($source, array $args, AppContext $context, ResolveInfo $info)
     {
-        if (! $this->hasInternalResolver()) {
+        if ( ! $this->hasInternalResolver()) {
             throw new LogicException('GraphQLField has no internal resolver.');
         }
         // dynamic methods using $this don't play nice

core/services/orm/TrashLogger.php

Indentation +123 added lines, -123 removed lines

@@ -18,127 +18,127 @@
  */
 class TrashLogger
 {
-    /**
-     * extra meta key for tracking when entities are trashed and by who
-     *
-     * @type string
-     */
-    const EXTRA_META_KEY_ENTITY_TRASHED = 'entity-trashed';
-
-    /**
-     * extra meta key for tracking when entities are restored and by who
-     *
-     * @type string
-     */
-    const EXTRA_META_KEY_ENTITY_RESTORED = 'entity-restored';
-
-    /**
-     * extra meta key for tracking when entities are deleted and by who
-     *
-     * @type string
-     */
-    const EXTRA_META_KEY_ENTITY_DELETED = 'entity-deleted';
-
-
-    public function __construct()
-    {
-        add_action('AHEE__EE_Soft_Delete_Base_Class__delete_or_restore__after', [$this, 'logSoftDelete'], 10, 3);
-        add_action('AHEE__EE_Base_Class__delete_permanently__end', [$this, 'logHardDelete'], 10, 2);
-        add_action(
-            'AHEE__Events_Admin_Page___permanently_delete_event__after_event_deleted',
-            [$this, 'logBatchDelete'],
-            10,
-            2
-        );
-    }
-
-
-    /**
-     * @param EE_Soft_Delete_Base_Class $entity
-     * @param bool                      $delete
-     * @param bool                      $result
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function logSoftDelete(EE_Soft_Delete_Base_Class $entity, $delete, $result)
-    {
-        // which base meta key to use... trashed or restored?
-        $action = $delete
-            ? TrashLogger::EXTRA_META_KEY_ENTITY_TRASHED
-            : TrashLogger::EXTRA_META_KEY_ENTITY_RESTORED;
-
-        $entity_class = $this->getEntityClass($entity);
-        $this->logDelete($entity->ID(), $entity_class, $action, $result);
-    }
-
-
-    /**
-     * @param EE_Base_Class $entity
-     * @param bool          $result
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function logHardDelete(EE_Base_Class $entity, $result)
-    {
-        $entity_class = $this->getEntityClass($entity);
-        $this->logDelete($entity->ID(), $entity_class, TrashLogger::EXTRA_META_KEY_ENTITY_DELETED, $result);
-    }
-
-
-    /**
-     * @param int|string $entity_ID
-     * @param string     $entity_type
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function logBatchDelete($entity_ID, $entity_type)
-    {
-        $entity_type = str_replace(['EE_', 'EEM_'], '', $entity_type);
-        $this->logDelete($entity_ID, $entity_type, TrashLogger::EXTRA_META_KEY_ENTITY_DELETED, true);
-    }
-
-
-    /**
-     * converts entity class like 'EE_Registration' to 'registration'
-     *
-     * @param EE_Base_Class $entity
-     * @return string
-     */
-    private function getEntityClass(EE_Base_Class $entity)
-    {
-        return str_replace('EE_', '', get_class($entity));
-    }
-
-
-    /**
-     * @param string $entity_ID
-     * @param string $entity_type
-     * @param string $action
-     * @param bool   $result
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    private function logDelete($entity_ID, $entity_type, $action, $result)
-    {
-        // if trash/restore/delete was not successful, then get out
-        if (! $result) {
-            return;
-        }
-
-        // convert 'entity-deleted' to just 'deleted'
-        $action       = str_replace('entity-', '', $action);
-        $current_user = wp_get_current_user();
-        $user_name    = $current_user->ID ? $current_user->display_name : 'unknown user';
-        $timestamp    = date("D M j, Y @ g:i:s a", current_time('timestamp'));
-
-        $log = EE_Change_Log::new_instance(
-            [
-                'OBJ_ID'      => $entity_ID,
-                'OBJ_type'    => $entity_type,
-                'LOG_type'    => EEM_Change_Log::type_delete,
-                'LOG_message' => "$action by $user_name on $timestamp",
-            ]
-        );
-        $log->save();
-    }
+	/**
+	 * extra meta key for tracking when entities are trashed and by who
+	 *
+	 * @type string
+	 */
+	const EXTRA_META_KEY_ENTITY_TRASHED = 'entity-trashed';
+
+	/**
+	 * extra meta key for tracking when entities are restored and by who
+	 *
+	 * @type string
+	 */
+	const EXTRA_META_KEY_ENTITY_RESTORED = 'entity-restored';
+
+	/**
+	 * extra meta key for tracking when entities are deleted and by who
+	 *
+	 * @type string
+	 */
+	const EXTRA_META_KEY_ENTITY_DELETED = 'entity-deleted';
+
+
+	public function __construct()
+	{
+		add_action('AHEE__EE_Soft_Delete_Base_Class__delete_or_restore__after', [$this, 'logSoftDelete'], 10, 3);
+		add_action('AHEE__EE_Base_Class__delete_permanently__end', [$this, 'logHardDelete'], 10, 2);
+		add_action(
+			'AHEE__Events_Admin_Page___permanently_delete_event__after_event_deleted',
+			[$this, 'logBatchDelete'],
+			10,
+			2
+		);
+	}
+
+
+	/**
+	 * @param EE_Soft_Delete_Base_Class $entity
+	 * @param bool                      $delete
+	 * @param bool                      $result
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function logSoftDelete(EE_Soft_Delete_Base_Class $entity, $delete, $result)
+	{
+		// which base meta key to use... trashed or restored?
+		$action = $delete
+			? TrashLogger::EXTRA_META_KEY_ENTITY_TRASHED
+			: TrashLogger::EXTRA_META_KEY_ENTITY_RESTORED;
+
+		$entity_class = $this->getEntityClass($entity);
+		$this->logDelete($entity->ID(), $entity_class, $action, $result);
+	}
+
+
+	/**
+	 * @param EE_Base_Class $entity
+	 * @param bool          $result
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function logHardDelete(EE_Base_Class $entity, $result)
+	{
+		$entity_class = $this->getEntityClass($entity);
+		$this->logDelete($entity->ID(), $entity_class, TrashLogger::EXTRA_META_KEY_ENTITY_DELETED, $result);
+	}
+
+
+	/**
+	 * @param int|string $entity_ID
+	 * @param string     $entity_type
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function logBatchDelete($entity_ID, $entity_type)
+	{
+		$entity_type = str_replace(['EE_', 'EEM_'], '', $entity_type);
+		$this->logDelete($entity_ID, $entity_type, TrashLogger::EXTRA_META_KEY_ENTITY_DELETED, true);
+	}
+
+
+	/**
+	 * converts entity class like 'EE_Registration' to 'registration'
+	 *
+	 * @param EE_Base_Class $entity
+	 * @return string
+	 */
+	private function getEntityClass(EE_Base_Class $entity)
+	{
+		return str_replace('EE_', '', get_class($entity));
+	}
+
+
+	/**
+	 * @param string $entity_ID
+	 * @param string $entity_type
+	 * @param string $action
+	 * @param bool   $result
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	private function logDelete($entity_ID, $entity_type, $action, $result)
+	{
+		// if trash/restore/delete was not successful, then get out
+		if (! $result) {
+			return;
+		}
+
+		// convert 'entity-deleted' to just 'deleted'
+		$action       = str_replace('entity-', '', $action);
+		$current_user = wp_get_current_user();
+		$user_name    = $current_user->ID ? $current_user->display_name : 'unknown user';
+		$timestamp    = date("D M j, Y @ g:i:s a", current_time('timestamp'));
+
+		$log = EE_Change_Log::new_instance(
+			[
+				'OBJ_ID'      => $entity_ID,
+				'OBJ_type'    => $entity_type,
+				'LOG_type'    => EEM_Change_Log::type_delete,
+				'LOG_message' => "$action by $user_name on $timestamp",
+			]
+		);
+		$log->save();
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -121,7 +121,7 @@
     private function logDelete($entity_ID, $entity_type, $action, $result)
     {
         // if trash/restore/delete was not successful, then get out
-        if (! $result) {
+        if ( ! $result) {
             return;
         }
 

admin_pages/registrations/templates/attendee_registrations_main_meta_box.template.php

Spacing +9 added lines, -9 removed lines

@@ -32,9 +32,9 @@ 
                                 'espresso_events_edit',
                                 $EVT_ID
                             )
-                                ? '<a href="' . esc_url_raw($event_url) . '"  aria-label="'
-                                  . esc_attr__('Edit Event', 'event_espresso') . '">'
-                                  . esc_html($registration->event_name()) . '</a>'
+                                ? '<a href="'.esc_url_raw($event_url).'"  aria-label="'
+                                  . esc_attr__('Edit Event', 'event_espresso').'">'
+                                  . esc_html($registration->event_name()).'</a>'
                                 : esc_html($registration->event_name());
                             ?>
                         </td>
@@ -50,9 +50,9 @@ 
                                 'espresso_registrations_view_registration',
                                 $REG_ID
                             )
-                                ? '<a href="' . esc_url_raw($reg_url) . '" aria-label="'
-                                  . esc_attr__('View Registration Details', 'event_espresso') . '">'
-                                  . esc_html__('View Registration', 'event_espresso') . '</a>'
+                                ? '<a href="'.esc_url_raw($reg_url).'" aria-label="'
+                                  . esc_attr__('View Registration Details', 'event_espresso').'">'
+                                  . esc_html__('View Registration', 'event_espresso').'</a>'
                                 : $REG_ID;
                             ?>
                         </td>
@@ -67,12 +67,12 @@ 
                                 'ee_read_transaction',
                                 'espresso_transactions_view_transaction'
                             )
-                                ? '<a href="' . esc_url_raw($txn_url) . '" aria-label="'
-                                  . esc_attr__('View Transaction Details', 'event_espresso') . '">'
+                                ? '<a href="'.esc_url_raw($txn_url).'" aria-label="'
+                                  . esc_attr__('View Transaction Details', 'event_espresso').'">'
                                   . sprintf(
                                       esc_html__('View Transaction %d', 'event_espresso'),
                                       $TXN_ID
-                                  ) .
+                                  ).
                                   '</a>'
                                 : $TXN_ID;
                             ?>

Indentation +51 added lines, -51 removed lines

@@ -26,67 +26,67 @@
                     <tr>
                         <td class="jst-left">
                             <?php
-                            $EVT_ID = absint($registration->event_ID());
-                            $event_url = add_query_arg(
-                                ['action' => 'edit', 'post' => $EVT_ID],
-                                admin_url('admin.php?page=espresso_events')
-                            );
-                            echo EE_Registry::instance()->CAP->current_user_can(
-                                'ee_edit_event',
-                                'espresso_events_edit',
-                                $EVT_ID
-                            )
-                                ? '<a href="' . esc_url_raw($event_url) . '"  aria-label="'
-                                  . esc_attr__('Edit Event', 'event_espresso') . '">'
-                                  . esc_html($registration->event_name()) . '</a>'
-                                : esc_html($registration->event_name());
-                            ?>
+							$EVT_ID = absint($registration->event_ID());
+							$event_url = add_query_arg(
+								['action' => 'edit', 'post' => $EVT_ID],
+								admin_url('admin.php?page=espresso_events')
+							);
+							echo EE_Registry::instance()->CAP->current_user_can(
+								'ee_edit_event',
+								'espresso_events_edit',
+								$EVT_ID
+							)
+								? '<a href="' . esc_url_raw($event_url) . '"  aria-label="'
+								  . esc_attr__('Edit Event', 'event_espresso') . '">'
+								  . esc_html($registration->event_name()) . '</a>'
+								: esc_html($registration->event_name());
+							?>
                         </td>
                         <td class="jst-left">
                             <?php
-                            $REG_ID = absint($registration->ID());
-                            $reg_url = EE_Admin_Page::add_query_args_and_nonce(
-                                ['action' => 'view_registration', '_REG_ID' => $REG_ID],
-                                REG_ADMIN_URL
-                            );
-                            echo EE_Registry::instance()->CAP->current_user_can(
-                                'ee_read_registration',
-                                'espresso_registrations_view_registration',
-                                $REG_ID
-                            )
-                                ? '<a href="' . esc_url_raw($reg_url) . '" aria-label="'
-                                  . esc_attr__('View Registration Details', 'event_espresso') . '">'
-                                  . esc_html__('View Registration', 'event_espresso') . '</a>'
-                                : $REG_ID;
-                            ?>
+							$REG_ID = absint($registration->ID());
+							$reg_url = EE_Admin_Page::add_query_args_and_nonce(
+								['action' => 'view_registration', '_REG_ID' => $REG_ID],
+								REG_ADMIN_URL
+							);
+							echo EE_Registry::instance()->CAP->current_user_can(
+								'ee_read_registration',
+								'espresso_registrations_view_registration',
+								$REG_ID
+							)
+								? '<a href="' . esc_url_raw($reg_url) . '" aria-label="'
+								  . esc_attr__('View Registration Details', 'event_espresso') . '">'
+								  . esc_html__('View Registration', 'event_espresso') . '</a>'
+								: $REG_ID;
+							?>
                         </td>
                         <td class="jst-left">
                             <?php
-                            $TXN_ID = absint($registration->transaction_ID());
-                            $txn_url = EE_Admin_Page::add_query_args_and_nonce(
-                                ['action' => 'view_transaction', 'TXN_ID' => $TXN_ID],
-                                TXN_ADMIN_URL
-                            );
-                            echo EE_Registry::instance()->CAP->current_user_can(
-                                'ee_read_transaction',
-                                'espresso_transactions_view_transaction'
-                            )
-                                ? '<a href="' . esc_url_raw($txn_url) . '" aria-label="'
-                                  . esc_attr__('View Transaction Details', 'event_espresso') . '">'
-                                  . sprintf(
-                                      esc_html__('View Transaction %d', 'event_espresso'),
-                                      $TXN_ID
-                                  ) .
-                                  '</a>'
-                                : $TXN_ID;
-                            ?>
+							$TXN_ID = absint($registration->transaction_ID());
+							$txn_url = EE_Admin_Page::add_query_args_and_nonce(
+								['action' => 'view_transaction', 'TXN_ID' => $TXN_ID],
+								TXN_ADMIN_URL
+							);
+							echo EE_Registry::instance()->CAP->current_user_can(
+								'ee_read_transaction',
+								'espresso_transactions_view_transaction'
+							)
+								? '<a href="' . esc_url_raw($txn_url) . '" aria-label="'
+								  . esc_attr__('View Transaction Details', 'event_espresso') . '">'
+								  . sprintf(
+									  esc_html__('View Transaction %d', 'event_espresso'),
+									  $TXN_ID
+								  ) .
+								  '</a>'
+								: $TXN_ID;
+							?>
                         </td>
                         <td class="jst-left"><?php echo esc_html($registration->reg_code()); ?></td>
                         <td class="jst-rght">
                             <?php echo wp_kses(
-                                EEH_Template::format_currency($registration->final_price()),
-                                AllowedTags::getAllowedTags()
-                            ); ?>
+								EEH_Template::format_currency($registration->final_price()),
+								AllowedTags::getAllowedTags()
+							); ?>
                         </td>
                     </tr>
                 <?php endforeach; ?>

caffeinated/admin/extend/events/qtips/EE_Event_Editor_Tips.lib.php

Spacing +8 added lines, -8 removed lines

@@ -69,7 +69,7 @@ 
             ),
             7  => array(
                 'content_id' => 'tkt-status-archived',
-                'target'     => '.ticket-row .tkt-status-' . EE_Ticket::archived,
+                'target'     => '.ticket-row .tkt-status-'.EE_Ticket::archived,
                 'content'    => $this->_ticket_status_legend(EE_Ticket::archived),
                 'options'    => array(
                     'position' => array(
@@ -82,7 +82,7 @@ 
             ),
             8  => array(
                 'content_id' => 'tkt-status-expired',
-                'target'     => '.ticket-row .tkt-status-' . EE_Ticket::expired,
+                'target'     => '.ticket-row .tkt-status-'.EE_Ticket::expired,
                 'content'    => $this->_ticket_status_legend(EE_Ticket::expired),
                 'options'    => array(
                     'position' => array(
@@ -95,7 +95,7 @@ 
             ),
             9  => array(
                 'content_id' => 'tkt-status-sold_out',
-                'target'     => '.ticket-row .tkt-status-' . EE_Ticket::sold_out,
+                'target'     => '.ticket-row .tkt-status-'.EE_Ticket::sold_out,
                 'content'    => $this->_ticket_status_legend(EE_Ticket::sold_out),
                 'options'    => array(
                     'position' => array(
@@ -108,7 +108,7 @@ 
             ),
             10 => array(
                 'content_id' => 'tkt-status-pending',
-                'target'     => '.ticket-row .tkt-status-' . EE_Ticket::pending,
+                'target'     => '.ticket-row .tkt-status-'.EE_Ticket::pending,
                 'content'    => $this->_ticket_status_legend(EE_Ticket::pending),
                 'options'    => array(
                     'position' => array(
@@ -121,7 +121,7 @@ 
             ),
             11 => array(
                 'content_id' => 'tkt-status-onsale',
-                'target'     => '.ticket-row .tkt-status-' . EE_Ticket::onsale,
+                'target'     => '.ticket-row .tkt-status-'.EE_Ticket::onsale,
                 'content'    => $this->_ticket_status_legend(EE_Ticket::onsale),
                 'options'    => array(
                     'position' => array(
@@ -171,12 +171,12 @@ 
                        'Clicking the taxable ticket toggle checkbox has enabled taxes for this ticket. What this means is that when a person purchases this ticket, the tax will be applied to all prices on this ticket. You can edit the existing tax price modifier that was setup in Event Espresso by going to  %sDefault Pricing Admin Page%s (labelled "Pricing" in the Event Espresso Menu)',
                        'event_espresso'
                    ),
-                   '<a href="' . $price_admin_link . '" aria-label="' . esc_attr__(
+                   '<a href="'.$price_admin_link.'" aria-label="'.esc_attr__(
                        'Pricing Admin Page',
                        'event_espresso'
-                   ) . '">',
+                   ).'">',
                    '</a>'
-               ) . '</p>';
+               ).'</p>';
     }
 
     /**

Indentation +124 added lines, -124 removed lines

@@ -13,133 +13,133 @@
  */
 class EE_Event_Editor_Tips extends EE_Qtip_Config
 {
-    protected function _set_tips_array()
-    {
-        $this->_qtipsa = array(
-            0  => array(
-                'content_id' => 'about-taxable-toggle',
-                'target'     => '.TKT-taxable-checkbox',
-                'content'    => $this->_get_taxable_info_content(),
-                'options'    => array(
-                    'show_only_once' => true,
-                    'content'        => array(
-                        'title' => esc_html__('Taxable Ticket Toggle', 'event_espresso'),
-                        'button' => true,
-                    ),
-                    'show'           => array(
-                        'event' => 'click',
-                    ),
-                    'hide'           => array(
-                        'event' => false,
-                    ),
-                    'style'          => array(
-                        'classes' => '',
-                    ),
-                )// defaults
-            ),
-            7  => array(
-                'content_id' => 'tkt-status-archived',
-                'target'     => '.ticket-row .tkt-status-' . EE_Ticket::archived,
-                'content'    => $this->_ticket_status_legend(EE_Ticket::archived),
-                'options'    => array(
-                    'position' => array(
-                        'target' => 'mouse',
-                        'adjust' => array(
-                            'mouse' => false,
-                        ),
-                    ),
-                ),
-            ),
-            8  => array(
-                'content_id' => 'tkt-status-expired',
-                'target'     => '.ticket-row .tkt-status-' . EE_Ticket::expired,
-                'content'    => $this->_ticket_status_legend(EE_Ticket::expired),
-                'options'    => array(
-                    'position' => array(
-                        'target' => 'mouse',
-                        'adjust' => array(
-                            'mouse' => false,
-                        ),
-                    ),
-                ),
-            ),
-            9  => array(
-                'content_id' => 'tkt-status-sold_out',
-                'target'     => '.ticket-row .tkt-status-' . EE_Ticket::sold_out,
-                'content'    => $this->_ticket_status_legend(EE_Ticket::sold_out),
-                'options'    => array(
-                    'position' => array(
-                        'target' => 'mouse',
-                        'adjust' => array(
-                            'mouse' => false,
-                        ),
-                    ),
-                ),
-            ),
-            10 => array(
-                'content_id' => 'tkt-status-pending',
-                'target'     => '.ticket-row .tkt-status-' . EE_Ticket::pending,
-                'content'    => $this->_ticket_status_legend(EE_Ticket::pending),
-                'options'    => array(
-                    'position' => array(
-                        'target' => 'mouse',
-                        'adjust' => array(
-                            'mouse' => false,
-                        ),
-                    ),
-                ),
-            ),
-            11 => array(
-                'content_id' => 'tkt-status-onsale',
-                'target'     => '.ticket-row .tkt-status-' . EE_Ticket::onsale,
-                'content'    => $this->_ticket_status_legend(EE_Ticket::onsale),
-                'options'    => array(
-                    'position' => array(
-                        'target' => 'mouse',
-                        'adjust' => array(
-                            'mouse' => false,
-                        ),
-                    ),
-                ),
-            ),
-        );
-    }
+	protected function _set_tips_array()
+	{
+		$this->_qtipsa = array(
+			0  => array(
+				'content_id' => 'about-taxable-toggle',
+				'target'     => '.TKT-taxable-checkbox',
+				'content'    => $this->_get_taxable_info_content(),
+				'options'    => array(
+					'show_only_once' => true,
+					'content'        => array(
+						'title' => esc_html__('Taxable Ticket Toggle', 'event_espresso'),
+						'button' => true,
+					),
+					'show'           => array(
+						'event' => 'click',
+					),
+					'hide'           => array(
+						'event' => false,
+					),
+					'style'          => array(
+						'classes' => '',
+					),
+				)// defaults
+			),
+			7  => array(
+				'content_id' => 'tkt-status-archived',
+				'target'     => '.ticket-row .tkt-status-' . EE_Ticket::archived,
+				'content'    => $this->_ticket_status_legend(EE_Ticket::archived),
+				'options'    => array(
+					'position' => array(
+						'target' => 'mouse',
+						'adjust' => array(
+							'mouse' => false,
+						),
+					),
+				),
+			),
+			8  => array(
+				'content_id' => 'tkt-status-expired',
+				'target'     => '.ticket-row .tkt-status-' . EE_Ticket::expired,
+				'content'    => $this->_ticket_status_legend(EE_Ticket::expired),
+				'options'    => array(
+					'position' => array(
+						'target' => 'mouse',
+						'adjust' => array(
+							'mouse' => false,
+						),
+					),
+				),
+			),
+			9  => array(
+				'content_id' => 'tkt-status-sold_out',
+				'target'     => '.ticket-row .tkt-status-' . EE_Ticket::sold_out,
+				'content'    => $this->_ticket_status_legend(EE_Ticket::sold_out),
+				'options'    => array(
+					'position' => array(
+						'target' => 'mouse',
+						'adjust' => array(
+							'mouse' => false,
+						),
+					),
+				),
+			),
+			10 => array(
+				'content_id' => 'tkt-status-pending',
+				'target'     => '.ticket-row .tkt-status-' . EE_Ticket::pending,
+				'content'    => $this->_ticket_status_legend(EE_Ticket::pending),
+				'options'    => array(
+					'position' => array(
+						'target' => 'mouse',
+						'adjust' => array(
+							'mouse' => false,
+						),
+					),
+				),
+			),
+			11 => array(
+				'content_id' => 'tkt-status-onsale',
+				'target'     => '.ticket-row .tkt-status-' . EE_Ticket::onsale,
+				'content'    => $this->_ticket_status_legend(EE_Ticket::onsale),
+				'options'    => array(
+					'position' => array(
+						'target' => 'mouse',
+						'adjust' => array(
+							'mouse' => false,
+						),
+					),
+				),
+			),
+		);
+	}
 
 
-    private function _get_taxable_info_content()
-    {
-        $price_admin_link = EE_Admin_Page::add_query_args_and_nonce(array('action' => 'default'), PRICING_ADMIN_URL);
-        return '<p>'
-               . sprintf(
-                   esc_html__(
-                       'Clicking the taxable ticket toggle checkbox has enabled taxes for this ticket. What this means is that when a person purchases this ticket, the tax will be applied to all prices on this ticket. You can edit the existing tax price modifier that was setup in Event Espresso by going to  %sDefault Pricing Admin Page%s (labelled "Pricing" in the Event Espresso Menu)',
-                       'event_espresso'
-                   ),
-                   '<a href="' . $price_admin_link . '" aria-label="' . esc_attr__(
-                       'Pricing Admin Page',
-                       'event_espresso'
-                   ) . '">',
-                   '</a>'
-               ) . '</p>';
-    }
+	private function _get_taxable_info_content()
+	{
+		$price_admin_link = EE_Admin_Page::add_query_args_and_nonce(array('action' => 'default'), PRICING_ADMIN_URL);
+		return '<p>'
+			   . sprintf(
+				   esc_html__(
+					   'Clicking the taxable ticket toggle checkbox has enabled taxes for this ticket. What this means is that when a person purchases this ticket, the tax will be applied to all prices on this ticket. You can edit the existing tax price modifier that was setup in Event Espresso by going to  %sDefault Pricing Admin Page%s (labelled "Pricing" in the Event Espresso Menu)',
+					   'event_espresso'
+				   ),
+				   '<a href="' . $price_admin_link . '" aria-label="' . esc_attr__(
+					   'Pricing Admin Page',
+					   'event_espresso'
+				   ) . '">',
+				   '</a>'
+			   ) . '</p>';
+	}
 
-    /**
-     * output the relevant ee-status-legend with the designated status highlighted.
-     *
-     * @param  EE_Ticket constant $status What status is set (by class)
-     * @return string         The status legend with the related status highlighted
-     */
-    private function _ticket_status_legend($status)
-    {
+	/**
+	 * output the relevant ee-status-legend with the designated status highlighted.
+	 *
+	 * @param  EE_Ticket constant $status What status is set (by class)
+	 * @return string         The status legend with the related status highlighted
+	 */
+	private function _ticket_status_legend($status)
+	{
 
-        $status_array = array(
-            'archived' => EE_Ticket::archived,
-            'expired'  => EE_Ticket::expired,
-            'sold_out' => EE_Ticket::sold_out,
-            'pending'  => EE_Ticket::pending,
-            'onsale'   => EE_Ticket::onsale,
-        );
+		$status_array = array(
+			'archived' => EE_Ticket::archived,
+			'expired'  => EE_Ticket::expired,
+			'sold_out' => EE_Ticket::sold_out,
+			'pending'  => EE_Ticket::pending,
+			'onsale'   => EE_Ticket::onsale,
+		);
 
-        return EEH_Template::status_legend($status_array, $status);
-    }
+		return EEH_Template::status_legend($status_array, $status);
+	}
 }

core/domain/services/commands/registration/CreateRegistrationCommand.php

Spacing +2 added lines, -2 removed lines

@@ -90,7 +90,7 @@ 
         $this->reg_status       = $reg_status;
         // grab the related ticket object for this line_item if one wasn't already supplied
         $this->ticket = $ticket instanceof EE_Ticket ? $ticket : $this->ticket_line_item->ticket();
-        if (! $this->ticket instanceof EE_Ticket) {
+        if ( ! $this->ticket instanceof EE_Ticket) {
             throw new InvalidEntityException(
                 is_object($this->ticket) ? get_class($this->ticket) : gettype($this->ticket),
                 'EE_Ticket',
@@ -109,7 +109,7 @@ 
      */
     public function getCapCheck()
     {
-        if (! $this->cap_check instanceof CapCheckInterface) {
+        if ( ! $this->cap_check instanceof CapCheckInterface) {
             // need cap for non-AJAX admin requests
             $this->cap_check = ! (defined('DOING_AJAX') && DOING_AJAX) && is_admin()
                 ? new CapCheck('ee_edit_registrations', 'create_new_registration')

Indentation +154 added lines, -154 removed lines

@@ -26,158 +26,158 @@
  */
 class CreateRegistrationCommand extends Command implements CommandRequiresCapCheckInterface
 {
-    /**
-     * @var EE_Transaction $transaction
-     */
-    private $transaction;
-
-    /**
-     * @var EE_Ticket $ticket
-     */
-    private $ticket;
-
-    /**
-     * @var EE_Line_Item $ticket_line_item
-     */
-    private $ticket_line_item;
-
-    /**
-     * @var int $reg_count
-     */
-    private $reg_count;
-
-    /**
-     * @var int $reg_group_size
-     */
-    private $reg_group_size;
-
-    /**
-     * @var string $reg_status
-     */
-    private $reg_status;
-
-    /**
-     * @var EE_Registration $registration
-     */
-    protected $registration;
-
-
-    /**
-     * CreateRegistrationCommand constructor.
-     *
-     * @param EE_Transaction $transaction
-     * @param EE_Line_Item   $ticket_line_item
-     * @param int            $reg_count
-     * @param int            $reg_group_size
-     * @param string         $reg_status
-     * @param EE_Ticket|null $ticket
-     * @throws InvalidEntityException
-     * @throws EE_Error
-     */
-    public function __construct(
-        EE_Transaction $transaction,
-        EE_Line_Item $ticket_line_item,
-        $reg_count = 1,
-        $reg_group_size = 0,
-        $reg_status = RegStatus::INCOMPLETE,
-        EE_Ticket $ticket = null
-    ) {
-        defined('EVENT_ESPRESSO_VERSION') || exit;
-        $this->transaction      = $transaction;
-        $this->ticket_line_item = $ticket_line_item;
-        $this->reg_count        = absint($reg_count);
-        $this->reg_group_size   = absint($reg_group_size);
-        $this->reg_status       = $reg_status;
-        // grab the related ticket object for this line_item if one wasn't already supplied
-        $this->ticket = $ticket instanceof EE_Ticket ? $ticket : $this->ticket_line_item->ticket();
-        if (! $this->ticket instanceof EE_Ticket) {
-            throw new InvalidEntityException(
-                is_object($this->ticket) ? get_class($this->ticket) : gettype($this->ticket),
-                'EE_Ticket',
-                sprintf(
-                    esc_html__('Line item %s did not contain a valid ticket', 'event_espresso'),
-                    $ticket_line_item->ID()
-                )
-            );
-        }
-    }
-
-
-    /**
-     * @return CapCheckInterface
-     * @throws InvalidDataTypeException
-     */
-    public function getCapCheck()
-    {
-        if (! $this->cap_check instanceof CapCheckInterface) {
-            // need cap for non-AJAX admin requests
-            $this->cap_check = ! (defined('DOING_AJAX') && DOING_AJAX) && is_admin()
-                ? new CapCheck('ee_edit_registrations', 'create_new_registration')
-                : new PublicCapabilities('', 'create_new_registration');
-        }
-        return $this->cap_check;
-    }
-
-
-    /**
-     * @return EE_Transaction
-     */
-    public function transaction()
-    {
-        return $this->transaction;
-    }
-
-
-    /**
-     * @return EE_Ticket
-     */
-    public function ticket()
-    {
-        return $this->ticket;
-    }
-
-
-    /**
-     * @return EE_Line_Item
-     */
-    public function ticketLineItem()
-    {
-        return $this->ticket_line_item;
-    }
-
-
-    /**
-     * @return int
-     */
-    public function regCount()
-    {
-        return $this->reg_count;
-    }
-
-
-    /**
-     * @return int
-     */
-    public function regGroupSize()
-    {
-        return $this->reg_group_size;
-    }
-
-
-    /**
-     * @return string
-     */
-    public function regStatus()
-    {
-        return $this->reg_status;
-    }
-
-
-    /**
-     * @return EE_Registration
-     */
-    public function registration()
-    {
-        return $this->registration;
-    }
+	/**
+	 * @var EE_Transaction $transaction
+	 */
+	private $transaction;
+
+	/**
+	 * @var EE_Ticket $ticket
+	 */
+	private $ticket;
+
+	/**
+	 * @var EE_Line_Item $ticket_line_item
+	 */
+	private $ticket_line_item;
+
+	/**
+	 * @var int $reg_count
+	 */
+	private $reg_count;
+
+	/**
+	 * @var int $reg_group_size
+	 */
+	private $reg_group_size;
+
+	/**
+	 * @var string $reg_status
+	 */
+	private $reg_status;
+
+	/**
+	 * @var EE_Registration $registration
+	 */
+	protected $registration;
+
+
+	/**
+	 * CreateRegistrationCommand constructor.
+	 *
+	 * @param EE_Transaction $transaction
+	 * @param EE_Line_Item   $ticket_line_item
+	 * @param int            $reg_count
+	 * @param int            $reg_group_size
+	 * @param string         $reg_status
+	 * @param EE_Ticket|null $ticket
+	 * @throws InvalidEntityException
+	 * @throws EE_Error
+	 */
+	public function __construct(
+		EE_Transaction $transaction,
+		EE_Line_Item $ticket_line_item,
+		$reg_count = 1,
+		$reg_group_size = 0,
+		$reg_status = RegStatus::INCOMPLETE,
+		EE_Ticket $ticket = null
+	) {
+		defined('EVENT_ESPRESSO_VERSION') || exit;
+		$this->transaction      = $transaction;
+		$this->ticket_line_item = $ticket_line_item;
+		$this->reg_count        = absint($reg_count);
+		$this->reg_group_size   = absint($reg_group_size);
+		$this->reg_status       = $reg_status;
+		// grab the related ticket object for this line_item if one wasn't already supplied
+		$this->ticket = $ticket instanceof EE_Ticket ? $ticket : $this->ticket_line_item->ticket();
+		if (! $this->ticket instanceof EE_Ticket) {
+			throw new InvalidEntityException(
+				is_object($this->ticket) ? get_class($this->ticket) : gettype($this->ticket),
+				'EE_Ticket',
+				sprintf(
+					esc_html__('Line item %s did not contain a valid ticket', 'event_espresso'),
+					$ticket_line_item->ID()
+				)
+			);
+		}
+	}
+
+
+	/**
+	 * @return CapCheckInterface
+	 * @throws InvalidDataTypeException
+	 */
+	public function getCapCheck()
+	{
+		if (! $this->cap_check instanceof CapCheckInterface) {
+			// need cap for non-AJAX admin requests
+			$this->cap_check = ! (defined('DOING_AJAX') && DOING_AJAX) && is_admin()
+				? new CapCheck('ee_edit_registrations', 'create_new_registration')
+				: new PublicCapabilities('', 'create_new_registration');
+		}
+		return $this->cap_check;
+	}
+
+
+	/**
+	 * @return EE_Transaction
+	 */
+	public function transaction()
+	{
+		return $this->transaction;
+	}
+
+
+	/**
+	 * @return EE_Ticket
+	 */
+	public function ticket()
+	{
+		return $this->ticket;
+	}
+
+
+	/**
+	 * @return EE_Line_Item
+	 */
+	public function ticketLineItem()
+	{
+		return $this->ticket_line_item;
+	}
+
+
+	/**
+	 * @return int
+	 */
+	public function regCount()
+	{
+		return $this->reg_count;
+	}
+
+
+	/**
+	 * @return int
+	 */
+	public function regGroupSize()
+	{
+		return $this->reg_group_size;
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function regStatus()
+	{
+		return $this->reg_status;
+	}
+
+
+	/**
+	 * @return EE_Registration
+	 */
+	public function registration()
+	{
+		return $this->registration;
+	}
 }

core/services/helpers/DecimalValues.php

Indentation +53 added lines, -53 removed lines

@@ -6,57 +6,57 @@
 
 class DecimalValues
 {
-    /**
-     * number of decimal places to round numbers to when performing calculations
-     *
-     * @var integer
-     */
-    protected $decimal_precision = 6;
-
-    /**
-     * number of decimal places to round numbers to for display
-     *
-     * @var integer
-     */
-    protected $locale_precision = 6;
-
-
-    /**
-     * @param EE_Currency_Config $currency_config
-     */
-    public function __construct(EE_Currency_Config $currency_config)
-    {
-        $this->locale_precision = $currency_config->dec_plc;
-    }
-
-    /**
-     * strips formatting, rounds the provided number, and returns a float
-     * if $round is set to true, then the decimal precision for the site locale will be used,
-     * otherwise the default decimal precision of 6 will be used
-     *
-     * @param float|int|string $number unformatted number value, ex: 1234.5678956789
-     * @param bool             $round  whether to round the price off according to the locale settings
-     * @return float                      rounded value, ex: 1,234.567896
-     */
-    public function roundDecimalValue($number, bool $round = false): float
-    {
-        $precision = $round ? $this->locale_precision : $this->decimal_precision;
-        return round(
-            $this->filterDecimalValue($number),
-            $precision ?? $this->decimal_precision,
-            PHP_ROUND_HALF_UP
-        );
-    }
-
-
-    /**
-     * Removes all characters except digits, +- and .
-     *
-     * @param float|int|string $number
-     * @return float
-     */
-    public function filterDecimalValue($number): float
-    {
-        return (float) $number;
-    }
+	/**
+	 * number of decimal places to round numbers to when performing calculations
+	 *
+	 * @var integer
+	 */
+	protected $decimal_precision = 6;
+
+	/**
+	 * number of decimal places to round numbers to for display
+	 *
+	 * @var integer
+	 */
+	protected $locale_precision = 6;
+
+
+	/**
+	 * @param EE_Currency_Config $currency_config
+	 */
+	public function __construct(EE_Currency_Config $currency_config)
+	{
+		$this->locale_precision = $currency_config->dec_plc;
+	}
+
+	/**
+	 * strips formatting, rounds the provided number, and returns a float
+	 * if $round is set to true, then the decimal precision for the site locale will be used,
+	 * otherwise the default decimal precision of 6 will be used
+	 *
+	 * @param float|int|string $number unformatted number value, ex: 1234.5678956789
+	 * @param bool             $round  whether to round the price off according to the locale settings
+	 * @return float                      rounded value, ex: 1,234.567896
+	 */
+	public function roundDecimalValue($number, bool $round = false): float
+	{
+		$precision = $round ? $this->locale_precision : $this->decimal_precision;
+		return round(
+			$this->filterDecimalValue($number),
+			$precision ?? $this->decimal_precision,
+			PHP_ROUND_HALF_UP
+		);
+	}
+
+
+	/**
+	 * Removes all characters except digits, +- and .
+	 *
+	 * @param float|int|string $number
+	 * @return float
+	 */
+	public function filterDecimalValue($number): float
+	{
+		return (float) $number;
+	}
 }