Slince\Shopify\Hydrator\Hydrator

Complexity

Total Complexity

3

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	extract()	0	4	1
A	hydrate()	0	4	1

<?php

/*
 * This file is part of the slince/shopify-api-php
 *
 * (c) Slince <taosikai@yeah.net>
 *
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace Slince\Shopify\Hydrator;

use JMS\Serializer\Context;
use JMS\Serializer\SerializationContext;
use JMS\Serializer\Serializer;
use JMS\Serializer\SerializerBuilder;

class Hydrator implements HydratorInterface
{
    /**
     * @var Serializer
     */
    protected $serializer;

    /**
     * @var Context
     */
    protected $serializationContext;

    public function __construct($cacheDir, $metaDir)
    {
        $this->serializer = SerializerBuilder::create()
            ->setCacheDir($cacheDir)
            ->addMetadataDir($metaDir)
            ->build();
        $this->serializationContext = SerializationContext::create()->setSerializeNull(true);
    }

    /**
     * {@inheritdoc}
     */
    public function extract($object)
    {
        return $this->serializer->toArray($object, $this->serializationContext);

$this->serializationContext is of type object<JMS\Serializer\Context>, but the function expects a null|object<JMS\Serializer\SerializationContext>.

    }

    /**
     * {@inheritdoc}
     */
    public function hydrate($target, array $data)
    {
        return $this->serializer->fromArray($data, $target);

It seems like $target defined by parameter $target on line 51 can also be of type object; however, JMS\Serializer\Serializer::fromArray() does only seem to accept string, maybe add an additional type check?

    }
}