TicketMutation::addDefaultPrices() - Code Metrics - Inspection of "Daily Inspection: Refactor RelationsSelector and u..." - eventespresso/event-espresso-core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — EDTR/master (cfb9e9)

unknown

created 2019-12-03 21:08 UTC

TicketMutation::addDefaultPrices() A

↳ Parent: TicketMutation

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
nc	2
nop	2
dl	0
loc	9
rs	9.9666
c	0
b	0
f	0

<?php

namespace EventEspresso\core\domain\services\graphql\data\mutations;

use GraphQLRelay\Relay;
use DateTime;
use EEM_Price;
use EEM_Ticket;
use EE_Ticket;
use EventEspresso\core\domain\services\assets\EspressoEditorAssetManager;
use EventEspresso\core\domain\services\converters\RestApiSpoofer;
use EventEspresso\core\exceptions\InvalidDataTypeException;
use EventEspresso\core\exceptions\InvalidInterfaceException;
use EventEspresso\core\exceptions\ModelConfigurationException;
use EventEspresso\core\exceptions\RestPasswordIncorrectException;
use EventEspresso\core\exceptions\RestPasswordRequiredException;
use EventEspresso\core\exceptions\UnexpectedEntityException;
use EventEspresso\core\libraries\rest_api\RestException;
use InvalidArgumentException;
use ReflectionException;

/**
 * Class TicketMutation
 *
 * @package       Event Espresso
 * @author        Manzoor Wani
 */
class TicketMutation
{

    /**
     * Maps the GraphQL input to a format that the model functions can use
     *
     * @param array $input Data coming from the GraphQL mutation query input
     * @return array
     */
    public static function prepareFields(array $input)
    {

        $args = [];

        if (! empty($input['name'])) {
            $args['TKT_name'] = sanitize_text_field($input['name']);
        }

        if (! empty($input['description'])) {
            $args['TKT_description'] = sanitize_text_field($input['description']);
        }

        if (! empty($input['price'])) {
            $args['TKT_price'] = floatval($input['price']);
        }

        if (! empty($input['startDate'])) {
            $args['TKT_start_date'] = new DateTime(sanitize_text_field($input['startDate']));
        }

        if (! empty($input['endDate'])) {
            $args['TKT_end_date'] = new DateTime(sanitize_text_field($input['endDate']));
        }

        if (! empty($input['datetimes'])) {
            $args['datetimes'] = array_map('sanitize_text_field', (array) $input['datetimes']);
        }

        if (! empty($input['prices'])) {
            $args['prices'] = array_map('sanitize_text_field', (array) $input['prices']);
        }

        // Likewise the other fields...

        return $args;
    }

    /**
     * Sets the related datetimes for the given ticket.
     *
     * @param EE_Ticket $entity    The Ticket instance.
     * @param array     $datetimes Array of datetime IDs to relate.
     */
    public static function setRelatedDatetimes($entity, array $datetimes)
    {
        $relationName = 'Datetime';
        // Remove all the existing related datetimes

        $entity->_remove_relations($relationName);
        // @todo replace loop with single query
        foreach ($datetimes as $ID) {
            $parts = Relay::fromGlobalId($ID);
            if (! empty($parts['id']) && absint($parts['id'])) {
                $entity->_add_relation_to(
                    $parts['id'],
                    $relationName
                );
            }
        }
    }

    /**
     * Sets the related prices for the given ticket.
     *
     * @param EE_Ticket $entity The Ticket instance.
     * @param array     $prices Array of entity IDs to relate.
     */
    public static function setRelatedPrices($entity, array $prices)
    {
        $relationName = 'Price';
        // Remove all the existing related entities
        $entity->_remove_relations($relationName);

        // @todo replace loop with single query
        foreach ($prices as $ID) {
            $parts = Relay::fromGlobalId($ID);
            if (! empty($parts['id']) && absint($parts['id'])) {
                $entity->_add_relation_to(
                    $parts['id'],
                    $relationName
                );
            }
        }
    }


    /**
     * @param EE_Ticket $ticket_entity
     * @param EEM_Ticket $ticket_model
     * @throws DomainException
     * @throws EE_Error
     * @throws InvalidArgumentException
     * @throws InvalidDataTypeException
     * @throws InvalidInterfaceException
     * @throws ModelConfigurationException
     * @throws ReflectionException
     * @throws RestException
     * @throws UnexpectedEntityException
     * @since $VID:$
     */
    public static function addDefaultPrices(EE_Ticket $ticket_entity, EEM_Ticket $ticket_model)
    {
        $price_model = EEM_Price::instance();
        $default_prices = $price_model->get_all_default_prices();
        foreach ($default_prices as $default_price) {
$collection = json_decode($data, true);
if ( ! is_array($collection)) {
    throw new \RuntimeException('$collection must be an array.');
}

foreach ($collection as $item) { /** ... */ }
            $default_price->save();
            $default_price->_add_relation_to($ticket_entity, 'Ticket');
        }
    }
}


1		<?php
2
3		namespace EventEspresso\core\domain\services\graphql\data\mutations;
4
5		use GraphQLRelay\Relay;
6		use DateTime;
7		use EEM_Price;
8		use EEM_Ticket;
9		use EE_Ticket;
10		use EventEspresso\core\domain\services\assets\EspressoEditorAssetManager;
11		use EventEspresso\core\domain\services\converters\RestApiSpoofer;
12		use EventEspresso\core\exceptions\InvalidDataTypeException;
13		use EventEspresso\core\exceptions\InvalidInterfaceException;
14		use EventEspresso\core\exceptions\ModelConfigurationException;
15		use EventEspresso\core\exceptions\RestPasswordIncorrectException;
16		use EventEspresso\core\exceptions\RestPasswordRequiredException;
17		use EventEspresso\core\exceptions\UnexpectedEntityException;
18		use EventEspresso\core\libraries\rest_api\RestException;
19		use InvalidArgumentException;
20		use ReflectionException;
21
22		/**
23		* Class TicketMutation
24		*
25		* @package Event Espresso
26		* @author Manzoor Wani
27		*/
28		class TicketMutation
29		{
30
31		/**
32		* Maps the GraphQL input to a format that the model functions can use
33		*
34		* @param array $input Data coming from the GraphQL mutation query input
35		* @return array
36		*/
37		public static function prepareFields(array $input)
38		{
39
40		$args = [];
41
42		if (! empty($input['name'])) {
43		$args['TKT_name'] = sanitize_text_field($input['name']);
44		}
45
46		if (! empty($input['description'])) {
47		$args['TKT_description'] = sanitize_text_field($input['description']);
48		}
49
50		if (! empty($input['price'])) {
51		$args['TKT_price'] = floatval($input['price']);
52		}
53
54	View Code Duplication	if (! empty($input['startDate'])) {
55		$args['TKT_start_date'] = new DateTime(sanitize_text_field($input['startDate']));
56		}
57
58	View Code Duplication	if (! empty($input['endDate'])) {
59		$args['TKT_end_date'] = new DateTime(sanitize_text_field($input['endDate']));
60		}
61
62	View Code Duplication	if (! empty($input['datetimes'])) {
63		$args['datetimes'] = array_map('sanitize_text_field', (array) $input['datetimes']);
64		}
65
66	View Code Duplication	if (! empty($input['prices'])) {
67		$args['prices'] = array_map('sanitize_text_field', (array) $input['prices']);
68		}
69
70		// Likewise the other fields...
71
72		return $args;
73		}
74
75		/**
76		* Sets the related datetimes for the given ticket.
77		*
78		* @param EE_Ticket $entity The Ticket instance.
79		* @param array $datetimes Array of datetime IDs to relate.
80		*/
81	View Code Duplication	public static function setRelatedDatetimes($entity, array $datetimes)
82		{
83		$relationName = 'Datetime';
84		// Remove all the existing related datetimes
85
86		$entity->_remove_relations($relationName);
87		// @todo replace loop with single query
88		foreach ($datetimes as $ID) {
89		$parts = Relay::fromGlobalId($ID);
90		if (! empty($parts['id']) && absint($parts['id'])) {
91		$entity->_add_relation_to(
92		$parts['id'],
93		$relationName
94		);
95		}
96		}
97		}
98
99		/**
100		* Sets the related prices for the given ticket.
101		*
102		* @param EE_Ticket $entity The Ticket instance.
103		* @param array $prices Array of entity IDs to relate.
104		*/
105	View Code Duplication	public static function setRelatedPrices($entity, array $prices)
106		{
107		$relationName = 'Price';
108		// Remove all the existing related entities
109		$entity->_remove_relations($relationName);
110
111		// @todo replace loop with single query
112		foreach ($prices as $ID) {
113		$parts = Relay::fromGlobalId($ID);
114		if (! empty($parts['id']) && absint($parts['id'])) {
115		$entity->_add_relation_to(
116		$parts['id'],
117		$relationName
118		);
119		}
120		}
121		}
122
123
124		/**
125		* @param EE_Ticket $ticket_entity
126		* @param EEM_Ticket $ticket_model
127		* @throws DomainException
128		* @throws EE_Error
129		* @throws InvalidArgumentException
130		* @throws InvalidDataTypeException
131		* @throws InvalidInterfaceException
132		* @throws ModelConfigurationException
133		* @throws ReflectionException
134		* @throws RestException
135		* @throws UnexpectedEntityException
136		* @since $VID:$
137		*/
138		public static function addDefaultPrices(EE_Ticket $ticket_entity, EEM_Ticket $ticket_model)
139		{
140		$price_model = EEM_Price::instance();
141		$default_prices = $price_model->get_all_default_prices();
142		foreach ($default_prices as $default_price) {
		0 ignored issues – show Bug introduced 2019-12-03 21:26 UTC by Report Bug Copy Issue Report The expression `$default_prices` of type `integer\|array<integer,object<EE_Base_Class>>` is not guaranteed to be traversable. How about adding an additional type check? There are different options of fixing this problem. If you want to be on the safe side, you can add an additional type-check: $collection = json_decode($data, true); if ( ! is_array($collection)) { throw new \RuntimeException('$collection must be an array.'); } foreach ($collection as $item) { /** ... / } If you are sure that the expression is traversable, you might want to add a doc comment cast* to improve IDE auto-completion and static analysis: /** @var array $collection / $collection = json_decode($data, true); foreach ($collection as $item) { /* .. / } Mark the issue as a false-positive*: Just hover the remove button, in the top-right corner of this issue for more options. Loading history...
143		$default_price->save();
144		$default_price->_add_relation_to($ticket_entity, 'Ticket');
145		}
146		}
147		}
148

eventespresso / event-espresso-core

Branch — EDTR/master (cfb9e9)

TicketMutation::addDefaultPrices() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like