codeigniter 代码触发器4:在var_dump()中发布请求有效的空值上调用成员函数

svgewumm  于 2022-12-07  发布在  其他
关注(0)|答案(1)|浏览(115)

对CodeIgniter非常陌生,所以请友好!

我的两个用户身份验证表单有问题:users/register.phpusers/login.php,其中我无法将post输入传递给模型中的函数。
到目前为止,我在注册表单上得到了错误Call to member function addUser() on null,在登录表单上得到了一个验证错误,表明用户名/密码与数据库中的任何凭据都不匹配。这两个错误似乎都是因为post为空,尽管它不是。
我已经在$login上做了一个var_dump,它被定义为$login = $this->request->getPost(),并且检查了Firefox Developers Browser中的请求,以找到正确显示的所有post数据。我被难住了。为什么我不能将这个数组传递给我的模型?
下面是login.php的post请求的屏幕截图(registration.php也是如此,不包括在内)。

这些是我的路线:

// Login and Registration
$routes->match(['get', 'post'], 'users/register', 'Users::register');
$routes->match(['get', 'post'], 'users/login', 'Users::login', ["filter" => "noauth"]);

下面是我的模型UserModel.php的全部内容:

class UserModel extends Model
{
    protected $DBGroup          = 'default';
    protected $table            = 'users';
    protected $primaryKey       = 'username';
    protected $useAutoIncrement = false;
    protected $insertID         = 0;
    protected $returnType       = 'object';
    protected $useSoftDelete    = false;
    protected $allowedFields    = [
        'username',
        'password',
        'id',
        'role',
        'profile_image',
        'profile_views',
        'last_login',
        'about_me',
        'age',
        'gender',
        'occupation',
        'hometown',
        'country',
        'fav_shape',
        'fav_color',
        'created',
        'modified',
    ];

    // Dates
    protected $useTimestamps    = true;
    protected $dateFormat       = 'datetime';
    protected $createdField     = 'created';
    protected $modifiedField    = 'modified';

    // Callbacks
    protected $allowCallbacks   = true;
    protected $beforeInsert     = ['beforeInsert'];

    public function __construct() 
    {
        parent::__construct(); 
    }

    protected function beforeInsert(array $data)
    {
        $data = $this->passwordHash($data);

        return $data;
    }

    protected function passwordHash(array $data)
    {
        if (isset($data['password'])) {

            $data['password'] = password_hash($data['password'], PASSWORD_DEFAULT);
        }

        return $data;
    }

       public function lastLogin($username)
    {
        $this->builder()
            ->where('username', $username)
            ->update('last_login', date('Y-m-d H:i:s'));
    }

    public function addUser($newUser)
    {
        $builder = $this->builder()
            ->set($newUser)
            ->insert();

        if ($builder->affected_rows() == 1) {

            return TRUE;
        
        } else {

            return FALSE;
        }
    }

    public function getUser($username)
    {
        $builder = $this->builder()
            ->where(['username' => $username])
            ->limit(1);

        if ($builder->countAllResults() === 1) {
            
            return $builder->get()->getRow();
        
        } else {

            return FALSE;
        }
    }
}

以下是我的控制器Users.php的摘录:

class Users extends BaseController
{
    protected $userModel;

    public function __construct() 
    {
        $userModel = new UserModel();
    }

    public function login()
    {
        $validation = \Config\Services::validation();

        // Set session variable
        $session = session();

        if ($this->request->getMethod() === 'post' && ! empty($_POST)) {   
            $validation->getRuleGroup('login');
            $validation->setRuleGroup('login');
            $validation->withRequest($this->request)->run(); 

            $recaptchaResponse = trim($this->request->getVar('g-recaptcha-response'));

            $userIp = $this->request->getIPAddress();

            $secret = env('recaptcha2_secretkey');

            $credential = [
                'secret'    => $secret,
                'response'  => $recaptchaResponse,
                'remoteip'  => $userIp,
            ];

            $verify = curl_init();

            curl_setopt($verify, CURLOPT_URL, 'https://www.google.com/recaptcha/api/siteverify');
            curl_setopt($verify, CURLOPT_POST, TRUE);
            curl_setopt($verify, CURLOPT_POSTFIELDS, http_build_query($credential));
            curl_setopt($verify, CURLOPT_SSL_VERIFYPEER, FALSE);
            curl_setopt($verify, CURLOPT_RETURNTRANSFER, TRUE);

            $response = curl_exec($verify);

            $status = json_decode($response, TRUE);

            curl_close($verify);

            if (empty($validation->getErrors()) && $status['success']) {
                $login = $this->request->getPost();

                $user = $this->userModel->getUser($login['username']);

                // Storing session values
                $this->setUserSession($user);

                // Storing success message
                $session->setFlashdata('success', 'You have successfully logged in!');

                // Update last login datetime
                $this->userModel->lastLogin($login['username']);

                // Redirecting to dashboard after login
                if ($user['role'] == 1) {

                    return redirect()->to('admin/dashboard');
                
                } elseif ($user['role'] == 0) {

                    return redirect()->to('members/dashboard');
                }
            } else {

                $data = [
                    'title'     => 'Login',
                    'errors'    => $validation->getErrors(),
                ];

                echo view('templates/index_header', $data);
                echo view('users/login');
                echo view('templates/footer', $data);
            }   
        } else {
            $data = [
                'title' => 'Login',
            ];

                echo view('templates/index_header', $data);
                echo view('users/login');
                echo view('templates/footer', $data);  
        }
    }

    /**
     * Sets session with user id, username, isLoggedIn, and role for use in member/admin site
     * @param model user data
     * @return boole if session was set successfully
     */
    private function setUserSession($user)
    {
        $data = [
            'id' => $user->id,
            'username' => $user->username,
            'profile_image' => $user->profile_image,
            'isLoggedIn' => true,
            'role' => $user->role,
        ];

        if (session()->set($data)) {
            
            return true;
        
        } else {

            return false;
        }
    }

    public function register() 
    {
        $validation = \Config\Services::validation();

        if ($this->request->getMethod() == 'post' && ! empty($_POST)) {

            $validation->getRuleGroup('registration');
            $validation->setRuleGroup('registration');
            $validation->withRequest($this->request)->run(); 

            if (empty($validation->getErrors())) {

                $newUser = $this->request->getPost();

                if ($this->userModel->addUser($newUser)) {

                    $this->session->setFlashdata('success', 'Successful Registration');

                    $data['title'] = 'Login';

                    echo view('templates/index_header', $data);
                    echo view('users/login');
                    echo view('templates/footer', $data);

                } else {

                    $this->session->setFlashdata('error', 'Something went wrong with your registration! Please try again.');
                }

            } else {

                $data = [];

                $data = [
                    'title'     => 'Register',
                    'script'    => 'js/click_link',
                    'errors'    => $validation->getErrors(),
                ];
                
                echo view('templates/index_header', $data);
                echo view('users/register', $data);
                echo view('templates/footer', $data);
            }
        } else {
            $data = [
                'title'         => 'Register',
                'script'        => 'js/click_link',
            ];
        
            echo view('templates/index_header', $data);
            echo view('users/register', $data);
            echo view('templates/footer', $data);
        }
    }
}

下面是我在Config\Validation中的验证规则:

/**
     *  Registration
     */
    public $registration = [
        'username'      => 'required|is_unique[users.username,username]|min_length[5]|max_length[25]|alpha_dash|badWordsFilter[username]',
        'password'      => 'required|min_length[8]|max_length[255]|regex_match[/^(?=.*[!@#$%^&*-])(?=.*[0-9])(?=.*[A-Z]).{8,255}$/]',
        'pass_confirm'  => 'required|matches[password]',
        'about_me'      => 'permit_empty|max_length[250]|alpha_numeric_punct|badWordsFilter[about_me]',
        'occupation'    => 'permit_empty|max_length[50]|alpha_space|badWordsFilter[occupation]',
        'hometown'      => 'permit_empty|max_length[50]|alpha_space|badWordsFilter[hometown]',
        'age'           => 'permit_empty|less_than[100]|greater_than[0]|numeric',
        'country'       => 'permit_empty',
    ];
/**
     *  Password Verification
     */
    public $login = [
        'password'      => 'required|validateUser[username,password]',
    ];

这是我验证用户名和密码凭据的自定义规则User_rules

class User_rules 
{
    /**
     * Checks if input username exists in database and then checks whether the input password matches the hash for that username
     * @param string $str is the input password
     * @param string $fields are the associated form fields that are being used
     * @param array $data is an array containing the values for the fields indexed by field names
     * @return boolean true or false depending on if the user exists and the password matches the hashed password stored in the database
     */
    public function validateUser(string $str, string $fields, array $data)
    {
        $userModel = new UserModel();

        $user = $userModel->getUser($data['username']);

        if(!$user) {

            return FALSE;
        }

        return password_verify($data['password'], $user->password);
    }

最后,我对login.php的看法是:

<div class='form-container'>

    <?= form_open('users/login',['autocomplete' => FALSE]); ?>

        <div class='form-header'>
            <h2>Login</h2>
        </div>
        
        <div class='form-body'>

            <div class='form-row'>
                <div class='input-container'>
                    <i class='fas fa-user'></i>

                    <?php $attributes = [
                        'type'      => 'text',
                        'name'      => 'username',
                        'class'     => 'input-field',
                        'id'        => 'username',
                        'placeholder'   => 'Username',
                        'required'      => TRUE,
                    ]; ?>
                    <?= form_input($attributes); ?>
                
                </div>
            </div>

            <div class='form-row'>
                <div class='input-container'>
                    <i class='fas fa-lock'></i>

                    <?php $attributes = [
                        'type'          => 'password',
                        'name'          => 'password',
                        'class'         => 'input-field',
                        'placeholder'   => 'Password',
                        'required'      => TRUE,
                    ]; ?>
                    <?= form_input($attributes); ?>
                  
                </div`>
            </div>
        </div>
                
        <div class='captcha-container'>
            <div class='g-recaptcha' data-sitekey='<?= env('recaptcha2_sitekey'); ?>'></div>
        </div>

        <div class='form-footer'>

            <?php $submit = [
                    'name'      => 'loginSubmit',
                    'value'     => 'Login',
                    'class'     => 'submit-btn',
            ];?>
            <?= form_submit($submit); ?>
    
        </div>

        <h4 style='text-align: center'>Not a member yet? Register 
            <a href= <?= site_url('users/register'); ?> title = 'Register'> HERE</a>
        </h4>

     <?= form_close(); ?>
</div>
zvokhttg

zvokhttg1#

这是一个愚蠢的错误。有人在codeigniter论坛上回答了我的问题在这里:CodeIgniter论坛
基本上,在我的构造函数中,我需要$this->userModel = new UserModel();而不是$userModel = new UserModel();

相关问题