UserEventHandler::modelAfterSave() - Code Metrics - Inspection of "Send create account message if create new notify u..." - CakeCMS/Community - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7c1383...8fd89c )

by Cheren

created 2018-02-27 21:09 UTC

UserEventHandler::modelAfterSave() A

↳ Parent: UserEventHandler

Complexity

Conditions	4
Paths	3

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	11
rs	9.2
c	0
b	0
f	0
cc	4
eloc	5
nc	3
nop	1

<?php
/**
 * CakeCMS Community
 *
 * This file is part of the of the simple cms based on CakePHP 3.
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @package     Community
 * @license     MIT
 * @copyright   MIT License http://www.opensource.org/licenses/mit-license.php
 * @link        https://github.com/CakeCMS/Community".
 * @author      Sergey Kalistratov <[email protected]>
 */

namespace Community\Event;

use Cake\Event\Event;
use JBZoo\Utils\Filter;
use Community\Notify\Email;
use Community\Model\Entity\User;
use Cake\Datasource\EntityInterface;
use Cake\Event\EventListenerInterface;

/**
 * Class UserEventHandler
 *
 * @package Community\Event
 */
class UserEventHandler implements EventListenerInterface
{

    /**
     * Returns a list of events this object is implementing.
     *
     * @return  array
     */
    public function implementedEvents()
    {
        return [
            'Model.User.beforeSave' => 'modelBeforeSave',
            'Model.User.afterSave'  => 'modelAfterSave'
        ];
    }

    /**
     * Global before save user data.
     *
     * @param   Event $event
     *
     * @return  void
     */
    public function modelBeforeSave(Event $event)
    {
        /** @var User $user */
        $user = $event->getData('user');
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
    }

    /**
     * Global after save user data.
     *
     * @param   Event $event
     *
     * @return  void
     *
     * TODO : Use return when send message?
     */
    public function modelAfterSave(Event $event)
    {
        /** @var User|bool $user */
        $user = $event->getData('user');

        if ($user instanceof EntityInterface) {
            if ($user->isNew() && Filter::bool($user->get('notify'))) {
                $this->_getMailer($user)->sendCreateMessage();
            }
        }
    }

    /**
     * Get mailer object.
     *
     * @param   EntityInterface $entity
     * @return  Email
     */
    protected function _getMailer(EntityInterface $entity)
    {
        return new Email($entity);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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


1			<?php
2			/**
3			* CakeCMS Community
4			*
5			* This file is part of the of the simple cms based on CakePHP 3.
6			* For the full copyright and license information, please view the LICENSE
7			* file that was distributed with this source code.
8			*
9			* @package Community
10			* @license MIT
11			* @copyright MIT License http://www.opensource.org/licenses/mit-license.php
12			* @link https://github.com/CakeCMS/Community".
13			* @author Sergey Kalistratov <[email protected]>
14			*/
15
16			namespace Community\Event;
17
18			use Cake\Event\Event;
19			use JBZoo\Utils\Filter;
20			use Community\Notify\Email;
21			use Community\Model\Entity\User;
22			use Cake\Datasource\EntityInterface;
23			use Cake\Event\EventListenerInterface;
24
25			/**
26			* Class UserEventHandler
27			*
28			* @package Community\Event
29			*/
30			class UserEventHandler implements EventListenerInterface
31			{
32
33			/**
34			* Returns a list of events this object is implementing.
35			*
36			* @return array
37			*/
38			public function implementedEvents()
39			{
40			return [
41			'Model.User.beforeSave' => 'modelBeforeSave',
42			'Model.User.afterSave' => 'modelAfterSave'
43			];
44			}
45
46			/**
47			* Global before save user data.
48			*
49			* @param Event $event
50			*
51			* @return void
52			*/
53			public function modelBeforeSave(Event $event)
54			{
55			/** @var User $user */
56			$user = $event->getData('user');
			0 ignored issues – show Unused Code introduced 2018-02-27 21:14 UTC by Report Bug Copy Issue Report `$user` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
57			}
58
59			/**
60			* Global after save user data.
61			*
62			* @param Event $event
63			*
64			* @return void
65			*
66			* TODO : Use return when send message?
67			*/
68			public function modelAfterSave(Event $event)
69			{
70			/** @var User\|bool $user */
71			$user = $event->getData('user');
72
73			if ($user instanceof EntityInterface) {
74			if ($user->isNew() && Filter::bool($user->get('notify'))) {
75			$this->_getMailer($user)->sendCreateMessage();
76			}
77			}
78			}
79
80			/**
81			* Get mailer object.
82			*
83			* @param EntityInterface $entity
84			* @return Email
85			*/
86			protected function _getMailer(EntityInterface $entity)
87			{
88			return new Email($entity);
			0 ignored issues – show Documentation introduced 2018-02-27 21:14 UTC by Report Bug Copy Issue Report `$entity` is of type `object<Cake\Datasource\EntityInterface>`, but the function expects a `object<Cake\ORM\Entity>\|array`. 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...
89			}
90			}
91

CakeCMS / Community

Push — master ( 7c1383...8fd89c )

UserEventHandler::modelAfterSave() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like