Skip to content

Instantly share code, notes, and snippets.

@mbabker
Last active July 21, 2016 18:30
Show Gist options
  • Star 1 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save mbabker/c7382e40e1914ecc950032f85190b304 to your computer and use it in GitHub Desktop.
Save mbabker/c7382e40e1914ecc950032f85190b304 to your computer and use it in GitHub Desktop.
Joomla! Prepared Statement Support

Joomla developers! Your support is needed to help make prepared statements in Joomla's database API a reality. I've started on some of this work but need help with testing and review.

Joomla's PDO driver already has provisions for prepared statement support written into it and the unsupported Oracle and SQLite drivers already make use of this.

In the Joomla! Framework Database Package I've started on work to expand this support. First, the PDO MySQL driver now has bound variable support and will fully support prepared statements.

I've opened pull requests for the MySQLi and PostgreSQL drivers to add support for prepared statements using the already defined interfaces in Joomla's database layer. Also, there is a pull request adding a PDO based PostgreSQL driver to the available options, also with prepared statement support already integrated.

This leaves only the SQL Server driver (in the Framework stack) without an active pull request to add this support.

Once these changes are well tested and reviewed, it is my intent to backport the same changes to the Joomla! CMS classes and add prepared statement support for all but one of the available drivers. I say most since the CMS still ships with and supports a database driver for the deprecated ext/mysql. Without creating an interface to emulate prepared statements, unfortunately the best we can do in the CMS is provide the API to allow prepared statements to be used but cannot actually start using them in the core APIs until support for the deprecated driver is removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment