RouteAction::run() - Code Metrics - Inspection of "RouteAction trait" - jasny/controller - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Pull Request — master (#3)

unknown

created 2016-10-18 20:55 UTC

RouteAction::run() B

↳ Parent: RouteAction

Complexity

Conditions	6
Paths	6

Size

Total Lines	21
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	21
rs	8.7624
c	0
b	0
f	0
cc	6
eloc	13
nc	6
nop	0

<?php

namespace Jasny\Controller;

use Psr\Http\Message\ResponseInterface;

/**
 * Execute controller on given route
 */
trait RouteAction
{
    /**
     * Run the controller
     *
     * @return ResponseInterface
     */
    public function run() {
        $request = $this->getRequest();
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        if (!$request) {
            throw new \RuntimeException("Request object is not set for controller");            
        }

        $route = $request->getAttribute('route');
        $method = $this->getActionMethod(isset($route->action) ? $route->action : null);
        
        if (!method_exists($this, $method)) {
            throw new \RuntimeException("No method $method in conrtoller to route to");
        }

        $args = isset($route->args) ? 
            $route->args :
            $this->getFunctionArgs($route, new \ReflectionMethod($this, $method));

        $response = call_user_func_array([$this, $method], $args);

        return $response ?: $this->getResponse();        
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    /**
     * Get the method name of the action
     * 
     * @param string $action
     * @return string
     */
    protected function getActionMethod($action)
    {
        return \Jasny\camelcase($action) . 'Action';
    }

    /**
     * Get the arguments for a function from a route using reflection
     * 
     * @param object $route
     * @param \ReflectionFunctionAbstract $refl
     * @return array
     */
    protected function getFunctionArgs($route, \ReflectionFunctionAbstract $refl)
    {
        $args = [];
        $params = $refl->getParameters();

        foreach ($params as $param) {
            $key = $param->name;

            if (property_exists($route, $key)) {
                $value = $route->{$key};
            } else {
                if (!$param->isOptional()) {
                    $fn = $refl instanceof \ReflectionMethod
                        ? $refl->getDeclaringClass()->getName() . ':' . $refl->getName()

                        : $refl->getName();

                    throw new \RuntimeException("Missing argument '$key' for $fn()");
                }
                
                $value = $param->isDefaultValueAvailable() ? $param->getDefaultValue() : null;
            }

            $args[$key] = $value;
        }
        
        return $args;
    }
}



1			<?php
2
3			namespace Jasny\Controller;
4
5			use Psr\Http\Message\ResponseInterface;
6
7			/**
8			* Execute controller on given route
9			*/
10			trait RouteAction
11			{
12			/**
13			* Run the controller
14			*
15			* @return ResponseInterface
16			*/
17			public function run() {
18			$request = $this->getRequest();
			0 ignored issues – show Bug introduced 2016-10-18 21:05 UTC by Report Bug Copy Issue Report It seems like `getRequest()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
19			if (!$request) {
20			throw new \RuntimeException("Request object is not set for controller");
21			}
22
23			$route = $request->getAttribute('route');
24			$method = $this->getActionMethod(isset($route->action) ? $route->action : null);
25
26			if (!method_exists($this, $method)) {
27			throw new \RuntimeException("No method $method in conrtoller to route to");
28			}
29
30			$args = isset($route->args) ?
31			$route->args :
32			$this->getFunctionArgs($route, new \ReflectionMethod($this, $method));
33
34			$response = call_user_func_array([$this, $method], $args);
35
36			return $response ?: $this->getResponse();
			0 ignored issues – show Bug introduced 2016-10-18 21:05 UTC by Report Bug Copy Issue Report It seems like `getResponse()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
37			}
38
39			/**
40			* Get the method name of the action
41			*
42			* @param string $action
43			* @return string
44			*/
45			protected function getActionMethod($action)
46			{
47			return \Jasny\camelcase($action) . 'Action';
48			}
49
50			/**
51			* Get the arguments for a function from a route using reflection
52			*
53			* @param object $route
54			* @param \ReflectionFunctionAbstract $refl
55			* @return array
56			*/
57			protected function getFunctionArgs($route, \ReflectionFunctionAbstract $refl)
58			{
59			$args = [];
60			$params = $refl->getParameters();
61
62			foreach ($params as $param) {
63			$key = $param->name;
64
65			if (property_exists($route, $key)) {
66			$value = $route->{$key};
67			} else {
68			if (!$param->isOptional()) {
69			$fn = $refl instanceof \ReflectionMethod
70			? $refl->getDeclaringClass()->getName() . ':' . $refl->getName()
			0 ignored issues – show introduced 2016-10-18 21:05 UTC by Report Bug Copy Issue Report Consider using `$refl->class`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history... Bug introduced 2016-10-18 21:05 UTC by Report Bug Copy Issue Report Consider using `$refl->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
71			: $refl->getName();
72
73			throw new \RuntimeException("Missing argument '$key' for $fn()");
74			}
75
76			$value = $param->isDefaultValueAvailable() ? $param->getDefaultValue() : null;
77			}
78
79			$args[$key] = $value;
80			}
81
82			return $args;
83			}
84			}
85
86

jasny / controller

Pull Request — master (#3)

RouteAction::run() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like