veejay\jsonrpc\tests\ProtectedHelper

Complexity

Total Complexity

8

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	catchExceptionCode()	0	7	2
A	callMethod()	0	7	2
A	getProperty()	0	7	2
A	setProperty()	0	9	2

<?php

namespace veejay\jsonrpc\tests;

use Exception;
use ReflectionClass;

final class ProtectedHelper
{
    /**
     * Get protected property.
     * @param string|object $object
     * @param string $property
     * @return mixed|bool
     */
    public static function getProperty($object, string $property)
    {
        $reflection = new ReflectionClass($object);
        if (!$reflection->hasProperty($property)) return false;
        $property = $reflection->getProperty($property);
        $property->setAccessible(true);
        return $property->getValue($object);

It seems like $object can also be of type string; however, parameter $object of ReflectionProperty::getValue() does only seem to accept null|object, maybe add an additional type check?

    }

    /**
     * Set protected property.
     * @param string|object $object
     * @param string $property
     * @param mixed $value
     * @return void
     */
    public static function setProperty($object, string $property, $value)
    {
        $reflection = new ReflectionClass($object);
        $property = $reflection->getProperty($property);
        $property->setAccessible(true);
        if ($property->isStatic()) {
            $property->setValue($value);
        } else {
            $property->setValue($object, $value);
        }
    }

    /**
     * Call protected method.
     * @param string|object $object
     * @param string $methodName
     * @param array $params
     * @return mixed|bool
     */
    public static function callMethod($object, string $methodName, array $params = [])
    {
        $reflection = new ReflectionClass(get_class($object));

It seems like $object can also be of type string; however, parameter $object of get_class() does only seem to accept object, maybe add an additional type check?

        if (!$reflection->hasMethod($methodName)) return false;
        $method = $reflection->getMethod($methodName);
        $method->setAccessible(true);
        return $method->invokeArgs($object, $params);

It seems like $object can also be of type string; however, parameter $object of ReflectionMethod::invokeArgs() does only seem to accept null|object, maybe add an additional type check?

    }

    /**
     * Catch exception and return the code.
     * @param callable $callback
     * @return int
     */
    public static function catchExceptionCode(callable $callback): int
    {
        try {
            $callback();
            return 0;
        } catch (Exception $e) {
            return $e->getCode();
        }
    }
}