Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Wierd and Annoying error: Call to undefined function mysql_query() [duplicate]

I've been at this for more than an hour now, I can connect to my database and all (doesn't give any errors) but i get the following error when i try to use "mysql_query($query);"

Call to undefined function mysql_query()

I've already tried enabling:

extension=php_mysql.dll
extension=php_mysqli.dll

and I've added:

extension_dir = "ext"
extension=php_mysql.dll

at the end of the import list.

Between every change I made I restarted all processes in exampp and after every single solution that I seperately tried it still gives this annoying error.

I've tried looking online, but the only error similar to mine would be:

Call to undefined function mysql_connect()

which at no point I've have had any issue with, never seen it.

I'm asking it here because I am being spammed on the google search results with the problem above; Not the one I am having.

I would be greatful if someone could help me with this, the tutorials I've found online show the exact same code that I have written.

phpInfo: top phpInfo: sql

like image 524
user2870011 Avatar asked Jan 26 '16 23:01

user2870011


2 Answers

PHP 7 has gotten rid of mysql_query() because it's problematic in a variety of ways! The API does not encourage good practices, the official line is that it's unmaintained, and NO PREPARED STATEMENTS!?! It essentially is the biggest problem in PHP that encourages bad practices leading to sql injection, and that's a major big bad situation.

However, I do work with legacy codebases, so I have to deal with the same situation as you in some cases. If you have a small codebase, just update your db connection method. If you have a large codebase, here is what I recommend:

  • Rollback your php version for this codebase to php 5.6, it will be supported for a bit more of 2016.
  • Take your time to update to PDO (you can create a wrapper around PDO to make it less verbose and still allow prepared queries).
  • Ignore mysqli. If it takes you 5 minutes to upgrade from mysql_*, you are probably doing it wrong, and leaving yourself open to sql-injection. Just go for PDO and start using prepared queries so you can sleep at night.
  • If you still want to use php 7 in more modern projects, spin up a container instance with the older php 5.6 legacy codebases on it.
like image 185
Kzqai Avatar answered Oct 19 '22 04:10

Kzqai


The answer is simple and this information has been available long before the release of PHP 7. It has been removed and they suggest to move over to mysqli or PDO. For a complete list of changes you need to know about for migration see this guide.


Your options:

  • Adjust your code to mysqli, which is quite the same just a little different. This wouldn't take long to adjust your code to.
  • Switch over to PDO, quite different but more flexible and has my preference.
  • The mysql_* have been removed, meaning they are open to be redefined. You can create wrapper functions that refer to MySqli or PDO instead.
  • Switch back to version 6 of PHP.

Why has it been removed?

  • It's not under development.
  • The mysql_* functions provide just a piece of functionality of what MySQL really has to offer. (think about transactions, prepared statements, asynchronous queries, etc)
  • People are still writing (even today) insecure code with those functions.

I'm not saying that using MySqli or PDO will magically prevent MySQL injections but at least they provide native support against those kind of attacks. The rest is up to you; the programmer, to make sure to point data where it needs to go.

like image 4
Xorifelse Avatar answered Oct 19 '22 05:10

Xorifelse