Routes::__call() - Code Metrics - CapMousse/React-Restify - Measure and Improve Code Quality continuously with Scrutinizer

Routes::__call() A
last analyzed 2017-09-06 07:50 UTC

↳ Parent: Routes

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
c	1
b	1
f	0
dl	0
loc	4
rs	10
cc	1
eloc	2
nc	1
nop	2

<?php

namespace CapMousse\ReactRestify\Routing;

use CapMousse\ReactRestify\Evenement\EventEmitter;
use CapMousse\ReactRestify\Routing\ControllerContainer;
use CapMousse\ReactRestify\Traits\EventTrait;

class Routes extends EventEmitter
{
    use EventTrait;
    /**
     * Router instance
     * @var \CapMousse\ReactRestify\Routing\Router
     */
    private $router;

    /**
     * Group prefix
     * @var String
     */
    private $prefix;

    /**
     * Routes of the group
     * @var array
     */
    public $routes = [];

    /**
     * Create a new group
     *
     * @param \CapMousse\ReactRestify\Routing\Router $router
     * @param String                                 $prefix
     * @param Callable                               $callback
     */
    public function __construct($router, $prefix, $callback)
    {
        $this->router = $router;
        $this->prefix = $prefix;

        $callback($this);
    }

    /**
     * Create a new route for the group
     * @param String   $method
     * @param String   $route
     * @param Callable $callback
     */
    public function addRoute($method, $route, $callback)
    {
        $route = $this->router->addRoute(strtoupper($method), $this->prefix . '/' . $route, $callback);

        $route->onAny(function($event, $arguments){
            $this->emit($event, $arguments);
        });

        $this->routes[] = $route;

        return $route;
    }

    /**
     * Add a new group of routes
     * @param string   $prefix
     * @param Callable $callback
     *
     * return \CapMousse\ReactRestify\Routing\Group
     */
    public function group($prefix, $callback)
    {
        $group = $this->router->addGroup($this->prefix . '/' . $prefix, $callback);

        $group->onAny(function($event, $arguments){
            $this->emit($event, $arguments);
        });
    }

    /**
     * @param string $name      method to call
     * @param array  $arguments
     */
    public function __call($name, $arguments)
    {
        return $this->addRoute($name, ...$arguments);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }
}


1			<?php
2
3			namespace CapMousse\ReactRestify\Routing;
4
5			use CapMousse\ReactRestify\Evenement\EventEmitter;
6			use CapMousse\ReactRestify\Routing\ControllerContainer;
7			use CapMousse\ReactRestify\Traits\EventTrait;
8
9			class Routes extends EventEmitter
10			{
11			use EventTrait;
12			/**
13			* Router instance
14			* @var \CapMousse\ReactRestify\Routing\Router
15			*/
16			private $router;
17
18			/**
19			* Group prefix
20			* @var String
21			*/
22			private $prefix;
23
24			/**
25			* Routes of the group
26			* @var array
27			*/
28			public $routes = [];
29
30			/**
31			* Create a new group
32			*
33			* @param \CapMousse\ReactRestify\Routing\Router $router
34			* @param String $prefix
35			* @param Callable $callback
36			*/
37			public function __construct($router, $prefix, $callback)
38			{
39			$this->router = $router;
40			$this->prefix = $prefix;
41
42			$callback($this);
43			}
44
45			/**
46			* Create a new route for the group
47			* @param String $method
48			* @param String $route
49			* @param Callable $callback
50			*/
51			public function addRoute($method, $route, $callback)
52			{
53			$route = $this->router->addRoute(strtoupper($method), $this->prefix . '/' . $route, $callback);
54
55			$route->onAny(function($event, $arguments){
56			$this->emit($event, $arguments);
57			});
58
59			$this->routes[] = $route;
60
61			return $route;
62			}
63
64			/**
65			* Add a new group of routes
66			* @param string $prefix
67			* @param Callable $callback
68			*
69			* return \CapMousse\ReactRestify\Routing\Group
70			*/
71			public function group($prefix, $callback)
72			{
73			$group = $this->router->addGroup($this->prefix . '/' . $prefix, $callback);
74
75			$group->onAny(function($event, $arguments){
76			$this->emit($event, $arguments);
77			});
78			}
79
80			/**
81			* @param string $name method to call
82			* @param array $arguments
83			*/
84			public function __call($name, $arguments)
85			{
86			return $this->addRoute($name, ...$arguments);
			0 ignored issues – show Bug introduced 2017-09-06 07:22 UTC by Report Bug Copy Issue Report The call to `addRoute()` misses a required argument `$callback`. This check looks for function calls that miss required arguments. Loading history... Documentation introduced 2017-09-06 07:22 UTC by Report Bug Copy Issue Report `$arguments` is of type `array`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
87			}
88			}
89

CapMousse / React-Restify

GitHub Access Token became invalid

Routes::__call() A last analyzed 2017-09-06 07:50 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

Routes::__call() A
last analyzed 2017-09-06 07:50 UTC