OpenCart has error in core mysqli client library

Standard

Today, while studying the architecture and code of OpenCart v1.5.5.1, I just found an issue with it and wanted to bring in everyone’s notice.

Open the file system/database/mysqli.php and look for the method “query”. It has an unmatched if…else statement. This also proves that OpenCart is not using “mysqli” library. They are still using “mysql” library for database communication.

Here is a screen shot from my system using Eclipse editor:

OpenCart mysqli class error in code

Also, from the screenshot above, what is “mysql_error()” doing in the mysqli communication class?

If you feel that you want to comment on this, please leave your comment.

Thank you!

Advertisements

4 thoughts on “OpenCart has error in core mysqli client library

  1. This is fairly evident. You can see that when you install it that there is only the MySQL connector available. It’s sad, really, that they haven’t moved it to MySQLi considering PHP is deprecating the extension. Maybe it will come soon.

I will be happy to answer your queries

Fill in your details below or click an icon to log in:

WordPress.com Logo

You are commenting using your WordPress.com account. Log Out / Change )

Twitter picture

You are commenting using your Twitter account. Log Out / Change )

Facebook photo

You are commenting using your Facebook account. Log Out / Change )

Google+ photo

You are commenting using your Google+ account. Log Out / Change )

Connecting to %s