EventTrait::after() - Code Metrics - Inspection of "Refactor route calling" - CapMousse/React-Restify - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#13)

by Jérémy

created 2017-02-15 19:14 UTC

EventTrait::after() A

↳ Parent: EventTrait

Complexity

Conditions	3
Paths	2

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	14
rs	9.4285
cc	3
eloc	8
nc	2
nop	1

<?php

namespace CapMousse\ReactRestify\Traits;

use CapMousse\ReactRestify\Routing\ControllerContainer;

trait EventTrait {
    /**
     * On event fired
     * @param  string   $event    
     * @param  callable $callback [description]
     * @return void
     */
    abstract function on($event, callable $callback);


    /**
     * Emit event
     * @param  string $event     
     * @param  array  $arguments 
     * @return void
     */
    abstract function emit($event, array $arguments = []);


    /**
     * Helper to listing to after event
     *
     * @param  Callable $callback
     * @return Void
     */
    public function after($callback)
    {
        if (is_string($callback)) {
            $callback = ControllerContainer::resolveController($callback);
        }

        $this->on('after', function ($request, $response) use (&$callback) {
            try {
                call_user_func_array($callback, func_get_args());
            } catch (\Exception $e) {
                $this->emit('error', [$request, $response, $e->getMessage()]);
            }
        });
    }
}

1			<?php
2
3			namespace CapMousse\ReactRestify\Traits;
4
5			use CapMousse\ReactRestify\Routing\ControllerContainer;
6
7			trait EventTrait {
8			/**
9			* On event fired
10			* @param string $event
11			* @param callable $callback [description]
12			* @return void
13			*/
14			abstract function on($event, callable $callback);
			0 ignored issues – show Best Practice introduced 2017-02-15 19:16 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
15
16			/**
17			* Emit event
18			* @param string $event
19			* @param array $arguments
20			* @return void
21			*/
22			abstract function emit($event, array $arguments = []);
			0 ignored issues – show Best Practice introduced 2017-02-15 19:16 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
23
24			/**
25			* Helper to listing to after event
26			*
27			* @param Callable $callback
28			* @return Void
29			*/
30			public function after($callback)
31			{
32			if (is_string($callback)) {
33			$callback = ControllerContainer::resolveController($callback);
34			}
35
36			$this->on('after', function ($request, $response) use (&$callback) {
37			try {
38			call_user_func_array($callback, func_get_args());
39			} catch (\Exception $e) {
40			$this->emit('error', [$request, $response, $e->getMessage()]);
41			}
42			});
43			}
44			}

CapMousse / React-Restify

GitHub Access Token became invalid

Pull Request — master (#13)

EventTrait::after() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like