Dtc\QueueBundle\Util\Util::copy()

Complexity

Conditions	10
Paths	17

Size

Total Lines	34
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public static function copy($obj1, $obj2)
    {
        if (!is_object($obj1)) {
            throw new \Exception('$obj1 must be an object, not '.gettype($obj1));
        }
        if (!is_object($obj2)) {
            throw new \Exception('$obj2 must be an object, not '.gettype($obj2));
        }
        $reflection1 = new \ReflectionObject($obj1);
        $reflection2 = new \ReflectionObject($obj2);

        $methods = $reflection1->getMethods(\ReflectionMethod::IS_PUBLIC);
        $publicVars = $reflection1->getProperties(\ReflectionProperty::IS_PUBLIC);
        foreach ($methods as $method) {
            $methodName = $method->getName();

Consider using $method->name. There is an issue with getName() and APC-enabled PHP versions.

            if (0 === strpos($methodName, 'get')) {
                $getMethod = $methodName;
                $setMethod = $methodName;
                $setMethod[0] = 's';
                if ($reflection2->hasMethod($setMethod)) {
                    $value = $obj1->$getMethod();
                    if (null !== $value) {
                        $obj2->$setMethod($value);
                    }
                }
            }
        }
        foreach ($publicVars as $property) {
            $propertyName = $property->getName();
            if ($reflection2->hasPropery($propertyName) && $reflection2->getProperty($propertyName)->isPublic()) {

The method hasPropery() does not exist on ReflectionObject. Did you maybe mean hasProperty()?

                $obj2->$propertyName = $obj1->$propertyName;
            }
        }
    }
}