ObjectMapper::map() - Code Metrics - Inspection of "Merge pull request #84 from fathomminds/staging" - fathomminds/php-rest-models - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( af979b...597647 )

by Csaba

created 2018-01-28 14:14 UTC

ObjectMapper::map() B

↳ Parent: ObjectMapper

Complexity

Conditions	5
Paths	5

Size

Total Lines	25
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	0
loc	25
c	1
b	0
f	0
ccs	0
cts	16
cp	0
rs	8.439
cc	5
eloc	16
nc	5
nop	2
crap	30

<?php
namespace Fathomminds\Rest\Helpers;

use Fathomminds\Rest\Exceptions\RestException;

abstract class ObjectMapper
{
    public static function map($object, $map)
    {
        if (gettype($object) !== 'object') {
            throw new RestException('ObjectMapper map method expects object as first parameter', [
                'parameter' => $object,
            ]);
        }
        if (!is_array($map)) {
            throw new RestException('ObjectMapper map method expects array as second parameter', [
                'parameter' => $map,
            ]);
        }
        $mappedObject = new \StdClass();
        foreach ($map as $targetFieldName => $sourceFieldName) {
            list($propertyExists, $propertyValue) = static::getPropertyValue($object, $sourceFieldName);
class YourClass
{
    private static function getTemperature() {
        return "3422 °C";
}

public static function getSomeVariable()
{
    return static::getTemperature();
}
            if ($propertyExists) {
                $mappedObject = static::setPropertyValue(
class YourClass
{
    private static function getTemperature() {
        return "3422 °C";
}

public static function getSomeVariable()
{
    return static::getTemperature();
}
                    $mappedObject,
                    $targetFieldName,
                    $propertyValue
                );
            }
        }
        return $mappedObject;
    }

    private static function setPropertyValue($mappedObject, $targetFieldName, $propertyValue)
    {
        $fieldNameArr = explode('.', $targetFieldName);
        $fieldName = array_shift($fieldNameArr);
        if (count($fieldNameArr) !== 0) {
            if (!property_exists($mappedObject, $fieldName)) {
                $mappedObject->{$fieldName} = new \StdClass();
            }
            $mappedObject->{$fieldName} = static::setPropertyValue(
class YourClass
{
    private static function getTemperature() {
        return "3422 °C";
}

public static function getSomeVariable()
{
    return static::getTemperature();
}
                $mappedObject->{$fieldName},
                implode('.', $fieldNameArr),
                $propertyValue
            );
            return $mappedObject;
        }
        $mappedObject->{$fieldName} = $propertyValue;
        return $mappedObject;
    }

    private static function getPropertyValue($object, $sourceFieldName)
    {
        $fieldNameArr = explode('.', $sourceFieldName);
        $fieldName = array_shift($fieldNameArr);
        if (!property_exists($object, $fieldName)) {
            return [
                false,
                null
            ];
        }
        if (count($fieldNameArr) === 0) {
            return [
                true,
                json_decode(json_encode($object->{$fieldName}))
            ];
        }
        return static::getPropertyValue($object->{$fieldName}, implode('.', $fieldNameArr));
class YourClass
{
    private static function getTemperature() {
        return "3422 °C";
}

public static function getSomeVariable()
{
    return static::getTemperature();
}
    }
}


1			<?php
2			namespace Fathomminds\Rest\Helpers;
3
4			use Fathomminds\Rest\Exceptions\RestException;
5
6			abstract class ObjectMapper
7			{
8			public static function map($object, $map)
9			{
10			if (gettype($object) !== 'object') {
11			throw new RestException('ObjectMapper map method expects object as first parameter', [
12			'parameter' => $object,
13			]);
14			}
15			if (!is_array($map)) {
16			throw new RestException('ObjectMapper map method expects array as second parameter', [
17			'parameter' => $map,
18			]);
19			}
20			$mappedObject = new \StdClass();
21			foreach ($map as $targetFieldName => $sourceFieldName) {
22			list($propertyExists, $propertyValue) = static::getPropertyValue($object, $sourceFieldName);
			0 ignored issues – show Bug introduced 2017-11-03 16:11 UTC by Report Bug Copy Issue Report Since `getPropertyValue()` is declared private, calling it with `static` will lead to errors in possible sub-classes. You can either use `self`, or increase the visibility of `getPropertyValue()` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return static::getTemperature(); } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { private static function getTemperature() { return "-182 °C"; } } print YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return self::getTemperature(); } } Loading history...
23			if ($propertyExists) {
24			$mappedObject = static::setPropertyValue(
			0 ignored issues – show Bug introduced 2017-11-03 16:11 UTC by Report Bug Copy Issue Report Since `setPropertyValue()` is declared private, calling it with `static` will lead to errors in possible sub-classes. You can either use `self`, or increase the visibility of `setPropertyValue()` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return static::getTemperature(); } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { private static function getTemperature() { return "-182 °C"; } } print YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return self::getTemperature(); } } Loading history...
25			$mappedObject,
26			$targetFieldName,
27			$propertyValue
28			);
29			}
30			}
31			return $mappedObject;
32			}
33
34			private static function setPropertyValue($mappedObject, $targetFieldName, $propertyValue)
35			{
36			$fieldNameArr = explode('.', $targetFieldName);
37			$fieldName = array_shift($fieldNameArr);
38			if (count($fieldNameArr) !== 0) {
39			if (!property_exists($mappedObject, $fieldName)) {
40			$mappedObject->{$fieldName} = new \StdClass();
41			}
42			$mappedObject->{$fieldName} = static::setPropertyValue(
			0 ignored issues – show Bug introduced 2017-11-03 16:11 UTC by Report Bug Copy Issue Report Since `setPropertyValue()` is declared private, calling it with `static` will lead to errors in possible sub-classes. You can either use `self`, or increase the visibility of `setPropertyValue()` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return static::getTemperature(); } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { private static function getTemperature() { return "-182 °C"; } } print YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return self::getTemperature(); } } Loading history...
43			$mappedObject->{$fieldName},
44			implode('.', $fieldNameArr),
45			$propertyValue
46			);
47			return $mappedObject;
48			}
49			$mappedObject->{$fieldName} = $propertyValue;
50			return $mappedObject;
51			}
52
53			private static function getPropertyValue($object, $sourceFieldName)
54			{
55			$fieldNameArr = explode('.', $sourceFieldName);
56			$fieldName = array_shift($fieldNameArr);
57			if (!property_exists($object, $fieldName)) {
58			return [
59			false,
60			null
61			];
62			}
63			if (count($fieldNameArr) === 0) {
64			return [
65			true,
66			json_decode(json_encode($object->{$fieldName}))
67			];
68			}
69			return static::getPropertyValue($object->{$fieldName}, implode('.', $fieldNameArr));
			0 ignored issues – show Bug introduced 2017-11-03 16:11 UTC by Report Bug Copy Issue Report Since `getPropertyValue()` is declared private, calling it with `static` will lead to errors in possible sub-classes. You can either use `self`, or increase the visibility of `getPropertyValue()` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return static::getTemperature(); } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { private static function getTemperature() { return "-182 °C"; } } print YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return self::getTemperature(); } } Loading history...
70			}
71			}
72

fathomminds / php-rest-models

Push — master ( af979b...597647 )

ObjectMapper::map() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like