Query working on phpmyadmin but not in php script

Question

I already tried other solutions from similar questions but it seems that nothing worked. Like the title says, the query I need to execute on my db works fine on phpMyAdmin, but it doesn't work when used inside a php function. This is the code in common_functions.php:

function eliminaRistorante($id_ristorante) {
    global $conn;

    // query su ristorante
    $sql = "DELETE FROM ristorante WHERE id_ristorante = $id_ristorante";

    echo $sql;

    if($conn->query($sql) === TRUE) {
        return true;
    } else {
        echo "Error: " . $sql . "<br>" . $conn->error;
        return false;
    }
}

This is the code in hold.php:

include('includes/logic/common_functions.php');

if (isset($_POST['del'])) {
    $id_ristorante = $_POST['del'];

    eliminaRistorante($id_ristorante);
    header('location: admin/ristorante.php');
}

I put an echo statement to see if the query was effectively correct. This is the output:

DELETE FROM ristorante WHERE id_ristorante = 5 Fatal error: Uncaught Error: Call to a member function query() on null in C:\xampp\htdocs\projects\db_ristorante\includes\logic\common_functions.php:57 Stack trace: #0 C:\xampp\htdocs\projects\db_ristorante\hold.php(8): eliminaRistorante('5') #1 {main} thrown in C:\xampp\htdocs\projects\db_ristorante\includes\logic\common_functions.php on line 57

Lastly, I tried these solutions: MySql query not in working in PHP but works in phpMyAdmin

Inside that question the user kindly shared other solutions that I considered too.

Call to a member function query() on null does not mean there was a problem with your query, it means that $conn was null. — C3roe
– C3roe, Commented Mar 18, 2021 at 11:46
Better to pass $conn into your function as a parameter than to use a global variable. Also you need to look at prepared statements rather than concatenating strings into queries. — droopsnoot
– droopsnoot, Commented Mar 18, 2021 at 11:48
Thanks to both I got the reason why the error popped up, I did as @droopsnoot suggested and passed $conn as a parameter, that solved the problem. — Sergio
– Sergio, Commented Mar 18, 2021 at 15:45

jpgerdeman · Accepted Answer · 2021-03-18 11:47:29Z

1

Uncaught Error: Call to a member function query() on null in

your $conn Object is null. You did not initialise a PDO instance.

Here is a introduction, on how to do that, PHP The Right Way - PDO

answered Mar 18, 2021 at 11:47

jpgerdeman

827 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Query working on phpmyadmin but not in php script

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related