AuthUserBehavior::errorSignup() - Code Metrics - yiicod/yii2-auth - Measure and Improve Code Quality continuously with Scrutinizer

AuthUserBehavior::errorSignup() A
last analyzed 2018-02-18 19:33 UTC

↳ Parent: AuthUserBehavior

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace yiicod\auth\controllers\behaviors;

/*
 * Auth behavior with event for controller action
 * @author Orlov Alexey <[email protected]>
 */

use Yii;
use yii\base\ActionEvent;
use yii\base\Behavior;
use yii\base\Event;
use yii\web\BadRequestHttpException;
use yiicod\auth\actions\webUser\LoginAction;
use yiicod\auth\actions\webUser\LogoutAction;
use yiicod\auth\actions\webUser\RequestPasswordResetAction;
use yiicod\auth\actions\webUser\ResetPasswordAction;
use yiicod\auth\actions\webUser\SignupAction;
use yiicod\auth\events\LoginEvent;
use yiicod\auth\events\RequestPasswordResetEvent;
use yiicod\auth\events\ResetPasswordEvent;
use yiicod\auth\events\SignupEvent;

class AuthUserBehavior extends Behavior
{
    /**
     * Assign all events
     */
    public function init()
    {
        // Signup
        Event::on(SignupAction::class, SignupAction::EVENT_BEFORE_SIGNUP, [$this, 'beforeSignup']);
        Event::on(SignupAction::class, SignupAction::EVENT_AFTER_SIGNUP, [$this, 'afterSignup']);
        Event::on(SignupAction::class, SignupAction::EVENT_ERROR_SIGNUP, [$this, 'errorSignup']);

        // Login
        Event::on(LoginAction::class, LoginAction::EVENT_BEFORE_LOGIN, [$this, 'beforeLogin']);
        Event::on(LoginAction::class, LoginAction::EVENT_AFTER_LOGIN, [$this, 'afterLogin']);
        Event::on(LoginAction::class, LoginAction::EVENT_ERROR_LOGIN, [$this, 'errorLogin']);

        // Logout
        Event::on(LogoutAction::class, LogoutAction::EVENT_BEFORE_LOGOUT, [$this, 'beforeLogout']);
        Event::on(LogoutAction::class, LogoutAction::EVENT_AFTER_LOGOUT, [$this, 'afterLogout']);

        // ResetPassword
        Event::on(ResetPasswordAction::class, ResetPasswordAction::EVENT_BEFORE_RESET_PASSWORD, [$this, 'beforeResetPassword']);
        Event::on(ResetPasswordAction::class, ResetPasswordAction::EVENT_AFTER_RESET_PASSWORD, [$this, 'afterResetPassword']);
        Event::on(ResetPasswordAction::class, ResetPasswordAction::EVENT_ERROR_RESET_PASSWORD, [$this, 'errorResetPassword']);

        // RequestPasswordReset
        Event::on(RequestPasswordResetAction::class, RequestPasswordResetAction::EVENT_BEFORE_REQUEST_PASSWORD_RESET, [$this, 'beforeRequestPasswordReset']);
        Event::on(RequestPasswordResetAction::class, RequestPasswordResetAction::EVENT_AFTER_REQUEST_PASSWORD_RESET, [$this, 'afterRequestPasswordReset']);
        Event::on(RequestPasswordResetAction::class, RequestPasswordResetAction::EVENT_ERROR_REQUEST_PASSWORD_RESET, [$this, 'errorRequestPasswordReset']);
    }

    /**
     * After login action event
     *
     * @param LoginEvent $event
     */
    public function afterLogin($event)
    {
        $event->action->controller->goHome();
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

        Yii::$app->end();
    }

    /**
     * After signup action event
     *
     * @param SignupEvent $event
     */
    public function afterSignUp($event)
    {
        if (Yii::$app->getUser()->login($event->user)) {
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            $event->action->controller->goHome();
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

            Yii::$app->end();
        }
    }

    /**
     * After RequestPasswordReset action event
     *
     * @param RequestPasswordResetEvent $event
     */
    public function afterRequestPasswordReset($event)
    {
        $mailerViewPath = Yii::$app->mailer->viewPath;

        Yii::$app->mailer->viewPath = '@yiicod/yii2-auth/mail';
        Yii::$app->mailer->compose('passwordResetToken', ['action' => $event->action, 'user' => $event->model->findUser()])
            ->setFrom([Yii::$app->params['supportEmail'] => Yii::$app->name . ' robot'])
            ->setTo($event->model->email)
            ->setSubject('Password reset for ' . Yii::$app->name)
            ->send();

        Yii::$app->getSession()->setFlash('success', 'Check your email for further instructions.');
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

        Yii::$app->mailer->viewPath = $mailerViewPath;

        $event->action->controller->goHome();
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

        Yii::$app->end();
    }

    /**
     * After ResetPassword action event
     *
     * @param ResetPasswordEvent $event
     */
    public function afterResetPassword($event)
    {
        Yii::$app->getSession()->setFlash('success', 'New password was saved.');
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

        $event->action->controller->goHome();
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

        Yii::$app->end();
    }

    /**
     * Before login action event
     *
     * @param LoginEvent $event
     *
     * @return mixed
     */
    public function beforeLogin($event)
    {
        if (false === Yii::$app->user->isGuest) {
            $event->action->controller->goHome();
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            Yii::$app->end();
        }
    }

    /**
     * Before signup action event
     *
     * @param SignupEvent $event
     */
    public function beforeSignup($event)

    {
    }

    /**
     * Before RequestPasswordReset action event
     *
     * @param RequestPasswordResetEvent $event
     */
    public function beforeRequestPasswordReset($event)

    {
    }

    /**
     * Before ResetPassword action event
     *
     * @param ResetPasswordEvent $event
     */
    public function beforeResetPassword($event)

    {
    }

    /**
     * error ResetPassword action event
     *
     * @param ResetPasswordEvent $event
     *
     * @throws BadRequestHttpException
     */
    public function errorResetPassword($event)
    {
        throw new BadRequestHttpException($event->e->getMessage());
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
    }

    /**
     * Error RequestPasswordReset action event
     *
     * @param RequestPasswordResetEvent $event
     */
    public function errorRequestPasswordReset($event)

    {
        Yii::$app->getSession()->setFlash('error', 'Sorry, we are unable to reset password for email provided.');
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
    }

    /**
     * @param ActionEvent $event
     */
    public function afterLogout($event)

    {
    }

    /**
     * @param ActionEvent $event
     */
    public function beforeLogout($event)

    {
    }

    /**
     * @param LoginEvent $event
     */
    public function errorLogin($event)

    {
    }

    /**
     * @param SignupEvent $event
     */
    public function errorSignup($event)

    {
    }
}


AuthUserBehavior::errorSignup() A
last analyzed 2018-02-18 19:33 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1			<?php
2
3			namespace yiicod\auth\controllers\behaviors;
4
5			/*
6			* Auth behavior with event for controller action
7			* @author Orlov Alexey <[email protected]>
8			*/
9
10			use Yii;
11			use yii\base\ActionEvent;
12			use yii\base\Behavior;
13			use yii\base\Event;
14			use yii\web\BadRequestHttpException;
15			use yiicod\auth\actions\webUser\LoginAction;
16			use yiicod\auth\actions\webUser\LogoutAction;
17			use yiicod\auth\actions\webUser\RequestPasswordResetAction;
18			use yiicod\auth\actions\webUser\ResetPasswordAction;
19			use yiicod\auth\actions\webUser\SignupAction;
20			use yiicod\auth\events\LoginEvent;
21			use yiicod\auth\events\RequestPasswordResetEvent;
22			use yiicod\auth\events\ResetPasswordEvent;
23			use yiicod\auth\events\SignupEvent;
24
25			class AuthUserBehavior extends Behavior
26			{
27			/**
28			* Assign all events
29			*/
30			public function init()
31			{
32			// Signup
33			Event::on(SignupAction::class, SignupAction::EVENT_BEFORE_SIGNUP, [$this, 'beforeSignup']);
34			Event::on(SignupAction::class, SignupAction::EVENT_AFTER_SIGNUP, [$this, 'afterSignup']);
35			Event::on(SignupAction::class, SignupAction::EVENT_ERROR_SIGNUP, [$this, 'errorSignup']);
36
37			// Login
38			Event::on(LoginAction::class, LoginAction::EVENT_BEFORE_LOGIN, [$this, 'beforeLogin']);
39			Event::on(LoginAction::class, LoginAction::EVENT_AFTER_LOGIN, [$this, 'afterLogin']);
40			Event::on(LoginAction::class, LoginAction::EVENT_ERROR_LOGIN, [$this, 'errorLogin']);
41
42			// Logout
43			Event::on(LogoutAction::class, LogoutAction::EVENT_BEFORE_LOGOUT, [$this, 'beforeLogout']);
44			Event::on(LogoutAction::class, LogoutAction::EVENT_AFTER_LOGOUT, [$this, 'afterLogout']);
45
46			// ResetPassword
47			Event::on(ResetPasswordAction::class, ResetPasswordAction::EVENT_BEFORE_RESET_PASSWORD, [$this, 'beforeResetPassword']);
48			Event::on(ResetPasswordAction::class, ResetPasswordAction::EVENT_AFTER_RESET_PASSWORD, [$this, 'afterResetPassword']);
49			Event::on(ResetPasswordAction::class, ResetPasswordAction::EVENT_ERROR_RESET_PASSWORD, [$this, 'errorResetPassword']);
50
51			// RequestPasswordReset
52			Event::on(RequestPasswordResetAction::class, RequestPasswordResetAction::EVENT_BEFORE_REQUEST_PASSWORD_RESET, [$this, 'beforeRequestPasswordReset']);
53			Event::on(RequestPasswordResetAction::class, RequestPasswordResetAction::EVENT_AFTER_REQUEST_PASSWORD_RESET, [$this, 'afterRequestPasswordReset']);
54			Event::on(RequestPasswordResetAction::class, RequestPasswordResetAction::EVENT_ERROR_REQUEST_PASSWORD_RESET, [$this, 'errorRequestPasswordReset']);
55			}
56
57			/**
58			* After login action event
59			*
60			* @param LoginEvent $event
61			*/
62			public function afterLogin($event)
63			{
64			$event->action->controller->goHome();
			0 ignored issues – show Bug introduced 2018-02-18 17:33 UTC by Report Bug Copy Issue Report The method `goHome` does only exist in `yii\web\Controller`, but not in `yii\base\Controller and yii\console\Controller`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
65
66			Yii::$app->end();
67			}
68
69			/**
70			* After signup action event
71			*
72			* @param SignupEvent $event
73			*/
74			public function afterSignUp($event)
75			{
76			if (Yii::$app->getUser()->login($event->user)) {
			0 ignored issues – show Bug introduced 2018-02-18 17:33 UTC by Report Bug Copy Issue Report The method `getUser` does only exist in `yii\web\Application`, but not in `yii\console\Application`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
77			$event->action->controller->goHome();
			0 ignored issues – show Bug introduced 2018-02-18 17:33 UTC by Report Bug Copy Issue Report The method `goHome` does only exist in `yii\web\Controller`, but not in `yii\base\Controller and yii\console\Controller`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
78
79			Yii::$app->end();
80			}
81			}
82
83			/**
84			* After RequestPasswordReset action event
85			*
86			* @param RequestPasswordResetEvent $event
87			*/
88			public function afterRequestPasswordReset($event)
89			{
90			$mailerViewPath = Yii::$app->mailer->viewPath;
91
92			Yii::$app->mailer->viewPath = '@yiicod/yii2-auth/mail';
93			Yii::$app->mailer->compose('passwordResetToken', ['action' => $event->action, 'user' => $event->model->findUser()])
94			->setFrom([Yii::$app->params['supportEmail'] => Yii::$app->name . ' robot'])
95			->setTo($event->model->email)
96			->setSubject('Password reset for ' . Yii::$app->name)
97			->send();
98
99			Yii::$app->getSession()->setFlash('success', 'Check your email for further instructions.');
			0 ignored issues – show Bug introduced 2018-02-18 17:33 UTC by Report Bug Copy Issue Report The method `getSession` does only exist in `yii\web\Application`, but not in `yii\console\Application`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
100
101			Yii::$app->mailer->viewPath = $mailerViewPath;
102
103			$event->action->controller->goHome();
			0 ignored issues – show Bug introduced 2018-02-18 17:33 UTC by Report Bug Copy Issue Report The method `goHome` does only exist in `yii\web\Controller`, but not in `yii\base\Controller and yii\console\Controller`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
104
105			Yii::$app->end();
106			}
107
108			/**
109			* After ResetPassword action event
110			*
111			* @param ResetPasswordEvent $event
112			*/
113			public function afterResetPassword($event)
114			{
115			Yii::$app->getSession()->setFlash('success', 'New password was saved.');
			0 ignored issues – show Bug introduced 2018-02-18 17:33 UTC by Report Bug Copy Issue Report The method `getSession` does only exist in `yii\web\Application`, but not in `yii\console\Application`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
116
117			$event->action->controller->goHome();
			0 ignored issues – show Bug introduced 2018-02-18 17:33 UTC by Report Bug Copy Issue Report The method `goHome` does only exist in `yii\web\Controller`, but not in `yii\base\Controller and yii\console\Controller`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
118
119			Yii::$app->end();
120			}
121
122			/**
123			* Before login action event
124			*
125			* @param LoginEvent $event
126			*
127			* @return mixed
128			*/
129			public function beforeLogin($event)
130			{
131			if (false === Yii::$app->user->isGuest) {
132			$event->action->controller->goHome();
			0 ignored issues – show Bug introduced 2018-02-18 17:33 UTC by Report Bug Copy Issue Report The method `goHome` does only exist in `yii\web\Controller`, but not in `yii\base\Controller and yii\console\Controller`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
133			Yii::$app->end();
134			}
135			}
136
137			/**
138			* Before signup action event
139			*
140			* @param SignupEvent $event
141			*/
142			public function beforeSignup($event)
			0 ignored issues – show Unused Code introduced 2018-02-18 17:33 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...
143			{
144			}
145
146			/**
147			* Before RequestPasswordReset action event
148			*
149			* @param RequestPasswordResetEvent $event
150			*/
151			public function beforeRequestPasswordReset($event)
			0 ignored issues – show Unused Code introduced 2018-02-18 17:33 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...
152			{
153			}
154
155			/**
156			* Before ResetPassword action event
157			*
158			* @param ResetPasswordEvent $event
159			*/
160			public function beforeResetPassword($event)
			0 ignored issues – show Unused Code introduced 2018-02-18 17:33 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...
161			{
162			}
163
164			/**
165			* error ResetPassword action event
166			*
167			* @param ResetPasswordEvent $event
168			*
169			* @throws BadRequestHttpException
170			*/
171			public function errorResetPassword($event)
172			{
173			throw new BadRequestHttpException($event->e->getMessage());
			0 ignored issues – show Documentation introduced 2018-02-18 17:33 UTC by Report Bug Copy Issue Report The property `e` does not exist on `object<yiicod\auth\events\ResetPasswordEvent>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
174			}
175
176			/**
177			* Error RequestPasswordReset action event
178			*
179			* @param RequestPasswordResetEvent $event
180			*/
181			public function errorRequestPasswordReset($event)
			0 ignored issues – show Unused Code introduced 2018-02-18 17:33 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...
182			{
183			Yii::$app->getSession()->setFlash('error', 'Sorry, we are unable to reset password for email provided.');
			0 ignored issues – show Bug introduced 2018-02-18 17:33 UTC by Report Bug Copy Issue Report The method `getSession` does only exist in `yii\web\Application`, but not in `yii\console\Application`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
184			}
185
186			/**
187			* @param ActionEvent $event
188			*/
189			public function afterLogout($event)
			0 ignored issues – show Unused Code introduced 2018-02-18 17:33 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...
190			{
191			}
192
193			/**
194			* @param ActionEvent $event
195			*/
196			public function beforeLogout($event)
			0 ignored issues – show Unused Code introduced 2018-02-18 17:33 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...
197			{
198			}
199
200			/**
201			* @param LoginEvent $event
202			*/
203			public function errorLogin($event)
			0 ignored issues – show Unused Code introduced 2018-02-18 17:33 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...
204			{
205			}
206
207			/**
208			* @param SignupEvent $event
209			*/
210			public function errorSignup($event)
			0 ignored issues – show Unused Code introduced 2018-02-18 17:33 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...
211			{
212			}
213			}
214

yiicod / yii2-auth

AuthUserBehavior::errorSignup() A last analyzed 2018-02-18 19:33 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

AuthUserBehavior::errorSignup() A
last analyzed 2018-02-18 19:33 UTC