eventespresso / event-espresso-core

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

Indentation +182 added lines, -182 removed lines

@@ -18,186 +18,186 @@
  */
 class DatetimeConnectionResolver extends AbstractConnectionResolver
 {
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_loader_name()
-    {
-        return 'espresso_datetime';
-    }
-
-    /**
-     * @return EEM_Datetime
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_query()
-    {
-        return EEM_Datetime::instance();
-    }
-
-    /**
-     * Return an array of item IDs from the query
-     *
-     * @return array
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_ids()
-    {
-        $results = $this->query->get_col($this->query_args);
-
-        return ! empty($results) ? $results : [];
-    }
-
-    /**
-     * Determine whether the Query should execute. If it's determined that the query should
-     * not be run based on context such as, but not limited to, who the user is, where in the
-     * ResolveTree the Query is, the relation to the node the Query is connected to, etc
-     *
-     * Return false to prevent the query from executing.
-     *
-     * @return bool
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function should_execute()
-    {
-        if (false === $this->should_execute) {
-            return false;
-        }
-
-        return $this->should_execute;
-    }
-
-    /**
-     * 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 InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_query_args()
-    {
-        $where_params = ['DTT_deleted' => ['IN', [true, false]]];
-        $query_args   = [];
-
-        $query_args['limit'] = $this->getLimit();
-
-        // Avoid multiple entries by join.
-        $query_args['group_by'] = 'DTT_ID';
-
-        /**
-         * 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['EVT_ID']) && is_array($input_fields['EVT_ID'])) {
-                $input_fields['EVT_ID'] = ['in', $input_fields['EVT_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']];
-            }
-        }
-
-        /**
-         * Determine where we're at in the Graph and adjust the query context appropriately.
-         *
-         * For example, if we're querying for datetime as a field of event query, this will automatically
-         * set the query to pull datetimes that belong to that event.
-         * We can set more cases for other source types.
-         */
-        if (is_object($this->source)) {
-            switch (true) {
-                // It's surely an event
-                case $this->source instanceof Post:
-                    $where_params['EVT_ID'] = $this->source->ID;
-                    break;
-                case $this->source instanceof EE_Event:
-                    $where_params['EVT_ID'] = $this->source->ID();
-                    break;
-                case $this->source instanceof EE_Ticket:
-                    $where_params['Ticket.TKT_ID'] = $this->source->ID();
-                    break;
-                case $this->source instanceof EE_Checkin:
-                    $where_params['Checkin.CHK_ID'] = $this->source->ID();
-                    break;
-            }
-        }
-
-        /**
-         * Merge the input_fields with the default query_args
-         */
-        if (! empty($input_fields)) {
-            $where_params = array_merge($where_params, $input_fields);
-        }
-
-        list($query_args, $where_params) = $this->mapOrderbyInputArgs($query_args, $where_params, 'DTT_ID');
-
-        if (! empty($this->args['where']['upcoming'])) {
-            $where_params['DTT_EVT_start'] = array(
-                '>',
-                EEM_Datetime::instance()->current_time_for_query('DTT_EVT_start')
-            );
-        }
-
-        if (! empty($this->args['where']['active'])) {
-            $where_params['DTT_EVT_start'] = array(
-                '<',
-                EEM_Datetime::instance()->current_time_for_query('DTT_EVT_start')
-            );
-            $where_params['DTT_EVT_end'] = array(
-                '>',
-                EEM_Datetime::instance()->current_time_for_query('DTT_EVT_end')
-            );
-        }
-
-        if (! empty($this->args['where']['expired'])) {
-            $where_params['DTT_EVT_end'] = array(
-                '<',
-                EEM_Datetime::instance()->current_time_for_query('DTT_EVT_end')
-            );
-        }
-
-        $query_args[] = $where_params;
-
-        /**
-         * Return the $query_args
-         */
-        return $query_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)
-    {
-        $arg_mapping = [
-            'event'      => 'EVT_ID',
-            'eventIn'    => 'EVT_ID',
-            'eventId'    => 'EVT_ID',
-            'eventIdIn'  => 'EVT_ID',
-            'ticket'     => 'Ticket.TKT_ID',
-            'ticketIn'   => 'Ticket.TKT_ID',
-            'ticketId'   => 'Ticket.TKT_ID',
-            'ticketIdIn' => 'Ticket.TKT_ID',
-        ];
-        return $this->sanitizeWhereArgsForInputFields(
-            $where_args,
-            $arg_mapping,
-            ['event', 'eventIn', 'ticket', 'ticketIn']
-        );
-    }
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_loader_name()
+	{
+		return 'espresso_datetime';
+	}
+
+	/**
+	 * @return EEM_Datetime
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_query()
+	{
+		return EEM_Datetime::instance();
+	}
+
+	/**
+	 * Return an array of item IDs from the query
+	 *
+	 * @return array
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_ids()
+	{
+		$results = $this->query->get_col($this->query_args);
+
+		return ! empty($results) ? $results : [];
+	}
+
+	/**
+	 * Determine whether the Query should execute. If it's determined that the query should
+	 * not be run based on context such as, but not limited to, who the user is, where in the
+	 * ResolveTree the Query is, the relation to the node the Query is connected to, etc
+	 *
+	 * Return false to prevent the query from executing.
+	 *
+	 * @return bool
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function should_execute()
+	{
+		if (false === $this->should_execute) {
+			return false;
+		}
+
+		return $this->should_execute;
+	}
+
+	/**
+	 * 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 InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_query_args()
+	{
+		$where_params = ['DTT_deleted' => ['IN', [true, false]]];
+		$query_args   = [];
+
+		$query_args['limit'] = $this->getLimit();
+
+		// Avoid multiple entries by join.
+		$query_args['group_by'] = 'DTT_ID';
+
+		/**
+		 * 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['EVT_ID']) && is_array($input_fields['EVT_ID'])) {
+				$input_fields['EVT_ID'] = ['in', $input_fields['EVT_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']];
+			}
+		}
+
+		/**
+		 * Determine where we're at in the Graph and adjust the query context appropriately.
+		 *
+		 * For example, if we're querying for datetime as a field of event query, this will automatically
+		 * set the query to pull datetimes that belong to that event.
+		 * We can set more cases for other source types.
+		 */
+		if (is_object($this->source)) {
+			switch (true) {
+				// It's surely an event
+				case $this->source instanceof Post:
+					$where_params['EVT_ID'] = $this->source->ID;
+					break;
+				case $this->source instanceof EE_Event:
+					$where_params['EVT_ID'] = $this->source->ID();
+					break;
+				case $this->source instanceof EE_Ticket:
+					$where_params['Ticket.TKT_ID'] = $this->source->ID();
+					break;
+				case $this->source instanceof EE_Checkin:
+					$where_params['Checkin.CHK_ID'] = $this->source->ID();
+					break;
+			}
+		}
+
+		/**
+		 * Merge the input_fields with the default query_args
+		 */
+		if (! empty($input_fields)) {
+			$where_params = array_merge($where_params, $input_fields);
+		}
+
+		list($query_args, $where_params) = $this->mapOrderbyInputArgs($query_args, $where_params, 'DTT_ID');
+
+		if (! empty($this->args['where']['upcoming'])) {
+			$where_params['DTT_EVT_start'] = array(
+				'>',
+				EEM_Datetime::instance()->current_time_for_query('DTT_EVT_start')
+			);
+		}
+
+		if (! empty($this->args['where']['active'])) {
+			$where_params['DTT_EVT_start'] = array(
+				'<',
+				EEM_Datetime::instance()->current_time_for_query('DTT_EVT_start')
+			);
+			$where_params['DTT_EVT_end'] = array(
+				'>',
+				EEM_Datetime::instance()->current_time_for_query('DTT_EVT_end')
+			);
+		}
+
+		if (! empty($this->args['where']['expired'])) {
+			$where_params['DTT_EVT_end'] = array(
+				'<',
+				EEM_Datetime::instance()->current_time_for_query('DTT_EVT_end')
+			);
+		}
+
+		$query_args[] = $where_params;
+
+		/**
+		 * Return the $query_args
+		 */
+		return $query_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)
+	{
+		$arg_mapping = [
+			'event'      => 'EVT_ID',
+			'eventIn'    => 'EVT_ID',
+			'eventId'    => 'EVT_ID',
+			'eventIdIn'  => 'EVT_ID',
+			'ticket'     => 'Ticket.TKT_ID',
+			'ticketIn'   => 'Ticket.TKT_ID',
+			'ticketId'   => 'Ticket.TKT_ID',
+			'ticketIdIn' => 'Ticket.TKT_ID',
+		];
+		return $this->sanitizeWhereArgsForInputFields(
+			$where_args,
+			$arg_mapping,
+			['event', 'eventIn', 'ticket', 'ticketIn']
+		);
+	}
 }

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

Indentation +69 added lines, -69 removed lines

@@ -14,85 +14,85 @@
  */
 class PriceTypeConnectionResolver extends AbstractConnectionResolver
 {
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_loader_name()
-    {
-        return 'espresso_priceType';
-    }
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_loader_name()
+	{
+		return 'espresso_priceType';
+	}
 
-    /**
-     * @return EEM_Price_Type
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_query()
-    {
-        return EEM_Price_Type::instance();
-    }
+	/**
+	 * @return EEM_Price_Type
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_query()
+	{
+		return EEM_Price_Type::instance();
+	}
 
 
-    /**
-     * Return an array of item IDs from the query
-     *
-     * @return array
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_ids()
-    {
-        $results = $this->query->get_col($this->query_args);
+	/**
+	 * Return an array of item IDs from the query
+	 *
+	 * @return array
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_ids()
+	{
+		$results = $this->query->get_col($this->query_args);
 
-        return ! empty($results) ? $results : [];
-    }
+		return ! empty($results) ? $results : [];
+	}
 
 
-    /**
-     * Determine whether the Query should execute. If it's determined that the query should
-     * not be run based on context such as, but not limited to, who the user is, where in the
-     * ResolveTree the Query is, the relation to the node the Query is connected to, etc
-     * Return false to prevent the query from executing.
-     *
-     * @return bool
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function should_execute()
-    {
-        if ($this->should_execute === false) {
-            return false;
-        }
+	/**
+	 * Determine whether the Query should execute. If it's determined that the query should
+	 * not be run based on context such as, but not limited to, who the user is, where in the
+	 * ResolveTree the Query is, the relation to the node the Query is connected to, etc
+	 * Return false to prevent the query from executing.
+	 *
+	 * @return bool
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function should_execute()
+	{
+		if ($this->should_execute === false) {
+			return false;
+		}
 
-        return $this->should_execute;
-    }
+		return $this->should_execute;
+	}
 
 
-    /**
-     * 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()
-    {
-        $where_params = [];
-        $query_args   = [];
+	/**
+	 * 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()
+	{
+		$where_params = [];
+		$query_args   = [];
 
-        $query_args['limit'] = $this->getLimit();
+		$query_args['limit'] = $this->getLimit();
 
-        // Avoid multiple entries by join.
-        $query_args['group_by'] = 'PRT_ID';
+		// Avoid multiple entries by join.
+		$query_args['group_by'] = 'PRT_ID';
 
-        /**
-         * Return the $query_args
-         */
-        return $query_args;
-    }
+		/**
+		 * Return the $query_args
+		 */
+		return $query_args;
+	}
 }

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

Indentation +141 added lines, -141 removed lines

@@ -16,145 +16,145 @@
  */
 class VenueConnectionResolver extends AbstractConnectionResolver
 {
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_loader_name()
-    {
-        return 'espresso_venue';
-    }
-
-    /**
-     * @return EEM_Venue
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_query()
-    {
-        return EEM_Venue::instance();
-    }
-
-
-    /**
-     * Return an array of item IDs from the query
-     *
-     * @return array
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_ids()
-    {
-        $results = $this->query->get_col($this->query_args);
-
-        return ! empty($results) ? $results : [];
-    }
-
-
-    /**
-     * Determine whether the Query should execute. If it's determined that the query should
-     * not be run based on context such as, but not limited to, who the user is, where in the
-     * ResolveTree the Query is, the relation to the node the Query is connected to, etc
-     * Return false to prevent the query from executing.
-     *
-     * @return bool
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function should_execute()
-    {
-        if (false === $this->should_execute) {
-            return false;
-        }
-
-        return $this->should_execute;
-    }
-
-
-    /**
-     * 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
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function get_query_args()
-    {
-        $query_args = [];
-
-        /**
-         * Prepare for later use
-         */
-        $last = ! empty($this->args['last']) ? $this->args['last'] : null;
-        $first = ! empty($this->args['first']) ? $this->args['first'] : null;
-
-        /**
-         * Set limit the highest value of $first and $last, with a (filterable) max of 100
-         */
-        $query_args['limit'] = min(
-            max(absint($first), absint($last), 10),
-            $this->query_amount
-        ) + 1;
-
-        /**
-         * 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->sanitize_input_fields($this->args['where']);
-        }
-
-        /**
-         * Determine where we're at in the Graph and adjust the query context appropriately.
-         * For example, if we're querying for datetime as a field of event query, this will automatically
-         * set the query to pull datetimes that belong to that event.
-         * We can set more cases for other source types.
-         */
-        if (is_object($this->source)) {
-            switch (true) {
-                // Assumed to be an event
-                case $this->source instanceof Post:
-                    $query_args[] = ['Event.EVT_ID' => $this->source->ID];
-                    break;
-                case $this->source instanceof EE_Event:
-                    $query_args[] = ['Event.EVT_ID' => $this->source->ID()];
-                    break;
-            }
-        }
-
-        /**
-         * Merge the input_fields with the default query_args
-         */
-        if (! empty($input_fields)) {
-            $query_args = array_merge($query_args, $input_fields);
-        }
-
-        /**
-         * Return the $query_args
-         */
-        return $query_args;
-    }
-
-
-    /**
-     * This sets up the "allowed" args, and translates the GraphQL-friendly keys to WP_Query
-     * friendly keys. There's probably a cleaner/more dynamic way to approach this, but
-     * this was quick. I'd be down to explore more dynamic ways to map this, but for
-     * now this gets the job done.
-     *
-     * @param array $query_args
-     * @return array
-     */
-    // phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
-    public function sanitize_input_fields(array $query_args)
-    {
-        $arg_mapping = [
-            'orderBy' => 'order_by',
-            'order'   => 'order',
-        ];
-
-        /**
-         * Return the Query Args
-         */
-        return ! empty($query_args) && is_array($query_args) ? $query_args : [];
-    }
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_loader_name()
+	{
+		return 'espresso_venue';
+	}
+
+	/**
+	 * @return EEM_Venue
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_query()
+	{
+		return EEM_Venue::instance();
+	}
+
+
+	/**
+	 * Return an array of item IDs from the query
+	 *
+	 * @return array
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_ids()
+	{
+		$results = $this->query->get_col($this->query_args);
+
+		return ! empty($results) ? $results : [];
+	}
+
+
+	/**
+	 * Determine whether the Query should execute. If it's determined that the query should
+	 * not be run based on context such as, but not limited to, who the user is, where in the
+	 * ResolveTree the Query is, the relation to the node the Query is connected to, etc
+	 * Return false to prevent the query from executing.
+	 *
+	 * @return bool
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function should_execute()
+	{
+		if (false === $this->should_execute) {
+			return false;
+		}
+
+		return $this->should_execute;
+	}
+
+
+	/**
+	 * 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
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function get_query_args()
+	{
+		$query_args = [];
+
+		/**
+		 * Prepare for later use
+		 */
+		$last = ! empty($this->args['last']) ? $this->args['last'] : null;
+		$first = ! empty($this->args['first']) ? $this->args['first'] : null;
+
+		/**
+		 * Set limit the highest value of $first and $last, with a (filterable) max of 100
+		 */
+		$query_args['limit'] = min(
+			max(absint($first), absint($last), 10),
+			$this->query_amount
+		) + 1;
+
+		/**
+		 * 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->sanitize_input_fields($this->args['where']);
+		}
+
+		/**
+		 * Determine where we're at in the Graph and adjust the query context appropriately.
+		 * For example, if we're querying for datetime as a field of event query, this will automatically
+		 * set the query to pull datetimes that belong to that event.
+		 * We can set more cases for other source types.
+		 */
+		if (is_object($this->source)) {
+			switch (true) {
+				// Assumed to be an event
+				case $this->source instanceof Post:
+					$query_args[] = ['Event.EVT_ID' => $this->source->ID];
+					break;
+				case $this->source instanceof EE_Event:
+					$query_args[] = ['Event.EVT_ID' => $this->source->ID()];
+					break;
+			}
+		}
+
+		/**
+		 * Merge the input_fields with the default query_args
+		 */
+		if (! empty($input_fields)) {
+			$query_args = array_merge($query_args, $input_fields);
+		}
+
+		/**
+		 * Return the $query_args
+		 */
+		return $query_args;
+	}
+
+
+	/**
+	 * This sets up the "allowed" args, and translates the GraphQL-friendly keys to WP_Query
+	 * friendly keys. There's probably a cleaner/more dynamic way to approach this, but
+	 * this was quick. I'd be down to explore more dynamic ways to map this, but for
+	 * now this gets the job done.
+	 *
+	 * @param array $query_args
+	 * @return array
+	 */
+	// phpcs:ignore PSR1.Methods.CamelCapsMethodName.NotCamelCaps
+	public function sanitize_input_fields(array $query_args)
+	{
+		$arg_mapping = [
+			'orderBy' => 'order_by',
+			'order'   => 'order',
+		];
+
+		/**
+		 * Return the Query Args
+		 */
+		return ! empty($query_args) && is_array($query_args) ? $query_args : [];
+	}
 }

core/domain/services/graphql/data/loaders/AbstractLoader.php

Indentation +59 added lines, -59 removed lines

@@ -9,70 +9,70 @@
  */
 abstract class AbstractLoader extends AbstractDataLoader
 {
-    protected $primaryKey;
+	protected $primaryKey;
 
-    /**
-     * @return EE_Base_Class
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    abstract protected function getQuery();
+	/**
+	 * @return EE_Base_Class
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	abstract protected function getQuery();
 
-    /**
-     * @param array $keys
-     *
-     * @return array
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    abstract protected function getWhereParams(array $keys);
+	/**
+	 * @param array $keys
+	 *
+	 * @return array
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	abstract protected function getWhereParams(array $keys);
 
-    /**
-     * Given array of keys, loads and returns a map consisting of keys from `keys` array and loaded
-     * values
-     *
-     * Note that order of returned values must match exactly the order of keys.
-     * If some entry is not available for given key - it must include null for the missing key.
-     *
-     * For example:
-     * loadKeys(['a', 'b', 'c']) -> ['a' => 'value1, 'b' => null, 'c' => 'value3']
-     *
-     * @param array $keys
-     *
-     * @return array
-     * @throws \Exception
-     */
-    public function loadKeys(array $keys)
-    {
-        if (empty($keys)) {
-            return $keys;
-        }
+	/**
+	 * Given array of keys, loads and returns a map consisting of keys from `keys` array and loaded
+	 * values
+	 *
+	 * Note that order of returned values must match exactly the order of keys.
+	 * If some entry is not available for given key - it must include null for the missing key.
+	 *
+	 * For example:
+	 * loadKeys(['a', 'b', 'c']) -> ['a' => 'value1, 'b' => null, 'c' => 'value3']
+	 *
+	 * @param array $keys
+	 *
+	 * @return array
+	 * @throws \Exception
+	 */
+	public function loadKeys(array $keys)
+	{
+		if (empty($keys)) {
+			return $keys;
+		}
 
-        $args = [
-            $this->getWhereParams($keys),
-        ];
+		$args = [
+			$this->getWhereParams($keys),
+		];
 
-        $query = $this->getQuery();
-        $result = $query->get_all($args);
+		$query = $this->getQuery();
+		$result = $query->get_all($args);
         
-        $loadedItems = [];
+		$loadedItems = [];
 
-        /**
-         * Loop over the items and return an array of loaded items,
-         * where the key is the ID and the value is the Object passed through
-         * the model layer.
-         */
-        foreach ($keys as $key) {
-            if (isset($result[ $key ])) {
-                $loadedItems[ $key ] = $result[ $key ];
-            } else {
-                $loadedItems[ $key ] = null;
-            }
-        }
-        return $loadedItems;
-    }
+		/**
+		 * Loop over the items and return an array of loaded items,
+		 * where the key is the ID and the value is the Object passed through
+		 * the model layer.
+		 */
+		foreach ($keys as $key) {
+			if (isset($result[ $key ])) {
+				$loadedItems[ $key ] = $result[ $key ];
+			} else {
+				$loadedItems[ $key ] = null;
+			}
+		}
+		return $loadedItems;
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -67,10 +67,10 @@
          * the model layer.
          */
         foreach ($keys as $key) {
-            if (isset($result[ $key ])) {
-                $loadedItems[ $key ] = $result[ $key ];
+            if (isset($result[$key])) {
+                $loadedItems[$key] = $result[$key];
             } else {
-                $loadedItems[ $key ] = null;
+                $loadedItems[$key] = null;
             }
         }
         return $loadedItems;

core/domain/services/graphql/data/loaders/PriceTypeLoader.php

Indentation +26 added lines, -26 removed lines

@@ -12,31 +12,31 @@
  */
 class PriceTypeLoader extends AbstractLoader
 {
-    /**
-     * @return EE_Base_Class
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    protected function getQuery()
-    {
-        return EEM_PriceType::instance();
-    }
+	/**
+	 * @return EE_Base_Class
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	protected function getQuery()
+	{
+		return EEM_PriceType::instance();
+	}
 
-    /**
-     * @param array $keys
-     *
-     * @return array
-     * @throws EE_Error
-     * @throws InvalidArgumentException
-     * @throws InvalidDataTypeException
-     * @throws InvalidInterfaceException
-     */
-    protected function getWhereParams(array $keys)
-    {
-        return [
-            'PRT_ID' => ['IN', $keys],
-        ];
-    }
+	/**
+	 * @param array $keys
+	 *
+	 * @return array
+	 * @throws EE_Error
+	 * @throws InvalidArgumentException
+	 * @throws InvalidDataTypeException
+	 * @throws InvalidInterfaceException
+	 */
+	protected function getWhereParams(array $keys)
+	{
+		return [
+			'PRT_ID' => ['IN', $keys],
+		];
+	}
 }