hiqdev\billing\hiapi\bill\BillHydrator

Complexity

Total Complexity

11

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	3
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
B	hydrate()	0	38	7
A	extract()	0	16	4

<?php
/**
 * API for Billing
 *
 * @link      https://github.com/hiqdev/billing-hiapi
 * @package   billing-hiapi
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2017-2018, HiQDev (http://hiqdev.com/)
 */

namespace hiqdev\billing\hiapi\bill;

use DateTimeImmutable;
use hiqdev\php\billing\bill\Bill;
use hiqdev\php\billing\bill\BillState;
use hiqdev\php\billing\charge\ChargeInterface;
use hiqdev\php\billing\customer\Customer;
use hiqdev\php\billing\plan\Plan;
use hiqdev\php\billing\target\Target;
use hiqdev\php\billing\type\Type;
use hiqdev\php\units\Quantity;
use hiqdev\yii\DataMapper\hydrator\GeneratedHydrator;
use Money\Money;

/**
 * Bill Hydrator.
 *
 * @author Andrii Vasyliev <sol@hiqdev.com>
 */
class BillHydrator extends GeneratedHydrator
{
    /**
     * {@inheritdoc}
     * @param object|Bill $object
     */
    public function hydrate(array $row, $object)
    {
        $row['type']        = $this->hydrator->create($row['type'],     Type::class);

The method create() does not exist on Zend\Hydrator\HydratorInterface. It seems like you code against a sub-type of Zend\Hydrator\HydratorInterface such as hiqdev\yii\DataMapper\hydrator\ConfigurableAggregateHydrator.

        $row['time']        = $this->hydrator->create($row['time'],     DateTimeImmutable::class);
        $row['sum']         = $this->hydrator->create($row['sum'],      Money::class);
        $row['quantity']    = $this->hydrator->create($row['quantity'], Quantity::class);
        $row['customer']    = $this->hydrator->create($row['customer'], Customer::class);
        if (isset($row['target'])) {
            $row['target']  = $this->hydrator->create($row['target'],   Target::class);
        }
        if (isset($row['plan'])) {
            $row['plan']    = $this->hydrator->create($row['plan'],     Plan::class);
        }
        if (isset($row['state'])) {
            $row['state']  = $this->hydrator->create($row['state'],   BillState::class);
        }

        $raw_charges = $row['charges'];
        unset($row['charges']);

        /** @var Bill $bill */
        $bill = parent::hydrate($row, $object);

        if (\is_array($raw_charges)) {
            $charges = [];
            foreach ($raw_charges as $key => $charge) {
                if ($charge instanceof ChargeInterface) {
                    $charge->setBill($bill);
                    $charges[$key] = $charge;
                } else {
                    $charge['bill'] = $bill;
                    $charges[$key] = $this->hydrator->hydrate($charge, ChargeInterface::class);

hiqdev\php\billing\charge\ChargeInterface::class of type string is incompatible with the type object expected by parameter $object of Zend\Hydrator\HydrationInterface::hydrate().

                }
            }
            $bill->setCharges($charges);
        }

        return $bill;
    }

    /**
     * {@inheritdoc}
     * @param object|Bill $object
     */
    public function extract($object)
    {
        return array_filter([
            'id'            => $object->getId(),
            'type'          => $this->hydrator->extract($object->getType()),
            'time'          => $this->hydrator->extract($object->getTime()),
            'sum'           => $this->hydrator->extract($object->getSum()),
            'quantity'      => $this->hydrator->extract($object->getQuantity()),
            'customer'      => $this->hydrator->extract($object->getCustomer()),
            'target'        => $object->getTarget() ? $this->hydrator->extract($object->getTarget()) : null,
            'plan'          => $object->getPlan() ? $this->hydrator->extract($object->getPlan()) : null,
            'charges'       => $this->hydrator->extractAll($object->getCharges()),

The method extractAll() does not exist on Zend\Hydrator\HydratorInterface. Did you maybe mean extract()?

            'state'         => $object->getState() ? $this->hydrator->extract($object->getState()) : null,
        ], static function ($value): bool {
            return $value !== null;
        }, ARRAY_FILTER_USE_BOTH);
    }
}