NotifiableBehavior::handle() - Code Metrics - Inspection of "Introduce database notifications" - tuyakhov/yii2-notifications - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#5)

by Anton

created 2018-07-07 10:51 UTC

NotifiableBehavior::handle() A

↳ Parent: NotifiableBehavior

Complexity

Conditions	4
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	20
rs	9.6
c	0
b	0
f	0
cc	4
nc	4
nop	1

<?php
/**
 * @copyright Anton Tuyakhov <[email protected]>
 */

namespace tuyakhov\notifications\behaviors;


use tuyakhov\notifications\NotifiableInterface;
use tuyakhov\notifications\NotificationInterface;
use tuyakhov\notifications\Notifier;
use yii\base\Behavior;
use yii\base\Event;
use yii\di\Instance;

class NotifiableBehavior extends Behavior
{
    public $notifications = [];

    /**
     * @var Notifier
     */
    public $notifier = 'notifier';

    /**
     * @throws \yii\base\InvalidConfigException
     */
    public function init()
    {
        parent::init();
        $this->notifier = Instance::ensure($this->notifier, 'tuyakhov\notifications\Notifier');
    }

    /**
     * @inheritdoc
     */
    public function attach($owner)
    {
        $this->owner = $owner;
        foreach ($this->notifications as $event => $notifications) {

            if (!is_array($notifications)) {
                $notifications = [$notifications];
            }
            foreach ($notifications as $notification) {
                $owner->on($event, [$this, 'handle'], ['notification' => $notification]);
            }
        }
    }

    /**
     * @inheritdoc
     */
    public function detach()
    {
        if ($this->owner) {
            foreach ($this->notifications as $event => $notifications) {

                if (!is_array($notifications)) {
                    $notifications = [$notifications];
                }
                foreach ($notifications as $notification) {
                    $this->owner->off($event, [$this, 'handle']);
                }
            }
            $this->owner = null;
        }
    }

    /**
     * Handles the event using public properties.
     * @param Event $event
     * @throws \yii\base\InvalidConfigException
     */
    public function handle(Event $event)
    {
        if (!isset($event->data['notification'])) {
            throw new \InvalidArgumentException('Can not find `notification` in event data');
        }
        if (!$this->owner instanceof NotifiableInterface) {
            throw new \RuntimeException('Owner should implement `NotifiableInterface`');
        }
        $notification = $event->data['notification'];
        $config = [];
$a = "a";
$ab = "ab";
$abc = "abc";
        foreach (get_object_vars($event) as $param => $value) {
            $config[$param] = $value;
        }
        $config['class'] = $notification;
        /**
         * @var $notification NotificationInterface
         */
        $notification = \Yii::createObject($config);
        $this->notifier->send($this->owner, $notification);
    }


}

1		<?php
2		/**
3		* @copyright Anton Tuyakhov <[email protected]>
4		*/
5
6		namespace tuyakhov\notifications\behaviors;
7
8
9		use tuyakhov\notifications\NotifiableInterface;
10		use tuyakhov\notifications\NotificationInterface;
11		use tuyakhov\notifications\Notifier;
12		use yii\base\Behavior;
13		use yii\base\Event;
14		use yii\di\Instance;
15
16		class NotifiableBehavior extends Behavior
17		{
18		public $notifications = [];
19
20		/**
21		* @var Notifier
22		*/
23		public $notifier = 'notifier';
24
25		/**
26		* @throws \yii\base\InvalidConfigException
27		*/
28		public function init()
29		{
30		parent::init();
31		$this->notifier = Instance::ensure($this->notifier, 'tuyakhov\notifications\Notifier');
32		}
33
34		/**
35		* @inheritdoc
36		*/
37		public function attach($owner)
38		{
39		$this->owner = $owner;
40	View Code Duplication	foreach ($this->notifications as $event => $notifications) {
		0 ignored issues – show Duplication introduced 2016-10-09 21:07 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
41		if (!is_array($notifications)) {
42		$notifications = [$notifications];
43		}
44		foreach ($notifications as $notification) {
45		$owner->on($event, [$this, 'handle'], ['notification' => $notification]);
46		}
47		}
48		}
49
50		/**
51		* @inheritdoc
52		*/
53		public function detach()
54		{
55		if ($this->owner) {
56	View Code Duplication	foreach ($this->notifications as $event => $notifications) {
		0 ignored issues – show Duplication introduced 2016-10-09 21:07 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
57		if (!is_array($notifications)) {
58		$notifications = [$notifications];
59		}
60		foreach ($notifications as $notification) {
61		$this->owner->off($event, [$this, 'handle']);
62		}
63		}
64		$this->owner = null;
65		}
66		}
67
68		/**
69		* Handles the event using public properties.
70		* @param Event $event
71		* @throws \yii\base\InvalidConfigException
72		*/
73		public function handle(Event $event)
74		{
75		if (!isset($event->data['notification'])) {
76		throw new \InvalidArgumentException('Can not find `notification` in event data');
77		}
78		if (!$this->owner instanceof NotifiableInterface) {
79		throw new \RuntimeException('Owner should implement `NotifiableInterface`');
80		}
81		$notification = $event->data['notification'];
82		$config = [];
		0 ignored issues – show Coding Style introduced 2016-10-09 21:07 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
83		foreach (get_object_vars($event) as $param => $value) {
84		$config[$param] = $value;
85		}
86		$config['class'] = $notification;
87		/**
88		* @var $notification NotificationInterface
89		*/
90		$notification = \Yii::createObject($config);
91		$this->notifier->send($this->owner, $notification);
92		}
93
94
95		}

tuyakhov / yii2-notifications

Pull Request — master (#5)

NotifiableBehavior::handle() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like