Dgame\Soap\Hydrator\Hydrator

Complexity

Total Complexity

6

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	hydrateDocument()	0	15	3
A	hydrateNode()	0	7	1
A	hydrate()	0	7	1

<?php

namespace Dgame\Soap\Hydrator;

use Dgame\Soap\Element;
use Dgame\Soap\XmlTranslator;
use DOMDocument;
use DOMNode;

/**
 * Class Hydrator
 * @package Dgame\Soap\Hydrator
 */
final class Hydrator
{
    /**
     * @var ClassMapper
     */
    private $mapper;

    /**
     * Hydrator constructor.
     *
     * @param ClassMapper $mapper
     */
    public function __construct(ClassMapper $mapper)
    {
        $this->mapper = $mapper;
    }

    /**
     * @param DOMDocument $document
     *
     * @return array
     */
    public function hydrateDocument(DOMDocument $document): array
    {
        $translator = new XmlTranslator();
        $elements   = $translator->translateDocument($document);

        $output = [];
        foreach ($elements as $element) {
            $hydrat = $this->hydrate($element);
            if ($hydrat->isValid()) {
                $output[] = $hydrat->getHydratable();
            }
        }

        return $output;
    }

    /**
     * @param DOMNode $node
     *
     * @return HydrateProcedure
     */
    public function hydrateNode(DOMNode $node): HydrateProcedure
    {
        $translator = new XmlTranslator();
        $element    = $translator->translateNode($node);

        return $this->hydrate($element);

It seems like $element defined by $translator->translateNode($node) on line 60 can be null; however, Dgame\Soap\Hydrator\Hydrator::hydrate() does not accept null, maybe add an additional type check?

    }

    /**
     * @param Element $element
     *
     * @return HydrateProcedure
     */
    private function hydrate(Element $element): HydrateProcedure
    {
        $hydrat = new HydrateProcedure($this->mapper);
        $element->hydration($hydrat);

        return $hydrat;
    }
}