eventespresso / event-espresso-core

core/domain/services/admin/entities/DefaultTickets.php

Spacing +3 added lines, -3 removed lines

@@ -54,14 +54,14 @@ 
      */
     public function create(EE_Base_Class $entity): array
     {
-        if (! $entity instanceof EE_Datetime) {
+        if ( ! $entity instanceof EE_Datetime) {
             throw new InvalidEntityException($entity, 'EE_Datetime');
         }
         $new_tickets     = [];
         $default_tickets = $this->ticket_model->get_all_default_tickets();
         if (is_array($default_tickets)) {
             foreach ($default_tickets as $default_ticket) {
-                if (! $default_ticket instanceof EE_Ticket) {
+                if ( ! $default_ticket instanceof EE_Ticket) {
                     throw new InvalidEntityException($default_ticket, 'EE_Ticket');
                 }
                 $existing_default_prices = $default_ticket->prices();
@@ -79,7 +79,7 @@ 
                     }
                 }
                 $this->default_prices->create($default_ticket_clone);
-                $new_tickets[ $default_ticket_clone->ID() ] = $default_ticket_clone;
+                $new_tickets[$default_ticket_clone->ID()] = $default_ticket_clone;
             }
         }
         return $new_tickets;

Indentation +57 added lines, -57 removed lines

@@ -21,66 +21,66 @@
  */
 class DefaultTickets implements DefaultEntityGeneratorInterface
 {
-    /**
-     * @var DefaultPrices $default_prices
-     */
-    protected $default_prices;
+	/**
+	 * @var DefaultPrices $default_prices
+	 */
+	protected $default_prices;
 
-    /**
-     * @var EEM_Ticket $ticket_model
-     */
-    protected $ticket_model;
+	/**
+	 * @var EEM_Ticket $ticket_model
+	 */
+	protected $ticket_model;
 
 
-    /**
-     * @param DefaultPrices $default_prices
-     * @param EEM_Ticket    $ticket_model
-     */
-    public function __construct(DefaultPrices $default_prices, EEM_Ticket $ticket_model)
-    {
-        $this->default_prices = $default_prices;
-        $this->ticket_model   = $ticket_model;
-    }
+	/**
+	 * @param DefaultPrices $default_prices
+	 * @param EEM_Ticket    $ticket_model
+	 */
+	public function __construct(DefaultPrices $default_prices, EEM_Ticket $ticket_model)
+	{
+		$this->default_prices = $default_prices;
+		$this->ticket_model   = $ticket_model;
+	}
 
 
-    /**
-     * @param EE_Datetime|EE_Base_Class $entity
-     * @return EE_Ticket[]
-     * @throws EE_Error
-     * @throws InvalidEntityException
-     * @throws ReflectionException
-     * @since $VID:$
-     */
-    public function create(EE_Base_Class $entity): array
-    {
-        if (! $entity instanceof EE_Datetime) {
-            throw new InvalidEntityException($entity, 'EE_Datetime');
-        }
-        $new_tickets     = [];
-        $default_tickets = $this->ticket_model->get_all_default_tickets();
-        if (is_array($default_tickets)) {
-            foreach ($default_tickets as $default_ticket) {
-                if (! $default_ticket instanceof EE_Ticket) {
-                    throw new InvalidEntityException($default_ticket, 'EE_Ticket');
-                }
-                $existing_default_prices = $default_ticket->prices();
-                // clone ticket, strip out ID, then save to get a new ID
-                $default_ticket_clone = clone $default_ticket;
-                $default_ticket_clone->set('TKT_ID', null);
-                $default_ticket_clone->set('TKT_is_default', false);
-                $default_ticket_clone->save();
-                $default_ticket_clone->_add_relation_to($entity, 'Datetime');
-                // temporarily adding relations to existing prices, but these will be cloned and removed
-                // when passed to DefaultPrices::create() below so that they the clones can be freely mutated
-                foreach ($existing_default_prices as $existing_default_price) {
-                    if ($existing_default_price instanceof EE_Price) {
-                        $existing_default_price->_add_relation_to($default_ticket_clone, 'Ticket');
-                    }
-                }
-                $this->default_prices->create($default_ticket_clone);
-                $new_tickets[ $default_ticket_clone->ID() ] = $default_ticket_clone;
-            }
-        }
-        return $new_tickets;
-    }
+	/**
+	 * @param EE_Datetime|EE_Base_Class $entity
+	 * @return EE_Ticket[]
+	 * @throws EE_Error
+	 * @throws InvalidEntityException
+	 * @throws ReflectionException
+	 * @since $VID:$
+	 */
+	public function create(EE_Base_Class $entity): array
+	{
+		if (! $entity instanceof EE_Datetime) {
+			throw new InvalidEntityException($entity, 'EE_Datetime');
+		}
+		$new_tickets     = [];
+		$default_tickets = $this->ticket_model->get_all_default_tickets();
+		if (is_array($default_tickets)) {
+			foreach ($default_tickets as $default_ticket) {
+				if (! $default_ticket instanceof EE_Ticket) {
+					throw new InvalidEntityException($default_ticket, 'EE_Ticket');
+				}
+				$existing_default_prices = $default_ticket->prices();
+				// clone ticket, strip out ID, then save to get a new ID
+				$default_ticket_clone = clone $default_ticket;
+				$default_ticket_clone->set('TKT_ID', null);
+				$default_ticket_clone->set('TKT_is_default', false);
+				$default_ticket_clone->save();
+				$default_ticket_clone->_add_relation_to($entity, 'Datetime');
+				// temporarily adding relations to existing prices, but these will be cloned and removed
+				// when passed to DefaultPrices::create() below so that they the clones can be freely mutated
+				foreach ($existing_default_prices as $existing_default_price) {
+					if ($existing_default_price instanceof EE_Price) {
+						$existing_default_price->_add_relation_to($default_ticket_clone, 'Ticket');
+					}
+				}
+				$this->default_prices->create($default_ticket_clone);
+				$new_tickets[ $default_ticket_clone->ID() ] = $default_ticket_clone;
+			}
+		}
+		return $new_tickets;
+	}
 }

core/domain/services/graphql/connection_resolvers/PriceConnectionResolver.php

Spacing +11 added lines, -11 removed lines

@@ -80,24 +80,24 @@ 
          * Collect the input_fields and sanitize them to prepare them for sending to the Query
          */
         $input_fields = [];
-        if (! empty($this->args['where'])) {
+        if ( ! empty($this->args['where'])) {
             $input_fields = $this->sanitizeInputFields($this->args['where']);
 
             // Use the proper operator.
-            if (! empty($input_fields['PRC_ID']) && is_array($input_fields['PRC_ID'])) {
+            if ( ! empty($input_fields['PRC_ID']) && is_array($input_fields['PRC_ID'])) {
                 $input_fields['PRC_ID'] = ['in', $input_fields['PRC_ID']];
             }
-            if (! empty($input_fields['Ticket.TKT_ID']) && is_array($input_fields['Ticket.TKT_ID'])) {
+            if ( ! empty($input_fields['Ticket.TKT_ID']) && is_array($input_fields['Ticket.TKT_ID'])) {
                 $input_fields['Ticket.TKT_ID'] = ['in', $input_fields['Ticket.TKT_ID']];
             }
-            if (! empty($input_fields['Price_Type.PBT_ID']) && is_array($input_fields['Price_Type.PBT_ID'])) {
+            if ( ! empty($input_fields['Price_Type.PBT_ID']) && is_array($input_fields['Price_Type.PBT_ID'])) {
                 $input_fields['Price_Type.PBT_ID'] = ['in', $input_fields['Price_Type.PBT_ID']];
             }
-            if (! empty($input_fields['Price_Type.PRT_ID']) && is_array($input_fields['Price_Type.PRT_ID'])) {
+            if ( ! empty($input_fields['Price_Type.PRT_ID']) && is_array($input_fields['Price_Type.PRT_ID'])) {
                 $input_fields['Price_Type.PRT_ID'] = ['in', $input_fields['Price_Type.PRT_ID']];
             }
             // if event ID is passed but not a ticket ID
-            if (! isset($input_fields['Ticket.TKT_ID']) && isset($input_fields['Event.EVT_ID'])) {
+            if ( ! isset($input_fields['Ticket.TKT_ID']) && isset($input_fields['Event.EVT_ID'])) {
                 $event_id = $input_fields['Event.EVT_ID'];
                 // Ensure that this doesn't go to the query.
                 // After all there is no DB relation between event and price
@@ -132,7 +132,7 @@ 
         /**
          * Merge the input_fields with the default query_args
          */
-        if (! empty($input_fields)) {
+        if ( ! empty($input_fields)) {
             $where_params = array_merge($where_params, $input_fields);
         }
 
@@ -141,7 +141,7 @@ 
         $default_prices_params = [];
 
         // If default ticket prices should be included.
-        if (! empty($this->args['where']['includeDefaultTicketsPrices'])) {
+        if ( ! empty($this->args['where']['includeDefaultTicketsPrices'])) {
             $default_ticket_ids = EEM_Ticket::instance()->get_col([
                 [
                     'TKT_is_default' => 1,
@@ -150,7 +150,7 @@ 
             ]);
 
             // if we have default tickets
-            if (! empty($default_ticket_ids)) {
+            if ( ! empty($default_ticket_ids)) {
                 $default_prices_params['OR'] = [
                     'Ticket.TKT_ID' => ['IN', $default_ticket_ids],
                 ];
@@ -158,14 +158,14 @@ 
         }
 
         // If default prices should be included.
-        if (! empty($this->args['where']['includeDefaultPrices'])) {
+        if ( ! empty($this->args['where']['includeDefaultPrices'])) {
             $default_prices_params['AND'] = [
                 'PRC_deleted'    => 0,
                 'PRC_is_default' => 1,
             ];
         }
 
-        if (! empty($default_prices_params)) {
+        if ( ! empty($default_prices_params)) {
             if (empty($where_params)) {
                 $where_params['OR'] = $default_prices_params;
             } else {

Indentation +215 added lines, -215 removed lines

@@ -17,219 +17,219 @@
  */
 class PriceConnectionResolver extends AbstractConnectionResolver
 {
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_loader_name(): string
-    {
-        return 'espresso_price';
-    }
-
-    /**
-     * @return EEM_Price
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     * @throws ReflectionException
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_query(): EEM_Price
-    {
-        return EEM_Price::instance();
-    }
-
-
-    /**
-     * Return an array of item IDs from the query
-     *
-     * @return array
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_ids(): array
-    {
-        $results = $this->query->get_col($this->query_args);
-
-        return ! empty($results) ? $results : [];
-    }
-
-
-    /**
-     * Here, we map the args from the input, then we make sure that we're only querying
-     * for IDs. The IDs are then passed down the resolve tree, and deferred resolvers
-     * handle batch resolution of the posts.
-     *
-     * @return array
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws ReflectionException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_query_args(): array
-    {
-        $where_params = [];
-        $query_args   = [];
-
-        $query_args['limit'] = $this->getLimit();
-
-        // Avoid multiple entries by join.
-        $query_args['group_by'] = 'PRC_ID';
-
-        $query_args['default_where_conditions'] = 'minimum';
-
-        /**
-         * Collect the input_fields and sanitize them to prepare them for sending to the Query
-         */
-        $input_fields = [];
-        if (! empty($this->args['where'])) {
-            $input_fields = $this->sanitizeInputFields($this->args['where']);
-
-            // Use the proper operator.
-            if (! empty($input_fields['PRC_ID']) && is_array($input_fields['PRC_ID'])) {
-                $input_fields['PRC_ID'] = ['in', $input_fields['PRC_ID']];
-            }
-            if (! empty($input_fields['Ticket.TKT_ID']) && is_array($input_fields['Ticket.TKT_ID'])) {
-                $input_fields['Ticket.TKT_ID'] = ['in', $input_fields['Ticket.TKT_ID']];
-            }
-            if (! empty($input_fields['Price_Type.PBT_ID']) && is_array($input_fields['Price_Type.PBT_ID'])) {
-                $input_fields['Price_Type.PBT_ID'] = ['in', $input_fields['Price_Type.PBT_ID']];
-            }
-            if (! empty($input_fields['Price_Type.PRT_ID']) && is_array($input_fields['Price_Type.PRT_ID'])) {
-                $input_fields['Price_Type.PRT_ID'] = ['in', $input_fields['Price_Type.PRT_ID']];
-            }
-            // if event ID is passed but not a ticket ID
-            if (! isset($input_fields['Ticket.TKT_ID']) && isset($input_fields['Event.EVT_ID'])) {
-                $event_id = $input_fields['Event.EVT_ID'];
-                // Ensure that this doesn't go to the query.
-                // After all there is no DB relation between event and price
-                unset($input_fields['Event.EVT_ID']);
-                // get all the datetimeIds of the event
-                $event_datetime_ids = EEM_Datetime::instance()->get_col([
-                    [
-                        'EVT_ID'      => $event_id,
-                    ],
-                    'default_where_conditions' => 'minimum'
-                ]);
-                // get all the related ticket Ids
-                $ticket_ids = EEM_Ticket::instance()->get_col([
-                    [
-                        'Datetime.DTT_ID' => ['IN', $event_datetime_ids],
-                    ],
-                    'default_where_conditions' => 'minimum'
-                ]);
-
-                // add tickets relation to the query
-                $input_fields['Ticket.TKT_ID'] = ['IN', $ticket_ids];
-            }
-        }
-
-        /**
-         * Determine where we're at in the Graph and adjust the query context appropriately.
-         */
-        if ($this->source instanceof EE_Ticket) {
-            $where_params['Ticket.TKT_ID'] = $this->source->ID();
-        }
-
-        /**
-         * Merge the input_fields with the default query_args
-         */
-        if (! empty($input_fields)) {
-            $where_params = array_merge($where_params, $input_fields);
-        }
-
-        [$query_args, $where_params] = $this->mapOrderbyInputArgs($query_args, $where_params, 'PRC_ID');
-
-        $default_prices_params = [];
-
-        // If default ticket prices should be included.
-        if (! empty($this->args['where']['includeDefaultTicketsPrices'])) {
-            $default_ticket_ids = EEM_Ticket::instance()->get_col([
-                [
-                    'TKT_is_default' => 1,
-                ],
-                'default_where_conditions' => 'minimum'
-            ]);
-
-            // if we have default tickets
-            if (! empty($default_ticket_ids)) {
-                $default_prices_params['OR'] = [
-                    'Ticket.TKT_ID' => ['IN', $default_ticket_ids],
-                ];
-            }
-        }
-
-        // If default prices should be included.
-        if (! empty($this->args['where']['includeDefaultPrices'])) {
-            $default_prices_params['AND'] = [
-                'PRC_deleted'    => 0,
-                'PRC_is_default' => 1,
-            ];
-        }
-
-        if (! empty($default_prices_params)) {
-            if (empty($where_params)) {
-                $where_params['OR'] = $default_prices_params;
-            } else {
-                $where_params = [
-                    'OR' => [
-                        'OR'  => $default_prices_params,
-                        'AND' => $where_params,
-                    ],
-                ];
-            }
-        }
-
-        $where_params = apply_filters(
-            'FHEE__EventEspresso_core_domain_services_graphql_connection_resolvers__price_where_params',
-            $where_params,
-            $this->source,
-            $this->args
-        );
-
-        $query_args[] = $where_params;
-
-        /**
-         * Return the $query_args
-         */
-        return apply_filters(
-            'FHEE__EventEspresso_core_domain_services_graphql_connection_resolvers__price_query_args',
-            $query_args,
-            $this->source,
-            $this->args
-        );
-    }
-
-
-    /**
-     * This sets up the "allowed" args, and translates the GraphQL-friendly keys to model
-     * friendly keys.
-     *
-     * @param array $where_args
-     * @return array
-     */
-    public function sanitizeInputFields(array $where_args): array
-    {
-        $arg_mapping = [
-            'in'              => 'PRC_ID',
-            'idIn'            => 'PRC_ID',
-            'isDefault'       => 'PRC_is_default',
-            'event'           => 'Event.EVT_ID',
-            'eventId'         => 'Event.EVT_ID', // priority.
-            'ticket'          => 'Ticket.TKT_ID',
-            'ticketIn'        => 'Ticket.TKT_ID',
-            'ticketIdIn'      => 'Ticket.TKT_ID',
-            'ticketId'        => 'Ticket.TKT_ID', // priority.
-            'priceType'       => 'Price_Type.PRT_ID',
-            'priceTypeIn'     => 'Price_Type.PRT_ID',
-            'priceTypeIdIn'   => 'Price_Type.PRT_ID',
-            'priceTypeId'     => 'Price_Type.PRT_ID', // priority.
-            'priceBaseType'   => 'Price_Type.PBT_ID',
-            'priceBaseTypeIn' => 'Price_Type.PBT_ID',
-        ];
-        return $this->sanitizeWhereArgsForInputFields(
-            $where_args,
-            $arg_mapping,
-            ['in', 'event', 'ticket', 'ticketIn', 'priceType', 'priceTypeIn']
-        );
-    }
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_loader_name(): string
+	{
+		return 'espresso_price';
+	}
+
+	/**
+	 * @return EEM_Price
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 * @throws ReflectionException
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_query(): EEM_Price
+	{
+		return EEM_Price::instance();
+	}
+
+
+	/**
+	 * Return an array of item IDs from the query
+	 *
+	 * @return array
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_ids(): array
+	{
+		$results = $this->query->get_col($this->query_args);
+
+		return ! empty($results) ? $results : [];
+	}
+
+
+	/**
+	 * Here, we map the args from the input, then we make sure that we're only querying
+	 * for IDs. The IDs are then passed down the resolve tree, and deferred resolvers
+	 * handle batch resolution of the posts.
+	 *
+	 * @return array
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws ReflectionException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_query_args(): array
+	{
+		$where_params = [];
+		$query_args   = [];
+
+		$query_args['limit'] = $this->getLimit();
+
+		// Avoid multiple entries by join.
+		$query_args['group_by'] = 'PRC_ID';
+
+		$query_args['default_where_conditions'] = 'minimum';
+
+		/**
+		 * Collect the input_fields and sanitize them to prepare them for sending to the Query
+		 */
+		$input_fields = [];
+		if (! empty($this->args['where'])) {
+			$input_fields = $this->sanitizeInputFields($this->args['where']);
+
+			// Use the proper operator.
+			if (! empty($input_fields['PRC_ID']) && is_array($input_fields['PRC_ID'])) {
+				$input_fields['PRC_ID'] = ['in', $input_fields['PRC_ID']];
+			}
+			if (! empty($input_fields['Ticket.TKT_ID']) && is_array($input_fields['Ticket.TKT_ID'])) {
+				$input_fields['Ticket.TKT_ID'] = ['in', $input_fields['Ticket.TKT_ID']];
+			}
+			if (! empty($input_fields['Price_Type.PBT_ID']) && is_array($input_fields['Price_Type.PBT_ID'])) {
+				$input_fields['Price_Type.PBT_ID'] = ['in', $input_fields['Price_Type.PBT_ID']];
+			}
+			if (! empty($input_fields['Price_Type.PRT_ID']) && is_array($input_fields['Price_Type.PRT_ID'])) {
+				$input_fields['Price_Type.PRT_ID'] = ['in', $input_fields['Price_Type.PRT_ID']];
+			}
+			// if event ID is passed but not a ticket ID
+			if (! isset($input_fields['Ticket.TKT_ID']) && isset($input_fields['Event.EVT_ID'])) {
+				$event_id = $input_fields['Event.EVT_ID'];
+				// Ensure that this doesn't go to the query.
+				// After all there is no DB relation between event and price
+				unset($input_fields['Event.EVT_ID']);
+				// get all the datetimeIds of the event
+				$event_datetime_ids = EEM_Datetime::instance()->get_col([
+					[
+						'EVT_ID'      => $event_id,
+					],
+					'default_where_conditions' => 'minimum'
+				]);
+				// get all the related ticket Ids
+				$ticket_ids = EEM_Ticket::instance()->get_col([
+					[
+						'Datetime.DTT_ID' => ['IN', $event_datetime_ids],
+					],
+					'default_where_conditions' => 'minimum'
+				]);
+
+				// add tickets relation to the query
+				$input_fields['Ticket.TKT_ID'] = ['IN', $ticket_ids];
+			}
+		}
+
+		/**
+		 * Determine where we're at in the Graph and adjust the query context appropriately.
+		 */
+		if ($this->source instanceof EE_Ticket) {
+			$where_params['Ticket.TKT_ID'] = $this->source->ID();
+		}
+
+		/**
+		 * Merge the input_fields with the default query_args
+		 */
+		if (! empty($input_fields)) {
+			$where_params = array_merge($where_params, $input_fields);
+		}
+
+		[$query_args, $where_params] = $this->mapOrderbyInputArgs($query_args, $where_params, 'PRC_ID');
+
+		$default_prices_params = [];
+
+		// If default ticket prices should be included.
+		if (! empty($this->args['where']['includeDefaultTicketsPrices'])) {
+			$default_ticket_ids = EEM_Ticket::instance()->get_col([
+				[
+					'TKT_is_default' => 1,
+				],
+				'default_where_conditions' => 'minimum'
+			]);
+
+			// if we have default tickets
+			if (! empty($default_ticket_ids)) {
+				$default_prices_params['OR'] = [
+					'Ticket.TKT_ID' => ['IN', $default_ticket_ids],
+				];
+			}
+		}
+
+		// If default prices should be included.
+		if (! empty($this->args['where']['includeDefaultPrices'])) {
+			$default_prices_params['AND'] = [
+				'PRC_deleted'    => 0,
+				'PRC_is_default' => 1,
+			];
+		}
+
+		if (! empty($default_prices_params)) {
+			if (empty($where_params)) {
+				$where_params['OR'] = $default_prices_params;
+			} else {
+				$where_params = [
+					'OR' => [
+						'OR'  => $default_prices_params,
+						'AND' => $where_params,
+					],
+				];
+			}
+		}
+
+		$where_params = apply_filters(
+			'FHEE__EventEspresso_core_domain_services_graphql_connection_resolvers__price_where_params',
+			$where_params,
+			$this->source,
+			$this->args
+		);
+
+		$query_args[] = $where_params;
+
+		/**
+		 * Return the $query_args
+		 */
+		return apply_filters(
+			'FHEE__EventEspresso_core_domain_services_graphql_connection_resolvers__price_query_args',
+			$query_args,
+			$this->source,
+			$this->args
+		);
+	}
+
+
+	/**
+	 * This sets up the "allowed" args, and translates the GraphQL-friendly keys to model
+	 * friendly keys.
+	 *
+	 * @param array $where_args
+	 * @return array
+	 */
+	public function sanitizeInputFields(array $where_args): array
+	{
+		$arg_mapping = [
+			'in'              => 'PRC_ID',
+			'idIn'            => 'PRC_ID',
+			'isDefault'       => 'PRC_is_default',
+			'event'           => 'Event.EVT_ID',
+			'eventId'         => 'Event.EVT_ID', // priority.
+			'ticket'          => 'Ticket.TKT_ID',
+			'ticketIn'        => 'Ticket.TKT_ID',
+			'ticketIdIn'      => 'Ticket.TKT_ID',
+			'ticketId'        => 'Ticket.TKT_ID', // priority.
+			'priceType'       => 'Price_Type.PRT_ID',
+			'priceTypeIn'     => 'Price_Type.PRT_ID',
+			'priceTypeIdIn'   => 'Price_Type.PRT_ID',
+			'priceTypeId'     => 'Price_Type.PRT_ID', // priority.
+			'priceBaseType'   => 'Price_Type.PBT_ID',
+			'priceBaseTypeIn' => 'Price_Type.PBT_ID',
+		];
+		return $this->sanitizeWhereArgsForInputFields(
+			$where_args,
+			$arg_mapping,
+			['in', 'event', 'ticket', 'ticketIn', 'priceType', 'priceTypeIn']
+		);
+	}
 }

core/domain/services/graphql/connection_resolvers/TicketConnectionResolver.php

Spacing +8 added lines, -8 removed lines

@@ -78,14 +78,14 @@ 
          * Collect the input_fields and sanitize them to prepare them for sending to the Query
          */
         $input_fields = [];
-        if (! empty($this->args['where'])) {
+        if ( ! empty($this->args['where'])) {
             $input_fields = $this->sanitizeInputFields($this->args['where']);
 
             // Use the proper operator.
-            if (! empty($input_fields['Datetime.DTT_ID']) && is_array($input_fields['Datetime.DTT_ID'])) {
+            if ( ! empty($input_fields['Datetime.DTT_ID']) && is_array($input_fields['Datetime.DTT_ID'])) {
                 $input_fields['Datetime.DTT_ID'] = ['IN', $input_fields['Datetime.DTT_ID']];
             }
-            if (! empty($input_fields['Datetime.EVT_ID']) && is_array($input_fields['Datetime.EVT_ID'])) {
+            if ( ! empty($input_fields['Datetime.EVT_ID']) && is_array($input_fields['Datetime.EVT_ID'])) {
                 $input_fields['Datetime.EVT_ID'] = ['IN', $input_fields['Datetime.EVT_ID']];
             }
         }
@@ -100,7 +100,7 @@ 
         /**
          * Merge the input_fields with the default query_args
          */
-        if (! empty($input_fields)) {
+        if ( ! empty($input_fields)) {
             $where_params = array_merge($where_params, $input_fields);
         }
 
@@ -108,16 +108,16 @@ 
 
         $search = isset($this->args['where']) ? $this->getSearchKeywords($this->args['where']) : '';
 
-        if (! empty($search)) {
+        if ( ! empty($search)) {
             // use OR operator to search in any of the fields
             $where_params['OR'] = array(
-                'TKT_name'        => array('LIKE', '%' . $search . '%'),
-                'TKT_description' => array('LIKE', '%' . $search . '%'),
+                'TKT_name'        => array('LIKE', '%'.$search.'%'),
+                'TKT_description' => array('LIKE', '%'.$search.'%'),
             );
         }
 
         // If default tickets should be included.
-        if (! empty($this->args['where']['includeDefaultTickets'])) {
+        if ( ! empty($this->args['where']['includeDefaultTickets'])) {
             /**
              * We need to get each ticket that
              * - satisfies $where_params above

Indentation +174 added lines, -174 removed lines

@@ -15,178 +15,178 @@
  */
 class TicketConnectionResolver extends AbstractConnectionResolver
 {
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_loader_name(): string
-    {
-        return 'espresso_ticket';
-    }
-
-    /**
-     * @return EEM_Ticket
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     * @throws ReflectionException
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_query(): EEM_Ticket
-    {
-        return EEM_Ticket::instance();
-    }
-
-
-    /**
-     * Return an array of item IDs from the query
-     *
-     * @return array
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_ids(): array
-    {
-        $results = $this->query->get_col($this->query_args);
-
-        return ! empty($results) ? $results : [];
-    }
-
-
-    /**
-     * Here, we map the args from the input, then we make sure that we're only querying
-     * for IDs. The IDs are then passed down the resolve tree, and deferred resolvers
-     * handle batch resolution of the posts.
-     *
-     * @return array
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws ReflectionException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_query_args(): array
-    {
-        $where_params = ['TKT_deleted' => ['IN', [true, false]]];
-        $query_args   = [];
-
-        $query_args['limit'] = $this->getLimit();
-
-        // Avoid multiple entries by join.
-        $query_args['group_by'] = 'TKT_ID';
-
-        $query_args['default_where_conditions'] = 'minimum';
-
-        /**
-         * Collect the input_fields and sanitize them to prepare them for sending to the Query
-         */
-        $input_fields = [];
-        if (! empty($this->args['where'])) {
-            $input_fields = $this->sanitizeInputFields($this->args['where']);
-
-            // Use the proper operator.
-            if (! empty($input_fields['Datetime.DTT_ID']) && is_array($input_fields['Datetime.DTT_ID'])) {
-                $input_fields['Datetime.DTT_ID'] = ['IN', $input_fields['Datetime.DTT_ID']];
-            }
-            if (! empty($input_fields['Datetime.EVT_ID']) && is_array($input_fields['Datetime.EVT_ID'])) {
-                $input_fields['Datetime.EVT_ID'] = ['IN', $input_fields['Datetime.EVT_ID']];
-            }
-        }
-
-        /**
-         * Determine where we're at in the Graph and adjust the query context appropriately.
-         */
-        if ($this->source instanceof EE_Datetime) {
-            $where_params['Datetime.DTT_ID'] = $this->source->ID();
-        }
-
-        /**
-         * Merge the input_fields with the default query_args
-         */
-        if (! empty($input_fields)) {
-            $where_params = array_merge($where_params, $input_fields);
-        }
-
-        [$query_args, $where_params] = $this->mapOrderbyInputArgs($query_args, $where_params, 'TKT_ID');
-
-        $search = isset($this->args['where']) ? $this->getSearchKeywords($this->args['where']) : '';
-
-        if (! empty($search)) {
-            // use OR operator to search in any of the fields
-            $where_params['OR'] = array(
-                'TKT_name'        => array('LIKE', '%' . $search . '%'),
-                'TKT_description' => array('LIKE', '%' . $search . '%'),
-            );
-        }
-
-        // If default tickets should be included.
-        if (! empty($this->args['where']['includeDefaultTickets'])) {
-            /**
-             * We need to get each ticket that
-             * - satisfies $where_params above
-             * OR
-             * - it's a default ticket
-             */
-            $where_params = [
-                'OR' => [
-                    // use extra OR instead of AND to avoid it getting overridden
-                    'OR' => [
-                        'AND' => [
-                            'TKT_deleted'    => ['IN', [true, false]],
-                            'TKT_is_default' => 1,
-                        ]
-                    ],
-                    'AND' => $where_params,
-                ],
-            ];
-        }
-
-        $where_params = apply_filters(
-            'FHEE__EventEspresso_core_domain_services_graphql_connection_resolvers__ticket_where_params',
-            $where_params,
-            $this->source,
-            $this->args
-        );
-
-        $query_args[] = $where_params;
-
-        /**
-         * Return the $query_args
-         */
-        return apply_filters(
-            'FHEE__EventEspresso_core_domain_services_graphql_connection_resolvers__ticket_query_args',
-            $query_args,
-            $this->source,
-            $this->args
-        );
-    }
-
-
-    /**
-     * This sets up the "allowed" args, and translates the GraphQL-friendly keys to model
-     * friendly keys.
-     *
-     * @param array $where_args
-     * @return array
-     */
-    public function sanitizeInputFields(array $where_args): array
-    {
-        $arg_mapping = [
-            'event'        => 'Datetime.EVT_ID',
-            'eventIn'      => 'Datetime.EVT_ID',
-            'eventIdIn'    => 'Datetime.EVT_ID',
-            'eventId'      => 'Datetime.EVT_ID', // priority.
-            'datetime'     => 'Datetime.DTT_ID',
-            'datetimeIn'   => 'Datetime.DTT_ID',
-            'datetimeIdIn' => 'Datetime.DTT_ID',
-            'datetimeId'   => 'Datetime.DTT_ID', // priority.
-            'isDefault'    => 'TKT_is_default',
-            'isRequired'   => 'TKT_required',
-            'isTaxable'    => 'TKT_taxable',
-            'isTrashed'    => 'TKT_deleted',
-        ];
-        return $this->sanitizeWhereArgsForInputFields(
-            $where_args,
-            $arg_mapping,
-            ['datetime', 'datetimeIn', 'event', 'eventIn']
-        );
-    }
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_loader_name(): string
+	{
+		return 'espresso_ticket';
+	}
+
+	/**
+	 * @return EEM_Ticket
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 * @throws ReflectionException
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_query(): EEM_Ticket
+	{
+		return EEM_Ticket::instance();
+	}
+
+
+	/**
+	 * Return an array of item IDs from the query
+	 *
+	 * @return array
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_ids(): array
+	{
+		$results = $this->query->get_col($this->query_args);
+
+		return ! empty($results) ? $results : [];
+	}
+
+
+	/**
+	 * Here, we map the args from the input, then we make sure that we're only querying
+	 * for IDs. The IDs are then passed down the resolve tree, and deferred resolvers
+	 * handle batch resolution of the posts.
+	 *
+	 * @return array
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws ReflectionException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_query_args(): array
+	{
+		$where_params = ['TKT_deleted' => ['IN', [true, false]]];
+		$query_args   = [];
+
+		$query_args['limit'] = $this->getLimit();
+
+		// Avoid multiple entries by join.
+		$query_args['group_by'] = 'TKT_ID';
+
+		$query_args['default_where_conditions'] = 'minimum';
+
+		/**
+		 * Collect the input_fields and sanitize them to prepare them for sending to the Query
+		 */
+		$input_fields = [];
+		if (! empty($this->args['where'])) {
+			$input_fields = $this->sanitizeInputFields($this->args['where']);
+
+			// Use the proper operator.
+			if (! empty($input_fields['Datetime.DTT_ID']) && is_array($input_fields['Datetime.DTT_ID'])) {
+				$input_fields['Datetime.DTT_ID'] = ['IN', $input_fields['Datetime.DTT_ID']];
+			}
+			if (! empty($input_fields['Datetime.EVT_ID']) && is_array($input_fields['Datetime.EVT_ID'])) {
+				$input_fields['Datetime.EVT_ID'] = ['IN', $input_fields['Datetime.EVT_ID']];
+			}
+		}
+
+		/**
+		 * Determine where we're at in the Graph and adjust the query context appropriately.
+		 */
+		if ($this->source instanceof EE_Datetime) {
+			$where_params['Datetime.DTT_ID'] = $this->source->ID();
+		}
+
+		/**
+		 * Merge the input_fields with the default query_args
+		 */
+		if (! empty($input_fields)) {
+			$where_params = array_merge($where_params, $input_fields);
+		}
+
+		[$query_args, $where_params] = $this->mapOrderbyInputArgs($query_args, $where_params, 'TKT_ID');
+
+		$search = isset($this->args['where']) ? $this->getSearchKeywords($this->args['where']) : '';
+
+		if (! empty($search)) {
+			// use OR operator to search in any of the fields
+			$where_params['OR'] = array(
+				'TKT_name'        => array('LIKE', '%' . $search . '%'),
+				'TKT_description' => array('LIKE', '%' . $search . '%'),
+			);
+		}
+
+		// If default tickets should be included.
+		if (! empty($this->args['where']['includeDefaultTickets'])) {
+			/**
+			 * We need to get each ticket that
+			 * - satisfies $where_params above
+			 * OR
+			 * - it's a default ticket
+			 */
+			$where_params = [
+				'OR' => [
+					// use extra OR instead of AND to avoid it getting overridden
+					'OR' => [
+						'AND' => [
+							'TKT_deleted'    => ['IN', [true, false]],
+							'TKT_is_default' => 1,
+						]
+					],
+					'AND' => $where_params,
+				],
+			];
+		}
+
+		$where_params = apply_filters(
+			'FHEE__EventEspresso_core_domain_services_graphql_connection_resolvers__ticket_where_params',
+			$where_params,
+			$this->source,
+			$this->args
+		);
+
+		$query_args[] = $where_params;
+
+		/**
+		 * Return the $query_args
+		 */
+		return apply_filters(
+			'FHEE__EventEspresso_core_domain_services_graphql_connection_resolvers__ticket_query_args',
+			$query_args,
+			$this->source,
+			$this->args
+		);
+	}
+
+
+	/**
+	 * This sets up the "allowed" args, and translates the GraphQL-friendly keys to model
+	 * friendly keys.
+	 *
+	 * @param array $where_args
+	 * @return array
+	 */
+	public function sanitizeInputFields(array $where_args): array
+	{
+		$arg_mapping = [
+			'event'        => 'Datetime.EVT_ID',
+			'eventIn'      => 'Datetime.EVT_ID',
+			'eventIdIn'    => 'Datetime.EVT_ID',
+			'eventId'      => 'Datetime.EVT_ID', // priority.
+			'datetime'     => 'Datetime.DTT_ID',
+			'datetimeIn'   => 'Datetime.DTT_ID',
+			'datetimeIdIn' => 'Datetime.DTT_ID',
+			'datetimeId'   => 'Datetime.DTT_ID', // priority.
+			'isDefault'    => 'TKT_is_default',
+			'isRequired'   => 'TKT_required',
+			'isTaxable'    => 'TKT_taxable',
+			'isTrashed'    => 'TKT_deleted',
+		];
+		return $this->sanitizeWhereArgsForInputFields(
+			$where_args,
+			$arg_mapping,
+			['datetime', 'datetimeIn', 'event', 'eventIn']
+		);
+	}
 }

core/domain/services/admin/events/editor/EventEntityRelations.php

Spacing +7 added lines, -7 removed lines

@@ -56,7 +56,7 @@ 
         ];
         // Get the IDs of event datetimes.
         $datetimeIds = $this->datetime_model->get_col([
-            [ 'EVT_ID' => $eventId ],
+            ['EVT_ID' => $eventId],
             'default_where_conditions' => 'minimum',
         ]);
         foreach ($datetimeIds as $datetimeId) {
@@ -64,10 +64,10 @@ 
             foreach ($related_models as $key => $model) {
                 // Get the IDs of related entities for the datetime ID.
                 $Ids = $model->get_col([
-                    [ 'Datetime.DTT_ID' => $datetimeId ],
+                    ['Datetime.DTT_ID' => $datetimeId],
                     'default_where_conditions' => 'minimum',
                 ]);
-                $this->data['datetimes'][ $GID ][ $key ] = ! empty($Ids)
+                $this->data['datetimes'][$GID][$key] = ! empty($Ids)
                     ? $this->utilities->convertToGlobalId($model->item_name(), $Ids)
                     : [];
             }
@@ -104,10 +104,10 @@ 
             foreach ($related_models as $key => $model) {
                 // Get the IDs of related entities for the ticket ID.
                 $Ids = $model->get_col([
-                    [ 'Ticket.TKT_ID' => $ticketId ],
+                    ['Ticket.TKT_ID' => $ticketId],
                     'default_where_conditions' => 'minimum',
                 ]);
-                $this->data['tickets'][ $GID ][ $key ] = ! empty($Ids)
+                $this->data['tickets'][$GID][$key] = ! empty($Ids)
                     ? $this->utilities->convertToGlobalId($model->item_name(), $Ids)
                     : [];
             }
@@ -149,10 +149,10 @@ 
             foreach ($related_models as $key => $model) {
                 // Get the IDs of related entities for the price ID.
                 $Ids = $model->get_col([
-                    [ 'Price.PRC_ID' => $priceId ],
+                    ['Price.PRC_ID' => $priceId],
                     'default_where_conditions' => 'minimum',
                 ]);
-                $this->data['prices'][ $GID ][ $key ] = ! empty($Ids)
+                $this->data['prices'][$GID][$key] = ! empty($Ids)
                     ? $this->utilities->convertToGlobalId($model->item_name(), $Ids)
                     : [];
             }

Indentation +131 added lines, -131 removed lines

@@ -15,146 +15,146 @@
  */
 class EventEntityRelations extends EventEditorData
 {
-    /**
-     * @param array $data
-     */
-    private $data;
+	/**
+	 * @param array $data
+	 */
+	private $data;
 
-    /**
-     * @param int $eventId
-     * @return array
-     * @throws EE_Error
-     * @since $VID:$
-     */
-    public function getData(int $eventId)
-    {
-        $this->data = [
-            'datetimes' => [],
-            'tickets'   => [],
-            'prices'    => [],
-        ];
+	/**
+	 * @param int $eventId
+	 * @return array
+	 * @throws EE_Error
+	 * @since $VID:$
+	 */
+	public function getData(int $eventId)
+	{
+		$this->data = [
+			'datetimes' => [],
+			'tickets'   => [],
+			'prices'    => [],
+		];
 
-        $datetimeIds = $this->processDatetimes($eventId);
-        $ticketIds = $this->processTickets($datetimeIds);
-        $this->processPrices($ticketIds);
+		$datetimeIds = $this->processDatetimes($eventId);
+		$ticketIds = $this->processTickets($datetimeIds);
+		$this->processPrices($ticketIds);
 
-        return $this->data;
-    }
+		return $this->data;
+	}
 
 
-    /**
-     * @param int $eventId
-     * @return array
-     * @throws EE_Error
-     * @since $VID:$
-     */
-    private function processDatetimes(int $eventId)
-    {
-        $related_models = [
-            'tickets' => $this->ticket_model,
-        ];
-        // Get the IDs of event datetimes.
-        $datetimeIds = $this->datetime_model->get_col([
-            [ 'EVT_ID' => $eventId ],
-            'default_where_conditions' => 'minimum',
-        ]);
-        foreach ($datetimeIds as $datetimeId) {
-            $GID = $this->utilities->convertToGlobalId($this->datetime_model->item_name(), $datetimeId);
-            foreach ($related_models as $key => $model) {
-                // Get the IDs of related entities for the datetime ID.
-                $Ids = $model->get_col([
-                    [ 'Datetime.DTT_ID' => $datetimeId ],
-                    'default_where_conditions' => 'minimum',
-                ]);
-                $this->data['datetimes'][ $GID ][ $key ] = ! empty($Ids)
-                    ? $this->utilities->convertToGlobalId($model->item_name(), $Ids)
-                    : [];
-            }
-        }
-        return $datetimeIds;
-    }
+	/**
+	 * @param int $eventId
+	 * @return array
+	 * @throws EE_Error
+	 * @since $VID:$
+	 */
+	private function processDatetimes(int $eventId)
+	{
+		$related_models = [
+			'tickets' => $this->ticket_model,
+		];
+		// Get the IDs of event datetimes.
+		$datetimeIds = $this->datetime_model->get_col([
+			[ 'EVT_ID' => $eventId ],
+			'default_where_conditions' => 'minimum',
+		]);
+		foreach ($datetimeIds as $datetimeId) {
+			$GID = $this->utilities->convertToGlobalId($this->datetime_model->item_name(), $datetimeId);
+			foreach ($related_models as $key => $model) {
+				// Get the IDs of related entities for the datetime ID.
+				$Ids = $model->get_col([
+					[ 'Datetime.DTT_ID' => $datetimeId ],
+					'default_where_conditions' => 'minimum',
+				]);
+				$this->data['datetimes'][ $GID ][ $key ] = ! empty($Ids)
+					? $this->utilities->convertToGlobalId($model->item_name(), $Ids)
+					: [];
+			}
+		}
+		return $datetimeIds;
+	}
 
 
-    /**
-     * @param array $datetimeIds
-     * @return array
-     * @throws EE_Error
-     * @since $VID:$
-     */
-    private function processTickets(array $datetimeIds)
-    {
-        $related_models = [
-            'datetimes' => $this->datetime_model,
-            'prices'    => $this->price_model,
-        ];
-        // Get the IDs of all datetime tickets.
-        $ticketIds = $this->ticket_model->get_col([
-            [
-                'OR' => [
-                    'Datetime.DTT_ID' => ['IN', $datetimeIds],
-                    'TKT_is_default' => 1,
-                ]
-            ],
-            'default_where_conditions' => 'minimum',
-        ]);
-        foreach ($ticketIds as $ticketId) {
-            $GID = $this->utilities->convertToGlobalId($this->ticket_model->item_name(), $ticketId);
+	/**
+	 * @param array $datetimeIds
+	 * @return array
+	 * @throws EE_Error
+	 * @since $VID:$
+	 */
+	private function processTickets(array $datetimeIds)
+	{
+		$related_models = [
+			'datetimes' => $this->datetime_model,
+			'prices'    => $this->price_model,
+		];
+		// Get the IDs of all datetime tickets.
+		$ticketIds = $this->ticket_model->get_col([
+			[
+				'OR' => [
+					'Datetime.DTT_ID' => ['IN', $datetimeIds],
+					'TKT_is_default' => 1,
+				]
+			],
+			'default_where_conditions' => 'minimum',
+		]);
+		foreach ($ticketIds as $ticketId) {
+			$GID = $this->utilities->convertToGlobalId($this->ticket_model->item_name(), $ticketId);
 
-            foreach ($related_models as $key => $model) {
-                // Get the IDs of related entities for the ticket ID.
-                $Ids = $model->get_col([
-                    [ 'Ticket.TKT_ID' => $ticketId ],
-                    'default_where_conditions' => 'minimum',
-                ]);
-                $this->data['tickets'][ $GID ][ $key ] = ! empty($Ids)
-                    ? $this->utilities->convertToGlobalId($model->item_name(), $Ids)
-                    : [];
-            }
-        }
-        return $ticketIds;
-    }
+			foreach ($related_models as $key => $model) {
+				// Get the IDs of related entities for the ticket ID.
+				$Ids = $model->get_col([
+					[ 'Ticket.TKT_ID' => $ticketId ],
+					'default_where_conditions' => 'minimum',
+				]);
+				$this->data['tickets'][ $GID ][ $key ] = ! empty($Ids)
+					? $this->utilities->convertToGlobalId($model->item_name(), $Ids)
+					: [];
+			}
+		}
+		return $ticketIds;
+	}
 
 
-    /**
-     * @param array $ticketIds
-     * @throws EE_Error
-     * @since $VID:$
-     */
-    private function processPrices(array $ticketIds)
-    {
-        $related_models = [
-            'tickets'    => $this->ticket_model,
-            'priceTypes' => $this->price_type_model,
-        ];
-        // Get the IDs of all ticket prices and default prices
-        $priceIds = $this->price_model->get_col([
-            [
-                'OR' => [
-                    // either the price is related to any of these tickets
-                    'Ticket.TKT_ID' => ['IN', $ticketIds],
-                    // or it's a default price and not trashed
-                    'AND' => [
-                        'PRC_deleted'    => 0,
-                        'PRC_is_default' => 1,
-                    ],
-                ],
-            ],
-            'group_by'                 => 'PRC_ID',
-            'default_where_conditions' => 'minimum',
-        ]);
-        foreach ($priceIds as $priceId) {
-            $GID = $this->utilities->convertToGlobalId($this->price_model->item_name(), $priceId);
+	/**
+	 * @param array $ticketIds
+	 * @throws EE_Error
+	 * @since $VID:$
+	 */
+	private function processPrices(array $ticketIds)
+	{
+		$related_models = [
+			'tickets'    => $this->ticket_model,
+			'priceTypes' => $this->price_type_model,
+		];
+		// Get the IDs of all ticket prices and default prices
+		$priceIds = $this->price_model->get_col([
+			[
+				'OR' => [
+					// either the price is related to any of these tickets
+					'Ticket.TKT_ID' => ['IN', $ticketIds],
+					// or it's a default price and not trashed
+					'AND' => [
+						'PRC_deleted'    => 0,
+						'PRC_is_default' => 1,
+					],
+				],
+			],
+			'group_by'                 => 'PRC_ID',
+			'default_where_conditions' => 'minimum',
+		]);
+		foreach ($priceIds as $priceId) {
+			$GID = $this->utilities->convertToGlobalId($this->price_model->item_name(), $priceId);
 
-            foreach ($related_models as $key => $model) {
-                // Get the IDs of related entities for the price ID.
-                $Ids = $model->get_col([
-                    [ 'Price.PRC_ID' => $priceId ],
-                    'default_where_conditions' => 'minimum',
-                ]);
-                $this->data['prices'][ $GID ][ $key ] = ! empty($Ids)
-                    ? $this->utilities->convertToGlobalId($model->item_name(), $Ids)
-                    : [];
-            }
-        }
-    }
+			foreach ($related_models as $key => $model) {
+				// Get the IDs of related entities for the price ID.
+				$Ids = $model->get_col([
+					[ 'Price.PRC_ID' => $priceId ],
+					'default_where_conditions' => 'minimum',
+				]);
+				$this->data['prices'][ $GID ][ $key ] = ! empty($Ids)
+					? $this->utilities->convertToGlobalId($model->item_name(), $Ids)
+					: [];
+			}
+		}
+	}
 }

core/domain/services/admin/entities/DefaultPrices.php

Spacing +10 added lines, -10 removed lines

@@ -67,7 +67,7 @@ 
      */
     public function create(EE_Base_Class $entity): array
     {
-        if (! $entity instanceof EE_Ticket) {
+        if ( ! $entity instanceof EE_Ticket) {
             throw new InvalidEntityException($entity, 'EE_Ticket');
         }
         $is_free                   = true;
@@ -91,9 +91,9 @@ 
             $has_base_price,
             $is_free
         );
-        if (! $has_base_price) {
+        if ( ! $has_base_price) {
             $new_base_price                            = $this->createNewBasePrice($entity);
-            $this->new_prices[ $new_base_price->ID() ] = $new_base_price;
+            $this->new_prices[$new_base_price->ID()] = $new_base_price;
         }
         $this->applyTaxes($entity, $is_free);
         $ticket_total = $entity->get_ticket_total_with_taxes(true);
@@ -113,7 +113,7 @@ 
      */
     private function applyTaxes(EE_Ticket $ticket, bool $is_free)
     {
-        if (! $is_free && $ticket->taxable() && ! empty($this->taxes)) {
+        if ( ! $is_free && $ticket->taxable() && ! empty($this->taxes)) {
             foreach ($this->taxes as $tax) {
                 // assign taxes but don't duplicate them because they operate globally
                 $ticket->set_taxable(true);
@@ -162,12 +162,12 @@ 
             $is_base_price = $default_price_clone->is_base_price();
             // add this price to ticket if it is a regular price modifier, ie: NOT a base price,
             // OR if it IS a base price but this ticket does NOT already have a base price
-            if (! $is_base_price || ! $has_base_price) {
+            if ( ! $is_base_price || ! $has_base_price) {
                 $default_price_clone->set('PRC_ID', null);
                 $default_price_clone->set('PRC_is_default', false);
                 $default_price_clone->save();
                 $default_price_clone->_add_relation_to($ticket, 'Ticket');
-                $this->new_prices[ $default_price_clone->ID() ] = $default_price_clone;
+                $this->new_prices[$default_price_clone->ID()] = $default_price_clone;
                 // then recheck that a base price has been set so that we don't add another one
                 $has_base_price = $is_base_price ? true : $has_base_price;
             }
@@ -192,7 +192,7 @@ 
                 ],
             ]
         );
-        if (! $base_price_type instanceof EE_Price_Type) {
+        if ( ! $base_price_type instanceof EE_Price_Type) {
             throw new RuntimeException(
                 esc_html__(
                     'A valid base price type could not be retrieved from the database.',
@@ -219,15 +219,15 @@ 
         $default_prices = [];
         if (is_array($prices)) {
             foreach ($prices as $key => $price) {
-                if (! $price instanceof EE_Price) {
+                if ( ! $price instanceof EE_Price) {
                     throw new InvalidEntityException($price, 'EE_Price');
                 }
                 // grab any taxes but don't do anything just yet
                 if ($price->is_tax()) {
-                    $this->taxes[ $price->ID() ] = $price;
+                    $this->taxes[$price->ID()] = $price;
                     continue;
                 }
-                $default_prices[ $price->ID() ] = $price;
+                $default_prices[$price->ID()] = $price;
             }
         }
         return $default_prices;

Indentation +219 added lines, -219 removed lines

@@ -24,223 +24,223 @@
  */
 class DefaultPrices implements DefaultEntityGeneratorInterface
 {
-    /**
-     * @var EEM_Price
-     */
-    protected $price_model;
-
-    /**
-     * @var EEM_Price_Type
-     */
-    protected $price_type_model;
-
-    /**
-     * @var EE_Price[]
-     */
-    protected $new_prices = [];
-
-    /**
-     * @var EE_Price[]
-     */
-    protected $taxes = [];
-
-
-    /**
-     * @param EEM_Price      $price_model
-     * @param EEM_Price_Type $price_type_model
-     */
-    public function __construct(EEM_Price $price_model, EEM_Price_Type $price_type_model)
-    {
-        $this->price_model      = $price_model;
-        $this->price_type_model = $price_type_model;
-    }
-
-
-    /**
-     * @param EE_Ticket|EE_Base_Class $entity
-     * @return EE_Price[]
-     * @throws EE_Error
-     * @throws InvalidInterfaceException
-     * @throws ReflectionException
-     * @since $VID:$
-     */
-    public function create(EE_Base_Class $entity): array
-    {
-        if (! $entity instanceof EE_Ticket) {
-            throw new InvalidEntityException($entity, 'EE_Ticket');
-        }
-        $is_free                   = true;
-        $has_base_price            = false;
-        $remove_existing_relations = true;
-        // first, let's get all of the default taxes for the site
-        $this->taxes = $this->price_model->getAllDefaultTaxes();
-        // then separate taxes from the other prices for the existing default ticket prices
-        $default_prices = $this->separateTaxesFromOtherPrices($entity->prices());
-        // but if that's empty, then let's get the default global prices
-        if (empty($default_prices)) {
-            $default_global_prices     = $this->price_model->get_all_default_prices();
-            $default_prices            = $this->separateTaxesFromOtherPrices($default_global_prices);
-            $remove_existing_relations = false;
-        }
-        // then clone and apply all of the default prices
-        [$has_base_price, $is_free] = $this->cloneDefaultPrices(
-            $entity,
-            $default_prices,
-            $remove_existing_relations,
-            $has_base_price,
-            $is_free
-        );
-        if (! $has_base_price) {
-            $new_base_price                            = $this->createNewBasePrice($entity);
-            $this->new_prices[ $new_base_price->ID() ] = $new_base_price;
-        }
-        $this->applyTaxes($entity, $is_free);
-        $ticket_total = $entity->get_ticket_total_with_taxes(true);
-        if ($ticket_total !== $entity->ticket_price()) {
-            $entity->set_price($ticket_total);
-            $entity->save();
-        }
-        return $this->new_prices;
-    }
-
-
-    /**
-     * @param EE_Ticket $ticket
-     * @param bool      $is_free
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    private function applyTaxes(EE_Ticket $ticket, bool $is_free)
-    {
-        if (! $is_free && $ticket->taxable() && ! empty($this->taxes)) {
-            foreach ($this->taxes as $tax) {
-                // assign taxes but don't duplicate them because they operate globally
-                $ticket->set_taxable(true);
-                $tax->_add_relation_to($ticket, 'Ticket');
-            }
-        }
-    }
-
-
-    /**
-     * @param EE_Ticket  $ticket
-     * @param EE_Price[] $default_prices
-     * @param bool       $remove_existing_relations
-     * @param bool       $has_base_price
-     * @param bool       $is_free
-     * @return bool[]
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    private function cloneDefaultPrices(
-        EE_Ticket $ticket,
-        array $default_prices,
-        bool $remove_existing_relations,
-        bool $has_base_price,
-        bool $is_free
-    ): array {
-        foreach ($default_prices as $default_price) {
-            // duplicate the default price so that it does not get mutated
-            $default_price_clone = clone $default_price;
-            if ($remove_existing_relations) {
-                $ticket->_remove_relation_to($default_price, 'Price');
-            }
-            if (
-                (
-                    // has non-zero base price
-                    $default_price_clone->is_base_price()
-                    && $default_price_clone->amount() > 0
-                )
-                || (
-                    // or has fixed amount surcharge
-                    $default_price_clone->is_surcharge()
-                    && ! $default_price_clone->is_percent()
-                )
-            ) {
-                $is_free = false;
-            }
-            $is_base_price = $default_price_clone->is_base_price();
-            // add this price to ticket if it is a regular price modifier, ie: NOT a base price,
-            // OR if it IS a base price but this ticket does NOT already have a base price
-            if (! $is_base_price || ! $has_base_price) {
-                $default_price_clone->set('PRC_ID', null);
-                $default_price_clone->set('PRC_is_default', false);
-
-                $order = $default_price_clone->get('PRC_order');
-
-                // enforce base price order to be 1 and 5 if the order is not set for a modifier
-                $order = $is_base_price ? 1 : $order;
-                $order = $order ?? 5;
-
-                $default_price_clone->set('PRC_order', $order);
-
-                $default_price_clone->save();
-                $default_price_clone->_add_relation_to($ticket, 'Ticket');
-                $this->new_prices[ $default_price_clone->ID() ] = $default_price_clone;
-                // then recheck that a base price has been set so that we don't add another one
-                $has_base_price = $is_base_price ? true : $has_base_price;
-            }
-        }
-        return [$has_base_price, $is_free];
-    }
-
-
-    /**
-     * @param EE_Ticket $ticket
-     * @return EE_Price
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    private function createNewBasePrice(EE_Ticket $ticket): EE_Price
-    {
-        $new_base_price  = $this->price_model->get_new_price();
-        $base_price_type = $this->price_type_model->get_one(
-            [
-                [
-                    'PBT_ID' => EEM_Price_Type::base_type_base_price,
-                ],
-            ]
-        );
-        if (! $base_price_type instanceof EE_Price_Type) {
-            throw new RuntimeException(
-                esc_html__(
-                    'A valid base price type could not be retrieved from the database.',
-                    'event_espresso'
-                )
-            );
-        }
-        $new_base_price->set('PRT_ID', $base_price_type->ID());
-        // set base price order to 1
-        $new_base_price->set('PRC_order', 1);
-        $new_base_price->set('PRC_is_default', false);
-        $new_base_price->save();
-        $new_base_price->_add_relation_to($ticket, 'Ticket');
-        return $new_base_price;
-    }
-
-
-    /**
-     * @param array $prices
-     * @return array
-     * @throws EE_Error
-     * @throws ReflectionException
-     */
-    private function separateTaxesFromOtherPrices(array $prices = []): array
-    {
-        $default_prices = [];
-        if (is_array($prices)) {
-            foreach ($prices as $key => $price) {
-                if (! $price instanceof EE_Price) {
-                    throw new InvalidEntityException($price, 'EE_Price');
-                }
-                // grab any taxes but don't do anything just yet
-                if ($price->is_tax()) {
-                    $this->taxes[ $price->ID() ] = $price;
-                    continue;
-                }
-                $default_prices[ $price->ID() ] = $price;
-            }
-        }
-        return $default_prices;
-    }
+	/**
+	 * @var EEM_Price
+	 */
+	protected $price_model;
+
+	/**
+	 * @var EEM_Price_Type
+	 */
+	protected $price_type_model;
+
+	/**
+	 * @var EE_Price[]
+	 */
+	protected $new_prices = [];
+
+	/**
+	 * @var EE_Price[]
+	 */
+	protected $taxes = [];
+
+
+	/**
+	 * @param EEM_Price      $price_model
+	 * @param EEM_Price_Type $price_type_model
+	 */
+	public function __construct(EEM_Price $price_model, EEM_Price_Type $price_type_model)
+	{
+		$this->price_model      = $price_model;
+		$this->price_type_model = $price_type_model;
+	}
+
+
+	/**
+	 * @param EE_Ticket|EE_Base_Class $entity
+	 * @return EE_Price[]
+	 * @throws EE_Error
+	 * @throws InvalidInterfaceException
+	 * @throws ReflectionException
+	 * @since $VID:$
+	 */
+	public function create(EE_Base_Class $entity): array
+	{
+		if (! $entity instanceof EE_Ticket) {
+			throw new InvalidEntityException($entity, 'EE_Ticket');
+		}
+		$is_free                   = true;
+		$has_base_price            = false;
+		$remove_existing_relations = true;
+		// first, let's get all of the default taxes for the site
+		$this->taxes = $this->price_model->getAllDefaultTaxes();
+		// then separate taxes from the other prices for the existing default ticket prices
+		$default_prices = $this->separateTaxesFromOtherPrices($entity->prices());
+		// but if that's empty, then let's get the default global prices
+		if (empty($default_prices)) {
+			$default_global_prices     = $this->price_model->get_all_default_prices();
+			$default_prices            = $this->separateTaxesFromOtherPrices($default_global_prices);
+			$remove_existing_relations = false;
+		}
+		// then clone and apply all of the default prices
+		[$has_base_price, $is_free] = $this->cloneDefaultPrices(
+			$entity,
+			$default_prices,
+			$remove_existing_relations,
+			$has_base_price,
+			$is_free
+		);
+		if (! $has_base_price) {
+			$new_base_price                            = $this->createNewBasePrice($entity);
+			$this->new_prices[ $new_base_price->ID() ] = $new_base_price;
+		}
+		$this->applyTaxes($entity, $is_free);
+		$ticket_total = $entity->get_ticket_total_with_taxes(true);
+		if ($ticket_total !== $entity->ticket_price()) {
+			$entity->set_price($ticket_total);
+			$entity->save();
+		}
+		return $this->new_prices;
+	}
+
+
+	/**
+	 * @param EE_Ticket $ticket
+	 * @param bool      $is_free
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	private function applyTaxes(EE_Ticket $ticket, bool $is_free)
+	{
+		if (! $is_free && $ticket->taxable() && ! empty($this->taxes)) {
+			foreach ($this->taxes as $tax) {
+				// assign taxes but don't duplicate them because they operate globally
+				$ticket->set_taxable(true);
+				$tax->_add_relation_to($ticket, 'Ticket');
+			}
+		}
+	}
+
+
+	/**
+	 * @param EE_Ticket  $ticket
+	 * @param EE_Price[] $default_prices
+	 * @param bool       $remove_existing_relations
+	 * @param bool       $has_base_price
+	 * @param bool       $is_free
+	 * @return bool[]
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	private function cloneDefaultPrices(
+		EE_Ticket $ticket,
+		array $default_prices,
+		bool $remove_existing_relations,
+		bool $has_base_price,
+		bool $is_free
+	): array {
+		foreach ($default_prices as $default_price) {
+			// duplicate the default price so that it does not get mutated
+			$default_price_clone = clone $default_price;
+			if ($remove_existing_relations) {
+				$ticket->_remove_relation_to($default_price, 'Price');
+			}
+			if (
+				(
+					// has non-zero base price
+					$default_price_clone->is_base_price()
+					&& $default_price_clone->amount() > 0
+				)
+				|| (
+					// or has fixed amount surcharge
+					$default_price_clone->is_surcharge()
+					&& ! $default_price_clone->is_percent()
+				)
+			) {
+				$is_free = false;
+			}
+			$is_base_price = $default_price_clone->is_base_price();
+			// add this price to ticket if it is a regular price modifier, ie: NOT a base price,
+			// OR if it IS a base price but this ticket does NOT already have a base price
+			if (! $is_base_price || ! $has_base_price) {
+				$default_price_clone->set('PRC_ID', null);
+				$default_price_clone->set('PRC_is_default', false);
+
+				$order = $default_price_clone->get('PRC_order');
+
+				// enforce base price order to be 1 and 5 if the order is not set for a modifier
+				$order = $is_base_price ? 1 : $order;
+				$order = $order ?? 5;
+
+				$default_price_clone->set('PRC_order', $order);
+
+				$default_price_clone->save();
+				$default_price_clone->_add_relation_to($ticket, 'Ticket');
+				$this->new_prices[ $default_price_clone->ID() ] = $default_price_clone;
+				// then recheck that a base price has been set so that we don't add another one
+				$has_base_price = $is_base_price ? true : $has_base_price;
+			}
+		}
+		return [$has_base_price, $is_free];
+	}
+
+
+	/**
+	 * @param EE_Ticket $ticket
+	 * @return EE_Price
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	private function createNewBasePrice(EE_Ticket $ticket): EE_Price
+	{
+		$new_base_price  = $this->price_model->get_new_price();
+		$base_price_type = $this->price_type_model->get_one(
+			[
+				[
+					'PBT_ID' => EEM_Price_Type::base_type_base_price,
+				],
+			]
+		);
+		if (! $base_price_type instanceof EE_Price_Type) {
+			throw new RuntimeException(
+				esc_html__(
+					'A valid base price type could not be retrieved from the database.',
+					'event_espresso'
+				)
+			);
+		}
+		$new_base_price->set('PRT_ID', $base_price_type->ID());
+		// set base price order to 1
+		$new_base_price->set('PRC_order', 1);
+		$new_base_price->set('PRC_is_default', false);
+		$new_base_price->save();
+		$new_base_price->_add_relation_to($ticket, 'Ticket');
+		return $new_base_price;
+	}
+
+
+	/**
+	 * @param array $prices
+	 * @return array
+	 * @throws EE_Error
+	 * @throws ReflectionException
+	 */
+	private function separateTaxesFromOtherPrices(array $prices = []): array
+	{
+		$default_prices = [];
+		if (is_array($prices)) {
+			foreach ($prices as $key => $price) {
+				if (! $price instanceof EE_Price) {
+					throw new InvalidEntityException($price, 'EE_Price');
+				}
+				// grab any taxes but don't do anything just yet
+				if ($price->is_tax()) {
+					$this->taxes[ $price->ID() ] = $price;
+					continue;
+				}
+				$default_prices[ $price->ID() ] = $price;
+			}
+		}
+		return $default_prices;
+	}
 }

core/db_models/EEM_Question_Group.model.php

Spacing +4 added lines, -4 removed lines

@@ -110,13 +110,13 @@
             '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_Public();
-        $this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
+        $this->_cap_restriction_generators[EEM_Base::caps_read_admin] =
             new EE_Restriction_Generator_Reg_Form('QSG_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
+        $this->_cap_restriction_generators[EEM_Base::caps_edit]       =
             new EE_Restriction_Generator_Reg_Form('QSG_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
+        $this->_cap_restriction_generators[EEM_Base::caps_delete]     =
             new EE_Restriction_Generator_Reg_Form('QSG_system');
 
         parent::__construct($timezone);

Indentation +124 added lines, -124 removed lines

@@ -9,136 +9,136 @@
  */
 class EEM_Question_Group extends EEM_Soft_Delete_Base
 {
-    const system_personal = 1;
+	const system_personal = 1;
 
-    const system_address = 2;
+	const system_address = 2;
 
-    /**
-     * private instance of the EEM_Question_Group object
-     *
-     * @var EEM_Question_Group
-     */
-    protected static $_instance = null;
+	/**
+	 * private instance of the EEM_Question_Group object
+	 *
+	 * @var EEM_Question_Group
+	 */
+	protected static $_instance = null;
 
 
-    /**
-     * EEM_Question_Group constructor.
-     *
-     * @param string|null $timezone
-     */
-    protected function __construct($timezone = null)
-    {
-        $this->singular_item = esc_html__('Question Group', 'event_espresso');
-        $this->plural_item   = esc_html__('Question Groups', 'event_espresso');
+	/**
+	 * EEM_Question_Group constructor.
+	 *
+	 * @param string|null $timezone
+	 */
+	protected function __construct($timezone = null)
+	{
+		$this->singular_item = esc_html__('Question Group', 'event_espresso');
+		$this->plural_item   = esc_html__('Question Groups', 'event_espresso');
 
-        $this->_tables          = [
-            'Question_Group' => new EE_Primary_Table('esp_question_group', 'QSG_ID'),
-        ];
-        $this->_fields          = [
-            'Question_Group' => [
-                'QSG_ID'              => new EE_Primary_Key_Int_Field(
-                    'QSG_ID',
-                    esc_html__('Question Group ID', 'event_espresso')
-                ),
-                'QSG_deleted'         => new EE_Trashed_Flag_Field(
-                    'QSG_deleted',
-                    esc_html__('Flag indicating this question group was deleted', 'event_espresso'),
-                    false,
-                    false
-                ),
-                'QSG_desc'            => new EE_Post_Content_Field(
-                    'QSG_desc',
-                    esc_html__('Description of Question Group', 'event_espresso'),
-                    true,
-                    ''
-                ),
-                'QSG_identifier'      => new EE_Plain_Text_Field(
-                    'QSG_identifier',
-                    esc_html__('Text ID for question Group', 'event_espresso'),
-                    false,
-                    ''
-                ),
-                'QSG_name'            => new EE_Plain_Text_Field(
-                    'QSG_name',
-                    esc_html__('Question Group Name', 'event_espresso'),
-                    false,
-                    ''
-                ),
-                'QSG_order'           => new EE_Integer_Field(
-                    'QSG_order',
-                    esc_html__('Order in which to show the question group', 'event_espresso'),
-                    true,
-                    0
-                ),
-                'QSG_show_group_desc' => new EE_Boolean_Field(
-                    'QSG_show_group_desc',
-                    esc_html__(
-                        'Flag indicating whether to show the group\s description on the registration page',
-                        'event_espresso'
-                    ),
-                    false,
-                    false
-                ),
-                'QSG_show_group_name' => new EE_Boolean_Field(
-                    'QSG_show_group_name',
-                    esc_html__(
-                        'Flag indicating whether to show the group\'s name on the registration page',
-                        'event_espresso'
-                    ),
-                    false,
-                    true
-                ),
-                'QSG_system'          => new EE_Integer_Field(
-                    'QSG_system',
-                    esc_html__(
-                        'Indicate IF this is a system group and if it is what system group it corresponds to.',
-                        'event_espresso'
-                    ),
-                    false,
-                    0
-                ),
-                'QSG_wp_user'         => new EE_WP_User_Field(
-                    'QSG_wp_user',
-                    esc_html__('Question Group Creator ID', 'event_espresso'),
-                    false
-                ),
-            ],
-        ];
-        $this->_model_relations = [
-            'Event'                => new EE_HABTM_Relation('Event_Question_Group'),
-            'Event_Question_Group' => new EE_Has_Many_Relation(),
-            'Question'             => new EE_HABTM_Relation('Question_Group_Question'),
-            '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_Public();
-        $this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
-            new EE_Restriction_Generator_Reg_Form('QSG_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
-            new EE_Restriction_Generator_Reg_Form('QSG_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
-            new EE_Restriction_Generator_Reg_Form('QSG_system');
+		$this->_tables          = [
+			'Question_Group' => new EE_Primary_Table('esp_question_group', 'QSG_ID'),
+		];
+		$this->_fields          = [
+			'Question_Group' => [
+				'QSG_ID'              => new EE_Primary_Key_Int_Field(
+					'QSG_ID',
+					esc_html__('Question Group ID', 'event_espresso')
+				),
+				'QSG_deleted'         => new EE_Trashed_Flag_Field(
+					'QSG_deleted',
+					esc_html__('Flag indicating this question group was deleted', 'event_espresso'),
+					false,
+					false
+				),
+				'QSG_desc'            => new EE_Post_Content_Field(
+					'QSG_desc',
+					esc_html__('Description of Question Group', 'event_espresso'),
+					true,
+					''
+				),
+				'QSG_identifier'      => new EE_Plain_Text_Field(
+					'QSG_identifier',
+					esc_html__('Text ID for question Group', 'event_espresso'),
+					false,
+					''
+				),
+				'QSG_name'            => new EE_Plain_Text_Field(
+					'QSG_name',
+					esc_html__('Question Group Name', 'event_espresso'),
+					false,
+					''
+				),
+				'QSG_order'           => new EE_Integer_Field(
+					'QSG_order',
+					esc_html__('Order in which to show the question group', 'event_espresso'),
+					true,
+					0
+				),
+				'QSG_show_group_desc' => new EE_Boolean_Field(
+					'QSG_show_group_desc',
+					esc_html__(
+						'Flag indicating whether to show the group\s description on the registration page',
+						'event_espresso'
+					),
+					false,
+					false
+				),
+				'QSG_show_group_name' => new EE_Boolean_Field(
+					'QSG_show_group_name',
+					esc_html__(
+						'Flag indicating whether to show the group\'s name on the registration page',
+						'event_espresso'
+					),
+					false,
+					true
+				),
+				'QSG_system'          => new EE_Integer_Field(
+					'QSG_system',
+					esc_html__(
+						'Indicate IF this is a system group and if it is what system group it corresponds to.',
+						'event_espresso'
+					),
+					false,
+					0
+				),
+				'QSG_wp_user'         => new EE_WP_User_Field(
+					'QSG_wp_user',
+					esc_html__('Question Group Creator ID', 'event_espresso'),
+					false
+				),
+			],
+		];
+		$this->_model_relations = [
+			'Event'                => new EE_HABTM_Relation('Event_Question_Group'),
+			'Event_Question_Group' => new EE_Has_Many_Relation(),
+			'Question'             => new EE_HABTM_Relation('Question_Group_Question'),
+			'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_Public();
+		$this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
+			new EE_Restriction_Generator_Reg_Form('QSG_system');
+		$this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
+			new EE_Restriction_Generator_Reg_Form('QSG_system');
+		$this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
+			new EE_Restriction_Generator_Reg_Form('QSG_system');
 
-        parent::__construct($timezone);
-    }
+		parent::__construct($timezone);
+	}
 
 
-    /**
-     * searches the db for the question group with the latest question order and returns that value.
-     *
-     * @return int
-     * @throws EE_Error
-     */
-    public function get_latest_question_group_order(): int
-    {
-        $max = $this->_get_all_wpdb_results(
-            [],
-            ARRAY_A,
-            [
-                'max_order' => ["MAX(QSG_order)", "%d"],
-            ]
-        );
-        return $max[0]['max_order'];
-    }
+	/**
+	 * searches the db for the question group with the latest question order and returns that value.
+	 *
+	 * @return int
+	 * @throws EE_Error
+	 */
+	public function get_latest_question_group_order(): int
+	{
+		$max = $this->_get_all_wpdb_results(
+			[],
+			ARRAY_A,
+			[
+				'max_order' => ["MAX(QSG_order)", "%d"],
+			]
+		);
+		return $max[0]['max_order'];
+	}
 }

core/db_models/EEM_Question_Option.model.php

Spacing +4 added lines, -4 removed lines

@@ -74,13 +74,13 @@
         $this->_model_chain_to_wp_user = 'Question';
 
         // 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_Public();
-        $this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
+        $this->_cap_restriction_generators[EEM_Base::caps_read_admin] =
             new EE_Restriction_Generator_Reg_Form('QSO_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
+        $this->_cap_restriction_generators[EEM_Base::caps_edit]       =
             new EE_Restriction_Generator_Reg_Form('QSO_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
+        $this->_cap_restriction_generators[EEM_Base::caps_delete]     =
             new EE_Restriction_Generator_Reg_Form('QSO_system');
 
         parent::__construct($timezone);

Indentation +70 added lines, -70 removed lines

@@ -9,80 +9,80 @@
  */
 class EEM_Question_Option extends EEM_Soft_Delete_Base
 {
-    // private instance of the Attendee object
-    protected static $_instance = null;
+	// private instance of the Attendee object
+	protected static $_instance = null;
 
 
-    protected function __construct($timezone = null)
-    {
-        $this->singular_item = esc_html__('Question Option', 'event_espresso');
-        $this->plural_item   = esc_html__('Question Options', 'event_espresso');
+	protected function __construct($timezone = null)
+	{
+		$this->singular_item = esc_html__('Question Option', 'event_espresso');
+		$this->plural_item   = esc_html__('Question Options', 'event_espresso');
 
-        $this->_tables          = [
-            'Question_Option' => new EE_Primary_Table('esp_question_option', 'QSO_ID'),
-        ];
-        $this->_fields          = [
-            'Question_Option' => [
-                'QSO_ID'      => new EE_Primary_Key_Int_Field(
-                    'QSO_ID',
-                    esc_html__('Question Option ID', 'event_espresso')
-                ),
-                'QST_ID'      => new EE_Foreign_Key_Int_Field(
-                    'QST_ID',
-                    esc_html__('Question ID', 'event_espresso'),
-                    false,
-                    0,
-                    'Question'
-                ),
-                'QSO_deleted' => new EE_Trashed_Flag_Field(
-                    'QSO_deleted',
-                    esc_html__('Flag indicating Option was trashed', 'event_espresso'),
-                    false,
-                    false
-                ),
-                'QSO_desc'    => new EE_Post_Content_Field(
-                    'QSO_desc',
-                    esc_html__('Question Option Description', 'event_espresso'),
-                    false,
-                    ''
-                ),
-                'QSO_order'   => new EE_Integer_Field(
-                    'QSO_order',
-                    esc_html__('Question Option Order', 'event_espresso'),
-                    false,
-                    0
-                ),
-                'QSO_system'  => new EE_Plain_Text_Field(
-                    'QSO_system',
-                    esc_html__('Internal string ID for question option', 'event_espresso'),
-                    true,
-                    null
-                ),
-                'QSO_value'   => new EE_Plain_Text_Field(
-                    'QSO_value',
-                    esc_html__("Question Option Value", "event_espresso"),
-                    false,
-                    ''
-                ),
-            ],
-        ];
-        $this->_model_relations = [
-            'Question' => new EE_Belongs_To_Relation(),
-        ];
+		$this->_tables          = [
+			'Question_Option' => new EE_Primary_Table('esp_question_option', 'QSO_ID'),
+		];
+		$this->_fields          = [
+			'Question_Option' => [
+				'QSO_ID'      => new EE_Primary_Key_Int_Field(
+					'QSO_ID',
+					esc_html__('Question Option ID', 'event_espresso')
+				),
+				'QST_ID'      => new EE_Foreign_Key_Int_Field(
+					'QST_ID',
+					esc_html__('Question ID', 'event_espresso'),
+					false,
+					0,
+					'Question'
+				),
+				'QSO_deleted' => new EE_Trashed_Flag_Field(
+					'QSO_deleted',
+					esc_html__('Flag indicating Option was trashed', 'event_espresso'),
+					false,
+					false
+				),
+				'QSO_desc'    => new EE_Post_Content_Field(
+					'QSO_desc',
+					esc_html__('Question Option Description', 'event_espresso'),
+					false,
+					''
+				),
+				'QSO_order'   => new EE_Integer_Field(
+					'QSO_order',
+					esc_html__('Question Option Order', 'event_espresso'),
+					false,
+					0
+				),
+				'QSO_system'  => new EE_Plain_Text_Field(
+					'QSO_system',
+					esc_html__('Internal string ID for question option', 'event_espresso'),
+					true,
+					null
+				),
+				'QSO_value'   => new EE_Plain_Text_Field(
+					'QSO_value',
+					esc_html__("Question Option Value", "event_espresso"),
+					false,
+					''
+				),
+			],
+		];
+		$this->_model_relations = [
+			'Question' => new EE_Belongs_To_Relation(),
+		];
 
-        $this->_caps_slug              = 'questions';
-        $this->_model_chain_to_wp_user = 'Question';
+		$this->_caps_slug              = 'questions';
+		$this->_model_chain_to_wp_user = 'Question';
 
-        // this model is generally available for reading
-        $this->_cap_restriction_generators[ EEM_Base::caps_read ]       =
-            new EE_Restriction_Generator_Public();
-        $this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
-            new EE_Restriction_Generator_Reg_Form('QSO_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
-            new EE_Restriction_Generator_Reg_Form('QSO_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
-            new EE_Restriction_Generator_Reg_Form('QSO_system');
+		// this model is generally available for reading
+		$this->_cap_restriction_generators[ EEM_Base::caps_read ]       =
+			new EE_Restriction_Generator_Public();
+		$this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
+			new EE_Restriction_Generator_Reg_Form('QSO_system');
+		$this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
+			new EE_Restriction_Generator_Reg_Form('QSO_system');
+		$this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
+			new EE_Restriction_Generator_Reg_Form('QSO_system');
 
-        parent::__construct($timezone);
-    }
+		parent::__construct($timezone);
+	}
 }

core/db_models/EEM_Question_Group_Question.model.php

Spacing +4 added lines, -4 removed lines

@@ -61,13 +61,13 @@
         $this->_model_chain_to_wp_user = 'Question_Group';
 
         // 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_Public();
-        $this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
+        $this->_cap_restriction_generators[EEM_Base::caps_read_admin] =
             new EE_Restriction_Generator_Reg_Form('Question_Group.QSG_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
+        $this->_cap_restriction_generators[EEM_Base::caps_edit]       =
             new EE_Restriction_Generator_Reg_Form('Question_Group.QSG_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
+        $this->_cap_restriction_generators[EEM_Base::caps_delete]     =
             new EE_Restriction_Generator_Reg_Form('Question_Group.QSG_system');
 
         // use the caps for question groups

Indentation +55 added lines, -55 removed lines

@@ -13,64 +13,64 @@
  */
 class EEM_Question_Group_Question extends EEM_Base
 {
-    // private instance of the Attendee object
-    protected static $_instance = null;
+	// private instance of the Attendee object
+	protected static $_instance = null;
 
 
-    protected function __construct($timezone = null)
-    {
-        $this->singular_item    = esc_html__('Question Group to Question Link', 'event_espresso');
-        $this->plural_item      = esc_html__('Question Group to Question Links', 'event_espresso');
-        $this->_tables          = [
-            'Question_Group_Question' => new EE_Primary_Table('esp_question_group_question', 'QGQ_ID'),
-        ];
-        $this->_fields          = [
-            'Question_Group_Question' => [
-                'QGQ_ID'    => new EE_Primary_Key_Int_Field(
-                    'QGQ_ID',
-                    esc_html__('Question Group to Question Link ID', 'event_espresso')
-                ),
-                'QSG_ID'    => new EE_Foreign_Key_Int_Field(
-                    'QSG_ID',
-                    esc_html__('Question Group ID', 'event_espresso'),
-                    false,
-                    0,
-                    'Question_Group'
-                ),
-                'QST_ID'    => new EE_Foreign_Key_Int_Field(
-                    'QST_ID',
-                    esc_html__('Question Id', 'event_espresso'),
-                    false,
-                    0,
-                    'Question'
-                ),
-                'QGQ_order' => new EE_Integer_Field(
-                    'QGQ_order',
-                    esc_html__('Question Group Question Order', 'event_espresso'),
-                    false,
-                    0
-                ),
-            ],
-        ];
-        $this->_model_relations = [
-            'Question_Group' => new EE_Belongs_To_Relation(),
-            'Question'       => new EE_Belongs_To_Relation(),
-        ];
+	protected function __construct($timezone = null)
+	{
+		$this->singular_item    = esc_html__('Question Group to Question Link', 'event_espresso');
+		$this->plural_item      = esc_html__('Question Group to Question Links', 'event_espresso');
+		$this->_tables          = [
+			'Question_Group_Question' => new EE_Primary_Table('esp_question_group_question', 'QGQ_ID'),
+		];
+		$this->_fields          = [
+			'Question_Group_Question' => [
+				'QGQ_ID'    => new EE_Primary_Key_Int_Field(
+					'QGQ_ID',
+					esc_html__('Question Group to Question Link ID', 'event_espresso')
+				),
+				'QSG_ID'    => new EE_Foreign_Key_Int_Field(
+					'QSG_ID',
+					esc_html__('Question Group ID', 'event_espresso'),
+					false,
+					0,
+					'Question_Group'
+				),
+				'QST_ID'    => new EE_Foreign_Key_Int_Field(
+					'QST_ID',
+					esc_html__('Question Id', 'event_espresso'),
+					false,
+					0,
+					'Question'
+				),
+				'QGQ_order' => new EE_Integer_Field(
+					'QGQ_order',
+					esc_html__('Question Group Question Order', 'event_espresso'),
+					false,
+					0
+				),
+			],
+		];
+		$this->_model_relations = [
+			'Question_Group' => new EE_Belongs_To_Relation(),
+			'Question'       => new EE_Belongs_To_Relation(),
+		];
 
-        $this->_model_chain_to_wp_user = 'Question_Group';
+		$this->_model_chain_to_wp_user = 'Question_Group';
 
-        // this model is generally available for reading
-        $this->_cap_restriction_generators[ EEM_Base::caps_read ]       =
-            new EE_Restriction_Generator_Public();
-        $this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
-            new EE_Restriction_Generator_Reg_Form('Question_Group.QSG_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
-            new EE_Restriction_Generator_Reg_Form('Question_Group.QSG_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
-            new EE_Restriction_Generator_Reg_Form('Question_Group.QSG_system');
+		// this model is generally available for reading
+		$this->_cap_restriction_generators[ EEM_Base::caps_read ]       =
+			new EE_Restriction_Generator_Public();
+		$this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
+			new EE_Restriction_Generator_Reg_Form('Question_Group.QSG_system');
+		$this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
+			new EE_Restriction_Generator_Reg_Form('Question_Group.QSG_system');
+		$this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
+			new EE_Restriction_Generator_Reg_Form('Question_Group.QSG_system');
 
-        // use the caps for question groups
-        $this->_caps_slug = 'question_groups';
-        parent::__construct($timezone);
-    }
+		// use the caps for question groups
+		$this->_caps_slug = 'question_groups';
+		parent::__construct($timezone);
+	}
 }

core/db_models/EEM_Question.model.php

Spacing +12 added lines, -12 removed lines

@@ -132,7 +132,7 @@ 
                 EEM_Question::QST_type_year          => esc_html__('Year', 'event_espresso'),
             ]
         );
-        $this->_question_descriptions         = apply_filters(
+        $this->_question_descriptions = apply_filters(
             'FHEE__EEM_Question__construct__question_descriptions',
             [
                 EEM_Question::QST_type_checkbox      => esc_html__(
@@ -205,7 +205,7 @@ 
                 ),
             ]
         );
-        $this->_question_type_categories      = (array) apply_filters(
+        $this->_question_type_categories = (array) apply_filters(
             'FHEE__EEM_Question__construct__question_type_categories',
             [
                 'text'               => [
@@ -326,13 +326,13 @@ 
             '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_Public();
-        $this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
+        $this->_cap_restriction_generators[EEM_Base::caps_read_admin] =
             new EE_Restriction_Generator_Reg_Form('QST_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
+        $this->_cap_restriction_generators[EEM_Base::caps_edit]       =
             new EE_Restriction_Generator_Reg_Form('QST_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
+        $this->_cap_restriction_generators[EEM_Base::caps_delete]     =
             new EE_Restriction_Generator_Reg_Form('QST_system');
 
         parent::__construct($timezone);
@@ -380,10 +380,10 @@ 
      */
     public function question_type_is_in_category(string $question_type, string $category): bool
     {
-        if (! isset($this->_question_type_categories[ $category ])) {
+        if ( ! isset($this->_question_type_categories[$category])) {
             return false;
         }
-        return in_array($question_type, $this->_question_type_categories[ $category ]);
+        return in_array($question_type, $this->_question_type_categories[$category]);
     }
 
 
@@ -395,8 +395,8 @@ 
      */
     public function question_types_in_category(string $category): array
     {
-        if (isset($this->_question_type_categories[ $category ])) {
-            return $this->_question_type_categories[ $category ];
+        if (isset($this->_question_type_categories[$category])) {
+            return $this->_question_type_categories[$category];
         }
         return [];
     }
@@ -560,8 +560,8 @@ 
     public function absolute_max_for_system_question(string $system_question_value)
     {
         $maxes = $this->system_question_maxes();
-        return isset($maxes[ $system_question_value ]) ?
-            $maxes[ $system_question_value ]
+        return isset($maxes[$system_question_value]) ?
+            $maxes[$system_question_value]
             : EE_INF;
     }
 

Indentation +573 added lines, -573 removed lines

@@ -9,578 +9,578 @@
  */
 class EEM_Question extends EEM_Soft_Delete_Base
 {
-    // constant used to indicate that the question type is CHECKBOX
-    const QST_type_checkbox = 'CHECKBOX';
+	// constant used to indicate that the question type is CHECKBOX
+	const QST_type_checkbox = 'CHECKBOX';
 
-    // constant used to indicate that the question type is COUNTRY
-    const QST_type_country = 'COUNTRY';
-
-    // constant used to indicate that the question type is DATE
-    const QST_type_date = 'DATE';
-
-    // constant used to indicate that the question type is a decimal (float)
-    const QST_type_decimal = 'DECIMAL';
-
-    // constant used to indicate that the question type is DROPDOWN
-    const QST_type_dropdown = 'DROPDOWN';
-
-    // constant used to indicate that the question type is an email input
-    const QST_type_email = 'EMAIL';
-
-    // constant used to indicate that the question type is an email input
-    const QST_type_email_confirm = 'EMAIL_CONFIRM';
-
-    // constant used to indicate that the question type is a TEXTAREA that allows simple html
-    const QST_type_html_textarea = 'HTML_TEXTAREA';
-
-    // constant used to indicate that the question type is an integer (whole number)
-    const QST_type_int = 'INTEGER';
-
-    // constant used to indicate that the question type is a multi-select
-    const QST_type_multi_select = 'MULTI_SELECT';
-
-    // constant used to indicate that the question type is RADIO_BTN
-    const QST_type_radio = 'RADIO_BTN';
-
-    // constant used to indicate that the question type is STATE
-    const QST_type_state = 'STATE';
-
-    // constant used to indicate that the question type is TEXT
-    const QST_type_text = 'TEXT';
-
-    // constant used to indicate that the question type is TEXTAREA
-    const QST_type_textarea = 'TEXTAREA';
-
-    // constant used to indicate that the question type is a valid URL
-    const QST_type_url = 'URL';
-
-    // constant used to indicate that the question type is a US-formatted phone number
-    const QST_type_us_phone = 'US_PHONE';
-
-    // constant used to indicate that the question type is a YEAR
-    const QST_type_year = 'YEAR';
-
-    /**
-     * lists all the question types which should be allowed. Ideally, this will be extensible.
-     *
-     * @var array $_allowed_question_types
-     */
-    protected $_allowed_question_types = [];
-
-
-    // private instance of the Attendee object
-    protected static $_instance = null;
-
-    /**
-     * brief descriptions for all the question types
-     *
-     * @var EEM_Question $_instance
-     */
-    protected $_question_descriptions;
-
-    /**
-     * Question types that are interchangeable, even after answers have been provided for them.
-     * Top-level keys are category slugs, next level is an array of question types. If question types
-     * aren't in this array, it is assumed they AREN'T interchangeable with any other question types.
-     *
-     * @var array   $_question_type_categories {
-     * @type string $text
-     * @type string $single                    -answer-enum
-     * @type string $multi                     -answer-enum
-     *                    }
-     */
-    protected $_question_type_categories = [];
-
-
-    /**
-     * Question types that should have an admin-defined max input length
-     *
-     * @var array
-     */
-    protected $question_types_with_max_length;
-
-
-    /**
-     * EEM_Question constructor.
-     *
-     * @param null $timezone
-     */
-    protected function __construct($timezone = null)
-    {
-        $this->singular_item                  = esc_html__('Question', 'event_espresso');
-        $this->plural_item                    = esc_html__('Questions', 'event_espresso');
-        $this->_allowed_question_types        = apply_filters(
-            'FHEE__EEM_Question__construct__allowed_question_types',
-            [
-                EEM_Question::QST_type_checkbox      => esc_html__('Checkboxes', 'event_espresso'),
-                EEM_Question::QST_type_country       => esc_html__('Country Dropdown', 'event_espresso'),
-                EEM_Question::QST_type_date          => esc_html__('Date Picker', 'event_espresso'),
-                EEM_Question::QST_type_decimal       => esc_html__('Number', 'event_espresso'),
-                EEM_Question::QST_type_dropdown      => esc_html__('Dropdown', 'event_espresso'),
-                EEM_Question::QST_type_email         => esc_html__('Email', 'event_espresso'),
-                EEM_Question::QST_type_email_confirm => esc_html__('Confirm Email', 'event_espresso'),
-                EEM_Question::QST_type_html_textarea => esc_html__('HTML Textarea', 'event_espresso'),
-                EEM_Question::QST_type_int           => esc_html__('Whole Number', 'event_espresso'),
-                EEM_Question::QST_type_multi_select  => esc_html__('Multi Select', 'event_espresso'),
-                EEM_Question::QST_type_radio         => esc_html__('Radio Buttons', 'event_espresso'),
-                EEM_Question::QST_type_state         => esc_html__('State/Province Dropdown', 'event_espresso'),
-                EEM_Question::QST_type_text          => esc_html__('Text', 'event_espresso'),
-                EEM_Question::QST_type_textarea      => esc_html__('Textarea', 'event_espresso'),
-                EEM_Question::QST_type_url           => esc_html__('URL', 'event_espresso'),
-                EEM_Question::QST_type_us_phone      => esc_html__('USA - Format Phone', 'event_espresso'),
-                EEM_Question::QST_type_year          => esc_html__('Year', 'event_espresso'),
-            ]
-        );
-        $this->_question_descriptions         = apply_filters(
-            'FHEE__EEM_Question__construct__question_descriptions',
-            [
-                EEM_Question::QST_type_checkbox      => esc_html__(
-                    'Allows multiple preset options to be selected',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_country       => esc_html__(
-                    'A dropdown that lists countries',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_date          => esc_html__(
-                    'A popup calendar that allows date selections',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_decimal       => esc_html__(
-                    'A text field that allows number values with decimals',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_dropdown      => esc_html__(
-                    'A dropdown that allows a single selection',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_email         => esc_html__(
-                    'A text field that must contain a valid Email address',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_email_confirm => esc_html__(
-                    'A text field that must contain a valid Email address and be equal to Email field',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_html_textarea => esc_html__(
-                    'A multi line text input field that allows HTML',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_int           => esc_html__(
-                    'A text field that only allows whole numbers (no decimals)',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_multi_select  => esc_html__(
-                    'A dropdown that allows multiple selections',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_radio         => esc_html__(
-                    'Allows a single preset option to be selected',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_state         => esc_html__(
-                    'A dropdown that lists states/provinces',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_text          => esc_html__(
-                    'A single line text input field',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_textarea      => esc_html__(
-                    'A multi line text input field',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_url           => esc_html__(
-                    'A text field that must contain a valid URL',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_us_phone      => esc_html__(
-                    'A text field that must contain a valid US phone number',
-                    'event_espresso'
-                ),
-                EEM_Question::QST_type_year          => esc_html__(
-                    'A dropdown that lists the last 100 years',
-                    'event_espresso'
-                ),
-            ]
-        );
-        $this->_question_type_categories      = (array) apply_filters(
-            'FHEE__EEM_Question__construct__question_type_categories',
-            [
-                'text'               => [
-                    EEM_Question::QST_type_date,
-                    EEM_Question::QST_type_decimal,
-                    EEM_Question::QST_type_email,
-                    EEM_Question::QST_type_email_confirm,
-                    EEM_Question::QST_type_html_textarea,
-                    EEM_Question::QST_type_int,
-                    EEM_Question::QST_type_text,
-                    EEM_Question::QST_type_textarea,
-                    EEM_Question::QST_type_url,
-                    EEM_Question::QST_type_us_phone,
-                    EEM_Question::QST_type_year,
-                ],
-                'single-answer-enum' => [
-                    EEM_Question::QST_type_dropdown,
-                    EEM_Question::QST_type_radio,
-                ],
-                'multi-answer-enum'  => [
-                    EEM_Question::QST_type_multi_select,
-                    EEM_Question::QST_type_checkbox,
-                ],
-            ]
-        );
-        $this->question_types_with_max_length = apply_filters(
-            'FHEE__EEM_Question___construct__question_types_with_max_length',
-            [
-                EEM_Question::QST_type_html_textarea,
-                EEM_Question::QST_type_text,
-                EEM_Question::QST_type_textarea,
-            ]
-        );
-
-        $this->_tables          = [
-            'Question' => new EE_Primary_Table('esp_question', 'QST_ID'),
-        ];
-        $this->_fields          = [
-            'Question' => [
-                'QST_ID'            => new EE_Primary_Key_Int_Field(
-                    'QST_ID',
-                    esc_html__('Question ID', 'event_espresso')
-                ),
-                'QST_admin_label'   => new EE_Plain_Text_Field(
-                    'QST_admin_label',
-                    esc_html__('Question Label (admin-only)', 'event_espresso'),
-                    true,
-                    ''
-                ),
-                'QST_admin_only'    => new EE_Boolean_Field(
-                    'QST_admin_only',
-                    esc_html__('Admin-Only Question?', 'event_espresso'),
-                    false,
-                    false
-                ),
-                'QST_deleted'       => new EE_Trashed_Flag_Field(
-                    'QST_deleted',
-                    esc_html__('Flag Indicating question was deleted', 'event_espresso'),
-                    false,
-                    false
-                ),
-                'QST_display_text'  => new EE_Post_Content_Field(
-                    'QST_display_text',
-                    esc_html__('Question Text', 'event_espresso'),
-                    true,
-                    ''
-                ),
-                'QST_max'           => new EE_Infinite_Integer_Field(
-                    'QST_max',
-                    esc_html__('Max Size', 'event_espresso'),
-                    false,
-                    EE_INF
-                ),
-                'QST_order'         => new EE_Integer_Field(
-                    'QST_order',
-                    esc_html__('Question Order', 'event_espresso'),
-                    false,
-                    0
-                ),
-                'QST_required'      => new EE_Boolean_Field(
-                    'QST_required',
-                    esc_html__('Required Question?', 'event_espresso'),
-                    false,
-                    false
-                ),
-                'QST_required_text' => new EE_Simple_HTML_Field(
-                    'QST_required_text',
-                    esc_html__('Text to Display if Not Provided', 'event_espresso'),
-                    true,
-                    ''
-                ),
-                'QST_system'        => new EE_Plain_Text_Field(
-                    'QST_system',
-                    esc_html__('Internal string ID for question', 'event_espresso'),
-                    false,
-                    ''
-                ),
-                'QST_type'          => new EE_Enum_Text_Field(
-                    'QST_type',
-                    esc_html__('Question Type', 'event_espresso'),
-                    false,
-                    'TEXT',
-                    $this->_allowed_question_types
-                ),
-                'QST_wp_user'       => new EE_WP_User_Field(
-                    'QST_wp_user',
-                    esc_html__('Question Creator ID', 'event_espresso'),
-                    false
-                ),
-            ],
-        ];
-        $this->_model_relations = [
-            'Answer'                  => new EE_Has_Many_Relation(),
-            'Question_Group'          => new EE_HABTM_Relation('Question_Group_Question'),
-            // for QST_order column
-            'Question_Group_Question' => new EE_Has_Many_Relation(),
-            'Question_Option'         => new EE_Has_Many_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_Public();
-        $this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
-            new EE_Restriction_Generator_Reg_Form('QST_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
-            new EE_Restriction_Generator_Reg_Form('QST_system');
-        $this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
-            new EE_Restriction_Generator_Reg_Form('QST_system');
-
-        parent::__construct($timezone);
-    }
-
-
-    /**
-     * Returns the list of allowed question types, which are normally:
-     * 'TEXT','TEXTAREA','RADIO_BTN','DROPDOWN','CHECKBOX','DATE' but they can be extended
-     *
-     * @return string[]
-     */
-    public function allowed_question_types(): array
-    {
-        return $this->_allowed_question_types;
-    }
-
-
-    /**
-     * Gets all the question types in the same category
-     *
-     * @param string $question_type one of EEM_Question::allowed_question_types(
-     * @return string[] like EEM_Question::allowed_question_types()
-     */
-    public function question_types_in_same_category(string $question_type): array
-    {
-        $question_types = [$question_type];
-        foreach ($this->_question_type_categories as $category => $question_types_in_category) {
-            if (in_array($question_type, $question_types_in_category)) {
-                $question_types = $question_types_in_category;
-                break;
-            }
-        }
-
-        return array_intersect_key($this->allowed_question_types(), array_flip($question_types));
-    }
-
-
-    /**
-     * Determines if the given question type is in the given question type category
-     *
-     * @param string $question_type one of EEM_Question::allowed_question_types()
-     * @param string $category      one of the top-level keys of EEM_Question::question_type_categories()
-     * @return boolean
-     */
-    public function question_type_is_in_category(string $question_type, string $category): bool
-    {
-        if (! isset($this->_question_type_categories[ $category ])) {
-            return false;
-        }
-        return in_array($question_type, $this->_question_type_categories[ $category ]);
-    }
-
-
-    /**
-     * Returns all the question types in the given category
-     *
-     * @param string $category
-     * @return array|mixed
-     */
-    public function question_types_in_category(string $category): array
-    {
-        if (isset($this->_question_type_categories[ $category ])) {
-            return $this->_question_type_categories[ $category ];
-        }
-        return [];
-    }
-
-
-    /**
-     * Returns all the question types that should have question options
-     *
-     * @return array
-     */
-    public function question_types_with_options(): array
-    {
-        return array_merge(
-            $this->question_types_in_category('single-answer-enum'),
-            $this->question_types_in_category('multi-answer-enum')
-        );
-    }
-
-
-    /**
-     * Returns the question type categories 2d array
-     *
-     * @return array see EEM_Question::_question_type_categories
-     */
-    public function question_type_categories(): array
-    {
-        return $this->_question_type_categories;
-    }
-
-
-    /**
-     * Returns an array of all the QST_system values that can be allowed in the system question group
-     * identified by $system_question_group_id
-     *
-     * @param string $system_question_group_id QSG_system
-     * @return array of system question names (QST_system)
-     */
-    public function allowed_system_questions_in_system_question_group(string $system_question_group_id): array
-    {
-        $question_system_ids = [];
-        switch ($system_question_group_id) {
-            case EEM_Question_Group::system_personal:
-                $question_system_ids = [
-                    EEM_Attendee::system_question_fname,
-                    EEM_Attendee::system_question_lname,
-                    EEM_Attendee::system_question_email,
-                    EEM_Attendee::system_question_email_confirm,
-                    EEM_Attendee::system_question_phone,
-                ];
-                break;
-            case EEM_Question_Group::system_address:
-                $question_system_ids = [
-                    EEM_Attendee::system_question_address,
-                    EEM_Attendee::system_question_address2,
-                    EEM_Attendee::system_question_city,
-                    EEM_Attendee::system_question_state,
-                    EEM_Attendee::system_question_country,
-                    EEM_Attendee::system_question_zip,
-                    EEM_Attendee::system_question_phone,
-                ];
-                break;
-        }
-        return apply_filters(
-            'FHEE__EEM_Question__system_questions_allowed_in_system_question_group__return',
-            $question_system_ids,
-            $system_question_group_id
-        );
-    }
-
-
-    /**
-     * Returns an array of all the QST_system values that are required in the system question group
-     * identified by $system_question_group_id
-     *
-     * @param string $system_question_group_id QSG_system
-     * @return array of system question names (QST_system)
-     */
-    public function required_system_questions_in_system_question_group(string $system_question_group_id): array
-    {
-        $question_system_ids = null;
-        switch ($system_question_group_id) {
-            case EEM_Question_Group::system_personal:
-                $question_system_ids = [
-                    EEM_Attendee::system_question_fname,
-                    EEM_Attendee::system_question_email,
-                ];
-                break;
-            default:
-                $question_system_ids = [];
-        }
-        return apply_filters(
-            'FHEE__EEM_Question__system_questions_required_in_system_question_group',
-            $question_system_ids,
-            $system_question_group_id
-        );
-    }
-
-
-    /**
-     * Gets an array for converting between QST_system and QST_IDs for system questions. Eg, if you want to know
-     * which system question QST_ID corresponds to the QST_system 'city', use
-     * EEM_Question::instance()->get_Question_ID_from_system_string('city');
-     *
-     * @param $QST_system
-     * @return int of QST_ID for the question that corresponds to that QST_system
-     * @throws EE_Error
-     */
-    public function get_Question_ID_from_system_string($QST_system): int
-    {
-        return $this->get_var([['QST_system' => $QST_system]]);
-    }
-
-
-    /**
-     * searches the db for the question with the latest question order and returns that value.
-     *
-     * @return int
-     * @throws EE_Error
-     */
-    public function get_latest_question_order(): int
-    {
-        $columns_to_select = [
-            'max_order' => ["MAX(QST_order)", "%d"],
-        ];
-        $max               = $this->_get_all_wpdb_results([], ARRAY_A, $columns_to_select);
-        return isset($max[0], $max[0]['max_order']) ? $max[0]['max_order'] : 0;
-    }
-
-
-    /**
-     * Returns an array where keys are system question QST_system values,
-     * and values are the highest question max the admin can set on the question
-     * (aka the "max max"; eg, a site admin can change the zip question to have a max
-     * of 5, but no larger than 12)
-     *
-     * @return array
-     */
-    public function system_question_maxes(): array
-    {
-        return [
-            'fname'         => 45,
-            'lname'         => 45,
-            'address'       => 255,
-            'address2'      => 255,
-            'city'          => 45,
-            'zip'           => 12,
-            'email'         => 255,
-            'email_confirm' => 255,
-            'phone'         => 45,
-        ];
-    }
-
-
-    /**
-     * Given a QST_system value, gets the question's largest allowable max input.
-     *
-     * @param string $system_question_value
-     * @return int|float
-     * @see Registration_Form_Admin_Page::system_question_maxes()
-     */
-    public function absolute_max_for_system_question(string $system_question_value)
-    {
-        $maxes = $this->system_question_maxes();
-        return isset($maxes[ $system_question_value ]) ?
-            $maxes[ $system_question_value ]
-            : EE_INF;
-    }
-
-
-    /**
-     * @return array
-     */
-    public function question_descriptions(): array
-    {
-        return $this->_question_descriptions;
-    }
-
-
-    /**
-     * Returns all the question types that should have an admin-defined max input length
-     *
-     * @return array
-     */
-    public function questionTypesWithMaxLength(): array
-    {
-        return (array) $this->question_types_with_max_length;
-    }
+	// constant used to indicate that the question type is COUNTRY
+	const QST_type_country = 'COUNTRY';
+
+	// constant used to indicate that the question type is DATE
+	const QST_type_date = 'DATE';
+
+	// constant used to indicate that the question type is a decimal (float)
+	const QST_type_decimal = 'DECIMAL';
+
+	// constant used to indicate that the question type is DROPDOWN
+	const QST_type_dropdown = 'DROPDOWN';
+
+	// constant used to indicate that the question type is an email input
+	const QST_type_email = 'EMAIL';
+
+	// constant used to indicate that the question type is an email input
+	const QST_type_email_confirm = 'EMAIL_CONFIRM';
+
+	// constant used to indicate that the question type is a TEXTAREA that allows simple html
+	const QST_type_html_textarea = 'HTML_TEXTAREA';
+
+	// constant used to indicate that the question type is an integer (whole number)
+	const QST_type_int = 'INTEGER';
+
+	// constant used to indicate that the question type is a multi-select
+	const QST_type_multi_select = 'MULTI_SELECT';
+
+	// constant used to indicate that the question type is RADIO_BTN
+	const QST_type_radio = 'RADIO_BTN';
+
+	// constant used to indicate that the question type is STATE
+	const QST_type_state = 'STATE';
+
+	// constant used to indicate that the question type is TEXT
+	const QST_type_text = 'TEXT';
+
+	// constant used to indicate that the question type is TEXTAREA
+	const QST_type_textarea = 'TEXTAREA';
+
+	// constant used to indicate that the question type is a valid URL
+	const QST_type_url = 'URL';
+
+	// constant used to indicate that the question type is a US-formatted phone number
+	const QST_type_us_phone = 'US_PHONE';
+
+	// constant used to indicate that the question type is a YEAR
+	const QST_type_year = 'YEAR';
+
+	/**
+	 * lists all the question types which should be allowed. Ideally, this will be extensible.
+	 *
+	 * @var array $_allowed_question_types
+	 */
+	protected $_allowed_question_types = [];
+
+
+	// private instance of the Attendee object
+	protected static $_instance = null;
+
+	/**
+	 * brief descriptions for all the question types
+	 *
+	 * @var EEM_Question $_instance
+	 */
+	protected $_question_descriptions;
+
+	/**
+	 * Question types that are interchangeable, even after answers have been provided for them.
+	 * Top-level keys are category slugs, next level is an array of question types. If question types
+	 * aren't in this array, it is assumed they AREN'T interchangeable with any other question types.
+	 *
+	 * @var array   $_question_type_categories {
+	 * @type string $text
+	 * @type string $single                    -answer-enum
+	 * @type string $multi                     -answer-enum
+	 *                    }
+	 */
+	protected $_question_type_categories = [];
+
+
+	/**
+	 * Question types that should have an admin-defined max input length
+	 *
+	 * @var array
+	 */
+	protected $question_types_with_max_length;
+
+
+	/**
+	 * EEM_Question constructor.
+	 *
+	 * @param null $timezone
+	 */
+	protected function __construct($timezone = null)
+	{
+		$this->singular_item                  = esc_html__('Question', 'event_espresso');
+		$this->plural_item                    = esc_html__('Questions', 'event_espresso');
+		$this->_allowed_question_types        = apply_filters(
+			'FHEE__EEM_Question__construct__allowed_question_types',
+			[
+				EEM_Question::QST_type_checkbox      => esc_html__('Checkboxes', 'event_espresso'),
+				EEM_Question::QST_type_country       => esc_html__('Country Dropdown', 'event_espresso'),
+				EEM_Question::QST_type_date          => esc_html__('Date Picker', 'event_espresso'),
+				EEM_Question::QST_type_decimal       => esc_html__('Number', 'event_espresso'),
+				EEM_Question::QST_type_dropdown      => esc_html__('Dropdown', 'event_espresso'),
+				EEM_Question::QST_type_email         => esc_html__('Email', 'event_espresso'),
+				EEM_Question::QST_type_email_confirm => esc_html__('Confirm Email', 'event_espresso'),
+				EEM_Question::QST_type_html_textarea => esc_html__('HTML Textarea', 'event_espresso'),
+				EEM_Question::QST_type_int           => esc_html__('Whole Number', 'event_espresso'),
+				EEM_Question::QST_type_multi_select  => esc_html__('Multi Select', 'event_espresso'),
+				EEM_Question::QST_type_radio         => esc_html__('Radio Buttons', 'event_espresso'),
+				EEM_Question::QST_type_state         => esc_html__('State/Province Dropdown', 'event_espresso'),
+				EEM_Question::QST_type_text          => esc_html__('Text', 'event_espresso'),
+				EEM_Question::QST_type_textarea      => esc_html__('Textarea', 'event_espresso'),
+				EEM_Question::QST_type_url           => esc_html__('URL', 'event_espresso'),
+				EEM_Question::QST_type_us_phone      => esc_html__('USA - Format Phone', 'event_espresso'),
+				EEM_Question::QST_type_year          => esc_html__('Year', 'event_espresso'),
+			]
+		);
+		$this->_question_descriptions         = apply_filters(
+			'FHEE__EEM_Question__construct__question_descriptions',
+			[
+				EEM_Question::QST_type_checkbox      => esc_html__(
+					'Allows multiple preset options to be selected',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_country       => esc_html__(
+					'A dropdown that lists countries',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_date          => esc_html__(
+					'A popup calendar that allows date selections',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_decimal       => esc_html__(
+					'A text field that allows number values with decimals',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_dropdown      => esc_html__(
+					'A dropdown that allows a single selection',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_email         => esc_html__(
+					'A text field that must contain a valid Email address',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_email_confirm => esc_html__(
+					'A text field that must contain a valid Email address and be equal to Email field',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_html_textarea => esc_html__(
+					'A multi line text input field that allows HTML',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_int           => esc_html__(
+					'A text field that only allows whole numbers (no decimals)',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_multi_select  => esc_html__(
+					'A dropdown that allows multiple selections',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_radio         => esc_html__(
+					'Allows a single preset option to be selected',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_state         => esc_html__(
+					'A dropdown that lists states/provinces',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_text          => esc_html__(
+					'A single line text input field',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_textarea      => esc_html__(
+					'A multi line text input field',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_url           => esc_html__(
+					'A text field that must contain a valid URL',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_us_phone      => esc_html__(
+					'A text field that must contain a valid US phone number',
+					'event_espresso'
+				),
+				EEM_Question::QST_type_year          => esc_html__(
+					'A dropdown that lists the last 100 years',
+					'event_espresso'
+				),
+			]
+		);
+		$this->_question_type_categories      = (array) apply_filters(
+			'FHEE__EEM_Question__construct__question_type_categories',
+			[
+				'text'               => [
+					EEM_Question::QST_type_date,
+					EEM_Question::QST_type_decimal,
+					EEM_Question::QST_type_email,
+					EEM_Question::QST_type_email_confirm,
+					EEM_Question::QST_type_html_textarea,
+					EEM_Question::QST_type_int,
+					EEM_Question::QST_type_text,
+					EEM_Question::QST_type_textarea,
+					EEM_Question::QST_type_url,
+					EEM_Question::QST_type_us_phone,
+					EEM_Question::QST_type_year,
+				],
+				'single-answer-enum' => [
+					EEM_Question::QST_type_dropdown,
+					EEM_Question::QST_type_radio,
+				],
+				'multi-answer-enum'  => [
+					EEM_Question::QST_type_multi_select,
+					EEM_Question::QST_type_checkbox,
+				],
+			]
+		);
+		$this->question_types_with_max_length = apply_filters(
+			'FHEE__EEM_Question___construct__question_types_with_max_length',
+			[
+				EEM_Question::QST_type_html_textarea,
+				EEM_Question::QST_type_text,
+				EEM_Question::QST_type_textarea,
+			]
+		);
+
+		$this->_tables          = [
+			'Question' => new EE_Primary_Table('esp_question', 'QST_ID'),
+		];
+		$this->_fields          = [
+			'Question' => [
+				'QST_ID'            => new EE_Primary_Key_Int_Field(
+					'QST_ID',
+					esc_html__('Question ID', 'event_espresso')
+				),
+				'QST_admin_label'   => new EE_Plain_Text_Field(
+					'QST_admin_label',
+					esc_html__('Question Label (admin-only)', 'event_espresso'),
+					true,
+					''
+				),
+				'QST_admin_only'    => new EE_Boolean_Field(
+					'QST_admin_only',
+					esc_html__('Admin-Only Question?', 'event_espresso'),
+					false,
+					false
+				),
+				'QST_deleted'       => new EE_Trashed_Flag_Field(
+					'QST_deleted',
+					esc_html__('Flag Indicating question was deleted', 'event_espresso'),
+					false,
+					false
+				),
+				'QST_display_text'  => new EE_Post_Content_Field(
+					'QST_display_text',
+					esc_html__('Question Text', 'event_espresso'),
+					true,
+					''
+				),
+				'QST_max'           => new EE_Infinite_Integer_Field(
+					'QST_max',
+					esc_html__('Max Size', 'event_espresso'),
+					false,
+					EE_INF
+				),
+				'QST_order'         => new EE_Integer_Field(
+					'QST_order',
+					esc_html__('Question Order', 'event_espresso'),
+					false,
+					0
+				),
+				'QST_required'      => new EE_Boolean_Field(
+					'QST_required',
+					esc_html__('Required Question?', 'event_espresso'),
+					false,
+					false
+				),
+				'QST_required_text' => new EE_Simple_HTML_Field(
+					'QST_required_text',
+					esc_html__('Text to Display if Not Provided', 'event_espresso'),
+					true,
+					''
+				),
+				'QST_system'        => new EE_Plain_Text_Field(
+					'QST_system',
+					esc_html__('Internal string ID for question', 'event_espresso'),
+					false,
+					''
+				),
+				'QST_type'          => new EE_Enum_Text_Field(
+					'QST_type',
+					esc_html__('Question Type', 'event_espresso'),
+					false,
+					'TEXT',
+					$this->_allowed_question_types
+				),
+				'QST_wp_user'       => new EE_WP_User_Field(
+					'QST_wp_user',
+					esc_html__('Question Creator ID', 'event_espresso'),
+					false
+				),
+			],
+		];
+		$this->_model_relations = [
+			'Answer'                  => new EE_Has_Many_Relation(),
+			'Question_Group'          => new EE_HABTM_Relation('Question_Group_Question'),
+			// for QST_order column
+			'Question_Group_Question' => new EE_Has_Many_Relation(),
+			'Question_Option'         => new EE_Has_Many_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_Public();
+		$this->_cap_restriction_generators[ EEM_Base::caps_read_admin ] =
+			new EE_Restriction_Generator_Reg_Form('QST_system');
+		$this->_cap_restriction_generators[ EEM_Base::caps_edit ]       =
+			new EE_Restriction_Generator_Reg_Form('QST_system');
+		$this->_cap_restriction_generators[ EEM_Base::caps_delete ]     =
+			new EE_Restriction_Generator_Reg_Form('QST_system');
+
+		parent::__construct($timezone);
+	}
+
+
+	/**
+	 * Returns the list of allowed question types, which are normally:
+	 * 'TEXT','TEXTAREA','RADIO_BTN','DROPDOWN','CHECKBOX','DATE' but they can be extended
+	 *
+	 * @return string[]
+	 */
+	public function allowed_question_types(): array
+	{
+		return $this->_allowed_question_types;
+	}
+
+
+	/**
+	 * Gets all the question types in the same category
+	 *
+	 * @param string $question_type one of EEM_Question::allowed_question_types(
+	 * @return string[] like EEM_Question::allowed_question_types()
+	 */
+	public function question_types_in_same_category(string $question_type): array
+	{
+		$question_types = [$question_type];
+		foreach ($this->_question_type_categories as $category => $question_types_in_category) {
+			if (in_array($question_type, $question_types_in_category)) {
+				$question_types = $question_types_in_category;
+				break;
+			}
+		}
+
+		return array_intersect_key($this->allowed_question_types(), array_flip($question_types));
+	}
+
+
+	/**
+	 * Determines if the given question type is in the given question type category
+	 *
+	 * @param string $question_type one of EEM_Question::allowed_question_types()
+	 * @param string $category      one of the top-level keys of EEM_Question::question_type_categories()
+	 * @return boolean
+	 */
+	public function question_type_is_in_category(string $question_type, string $category): bool
+	{
+		if (! isset($this->_question_type_categories[ $category ])) {
+			return false;
+		}
+		return in_array($question_type, $this->_question_type_categories[ $category ]);
+	}
+
+
+	/**
+	 * Returns all the question types in the given category
+	 *
+	 * @param string $category
+	 * @return array|mixed
+	 */
+	public function question_types_in_category(string $category): array
+	{
+		if (isset($this->_question_type_categories[ $category ])) {
+			return $this->_question_type_categories[ $category ];
+		}
+		return [];
+	}
+
+
+	/**
+	 * Returns all the question types that should have question options
+	 *
+	 * @return array
+	 */
+	public function question_types_with_options(): array
+	{
+		return array_merge(
+			$this->question_types_in_category('single-answer-enum'),
+			$this->question_types_in_category('multi-answer-enum')
+		);
+	}
+
+
+	/**
+	 * Returns the question type categories 2d array
+	 *
+	 * @return array see EEM_Question::_question_type_categories
+	 */
+	public function question_type_categories(): array
+	{
+		return $this->_question_type_categories;
+	}
+
+
+	/**
+	 * Returns an array of all the QST_system values that can be allowed in the system question group
+	 * identified by $system_question_group_id
+	 *
+	 * @param string $system_question_group_id QSG_system
+	 * @return array of system question names (QST_system)
+	 */
+	public function allowed_system_questions_in_system_question_group(string $system_question_group_id): array
+	{
+		$question_system_ids = [];
+		switch ($system_question_group_id) {
+			case EEM_Question_Group::system_personal:
+				$question_system_ids = [
+					EEM_Attendee::system_question_fname,
+					EEM_Attendee::system_question_lname,
+					EEM_Attendee::system_question_email,
+					EEM_Attendee::system_question_email_confirm,
+					EEM_Attendee::system_question_phone,
+				];
+				break;
+			case EEM_Question_Group::system_address:
+				$question_system_ids = [
+					EEM_Attendee::system_question_address,
+					EEM_Attendee::system_question_address2,
+					EEM_Attendee::system_question_city,
+					EEM_Attendee::system_question_state,
+					EEM_Attendee::system_question_country,
+					EEM_Attendee::system_question_zip,
+					EEM_Attendee::system_question_phone,
+				];
+				break;
+		}
+		return apply_filters(
+			'FHEE__EEM_Question__system_questions_allowed_in_system_question_group__return',
+			$question_system_ids,
+			$system_question_group_id
+		);
+	}
+
+
+	/**
+	 * Returns an array of all the QST_system values that are required in the system question group
+	 * identified by $system_question_group_id
+	 *
+	 * @param string $system_question_group_id QSG_system
+	 * @return array of system question names (QST_system)
+	 */
+	public function required_system_questions_in_system_question_group(string $system_question_group_id): array
+	{
+		$question_system_ids = null;
+		switch ($system_question_group_id) {
+			case EEM_Question_Group::system_personal:
+				$question_system_ids = [
+					EEM_Attendee::system_question_fname,
+					EEM_Attendee::system_question_email,
+				];
+				break;
+			default:
+				$question_system_ids = [];
+		}
+		return apply_filters(
+			'FHEE__EEM_Question__system_questions_required_in_system_question_group',
+			$question_system_ids,
+			$system_question_group_id
+		);
+	}
+
+
+	/**
+	 * Gets an array for converting between QST_system and QST_IDs for system questions. Eg, if you want to know
+	 * which system question QST_ID corresponds to the QST_system 'city', use
+	 * EEM_Question::instance()->get_Question_ID_from_system_string('city');
+	 *
+	 * @param $QST_system
+	 * @return int of QST_ID for the question that corresponds to that QST_system
+	 * @throws EE_Error
+	 */
+	public function get_Question_ID_from_system_string($QST_system): int
+	{
+		return $this->get_var([['QST_system' => $QST_system]]);
+	}
+
+
+	/**
+	 * searches the db for the question with the latest question order and returns that value.
+	 *
+	 * @return int
+	 * @throws EE_Error
+	 */
+	public function get_latest_question_order(): int
+	{
+		$columns_to_select = [
+			'max_order' => ["MAX(QST_order)", "%d"],
+		];
+		$max               = $this->_get_all_wpdb_results([], ARRAY_A, $columns_to_select);
+		return isset($max[0], $max[0]['max_order']) ? $max[0]['max_order'] : 0;
+	}
+
+
+	/**
+	 * Returns an array where keys are system question QST_system values,
+	 * and values are the highest question max the admin can set on the question
+	 * (aka the "max max"; eg, a site admin can change the zip question to have a max
+	 * of 5, but no larger than 12)
+	 *
+	 * @return array
+	 */
+	public function system_question_maxes(): array
+	{
+		return [
+			'fname'         => 45,
+			'lname'         => 45,
+			'address'       => 255,
+			'address2'      => 255,
+			'city'          => 45,
+			'zip'           => 12,
+			'email'         => 255,
+			'email_confirm' => 255,
+			'phone'         => 45,
+		];
+	}
+
+
+	/**
+	 * Given a QST_system value, gets the question's largest allowable max input.
+	 *
+	 * @param string $system_question_value
+	 * @return int|float
+	 * @see Registration_Form_Admin_Page::system_question_maxes()
+	 */
+	public function absolute_max_for_system_question(string $system_question_value)
+	{
+		$maxes = $this->system_question_maxes();
+		return isset($maxes[ $system_question_value ]) ?
+			$maxes[ $system_question_value ]
+			: EE_INF;
+	}
+
+
+	/**
+	 * @return array
+	 */
+	public function question_descriptions(): array
+	{
+		return $this->_question_descriptions;
+	}
+
+
+	/**
+	 * Returns all the question types that should have an admin-defined max input length
+	 *
+	 * @return array
+	 */
+	public function questionTypesWithMaxLength(): array
+	{
+		return (array) $this->question_types_with_max_length;
+	}
 }