eventespresso / event-espresso-core

core/db_models/fields/EE_Full_HTML_Field.php

Indentation +48 added lines, -48 removed lines

@@ -14,57 +14,57 @@
 class EE_Full_HTML_Field extends EE_Text_Field_Base
 {
 
-    /**
-     * @param string $table_column
-     * @param string $nicename
-     * @param bool   $nullable
-     * @param null   $default_value
-     */
-    public function __construct($table_column, $nicename, $nullable, $default_value = null)
-    {
-        parent::__construct($table_column, $nicename, $nullable, $default_value);
-        $this->setSchemaType('object');
-    }
+	/**
+	 * @param string $table_column
+	 * @param string $nicename
+	 * @param bool   $nullable
+	 * @param null   $default_value
+	 */
+	public function __construct($table_column, $nicename, $nullable, $default_value = null)
+	{
+		parent::__construct($table_column, $nicename, $nullable, $default_value);
+		$this->setSchemaType('object');
+	}
 
 
-    /**
-     * Does shortcodes and auto-paragraphs the content (unless schema is 'no_wpautop')
-     *
-     * @param type $value_on_field_to_be_outputted
-     * @param type $schema
-     * @return string
-     */
-    public function prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema = null)
-    {
-        if ($schema == 'form_input') {
-            return parent::prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema);
-        } elseif ($schema == 'no_wpautop') {
-            return do_shortcode(parent::prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema));
-        } else {
-            return wpautop(do_shortcode(parent::prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema)));
-        }
-    }
+	/**
+	 * Does shortcodes and auto-paragraphs the content (unless schema is 'no_wpautop')
+	 *
+	 * @param type $value_on_field_to_be_outputted
+	 * @param type $schema
+	 * @return string
+	 */
+	public function prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema = null)
+	{
+		if ($schema == 'form_input') {
+			return parent::prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema);
+		} elseif ($schema == 'no_wpautop') {
+			return do_shortcode(parent::prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema));
+		} else {
+			return wpautop(do_shortcode(parent::prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema)));
+		}
+	}
 
 
 
-    public function getSchemaProperties()
-    {
-        return array(
-            'raw' => array(
-                'description' =>  sprintf(
-                    esc_html__('%s - the value in the database.', 'event_espresso'),
-                    $this->get_nicename()
-                ),
-                'type' => 'string'
-            ),
-            'rendered' => array(
-                'description' =>  sprintf(
-                    esc_html__('%s - transformed for display.', 'event_espresso'),
-                    $this->get_nicename()
-                ),
-                'type' => 'string',
-                'readonly' => true
-            )
-        );
-    }
+	public function getSchemaProperties()
+	{
+		return array(
+			'raw' => array(
+				'description' =>  sprintf(
+					esc_html__('%s - the value in the database.', 'event_espresso'),
+					$this->get_nicename()
+				),
+				'type' => 'string'
+			),
+			'rendered' => array(
+				'description' =>  sprintf(
+					esc_html__('%s - transformed for display.', 'event_espresso'),
+					$this->get_nicename()
+				),
+				'type' => 'string',
+				'readonly' => true
+			)
+		);
+	}
 }

core/db_models/fields/EE_Infinite_Integer_Field.php

Indentation +69 added lines, -69 removed lines

@@ -10,78 +10,78 @@
 class EE_Infinite_Integer_Field extends EE_Model_Field_Base
 {
 
-    /**
-     * @param string $table_column
-     * @param string $nicename
-     * @param bool   $nullable
-     * @param null   $default_value
-     */
-    public function __construct($table_column, $nicename, $nullable, $default_value = null)
-    {
-        parent::__construct($table_column, $nicename, $nullable, $default_value);
-        $this->setSchemaType(array('integer', 'null'));
-    }
+	/**
+	 * @param string $table_column
+	 * @param string $nicename
+	 * @param bool   $nullable
+	 * @param null   $default_value
+	 */
+	public function __construct($table_column, $nicename, $nullable, $default_value = null)
+	{
+		parent::__construct($table_column, $nicename, $nullable, $default_value);
+		$this->setSchemaType(array('integer', 'null'));
+	}
 
 
-    public function prepare_for_use_in_db($value_of_field_on_model_object)
-    {
-        if ($value_of_field_on_model_object === EE_INF) {
-            return EE_INF_IN_DB;
-        } else {
-            return intval($value_of_field_on_model_object);
-        }
-    }
+	public function prepare_for_use_in_db($value_of_field_on_model_object)
+	{
+		if ($value_of_field_on_model_object === EE_INF) {
+			return EE_INF_IN_DB;
+		} else {
+			return intval($value_of_field_on_model_object);
+		}
+	}
 
-    public function prepare_for_set($value_inputted_for_field_on_model_object)
-    {
-        if (
-            $value_inputted_for_field_on_model_object === EE_INF_IN_DB ||
-            $value_inputted_for_field_on_model_object === EE_INF ||
-            $value_inputted_for_field_on_model_object === "EE_INF" ||
-            $value_inputted_for_field_on_model_object === ""
-        ) {
-            return EE_INF;
-        } else {
-            return intval($value_inputted_for_field_on_model_object);
-        }
-    }
+	public function prepare_for_set($value_inputted_for_field_on_model_object)
+	{
+		if (
+			$value_inputted_for_field_on_model_object === EE_INF_IN_DB ||
+			$value_inputted_for_field_on_model_object === EE_INF ||
+			$value_inputted_for_field_on_model_object === "EE_INF" ||
+			$value_inputted_for_field_on_model_object === ""
+		) {
+			return EE_INF;
+		} else {
+			return intval($value_inputted_for_field_on_model_object);
+		}
+	}
 
-    public function prepare_for_set_from_db($value_inputted_for_field_on_model_object)
-    {
-        $intval = intval($value_inputted_for_field_on_model_object);
-        if ($intval == EE_INF_IN_DB) {
-            return EE_INF;
-        } else {
-            return $intval;
-        }
-    }
+	public function prepare_for_set_from_db($value_inputted_for_field_on_model_object)
+	{
+		$intval = intval($value_inputted_for_field_on_model_object);
+		if ($intval == EE_INF_IN_DB) {
+			return EE_INF;
+		} else {
+			return $intval;
+		}
+	}
 
-    /**
-     * For outputting this field's value. If you want to output it into an input or something,
-     * use $schema=='input', as it will replace EE_INF with ''. If you want a readable version, use $schema=='text'
-     * as it will replace EE_INF with i18n Infinite
-     *
-     * @param type   $value_on_field_to_be_outputted
-     * @param string $schema input, symbol, text; or any string you want to show if the value equals EE_INF
-     * @return string
-     */
-    public function prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema = null)
-    {
-        if ($value_on_field_to_be_outputted === EE_INF) {
-            switch ($schema) {
-                case 'input':
-                case 'form_input':
-                    return '';
-                case 'symbol':
-                    return "∞";
-                case 'text':
-                case null:
-                    return esc_html__("Unlimited", "event_espresso");
-                default:
-                    return $schema;
-            }
-        } else {
-            return $value_on_field_to_be_outputted;
-        }
-    }
+	/**
+	 * For outputting this field's value. If you want to output it into an input or something,
+	 * use $schema=='input', as it will replace EE_INF with ''. If you want a readable version, use $schema=='text'
+	 * as it will replace EE_INF with i18n Infinite
+	 *
+	 * @param type   $value_on_field_to_be_outputted
+	 * @param string $schema input, symbol, text; or any string you want to show if the value equals EE_INF
+	 * @return string
+	 */
+	public function prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema = null)
+	{
+		if ($value_on_field_to_be_outputted === EE_INF) {
+			switch ($schema) {
+				case 'input':
+				case 'form_input':
+					return '';
+				case 'symbol':
+					return "∞";
+				case 'text':
+				case null:
+					return esc_html__("Unlimited", "event_espresso");
+				default:
+					return $schema;
+			}
+		} else {
+			return $value_on_field_to_be_outputted;
+		}
+	}
 }

core/db_models/fields/EE_Foreign_Key_Int_Field.php

Indentation +31 added lines, -31 removed lines

@@ -3,39 +3,39 @@
 class EE_Foreign_Key_Int_Field extends EE_Foreign_Key_Field_Base
 {
 
-    /**
-     * @param string       $table_column  name fo column for field
-     * @param string       $nice_name     should eb internationalized with esc_html__('blah','event_espresso')
-     * @param boolean      $nullable
-     * @param mixed        $default_value if this is a integer field, it should be an int.
-     *                                    if it's a string field, it should be a string
-     * @param string|array $model_name    eg 'Event','Answer','Term', etc.
-     *                                    Basically its the model class's name without the "EEM_"
-     */
-    public function __construct($table_column, $nice_name, $nullable, $default_value, $model_name)
-    {
-        parent::__construct($table_column, $nice_name, $nullable, $default_value, $model_name);
-        $this->setSchemaType('integer');
-    }
+	/**
+	 * @param string       $table_column  name fo column for field
+	 * @param string       $nice_name     should eb internationalized with esc_html__('blah','event_espresso')
+	 * @param boolean      $nullable
+	 * @param mixed        $default_value if this is a integer field, it should be an int.
+	 *                                    if it's a string field, it should be a string
+	 * @param string|array $model_name    eg 'Event','Answer','Term', etc.
+	 *                                    Basically its the model class's name without the "EEM_"
+	 */
+	public function __construct($table_column, $nice_name, $nullable, $default_value, $model_name)
+	{
+		parent::__construct($table_column, $nice_name, $nullable, $default_value, $model_name);
+		$this->setSchemaType('integer');
+	}
 
 
-    /**
-     * @param int|EE_Base_Class $value_inputted_for_field_on_model_object
-     * @return int
-     * @throws EE_Error
-     * @throws EE_Error
-     */
-    public function prepare_for_set($value_inputted_for_field_on_model_object): int
-    {
-        if ($this->is_model_obj_of_type_pointed_to($value_inputted_for_field_on_model_object)) {
-            $value_inputted_for_field_on_model_object = $value_inputted_for_field_on_model_object->ID();
-        }
-        return absint($value_inputted_for_field_on_model_object);
-    }
+	/**
+	 * @param int|EE_Base_Class $value_inputted_for_field_on_model_object
+	 * @return int
+	 * @throws EE_Error
+	 * @throws EE_Error
+	 */
+	public function prepare_for_set($value_inputted_for_field_on_model_object): int
+	{
+		if ($this->is_model_obj_of_type_pointed_to($value_inputted_for_field_on_model_object)) {
+			$value_inputted_for_field_on_model_object = $value_inputted_for_field_on_model_object->ID();
+		}
+		return absint($value_inputted_for_field_on_model_object);
+	}
 
 
-    public function prepare_for_set_from_db($value_found_in_db_for_model_object)
-    {
-        return intval($value_found_in_db_for_model_object);
-    }
+	public function prepare_for_set_from_db($value_found_in_db_for_model_object)
+	{
+		return intval($value_found_in_db_for_model_object);
+	}
 }

core/db_models/fields/EE_WP_User_Field.php

Indentation +34 added lines, -34 removed lines

@@ -13,39 +13,39 @@
  */
 class EE_WP_User_Field extends EE_Foreign_Key_Int_Field
 {
-    /**
-     * No need to provide a default or the model pointed to- the default is
-     * always get_current_user_id() and the model pointed to is always WP_User
-     *
-     * @param string  $table_column name fo column for field
-     * @param string  $nice_name     should eb internationalized with esc_html__('blah','event_espresso')
-     * @param boolean $nullable
-     */
-    public function __construct($table_column, $nice_name, $nullable)
-    {
-        parent::__construct($table_column, $nice_name, $nullable, get_current_user_id(), 'WP_User');
-    }
+	/**
+	 * No need to provide a default or the model pointed to- the default is
+	 * always get_current_user_id() and the model pointed to is always WP_User
+	 *
+	 * @param string  $table_column name fo column for field
+	 * @param string  $nice_name     should eb internationalized with esc_html__('blah','event_espresso')
+	 * @param boolean $nullable
+	 */
+	public function __construct($table_column, $nice_name, $nullable)
+	{
+		parent::__construct($table_column, $nice_name, $nullable, get_current_user_id(), 'WP_User');
+	}
 
-    /**
-     * Gets the default which is always the current user. This can't be set when initially
-     * constructing the model field because that's done before $current_user is set
-     *
-     * @return mixed
-     */
-    public function get_default_value()
-    {
-        if (did_action('init')) {
-            return get_current_user_id();
-        } else {
-            EE_Error::doing_it_wrong(
-                'EE_WP_User_Field::get_default_value',
-                esc_html__(
-                    'You cant get a default value for a wp_User_Field because the "init" action is called, because current_user global hasnt yet been setup. Consider doing your business logic on the "init" hook or later.',
-                    'event_espresso'
-                ),
-                '4.6.20'
-            );
-            return 1;
-        }
-    }
+	/**
+	 * Gets the default which is always the current user. This can't be set when initially
+	 * constructing the model field because that's done before $current_user is set
+	 *
+	 * @return mixed
+	 */
+	public function get_default_value()
+	{
+		if (did_action('init')) {
+			return get_current_user_id();
+		} else {
+			EE_Error::doing_it_wrong(
+				'EE_WP_User_Field::get_default_value',
+				esc_html__(
+					'You cant get a default value for a wp_User_Field because the "init" action is called, because current_user global hasnt yet been setup. Consider doing your business logic on the "init" hook or later.',
+					'event_espresso'
+				),
+				'4.6.20'
+			);
+			return 1;
+		}
+	}
 }

core/db_models/fields/EE_Money_Field.php

Indentation +78 added lines, -78 removed lines

@@ -7,92 +7,92 @@
 class EE_Money_Field extends EE_Float_Field
 {
 
-    /**
-     * @param string $table_column
-     * @param string $nicename
-     * @param bool   $nullable
-     * @param null   $default_value
-     */
-    public function __construct($table_column, $nicename, $nullable, $default_value = null)
-    {
-        parent::__construct($table_column, $nicename, $nullable, $default_value);
-        $this->setSchemaType('object');
-    }
+	/**
+	 * @param string $table_column
+	 * @param string $nicename
+	 * @param bool   $nullable
+	 * @param null   $default_value
+	 */
+	public function __construct($table_column, $nicename, $nullable, $default_value = null)
+	{
+		parent::__construct($table_column, $nicename, $nullable, $default_value);
+		$this->setSchemaType('object');
+	}
 
 
-    /**
-     * Schemas:
-     *    'localized_float': "3,023.00"
-     *    'no_currency_code': "$3,023.00"
-     *    null: "$3,023.00<span>USD</span>"
-     *
-     * @param string $value_on_field_to_be_outputted
-     * @param string $schema
-     * @return string
-     */
-    public function prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema = null)
-    {
-        $pretty_float = parent::prepare_for_pretty_echoing($value_on_field_to_be_outputted);
+	/**
+	 * Schemas:
+	 *    'localized_float': "3,023.00"
+	 *    'no_currency_code': "$3,023.00"
+	 *    null: "$3,023.00<span>USD</span>"
+	 *
+	 * @param string $value_on_field_to_be_outputted
+	 * @param string $schema
+	 * @return string
+	 */
+	public function prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema = null)
+	{
+		$pretty_float = parent::prepare_for_pretty_echoing($value_on_field_to_be_outputted);
 
-        if ($schema == 'localized_float') {
-            return $pretty_float;
-        }
-        if ($schema == 'no_currency_code') {
+		if ($schema == 'localized_float') {
+			return $pretty_float;
+		}
+		if ($schema == 'no_currency_code') {
 //          echo "schema no currency!";
-            $display_code = false;
-        } else {
-            $display_code = true;
-        }
-        // we don't use the $pretty_float because format_currency will take care of it.
-        return EEH_Template::format_currency($value_on_field_to_be_outputted, false, $display_code);
-    }
+			$display_code = false;
+		} else {
+			$display_code = true;
+		}
+		// we don't use the $pretty_float because format_currency will take care of it.
+		return EEH_Template::format_currency($value_on_field_to_be_outputted, false, $display_code);
+	}
 
-    /**
-     * If provided with a string, strips out money-related formatting to turn it into a proper float.
-     * Rounds the float to the correct number of decimal places for this country's currency.
-     * Also, interprets periods and commas according to the country's currency settings.
-     * So if you want to pass in a string that NEEDS to interpret periods as decimal marks, call floatval() on it first.
-     *
-     * @param string $value_inputted_for_field_on_model_object
-     * @return float
-     */
-    public function prepare_for_set($value_inputted_for_field_on_model_object)
-    {
-        // remove any currencies etc.
+	/**
+	 * If provided with a string, strips out money-related formatting to turn it into a proper float.
+	 * Rounds the float to the correct number of decimal places for this country's currency.
+	 * Also, interprets periods and commas according to the country's currency settings.
+	 * So if you want to pass in a string that NEEDS to interpret periods as decimal marks, call floatval() on it first.
+	 *
+	 * @param string $value_inputted_for_field_on_model_object
+	 * @return float
+	 */
+	public function prepare_for_set($value_inputted_for_field_on_model_object)
+	{
+		// remove any currencies etc.
 //      if(is_string($value_inputted_for_field_on_model_object)){
 //          $value_inputted_for_field_on_model_object = preg_replace("/[^0-9,.]/", "", $value_inputted_for_field_on_model_object);
 //      }
-        // now it's a float-style string or number
-        $float_val = parent::prepare_for_set($value_inputted_for_field_on_model_object);
-        // round to the correctly number of decimal places for this  currency
-        $rounded_value = round($float_val, EE_Registry::instance()->CFG->currency->dec_plc);
-        return $rounded_value;
-    }
+		// now it's a float-style string or number
+		$float_val = parent::prepare_for_set($value_inputted_for_field_on_model_object);
+		// round to the correctly number of decimal places for this  currency
+		$rounded_value = round($float_val, EE_Registry::instance()->CFG->currency->dec_plc);
+		return $rounded_value;
+	}
 
-    public function prepare_for_get($value_of_field_on_model_object)
-    {
-        $c = EE_Registry::instance()->CFG->currency;
-        return round(parent::prepare_for_get($value_of_field_on_model_object), $c->dec_plc);
-    }
+	public function prepare_for_get($value_of_field_on_model_object)
+	{
+		$c = EE_Registry::instance()->CFG->currency;
+		return round(parent::prepare_for_get($value_of_field_on_model_object), $c->dec_plc);
+	}
 
-    public function getSchemaProperties()
-    {
-        return array(
-            'raw' => array(
-                'description' =>  sprintf(
-                    esc_html__('%s - the raw value as it exists in the database as a simple float.', 'event_espresso'),
-                    $this->get_nicename()
-                ),
-                'type' => 'number',
-            ),
-            'pretty' => array(
-                'description' =>  sprintf(
-                    esc_html__('%s - formatted for display in the set currency and decimal places.', 'event_espresso'),
-                    $this->get_nicename()
-                ),
-                'type' => 'string',
-                'format' => 'money'
-            )
-        );
-    }
+	public function getSchemaProperties()
+	{
+		return array(
+			'raw' => array(
+				'description' =>  sprintf(
+					esc_html__('%s - the raw value as it exists in the database as a simple float.', 'event_espresso'),
+					$this->get_nicename()
+				),
+				'type' => 'number',
+			),
+			'pretty' => array(
+				'description' =>  sprintf(
+					esc_html__('%s - formatted for display in the set currency and decimal places.', 'event_espresso'),
+					$this->get_nicename()
+				),
+				'type' => 'string',
+				'format' => 'money'
+			)
+		);
+	}
 }

core/db_models/fields/EE_Primary_Key_Field_Base.php

Indentation +19 added lines, -19 removed lines

@@ -2,24 +2,24 @@
 
 abstract class EE_Primary_Key_Field_Base extends EE_Field_With_Model_Name
 {
-    /**
-     * Overrides parent so it doesn't need to provide so many non-applicable fields
-     *
-     * @param string $table_column
-     * @param string $nicename
-     */
-    public function __construct($table_column, $nicename, $default)
-    {
-        parent::__construct($table_column, $nicename, false, $default, '');
-    }
+	/**
+	 * Overrides parent so it doesn't need to provide so many non-applicable fields
+	 *
+	 * @param string $table_column
+	 * @param string $nicename
+	 */
+	public function __construct($table_column, $nicename, $default)
+	{
+		parent::__construct($table_column, $nicename, false, $default, '');
+	}
 
-    /**
-     * @param $table_alias
-     * @param $name
-     */
-    public function _construct_finalize($table_alias, $name, $model_name)
-    {
-        $this->_model_name_pointed_to = $model_name;
-        parent::_construct_finalize($table_alias, $name, $model_name);
-    }
+	/**
+	 * @param $table_alias
+	 * @param $name
+	 */
+	public function _construct_finalize($table_alias, $name, $model_name)
+	{
+		$this->_model_name_pointed_to = $model_name;
+		parent::_construct_finalize($table_alias, $name, $model_name);
+	}
 }

core/db_models/fields/EE_WP_Post_Status_Field.php

Indentation +97 added lines, -97 removed lines

@@ -3,114 +3,114 @@
 class EE_WP_Post_Status_Field extends EE_Enum_Text_Field
 {
 
-    protected $_wp_post_stati;
+	protected $_wp_post_stati;
 
 
-    /**
-     * constructor
-     *
-     * @param string  $table_column       column on table
-     * @param string  $nicename           nice name for column(field)
-     * @param bool    $nullable           is this field nullable
-     * @param string  $default_value      default status
-     * @param array   $new_stati          If additional stati are to be used other than the default WP statuses then
-     *                                    they can be registered via this property.  The format of the array should be
-     *                                    as follows: array(
-     *                                    'status_reference' => array(
-     *                                    'label' => esc_html__('Status Reference Label', 'event_espresso')
-     *                                    'public' => true, //'Whether posts of this status should be shown on the
-     *                                    frontend of the site'
-     *                                    'exclude_from_search' => false, //'Whether posts of this status should be
-     *                                    excluded from wp searches'
-     *                                    'show_in_admin_all_list' => true, //whether posts of this status are included
-     *                                    in queries for the admin "all" view in list table views.
-     *                                    'show_in_admin_status_list' => true, //Show in the list of statuses with post
-     *                                    counts at the top of the admin list tables (i.e. Status Reference(2) )
-     *                                    'label_count' => _n_noop( 'Status Reference <span class="count">(%s)</span>',
-     *                                    'Status References <span class="count">(%s)</span>' ), //the text to display
-     *                                    on the admin screen( or you won't see your status count ).
-     *                                    )
-     *                                    )
-     * @link http://codex.wordpress.org/Function_Reference/register_post_status for more info
-     * @param boolean $store_in_db_as_int By default, enums are stored as STRINGS in the DB. However, if this var is
-     *                                    set to true, it will be stored as an INT
-     */
-    public function __construct($table_column, $nicename, $nullable, $default_value, $new_stati = array())
-    {
-        $this->_register_new_stati($new_stati);
-        $this->_set_allowed_enum_values();
-        parent::__construct($table_column, $nicename, $nullable, $default_value, $this->_allowed_enum_values);
-    }
+	/**
+	 * constructor
+	 *
+	 * @param string  $table_column       column on table
+	 * @param string  $nicename           nice name for column(field)
+	 * @param bool    $nullable           is this field nullable
+	 * @param string  $default_value      default status
+	 * @param array   $new_stati          If additional stati are to be used other than the default WP statuses then
+	 *                                    they can be registered via this property.  The format of the array should be
+	 *                                    as follows: array(
+	 *                                    'status_reference' => array(
+	 *                                    'label' => esc_html__('Status Reference Label', 'event_espresso')
+	 *                                    'public' => true, //'Whether posts of this status should be shown on the
+	 *                                    frontend of the site'
+	 *                                    'exclude_from_search' => false, //'Whether posts of this status should be
+	 *                                    excluded from wp searches'
+	 *                                    'show_in_admin_all_list' => true, //whether posts of this status are included
+	 *                                    in queries for the admin "all" view in list table views.
+	 *                                    'show_in_admin_status_list' => true, //Show in the list of statuses with post
+	 *                                    counts at the top of the admin list tables (i.e. Status Reference(2) )
+	 *                                    'label_count' => _n_noop( 'Status Reference <span class="count">(%s)</span>',
+	 *                                    'Status References <span class="count">(%s)</span>' ), //the text to display
+	 *                                    on the admin screen( or you won't see your status count ).
+	 *                                    )
+	 *                                    )
+	 * @link http://codex.wordpress.org/Function_Reference/register_post_status for more info
+	 * @param boolean $store_in_db_as_int By default, enums are stored as STRINGS in the DB. However, if this var is
+	 *                                    set to true, it will be stored as an INT
+	 */
+	public function __construct($table_column, $nicename, $nullable, $default_value, $new_stati = array())
+	{
+		$this->_register_new_stati($new_stati);
+		$this->_set_allowed_enum_values();
+		parent::__construct($table_column, $nicename, $nullable, $default_value, $this->_allowed_enum_values);
+	}
 
 
-    /**
-     * This registers any new statuses sent via the $new_stati array on construct
-     *
-     * @access protected
-     * @param  array $new_stati statuses
-     * @return void
-     */
-    protected function _register_new_stati($new_stati)
-    {
+	/**
+	 * This registers any new statuses sent via the $new_stati array on construct
+	 *
+	 * @access protected
+	 * @param  array $new_stati statuses
+	 * @return void
+	 */
+	protected function _register_new_stati($new_stati)
+	{
 
-        foreach ((array) $new_stati as $status_key => $status_args) {
-            $args = array(
-                'label'                     => isset($status_args['label']) ? $status_args['label'] : $status_key,
-                'public'                    => isset($status_args['public']) && is_bool($status_args['public']) ? $status_args['public'] : true,
-                'exclude_from_search'       => isset($status_args['exclude_from_search']) && is_bool($status_args['exclude_from_search']) ? $status_args['exclude_from_search'] : false,
-                'show_in_admin_all_list'    => isset($status_args['show_in_admin_all_list']) && is_bool($status_args['show_in_admin_all_list']) ? $status_args['show_in_admin_all_list'] : false,
-                'show_in_admin_status_list' => isset($status_args['show_in_admin_status_list']) && is_bool($status_args['show_in_admin_status_list']) ? $status_args['show_in_admin_status_list'] : true,
-                'label_count'               => isset($status_args['label_count']) ? $status_args['label_count'] : '',
-            );
-            register_post_status($status_key, $status_args);
-        }
-    }
+		foreach ((array) $new_stati as $status_key => $status_args) {
+			$args = array(
+				'label'                     => isset($status_args['label']) ? $status_args['label'] : $status_key,
+				'public'                    => isset($status_args['public']) && is_bool($status_args['public']) ? $status_args['public'] : true,
+				'exclude_from_search'       => isset($status_args['exclude_from_search']) && is_bool($status_args['exclude_from_search']) ? $status_args['exclude_from_search'] : false,
+				'show_in_admin_all_list'    => isset($status_args['show_in_admin_all_list']) && is_bool($status_args['show_in_admin_all_list']) ? $status_args['show_in_admin_all_list'] : false,
+				'show_in_admin_status_list' => isset($status_args['show_in_admin_status_list']) && is_bool($status_args['show_in_admin_status_list']) ? $status_args['show_in_admin_status_list'] : true,
+				'label_count'               => isset($status_args['label_count']) ? $status_args['label_count'] : '',
+			);
+			register_post_status($status_key, $status_args);
+		}
+	}
 
 
-    /**
-     * This sets the _allowed_enum_values property using the $wp_post_stati array
-     *
-     * @access protected
-     * @regurn void
-     */
-    protected function _set_allowed_enum_values()
-    {
-        // first let's get the post_statuses
-        global $wp_post_statuses;
-        $this->_wp_post_stati = $wp_post_statuses;
+	/**
+	 * This sets the _allowed_enum_values property using the $wp_post_stati array
+	 *
+	 * @access protected
+	 * @regurn void
+	 */
+	protected function _set_allowed_enum_values()
+	{
+		// first let's get the post_statuses
+		global $wp_post_statuses;
+		$this->_wp_post_stati = $wp_post_statuses;
 
-        foreach ($this->_wp_post_stati as $post_status => $args_object) {
-            $this->_allowed_enum_values[ $post_status ] = $args_object->label;
-        }
-    }
+		foreach ($this->_wp_post_stati as $post_status => $args_object) {
+			$this->_allowed_enum_values[ $post_status ] = $args_object->label;
+		}
+	}
 
-    /**
-     * Before calling parent, first double-checks our list of acceptable post
-     * types is up-to-date
-     *
-     * @param string $value_inputted_for_field_on_model_object
-     * @return string
-     */
-    public function prepare_for_set($value_inputted_for_field_on_model_object)
-    {
-        $this->_set_allowed_enum_values();
-        return parent::prepare_for_set($value_inputted_for_field_on_model_object);
-    }
+	/**
+	 * Before calling parent, first double-checks our list of acceptable post
+	 * types is up-to-date
+	 *
+	 * @param string $value_inputted_for_field_on_model_object
+	 * @return string
+	 */
+	public function prepare_for_set($value_inputted_for_field_on_model_object)
+	{
+		$this->_set_allowed_enum_values();
+		return parent::prepare_for_set($value_inputted_for_field_on_model_object);
+	}
 
 
 
-    // helper methods for getting various $wp_post_statuses stuff.
+	// helper methods for getting various $wp_post_statuses stuff.
 
-    /**
-     * This just returns the status object for the given status
-     *
-     * @access public
-     * @see    wp_register_post_status in wp-includes/post.php for a list of properties of the status object
-     * @param  string $status What status object you want
-     * @return std_object         the status object or FALSE if it doesn't exist.
-     */
-    public function get_status_object($status)
-    {
-        return isset($this->_wp_post_stati[ $status ]) ? $this->_wp_post_stati[ $status ] : false;
-    }
+	/**
+	 * This just returns the status object for the given status
+	 *
+	 * @access public
+	 * @see    wp_register_post_status in wp-includes/post.php for a list of properties of the status object
+	 * @param  string $status What status object you want
+	 * @return std_object         the status object or FALSE if it doesn't exist.
+	 */
+	public function get_status_object($status)
+	{
+		return isset($this->_wp_post_stati[ $status ]) ? $this->_wp_post_stati[ $status ] : false;
+	}
 }

core/db_models/EEM_Price.model.php

Indentation +351 added lines, -351 removed lines

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

Spacing +10 added lines, -10 removed lines

@@ -31,7 +31,7 @@ 
      */
     protected function __construct(string $timezone = '')
     {
-        require_once(EE_MODELS . 'EEM_Price_Type.model.php');
+        require_once(EE_MODELS.'EEM_Price_Type.model.php');
         $this->singular_item = esc_html__('Price', 'event_espresso');
         $this->plural_item   = esc_html__('Prices', 'event_espresso');
 
@@ -118,23 +118,23 @@ 
             'WP_User'    => new EE_Belongs_To_Relation(),
         ];
         // this model is generally available for reading
-        $this->_cap_restriction_generators[ EEM_Base::caps_read ]
+        $this->_cap_restriction_generators[EEM_Base::caps_read]
             = new EE_Restriction_Generator_Default_Public(
             'PRC_is_default',
             'Ticket.Datetime.Event'
         );
         // account for default tickets in the caps
-        $this->_cap_restriction_generators[ EEM_Base::caps_read_admin ]
+        $this->_cap_restriction_generators[EEM_Base::caps_read_admin]
             = new EE_Restriction_Generator_Default_Protected(
             'PRC_is_default',
             'Ticket.Datetime.Event'
         );
-        $this->_cap_restriction_generators[ EEM_Base::caps_edit ]
+        $this->_cap_restriction_generators[EEM_Base::caps_edit]
             = new EE_Restriction_Generator_Default_Protected(
             'PRC_is_default',
             'Ticket.Datetime.Event'
         );
-        $this->_cap_restriction_generators[ EEM_Base::caps_delete ]
+        $this->_cap_restriction_generators[EEM_Base::caps_delete]
             = new EE_Restriction_Generator_Default_Protected(
             'PRC_is_default',
             'Ticket.Datetime.Event'
@@ -207,7 +207,7 @@ 
             'PRC_deleted'    => 0,
             'PRC_is_default' => 1,
         ];
-        if (! $include_taxes) {
+        if ( ! $include_taxes) {
             $_where['Price_Type.PBT_ID'] = ['!=', 4];
         }
         $_query_params = [
@@ -265,7 +265,7 @@ 
         );
         foreach ($all_taxes as $tax) {
             if ($tax instanceof EE_Price) {
-                $taxes[ $tax->order() ][ $tax->ID() ] = $tax;
+                $taxes[$tax->order()][$tax->ID()] = $tax;
             }
         }
         return $taxes;
@@ -291,7 +291,7 @@ 
             if ($default_prices) {
                 foreach ($default_prices as $price) {
                     if ($price instanceof EE_Price) {
-                        $array_of_price_objects[ $price->type() ][] = $price;
+                        $array_of_price_objects[$price->type()][] = $price;
                     }
                 }
                 return $array_of_price_objects;
@@ -308,10 +308,10 @@ 
             ]
         );
 
-        if (! empty($ticket_prices)) {
+        if ( ! empty($ticket_prices)) {
             foreach ($ticket_prices as $price) {
                 if ($price instanceof EE_Price) {
-                    $array_of_price_objects[ $price->type() ][] = $price;
+                    $array_of_price_objects[$price->type()][] = $price;
                 }
             }
         }

core/db_models/EEM_Change_Log.model.php

Indentation +233 added lines, -233 removed lines

@@ -10,99 +10,99 @@ 
 class EEM_Change_Log extends EEM_Base
 {
 
-    /**
-     * the related object was created log type
-     */
-    const type_create = 'create';
+	/**
+	 * the related object was created log type
+	 */
+	const type_create = 'create';
 
-    /**
-     * the related object was updated (changed, or soft-deleted)
-     */
-    const type_update = 'update';
+	/**
+	 * the related object was updated (changed, or soft-deleted)
+	 */
+	const type_update = 'update';
 
-    /**
-     * the related object was deleted permanently
-     */
-    const type_delete = 'delete';
+	/**
+	 * the related object was deleted permanently
+	 */
+	const type_delete = 'delete';
 
-    /**
-     * the related item had something worth noting happen on it, but
-     * only for the purposes of debugging problems
-     */
-    const type_debug = 'debug';
+	/**
+	 * the related item had something worth noting happen on it, but
+	 * only for the purposes of debugging problems
+	 */
+	const type_debug = 'debug';
 
-    /**
-     * the related item had an error occur on it
-     */
-    const type_error = 'error';
+	/**
+	 * the related item had an error occur on it
+	 */
+	const type_error = 'error';
 
-    /**
-     * the related item is regarding some gateway interaction, like an IPN
-     * or request to process a payment
-     */
-    const type_gateway = 'gateway';
+	/**
+	 * the related item is regarding some gateway interaction, like an IPN
+	 * or request to process a payment
+	 */
+	const type_gateway = 'gateway';
 
-    /**
-     * private instance of the EEM_Change_Log object
-     *
-     * @access private
-     * @var EEM_Change_Log $_instance
-     */
-    protected static $_instance;
+	/**
+	 * private instance of the EEM_Change_Log object
+	 *
+	 * @access private
+	 * @var EEM_Change_Log $_instance
+	 */
+	protected static $_instance;
 
 
-    /**
-     * constructor
-     *
-     * @param string|null $timezone
-     * @throws EE_Error
-     */
-    protected function __construct(string $timezone = '')
-    {
-        $this->singular_item       = esc_html__('Log', 'event_espresso');
-        $this->plural_item         = esc_html__('Logs', 'event_espresso');
-        $this->_tables             = [
-            'Log' => new EE_Primary_Table('esp_log', 'LOG_ID'),
-        ];
-        $models_this_can_attach_to = array_keys(EE_Registry::instance()->non_abstract_db_models);
-        $this->_fields             = [
-            'Log' => [
-                'LOG_ID'      => new EE_Primary_Key_Int_Field(
-                    'LOG_ID',
-                    esc_html__('Log ID', 'event_espresso')
-                ),
-                'LOG_time'    => new EE_Datetime_Field(
-                    'LOG_time',
-                    esc_html__("Log Time", 'event_espresso'),
-                    false,
-                    EE_Datetime_Field::now
-                ),
-                'OBJ_ID'      => new EE_Foreign_Key_String_Field(
-                    'OBJ_ID',
-                    esc_html__("Object ID (int or string)", 'event_espresso'),
-                    true,
-                    null,
-                    $models_this_can_attach_to
-                ),
-                'OBJ_type'    => new EE_Any_Foreign_Model_Name_Field(
-                    'OBJ_type',
-                    esc_html__("Object Type", 'event_espresso'),
-                    true,
-                    null,
-                    $models_this_can_attach_to
-                ),
-                'LOG_type'    => new EE_Plain_Text_Field(
-                    'LOG_type',
-                    esc_html__("Type of log entry", "event_espresso"),
-                    false,
-                    self::type_debug
-                ),
-                'LOG_message' => new EE_Maybe_Serialized_Text_Field(
-                    'LOG_message',
-                    esc_html__("Log Message (body)", 'event_espresso'),
-                    true
-                ),
-                /*
+	/**
+	 * constructor
+	 *
+	 * @param string|null $timezone
+	 * @throws EE_Error
+	 */
+	protected function __construct(string $timezone = '')
+	{
+		$this->singular_item       = esc_html__('Log', 'event_espresso');
+		$this->plural_item         = esc_html__('Logs', 'event_espresso');
+		$this->_tables             = [
+			'Log' => new EE_Primary_Table('esp_log', 'LOG_ID'),
+		];
+		$models_this_can_attach_to = array_keys(EE_Registry::instance()->non_abstract_db_models);
+		$this->_fields             = [
+			'Log' => [
+				'LOG_ID'      => new EE_Primary_Key_Int_Field(
+					'LOG_ID',
+					esc_html__('Log ID', 'event_espresso')
+				),
+				'LOG_time'    => new EE_Datetime_Field(
+					'LOG_time',
+					esc_html__("Log Time", 'event_espresso'),
+					false,
+					EE_Datetime_Field::now
+				),
+				'OBJ_ID'      => new EE_Foreign_Key_String_Field(
+					'OBJ_ID',
+					esc_html__("Object ID (int or string)", 'event_espresso'),
+					true,
+					null,
+					$models_this_can_attach_to
+				),
+				'OBJ_type'    => new EE_Any_Foreign_Model_Name_Field(
+					'OBJ_type',
+					esc_html__("Object Type", 'event_espresso'),
+					true,
+					null,
+					$models_this_can_attach_to
+				),
+				'LOG_type'    => new EE_Plain_Text_Field(
+					'LOG_type',
+					esc_html__("Type of log entry", "event_espresso"),
+					false,
+					self::type_debug
+				),
+				'LOG_message' => new EE_Maybe_Serialized_Text_Field(
+					'LOG_message',
+					esc_html__("Log Message (body)", 'event_espresso'),
+					true
+				),
+				/*
                  * Note: when querying for a change log's user, the OBJ_ID and OBJ_type fields are used,
                  * not the LOG_wp_user field. E.g.,
                  * `EEM_Change_Log::instance()->get_all(array(array('WP_User.ID'=>1)))` will actually return
@@ -111,164 +111,164 @@ 
                  *  If you want the latter, you can't use the model's magic joining. E.g, you would need to do
                  * `EEM_Change_Log::instance()->get_all(array(array('LOG_wp_user' => 1)))`.
                  */
-                'LOG_wp_user' => new EE_WP_User_Field(
-                    'LOG_wp_user',
-                    esc_html__("User who was logged in while this occurred", 'event_espresso'),
-                    true
-                ),
-            ],
-        ];
-        $this->_model_relations    = [];
-        foreach ($models_this_can_attach_to as $model) {
-            if ($model != 'Change_Log') {
-                $this->_model_relations[ $model ] = new EE_Belongs_To_Any_Relation();
-            }
-        }
-        // use completely custom caps for this
-        $this->_cap_restriction_generators = false;
-        // caps-wise this is all-or-nothing: if you have the default role you can access anything, otherwise nothing
-        foreach ($this->_cap_contexts_to_cap_action_map as $cap_context => $action) {
-            $this->_cap_restrictions[ $cap_context ][ EE_Restriction_Generator_Base::get_default_restrictions_cap() ]
-                = new EE_Return_None_Where_Conditions();
-        }
-        parent::__construct($timezone);
-    }
+				'LOG_wp_user' => new EE_WP_User_Field(
+					'LOG_wp_user',
+					esc_html__("User who was logged in while this occurred", 'event_espresso'),
+					true
+				),
+			],
+		];
+		$this->_model_relations    = [];
+		foreach ($models_this_can_attach_to as $model) {
+			if ($model != 'Change_Log') {
+				$this->_model_relations[ $model ] = new EE_Belongs_To_Any_Relation();
+			}
+		}
+		// use completely custom caps for this
+		$this->_cap_restriction_generators = false;
+		// caps-wise this is all-or-nothing: if you have the default role you can access anything, otherwise nothing
+		foreach ($this->_cap_contexts_to_cap_action_map as $cap_context => $action) {
+			$this->_cap_restrictions[ $cap_context ][ EE_Restriction_Generator_Base::get_default_restrictions_cap() ]
+				= new EE_Return_None_Where_Conditions();
+		}
+		parent::__construct($timezone);
+	}
 
 
-    /**
-     * @param string        $log_type !see the acceptable values of LOG_type in EEM__Change_Log::__construct
-     * @param mixed         $message  array|string of the message you want to record
-     * @param EE_Base_Class $related_model_obj
-     * @return EE_Change_Log
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function log(string $log_type, $message, EE_Base_Class $related_model_obj): EE_Change_Log
-    {
-        if ($related_model_obj instanceof EE_Base_Class) {
-            $obj_id   = $related_model_obj->ID();
-            $obj_type = $related_model_obj->get_model()->get_this_model_name();
-        } else {
-            $obj_id   = null;
-            $obj_type = null;
-        }
-        $log = EE_Change_Log::new_instance(
-            [
-                'LOG_type'    => $log_type,
-                'LOG_message' => $message,
-                'OBJ_ID'      => $obj_id,
-                'OBJ_type'    => $obj_type,
-            ]
-        );
-        $log->save();
-        return $log;
-    }
+	/**
+	 * @param string        $log_type !see the acceptable values of LOG_type in EEM__Change_Log::__construct
+	 * @param mixed         $message  array|string of the message you want to record
+	 * @param EE_Base_Class $related_model_obj
+	 * @return EE_Change_Log
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function log(string $log_type, $message, EE_Base_Class $related_model_obj): EE_Change_Log
+	{
+		if ($related_model_obj instanceof EE_Base_Class) {
+			$obj_id   = $related_model_obj->ID();
+			$obj_type = $related_model_obj->get_model()->get_this_model_name();
+		} else {
+			$obj_id   = null;
+			$obj_type = null;
+		}
+		$log = EE_Change_Log::new_instance(
+			[
+				'LOG_type'    => $log_type,
+				'LOG_message' => $message,
+				'OBJ_ID'      => $obj_id,
+				'OBJ_type'    => $obj_type,
+			]
+		);
+		$log->save();
+		return $log;
+	}
 
 
-    /**
-     * Adds a gateway log for the specified object, given its ID and type
-     *
-     * @param string $message
-     * @param mixed  $related_obj_id
-     * @param string $related_obj_type
-     * @return EE_Change_Log
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    public function gateway_log(string $message, $related_obj_id, string $related_obj_type): EE_Change_Log
-    {
-        if (! EE_Registry::instance()->is_model_name($related_obj_type)) {
-            throw new EE_Error(
-                sprintf(
-                    esc_html__(
-                        "'%s' is not a model name. A model name must be provided when making a gateway log. Eg, 'Payment', 'Payment_Method', etc",
-                        "event_espresso"
-                    ),
-                    $related_obj_type
-                )
-            );
-        }
-        $log = EE_Change_Log::new_instance(
-            [
-                'LOG_type'    => EEM_Change_Log::type_gateway,
-                'LOG_message' => $message,
-                'OBJ_ID'      => $related_obj_id,
-                'OBJ_type'    => $related_obj_type,
-            ]
-        );
-        $log->save();
-        return $log;
-    }
+	/**
+	 * Adds a gateway log for the specified object, given its ID and type
+	 *
+	 * @param string $message
+	 * @param mixed  $related_obj_id
+	 * @param string $related_obj_type
+	 * @return EE_Change_Log
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	public function gateway_log(string $message, $related_obj_id, string $related_obj_type): EE_Change_Log
+	{
+		if (! EE_Registry::instance()->is_model_name($related_obj_type)) {
+			throw new EE_Error(
+				sprintf(
+					esc_html__(
+						"'%s' is not a model name. A model name must be provided when making a gateway log. Eg, 'Payment', 'Payment_Method', etc",
+						"event_espresso"
+					),
+					$related_obj_type
+				)
+			);
+		}
+		$log = EE_Change_Log::new_instance(
+			[
+				'LOG_type'    => EEM_Change_Log::type_gateway,
+				'LOG_message' => $message,
+				'OBJ_ID'      => $related_obj_id,
+				'OBJ_type'    => $related_obj_type,
+			]
+		);
+		$log->save();
+		return $log;
+	}
 
 
-    /**
-     * Just gets the bare-bones wpdb results as an array in cases where efficiency is essential
-     *
-     * @param array $query_params
-     * @return array of arrays
-     * @throws EE_Error
-     * @throws ReflectionException
-     * @see https://github.com/eventespresso/event-espresso-core/tree/master/docs/G--Model-System/model-query-params.md
-     */
-    public function get_all_efficiently(array $query_params): array
-    {
-        return $this->_get_all_wpdb_results($query_params);
-    }
+	/**
+	 * Just gets the bare-bones wpdb results as an array in cases where efficiency is essential
+	 *
+	 * @param array $query_params
+	 * @return array of arrays
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 * @see https://github.com/eventespresso/event-espresso-core/tree/master/docs/G--Model-System/model-query-params.md
+	 */
+	public function get_all_efficiently(array $query_params): array
+	{
+		return $this->_get_all_wpdb_results($query_params);
+	}
 
 
-    /**
-     * Executes a database query to delete gateway logs. Does not affect model objects, so if you attempt to use
-     * models after this, they may be out-of-sync with the database
-     *
-     * @param DateTime $datetime
-     * @return false|int
-     * @throws EE_Error
-     */
-    public function delete_gateway_logs_older_than(DateTime $datetime)
-    {
-        global $wpdb;
-        return $wpdb->query(
-            $wpdb->prepare(
-                'DELETE FROM ' . $this->table() . ' WHERE LOG_type = %s AND LOG_time < %s',
-                EEM_Change_Log::type_gateway,
-                $datetime->format(EE_Datetime_Field::mysql_timestamp_format)
-            )
-        );
-    }
+	/**
+	 * Executes a database query to delete gateway logs. Does not affect model objects, so if you attempt to use
+	 * models after this, they may be out-of-sync with the database
+	 *
+	 * @param DateTime $datetime
+	 * @return false|int
+	 * @throws EE_Error
+	 */
+	public function delete_gateway_logs_older_than(DateTime $datetime)
+	{
+		global $wpdb;
+		return $wpdb->query(
+			$wpdb->prepare(
+				'DELETE FROM ' . $this->table() . ' WHERE LOG_type = %s AND LOG_time < %s',
+				EEM_Change_Log::type_gateway,
+				$datetime->format(EE_Datetime_Field::mysql_timestamp_format)
+			)
+		);
+	}
 
 
-    /**
-     * Returns the map of type to pretty label for identifiers used for `LOG_type`.  Client code can register their own
-     * map vai the given filter.
-     *
-     * @return array
-     */
-    public static function get_pretty_label_map_for_registered_types(): array
-    {
-        return apply_filters(
-            'FHEE__EEM_Change_Log__get_pretty_label_map_for_registered_types',
-            [
-                self::type_create  => esc_html__("Create", "event_espresso"),
-                self::type_update  => esc_html__("Update", "event_espresso"),
-                self::type_delete  => esc_html__("Delete", "event_espresso"),
-                self::type_debug   => esc_html__("Debug", "event_espresso"),
-                self::type_error   => esc_html__("Error", "event_espresso"),
-                self::type_gateway => esc_html__("Gateway Interaction (IPN or Direct Payment)", 'event_espresso'),
-            ]
-        );
-    }
+	/**
+	 * Returns the map of type to pretty label for identifiers used for `LOG_type`.  Client code can register their own
+	 * map vai the given filter.
+	 *
+	 * @return array
+	 */
+	public static function get_pretty_label_map_for_registered_types(): array
+	{
+		return apply_filters(
+			'FHEE__EEM_Change_Log__get_pretty_label_map_for_registered_types',
+			[
+				self::type_create  => esc_html__("Create", "event_espresso"),
+				self::type_update  => esc_html__("Update", "event_espresso"),
+				self::type_delete  => esc_html__("Delete", "event_espresso"),
+				self::type_debug   => esc_html__("Debug", "event_espresso"),
+				self::type_error   => esc_html__("Error", "event_espresso"),
+				self::type_gateway => esc_html__("Gateway Interaction (IPN or Direct Payment)", 'event_espresso'),
+			]
+		);
+	}
 
 
-    /**
-     * Return the pretty (localized) label for the given log type identifier.
-     *
-     * @param string $type_identifier
-     * @return string
-     */
-    public static function get_pretty_label_for_type(string $type_identifier): string
-    {
-        $type_identifier_map = self::get_pretty_label_map_for_registered_types();
-        // we fallback to the incoming type identifier if there is no localized label for it.
-        return $type_identifier_map[ $type_identifier ] ?? $type_identifier;
-    }
+	/**
+	 * Return the pretty (localized) label for the given log type identifier.
+	 *
+	 * @param string $type_identifier
+	 * @return string
+	 */
+	public static function get_pretty_label_for_type(string $type_identifier): string
+	{
+		$type_identifier_map = self::get_pretty_label_map_for_registered_types();
+		// we fallback to the incoming type identifier if there is no localized label for it.
+		return $type_identifier_map[ $type_identifier ] ?? $type_identifier;
+	}
 }

Spacing +6 added lines, -6 removed lines

@@ -118,17 +118,17 @@ 
                 ),
             ],
         ];
-        $this->_model_relations    = [];
+        $this->_model_relations = [];
         foreach ($models_this_can_attach_to as $model) {
             if ($model != 'Change_Log') {
-                $this->_model_relations[ $model ] = new EE_Belongs_To_Any_Relation();
+                $this->_model_relations[$model] = new EE_Belongs_To_Any_Relation();
             }
         }
         // use completely custom caps for this
         $this->_cap_restriction_generators = false;
         // caps-wise this is all-or-nothing: if you have the default role you can access anything, otherwise nothing
         foreach ($this->_cap_contexts_to_cap_action_map as $cap_context => $action) {
-            $this->_cap_restrictions[ $cap_context ][ EE_Restriction_Generator_Base::get_default_restrictions_cap() ]
+            $this->_cap_restrictions[$cap_context][EE_Restriction_Generator_Base::get_default_restrictions_cap()]
                 = new EE_Return_None_Where_Conditions();
         }
         parent::__construct($timezone);
@@ -177,7 +177,7 @@ 
      */
     public function gateway_log(string $message, $related_obj_id, string $related_obj_type): EE_Change_Log
     {
-        if (! EE_Registry::instance()->is_model_name($related_obj_type)) {
+        if ( ! EE_Registry::instance()->is_model_name($related_obj_type)) {
             throw new EE_Error(
                 sprintf(
                     esc_html__(
@@ -229,7 +229,7 @@ 
         global $wpdb;
         return $wpdb->query(
             $wpdb->prepare(
-                'DELETE FROM ' . $this->table() . ' WHERE LOG_type = %s AND LOG_time < %s',
+                'DELETE FROM '.$this->table().' WHERE LOG_type = %s AND LOG_time < %s',
                 EEM_Change_Log::type_gateway,
                 $datetime->format(EE_Datetime_Field::mysql_timestamp_format)
             )
@@ -269,6 +269,6 @@ 
     {
         $type_identifier_map = self::get_pretty_label_map_for_registered_types();
         // we fallback to the incoming type identifier if there is no localized label for it.
-        return $type_identifier_map[ $type_identifier ] ?? $type_identifier;
+        return $type_identifier_map[$type_identifier] ?? $type_identifier;
     }
 }