eventespresso / event-espresso-core

core/db_models/fields/EE_Post_Content_Field.php

Indentation +114 added lines, -114 removed lines

@@ -7,129 +7,129 @@
 class EE_Post_Content_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');
+	}
 
 
-    /**
-     * removes all tags which a WP Post wouldn't allow in its content normally
-     *
-     * @param string $value
-     * @return string
-     */
-    public function prepare_for_set($value)
-    {
-        if (! current_user_can('unfiltered_html')) {
-            $value = wp_kses("$value", wp_kses_allowed_html('post'));
-        }
-        return parent::prepare_for_set($value);
-    }
+	/**
+	 * removes all tags which a WP Post wouldn't allow in its content normally
+	 *
+	 * @param string $value
+	 * @return string
+	 */
+	public function prepare_for_set($value)
+	{
+		if (! current_user_can('unfiltered_html')) {
+			$value = wp_kses("$value", wp_kses_allowed_html('post'));
+		}
+		return parent::prepare_for_set($value);
+	}
 
 
 
-    /**
-     * Runs the content through `the_content`, or if prepares the content for placing in a form input
-     * @param string $value_on_field_to_be_outputted
-     * @param string   $schema possible values: 'form_input' or null (if null, will run through 'the_content')
-     * @return string
-     * @throws EE_Error when WP_DEBUG is on and recursive calling is detected
-     */
-    public function prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema = null)
-    {
-        switch ($schema) {
-            case 'form_input':
-                return parent::prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema);
-            case 'the_content':
-                if (doing_filter('the_content')) {
-                    if (defined('WP_DEBUG') && WP_DEBUG) {
-                        throw new EE_Error(
-                            sprintf(
-                                esc_html__('You have recursively called "%1$s" with %2$s set to %3$s which uses "%2$s" filter. You should use it with %2$s "%3$s" instead here.', 'event_espresso'),
-                                'EE_Post_Content_Field::prepare_for_pretty_echoing',
-                                '$schema',
-                                'the_content',
-                                'the_content_wp_core_only'
-                            )
-                        );
-                    } else {
-                        return $this->prepare_for_pretty_echoing($value_on_field_to_be_outputted, 'the_content_wp_core_only');
-                    }
-                }
-                return apply_filters(
-                    'the_content',
-                    parent::prepare_for_pretty_echoing(
-                        $value_on_field_to_be_outputted,
-                        $schema
-                    )
-                );
-            case 'the_content_wp_core_only':
-            default:
-                self::_setup_the_content_wp_core_only_filters();
-                $return_value = apply_filters(
-                    'the_content_wp_core_only',
-                    parent::prepare_for_pretty_echoing(
-                        $value_on_field_to_be_outputted,
-                        $schema
-                    )
-                );
-                // ya know what? adding these filters is super fast. Let's just
-                // avoid needing to maintain global state and set this up as-needed
-                remove_all_filters('the_content_wp_core_only');
-                do_action('AHEE__EE_Post_Content_Field__prepare_for_pretty_echoing__the_content_wp_core_only__done');
-                return $return_value;
-        }
-    }
+	/**
+	 * Runs the content through `the_content`, or if prepares the content for placing in a form input
+	 * @param string $value_on_field_to_be_outputted
+	 * @param string   $schema possible values: 'form_input' or null (if null, will run through 'the_content')
+	 * @return string
+	 * @throws EE_Error when WP_DEBUG is on and recursive calling is detected
+	 */
+	public function prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema = null)
+	{
+		switch ($schema) {
+			case 'form_input':
+				return parent::prepare_for_pretty_echoing($value_on_field_to_be_outputted, $schema);
+			case 'the_content':
+				if (doing_filter('the_content')) {
+					if (defined('WP_DEBUG') && WP_DEBUG) {
+						throw new EE_Error(
+							sprintf(
+								esc_html__('You have recursively called "%1$s" with %2$s set to %3$s which uses "%2$s" filter. You should use it with %2$s "%3$s" instead here.', 'event_espresso'),
+								'EE_Post_Content_Field::prepare_for_pretty_echoing',
+								'$schema',
+								'the_content',
+								'the_content_wp_core_only'
+							)
+						);
+					} else {
+						return $this->prepare_for_pretty_echoing($value_on_field_to_be_outputted, 'the_content_wp_core_only');
+					}
+				}
+				return apply_filters(
+					'the_content',
+					parent::prepare_for_pretty_echoing(
+						$value_on_field_to_be_outputted,
+						$schema
+					)
+				);
+			case 'the_content_wp_core_only':
+			default:
+				self::_setup_the_content_wp_core_only_filters();
+				$return_value = apply_filters(
+					'the_content_wp_core_only',
+					parent::prepare_for_pretty_echoing(
+						$value_on_field_to_be_outputted,
+						$schema
+					)
+				);
+				// ya know what? adding these filters is super fast. Let's just
+				// avoid needing to maintain global state and set this up as-needed
+				remove_all_filters('the_content_wp_core_only');
+				do_action('AHEE__EE_Post_Content_Field__prepare_for_pretty_echoing__the_content_wp_core_only__done');
+				return $return_value;
+		}
+	}
 
 
 
-    /**
-     * Verifies we've setup the standard WP core filters on  'the_content_wp_core_only' filter
-     */
-    protected static function _setup_the_content_wp_core_only_filters()
-    {
-        add_filter('the_content_wp_core_only', array( $GLOBALS['wp_embed'], 'run_shortcode'), 8);
-        add_filter('the_content_wp_core_only', array( $GLOBALS['wp_embed'], 'autoembed'), 8);
-        add_filter('the_content_wp_core_only', 'wptexturize', 10);
-        add_filter('the_content_wp_core_only', 'wpautop', 10);
-        add_filter('the_content_wp_core_only', 'shortcode_unautop', 10);
-        add_filter('the_content_wp_core_only', 'prepend_attachment', 10);
-        if (function_exists('wp_filter_content_tags')) {
-            add_filter('the_content_wp_core_only', 'wp_filter_content_tags', 10);
-        } elseif (function_exists('wp_make_content_images_responsive')) {
-            add_filter('the_content_wp_core_only', 'wp_make_content_images_responsive', 10);
-        }
-        add_filter('the_content_wp_core_only', 'do_shortcode', 11);
-        add_filter('the_content_wp_core_only', 'convert_smilies', 20);
-    }
+	/**
+	 * Verifies we've setup the standard WP core filters on  'the_content_wp_core_only' filter
+	 */
+	protected static function _setup_the_content_wp_core_only_filters()
+	{
+		add_filter('the_content_wp_core_only', array( $GLOBALS['wp_embed'], 'run_shortcode'), 8);
+		add_filter('the_content_wp_core_only', array( $GLOBALS['wp_embed'], 'autoembed'), 8);
+		add_filter('the_content_wp_core_only', 'wptexturize', 10);
+		add_filter('the_content_wp_core_only', 'wpautop', 10);
+		add_filter('the_content_wp_core_only', 'shortcode_unautop', 10);
+		add_filter('the_content_wp_core_only', 'prepend_attachment', 10);
+		if (function_exists('wp_filter_content_tags')) {
+			add_filter('the_content_wp_core_only', 'wp_filter_content_tags', 10);
+		} elseif (function_exists('wp_make_content_images_responsive')) {
+			add_filter('the_content_wp_core_only', 'wp_make_content_images_responsive', 10);
+		}
+		add_filter('the_content_wp_core_only', 'do_shortcode', 11);
+		add_filter('the_content_wp_core_only', 'convert_smilies', 20);
+	}
 
 
 
-    public function getSchemaProperties()
-    {
-        return array(
-            'raw' => array(
-                'description' =>  sprintf(
-                    esc_html__('%s - the content as it exists in the database.', 'event_espresso'),
-                    $this->get_nicename()
-                ),
-                'type' => 'string'
-            ),
-            'rendered' => array(
-                'description' =>  sprintf(
-                    esc_html__('%s - the content rendered for display.', 'event_espresso'),
-                    $this->get_nicename()
-                ),
-                'type' => 'string'
-            )
-        );
-    }
+	public function getSchemaProperties()
+	{
+		return array(
+			'raw' => array(
+				'description' =>  sprintf(
+					esc_html__('%s - the content as it exists in the database.', 'event_espresso'),
+					$this->get_nicename()
+				),
+				'type' => 'string'
+			),
+			'rendered' => array(
+				'description' =>  sprintf(
+					esc_html__('%s - the content rendered for display.', 'event_espresso'),
+					$this->get_nicename()
+				),
+				'type' => 'string'
+			)
+		);
+	}
 }

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;
                 }
             }
         }