Mysqli: Fatal error: Call to a member function prepare() on null

Question

I'm trying to convert a login function into class with some functions, it was working until I converted it into class, this is my code:

class merwaa_login {
public $conn;
public function __construct($conn) {
    $this->conn = $conn;
    if(isset($_POST['login-submit'])) {
    $username = merwaa_clean($_POST['username']);
    $password = merwaa_clean($_POST['password']);
    $sql = $conn->prepare("SELECT * FROM users WHERE username = ? OR email = ?;");
    $sql->bind_param('ss', $username, $username);
    $sql->execute();
    $all = $sql->get_result();
    $all = $all->fetch_assoc();
    if (count($all) === 0){
        echo 'البيانات المدخلة خاطئة!';
    }
    else{
        $hashed_pwd = $all['password'];
        if (password_verify($password,$hashed_pwd)) {
            echo 'كفو';
            $_SESSION['username'] = $username = $all['username'];
            if(isset($POST['rememberme'])) {
                $uid = $all['ID'];
                merwaa_set_jwt($username,$uid);
            }

        } else{
            echo 'خطأ';
        }
    }
}
}
}

May I've some grammar mistakes, please notice me about it.

score 0 · Accepted Answer · answered Sep 26 '15 at 18:39

0

Change this :-

$sql = $conn->prepare("SELECT * FROM users WHERE username = ? OR email = ?;");

To this :-

$sql = $this->conn->prepare("SELECT * FROM users WHERE username = ? OR email = ?;");

Or, you could change public $conn to global $conn. Then you won't need to make the above change.

answered Sep 26 '15 at 18:39

Akshay

2,244
3
15
34

I tried to change `public` to `global`, it says `Parse error: syntax error, unexpected 'global' (T_GLOBAL), expecting function (T_FUNCTION) in`, what the difference? – Maometto Sep 26 '15 at 18:59
when I use the first solution I got the sme `Fatal error: Call to a member function prepare() on null ` – Maometto Sep 26 '15 at 19:01
Are you even including the db connection file in your class file? – Akshay Sep 26 '15 at 19:06
Wow! I include it now, you slove it! thank you! – Maometto Sep 26 '15 at 19:31

score 0 · Answer 2 · answered Sep 26 '15 at 18:40

0

try to use $this->conn->prepare instead of $conn->prepare and a proper connection object should be passed to the constructor

answered Sep 26 '15 at 18:40

haseeb

682
5
16

the same `Fatal error: Call to a member function prepare() on null ` – Maometto Sep 26 '15 at 19:00
you need to pass a proper connection object $conn. like $marwa = new merwaa_login($conn) where $conn is a previously set connection object – haseeb Sep 26 '15 at 19:05
I did `$login = new merwaa_login($conn);` – Maometto Sep 26 '15 at 19:28
but now it's sloved! thanks! – Maometto Sep 26 '15 at 19:28

Mysqli: Fatal error: Call to a member function prepare() on null

2 Answers2