UserHydrator::__construct() - Code Metrics - Inspection of "Merge pull request #13 from alexz707/UserHydrator" - LM-Commons/LmcUser - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 45e13e...d296b3 )

by Mat

created 2021-01-19 11:24 UTC

UserHydrator::__construct() A

↳ Parent: UserHydrator

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	1

<?php

namespace LmcUser\Mapper;

use Laminas\Hydrator\HydratorInterface;
use LmcUser\Entity\UserInterface as UserEntityInterface;
use LmcUser\Mapper\Exception\InvalidArgumentException;

/**
 * Class UserHydrator
 */
class UserHydrator implements HydratorInterface
{
    /** @var HydratorInterface */
    private $hydrator;

    /**
     * UserHydrator constructor.
     *
     * @param HydratorInterface $hydrator
     */
    public function __construct(HydratorInterface $hydrator)
    {
        $this->hydrator = $hydrator;
    }

    /**
     * Extract values from an object
     *
     * @param  UserEntityInterface $object
     * @return array
     * @throws Exception\InvalidArgumentException
     */
    public function extract($object): array
    {
        if (!$object instanceof UserEntityInterface) {
            throw new InvalidArgumentException('$object must be an instance of LmcUser\Entity\UserInterface');
        }

        $data = $this->hydrator->extract($object);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        if ($data['id'] !== null) {
            $data = $this->mapField('id', 'user_id', $data);
        } else {
            unset($data['id']);
        }

        return $data;
    }

    /**
     * Hydrate $object with the provided $data.
     *
     * @param  array               $data
     * @param  UserEntityInterface $object
     * @return UserInterface
     * @throws Exception\InvalidArgumentException
     */
    public function hydrate(array $data, $object)
    {
        if (!$object instanceof UserEntityInterface) {
            throw new Exception\InvalidArgumentException('$object must be an instance of LmcUser\Entity\UserInterface');
        }

        $data = $this->mapField('user_id', 'id', $data);

        return $this->hydrator->hydrate($data, $object);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @param  string $keyFrom
     * @param  string $keyTo
     * @param  array  $array
     * @return array
     */
    protected function mapField($keyFrom, $keyTo, array $array)
    {
        $array[$keyTo] = $array[$keyFrom];
        unset($array[$keyFrom]);

        return $array;
    }
}


1			<?php
2
3			namespace LmcUser\Mapper;
4
5			use Laminas\Hydrator\HydratorInterface;
6			use LmcUser\Entity\UserInterface as UserEntityInterface;
7			use LmcUser\Mapper\Exception\InvalidArgumentException;
8
9			/**
10			* Class UserHydrator
11			*/
12			class UserHydrator implements HydratorInterface
13			{
14			/** @var HydratorInterface */
15			private $hydrator;
16
17			/**
18			* UserHydrator constructor.
19			*
20			* @param HydratorInterface $hydrator
21			*/
22			public function __construct(HydratorInterface $hydrator)
23			{
24			$this->hydrator = $hydrator;
25			}
26
27			/**
28			* Extract values from an object
29			*
30			* @param UserEntityInterface $object
31			* @return array
32			* @throws Exception\InvalidArgumentException
33			*/
34			public function extract($object): array
35			{
36			if (!$object instanceof UserEntityInterface) {
37			throw new InvalidArgumentException('$object must be an instance of LmcUser\Entity\UserInterface');
38			}
39
40			$data = $this->hydrator->extract($object);
			0 ignored issues – show Documentation introduced 2021-01-07 19:49 UTC by Report Bug Copy Issue Report `$object` is of type `object<LmcUser\Entity\UserInterface>`, but the function expects a `object<Laminas\Hydrator\object>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
41			if ($data['id'] !== null) {
42			$data = $this->mapField('id', 'user_id', $data);
43			} else {
44			unset($data['id']);
45			}
46
47			return $data;
48			}
49
50			/**
51			* Hydrate $object with the provided $data.
52			*
53			* @param array $data
54			* @param UserEntityInterface $object
55			* @return UserInterface
56			* @throws Exception\InvalidArgumentException
57			*/
58			public function hydrate(array $data, $object)
59			{
60			if (!$object instanceof UserEntityInterface) {
61			throw new Exception\InvalidArgumentException('$object must be an instance of LmcUser\Entity\UserInterface');
62			}
63
64			$data = $this->mapField('user_id', 'id', $data);
65
66			return $this->hydrator->hydrate($data, $object);
			0 ignored issues – show Documentation introduced 2021-01-07 19:49 UTC by Report Bug Copy Issue Report `$object` is of type `object<LmcUser\Entity\UserInterface>`, but the function expects a `object<Laminas\Hydrator\object>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
67			}
68
69			/**
70			* @param string $keyFrom
71			* @param string $keyTo
72			* @param array $array
73			* @return array
74			*/
75			protected function mapField($keyFrom, $keyTo, array $array)
76			{
77			$array[$keyTo] = $array[$keyFrom];
78			unset($array[$keyFrom]);
79
80			return $array;
81			}
82			}
83

LM-Commons / LmcUser

Push — master ( 45e13e...d296b3 )

UserHydrator::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like