Comments on: PDO For WordPress – First Public Release

By: Justin

Justin — Mon, 19 Nov 2007 23:07:54 +0000

thanks. i’ll cut your earlier comment a bit. it set me on the right track though.

an error had crept in to the mysql_* disabling code in db.php. i had forgotten to escape some round brackets in a regex.

the other error is, i think, also easily fixed. i’ll post revised code shortly

By: fnumatic

fnumatic — Mon, 19 Nov 2007 22:21:21 +0000

oh sorry that’s probably a way to much

my first issue was dealing with
function changeFiles_2_4() in db.php

you want to modify the mysql_get_server_info() statement. and this was happened twice per file.
should be:
_mysql_get_server_info()/*automatically changed by sqlite compat*/
but was:
__mysql_get_server_info()/*automatically changed by sqlite compat*//*automatically changed by sqlite compat*/

By: fnumatic

fnumatic — Mon, 19 Nov 2007 22:11:42 +0000

the sql issue:

Queries made or created this session were

—edited by Justin—

4. Prepare: UPDATE IGNORE wp_posts SET post_author = ? , post_date = ? , post_date_gmt = ? , post_content = ? , post_content_filtered = ? , post_title = ? , post_excerpt = ? , post_status = ? , post_type = ? , comment_status = ? , ping_status = ? , post_password = ? , post_name = ? , to_ping = ? , pinged = ? , post_modified = ? , post_modified_gmt = ? , post_parent = ? , menu_order = ? WHERE ID = 6
—
Error occurred at line 313 in Function prepareQuery.
Error message was: Problem preparing the PDO SQL Statement. Error was Array ( [0] => HY000 [1] => 1 [2] => near “wp_posts”: syntax error )

By: Justin

Justin — Mon, 19 Nov 2007 21:47:49 +0000

rather annoyingly, i’ve just downloaded wp2.3.1 and found a healthy number of errors in my code. i had built and tested on version 2.4-bleeding.

i’m looking into the errors and will post a cumulative bug fix.

By: Justin

Justin — Mon, 19 Nov 2007 21:09:39 +0000

Hi
you mention two issues:

1. schema.php being called twice per script; and
2. an SQL error.

dealing with each in turn

1. this surprises me. schema.php is only called by pdoEngine.php when the database does NOT exist as a file in the filesystem – and should only be callable once per installation. have you by any chance changed the definitions in wp-content/db.php? if not, can you post your changes?

2. this needs looking at and is difficult for me to reproduce as my installation does not suffer from these errors (this blog has been operating error free for weeks). to be certain, i’d need to know what the query was that was being mangled. can you add this definition to your wp-config.php file:
define(‘PDO_DEBUB’, true);
and post the queries that are bailed out? if you don’t fancy posting publicly, then mail me @ justin.adie NEAR adieandco.com