UserEventHandler::onSuccessActivate() - Code Metrics - Inspection of "Add site activate user profile" - CakeCMS/Community - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 76aee5...716e15 )

by Cheren

created 2018-03-01 19:47 UTC

UserEventHandler::onSuccessActivate() A

↳ Parent: UserEventHandler

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	3
rs	10
cc	1
eloc	1
nc	1
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.afterSave'             => 'onModelAfterSave',
            'Model.User.beforeSave'            => 'onModelBeforeSave',
            'Controller.Users.successActivate' => 'onSuccessActivate'
        ];
    }

    /**
     * On success user activation profile.
     *
     * @param Event $event
     *
     * @return void
     */
    public function onSuccessActivate(Event $event)

    {
    }

    /**
     * Global before save user data.
     *
     * @param   Event $event
     *
     * @return  void
     */
    public function onModelBeforeSave(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 onModelAfterSave(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.afterSave' => 'onModelAfterSave',
42			'Model.User.beforeSave' => 'onModelBeforeSave',
43			'Controller.Users.successActivate' => 'onSuccessActivate'
44			];
45			}
46
47			/**
48			* On success user activation profile.
49			*
50			* @param Event $event
51			*
52			* @return void
53			*/
54			public function onSuccessActivate(Event $event)
			0 ignored issues – show Unused Code introduced 2018-03-01 19:52 UTC by Report Bug Copy Issue Report The parameter `$event` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
55			{
56			}
57
58			/**
59			* Global before save user data.
60			*
61			* @param Event $event
62			*
63			* @return void
64			*/
65			public function onModelBeforeSave(Event $event)
66			{
67			/** @var User $user */
68			$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...
69			}
70
71			/**
72			* Global after save user data.
73			*
74			* @param Event $event
75			*
76			* @return void
77			*
78			* TODO : Use return when send message?
79			*/
80			public function onModelAfterSave(Event $event)
81			{
82			/** @var User\|bool $user */
83			$user = $event->getData('user');
84
85			if ($user instanceof EntityInterface) {
86			if ($user->isNew() && Filter::bool($user->get('notify'))) {
87			$this->_getMailer($user)->sendCreateMessage();
88			}
89			}
90			}
91
92			/**
93			* Get mailer object.
94			*
95			* @param EntityInterface $entity
96			* @return Email
97			*/
98			protected function _getMailer(EntityInterface $entity)
99			{
100			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...
101			}
102			}
103

CakeCMS / Community

Push — master ( 76aee5...716e15 )

UserEventHandler::onSuccessActivate() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like