PHP Recoverable Error – yii\base\ErrorException

Hi, i get this error when I tried adding an auto-login feature after the user signs up. I am not sure if I am doing it right but please save me guys

ERROR:




Argument 1 passed to yii\web\User::login() must implement interface yii\web\IdentityInterface, null given, called in C:\xampp\htdocs\jobsweep\frontend\controllers\SiteController.php on line 205 and defined



Here’s the controller:




public function actionSignup()

    {

        

        $model = new SignupForm();

        

        if ($model->load(Yii::$app->request->post())) {

            

            $user = $model->signup();

            

            Yii::$app->getUser()->login($user);  

            

            return $this->actionmyhome();

           

            

        }else{


                return $this->render('signup', [

                    'model' => $model,

                ]);

            

        }

    }



HERE is the model:





public function signup()

    {

        

        if (!$this->validate()) {

            return null;

        }

        

        $user = new User();

        $user->username = $this->email;

        $user->email = $this->email;

        $user->setPassword($this->password);

        $user->generateAuthKey();

        

      

        return $user->save() ? $user : null;





    }//end function




We are actually using a custom User (common\models\User) instead of the yii\web\User. Here it is:





<?php

namespace common\models;


use Yii;

use yii\base\NotSupportedException;

use yii\behaviors\TimestampBehavior;

use yii\db\ActiveRecord;

use yii\web\IdentityInterface;


/**

 * User model

 *

 * @property integer $id

 * @property string $username

 * @property string $password_hash

 * @property string $password_reset_token

 * @property string $email

 * @property string $auth_key

 * @property integer $status

 * @property integer $created_at

 * @property integer $updated_at

 * @property string $password write-only password

 */

class User extends ActiveRecord implements IdentityInterface

{

    const STATUS_DELETED = 0;

    const STATUS_ACTIVE = 10;




    /**

     * @inheritdoc

     */

    public static function tableName()

    {

        return '{{%user}}';

    }


    /**

     * @inheritdoc

     */

    public function behaviors()

    {

        return [

            TimestampBehavior::className(),

        ];

    }


    /**

     * @inheritdoc

     */

    public function rules()

    {

        return [

            ['status', 'default', 'value' => self::STATUS_ACTIVE],

            ['status', 'in', 'range' => [self::STATUS_ACTIVE, self::STATUS_DELETED]],

        ];

    }


    /**

     * @inheritdoc

     */

    public static function findIdentity($id)

    {

        return static::findOne(['id' => $id, 'status' => self::STATUS_ACTIVE]);

    }


    /**

     * @inheritdoc

     */

    public static function findIdentityByAccessToken($token, $type = null)

    {

        throw new NotSupportedException('"findIdentityByAccessToken" is not implemented.');

    }


    /**

     * Finds user by username

     *

     * @param string $username

     * @return static|null

     */

    public static function findByUsername($username)

    {

        return static::findOne(['username' => $username, 'status' => self::STATUS_ACTIVE]);

    }


    /**

     * Finds user by password reset token

     *

     * @param string $token password reset token

     * @return static|null

     */

    public static function findByPasswordResetToken($token)

    {

        if (!static::isPasswordResetTokenValid($token)) {

            return null;

        }


        return static::findOne([

            'password_reset_token' => $token,

            'status' => self::STATUS_ACTIVE,

        ]);

    }


    /**

     * Finds out if password reset token is valid

     *

     * @param string $token password reset token

     * @return boolean

     */

    public static function isPasswordResetTokenValid($token)

    {

        if (empty($token)) {

            return false;

        }


        $timestamp = (int) substr($token, strrpos($token, '_') + 1);

        $expire = Yii::$app->params['user.passwordResetTokenExpire'];

        return $timestamp + $expire >= time();

    }


    /**

     * @inheritdoc

     */

    public function getId()

    {

        return $this->getPrimaryKey();

    }


    /**

     * @inheritdoc

     */

    public function getAuthKey()

    {

        return $this->auth_key;

    }


    /**

     * @inheritdoc

     */

    public function validateAuthKey($authKey)

    {

        return $this->getAuthKey() === $authKey;

    }


    /**

     * Validates password

     *

     * @param string $password password to validate

     * @return boolean if password provided is valid for current user

     */

    public function validatePassword($password)

    {

        return Yii::$app->security->validatePassword($password, $this->password_hash);

    }


    /**

     * Generates password hash from password and sets it to the model

     *

     * @param string $password

     */

    public function setPassword($password)

    {

        $this->password_hash = Yii::$app->security->generatePasswordHash($password);

    }


    /**

     * Generates "remember me" authentication key

     */

    public function generateAuthKey()

    {

        $this->auth_key = Yii::$app->security->generateRandomString();

    }


    /**

     * Generates new password reset token

     */

    public function generatePasswordResetToken()

    {

        $this->password_reset_token = Yii::$app->security->generateRandomString() . '_' . time();

    }


    /**

     * Removes password reset token

     */

    public function removePasswordResetToken()

    {

        $this->password_reset_token = null;

    }

}




User component’s login function expects IdentityInterface as parameter, your signup function returns null if it fails validation or does not save which is the cause of the error




public function signup()

{

    if (!$this->validate()) {

        return null; 

    }


    // ...


    return $user->save() ? $user : null;

}




$user = $model->signup(); // this might return null judging by the above signup fn


Yii::$app->getUser()->login($user);



The signup model is already from the advanced app of Yii2. It’s actually saving the data, I checked the DB and it was there.

The code for the controller below is from Yii2 also,




public function actionSignup()

    {

        $model = new SignupForm();

        if ($model->load(Yii::$app->request->post())) {

            if ($user = $model->signup()) {

                if (Yii::$app->getUser()->login($user)) {

                    return $this->goHome();

                }

            }

        }


        return $this->render('signup', [

            'model' => $model,

        ]);

    }




This was the original one, I just revised the middle part and removed the if statements to see what the model returns back to the controller.

Now, im not sure what to do next. :(