Converter::__construct() - Code Metrics - Inspection of "Serialization makeover" - ongr-io/ElasticsearchBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — 6.1-dev (#889)

unknown

created 2019-10-23 21:38 UTC

Converter::__construct() A

↳ Parent: Converter::convertArrayToDocument()

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

/*
 * This file is part of the ONGR package.
 *
 * (c) NFQ Technologies UAB <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace ONGR\ElasticsearchBundle\Mapping;

use ONGR\ElasticsearchBundle\Result\ObjectIterator;

/**
 * This class converts array to document object.
 */
class Converter
{
    private $propertyMetadata = [];

    public function addClassMetadata(string $class, array $metadata): void
    {
        $this->propertyMetadata[$class] = $metadata;
    }

    public function convertArrayToDocument(string $namespace, array $raw)
    {
        if (!isset($this->propertyMetadata[$namespace])) {
            echo json_encode($raw, JSON_PRETTY_PRINT);

            return [];
        }

        return $this->denormalize($raw, $namespace);
    }

    public function convertDocumentToArray($document): array
    {
        $class = get_class($document);

        if (!isset($this->propertyMetadata[$class])) {
            throw new \Exception("Cannot convert object of class `$class` to array.");
        }

        return $this->normalize($document);
    }

    protected function normalize($document, $metadata = null)
    {
        $metadata = $metadata ?? $this->propertyMetadata[get_class($document)];
        $result = [];

        foreach ($metadata as $field => $fieldMeta) {
            $getter = $fieldMeta['getter'];
            $value = $fieldMeta['public'] ? $document->{$fieldMeta['name']} : $document->$getter();

            if ($fieldMeta['embeded']) {
                if (is_iterable($value)) {
                    foreach ($value as $item) {
                        $result[$field][] = $this->normalize($item, $fieldMeta['sub_properties']);
                    }
                } else {
                    $result[$field] = $this->normalize($item, $fieldMeta['sub_properties']);
foreach ($a as $b) {
}

// $b is defined here only if $a has elements, for example if $a is array()
// then $b would not be defined here. To avoid that, we recommend to set a
// default value for $b.


// Better
$b = 0; // or whatever default makes sense in your context
foreach ($a as $b) {
}

// $b is now guaranteed to be defined here.
                }
            } else {
                $result[$field] = $value;
            }
        }

        return $result;
    }

    protected function denormalize(array $raw, string $namespace)
    {
        $metadata = $this->propertyMetadata[$namespace];
        $object = new $namespace();

        foreach ($raw as $field => $value) {
            $fieldMeta = $metadata[$field];
            $setter = $fieldMeta['setter'];

            if ($fieldMeta['embeded']) {
                $this->addClassMetadata($fieldMeta['class'], $fieldMeta['sub_properties']);
                $object->$setter(new ObjectIterator($fieldMeta['class'], $value, $this));
            } else {
                if ($fieldMeta['public']) {
                    $object->{$fieldMeta['name']} = $value;
                } else {
                    if ($fieldMeta['identifier']) {
                        $setter = function ($field, $value) {
                            $this->$field = $value;
                        };

                        $setter = \Closure::bind($setter, $object, $object);
                        $setter($fieldMeta['name'], $value);
                    } else {
                        $object->$setter($value);
                    }
                }
            }
        }

        return $object;
    }
}


1			<?php
2
3			/*
4			* This file is part of the ONGR package.
5			*
6			* (c) NFQ Technologies UAB <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace ONGR\ElasticsearchBundle\Mapping;
13
14			use ONGR\ElasticsearchBundle\Result\ObjectIterator;
15
16			/**
17			* This class converts array to document object.
18			*/
19			class Converter
20			{
21			private $propertyMetadata = [];
22
23			public function addClassMetadata(string $class, array $metadata): void
24			{
25			$this->propertyMetadata[$class] = $metadata;
26			}
27
28			public function convertArrayToDocument(string $namespace, array $raw)
29			{
30			if (!isset($this->propertyMetadata[$namespace])) {
31			echo json_encode($raw, JSON_PRETTY_PRINT);
32
33			return [];
34			}
35
36			return $this->denormalize($raw, $namespace);
37			}
38
39			public function convertDocumentToArray($document): array
40			{
41			$class = get_class($document);
42
43			if (!isset($this->propertyMetadata[$class])) {
44			throw new \Exception("Cannot convert object of class `$class` to array.");
45			}
46
47			return $this->normalize($document);
48			}
49
50			protected function normalize($document, $metadata = null)
51			{
52			$metadata = $metadata ?? $this->propertyMetadata[get_class($document)];
53			$result = [];
54
55			foreach ($metadata as $field => $fieldMeta) {
56			$getter = $fieldMeta['getter'];
57			$value = $fieldMeta['public'] ? $document->{$fieldMeta['name']} : $document->$getter();
58
59			if ($fieldMeta['embeded']) {
60			if (is_iterable($value)) {
61			foreach ($value as $item) {
62			$result[$field][] = $this->normalize($item, $fieldMeta['sub_properties']);
63			}
64			} else {
65			$result[$field] = $this->normalize($item, $fieldMeta['sub_properties']);
			0 ignored issues – show Bug introduced 2019-10-23 21:39 UTC by Report Bug Copy Issue Report The variable `$item` seems to be defined by a `foreach` iteration on line `61`. Are you sure the iterator is never empty, otherwise this variable is not defined? It seems like you are relying on a variable being defined by an iteration: foreach ($a as $b) { } // $b is defined here only if $a has elements, for example if $a is array() // then $b would not be defined here. To avoid that, we recommend to set a // default value for $b. // Better $b = 0; // or whatever default makes sense in your context foreach ($a as $b) { } // $b is now guaranteed to be defined here. Loading history...
66			}
67			} else {
68			$result[$field] = $value;
69			}
70			}
71
72			return $result;
73			}
74
75			protected function denormalize(array $raw, string $namespace)
76			{
77			$metadata = $this->propertyMetadata[$namespace];
78			$object = new $namespace();
79
80			foreach ($raw as $field => $value) {
81			$fieldMeta = $metadata[$field];
82			$setter = $fieldMeta['setter'];
83
84			if ($fieldMeta['embeded']) {
85			$this->addClassMetadata($fieldMeta['class'], $fieldMeta['sub_properties']);
86			$object->$setter(new ObjectIterator($fieldMeta['class'], $value, $this));
87			} else {
88			if ($fieldMeta['public']) {
89			$object->{$fieldMeta['name']} = $value;
90			} else {
91			if ($fieldMeta['identifier']) {
92			$setter = function ($field, $value) {
93			$this->$field = $value;
94			};
95
96			$setter = \Closure::bind($setter, $object, $object);
97			$setter($fieldMeta['name'], $value);
98			} else {
99			$object->$setter($value);
100			}
101			}
102			}
103			}
104
105			return $object;
106			}
107			}
108

ongr-io / ElasticsearchBundle

Pull Request — 6.1-dev (#889)

Converter::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like