Thread Starter
ronbme
(@ronbme)
Rachel, this should help:
Changeset 22429
Timestamp:
11/07/12 19:30:59 (3 weeks ago)
Author:
nacin
Message:
Enforce a minimum of two arguments for wpdb::prepare(). The first argument is the query (or fragment thereof), which is required. Additional arguments are values to substitute into placeholders.
This will generate E_WARNINGs for insufficient arguments when prepare() is called with no additional arguments. This should discourage improper uses of prepare() under the guise of safely running a query.
props xknown. fixes #22262.
It looks like they are forcing wpdb to have a second argument.
Ron
Thanks Ron!! I’ll get it all fixed up for the next update.
I’ve got this all fixed for the next update! Thanks for the heads up!
Thread Starter
ronbme
(@ronbme)
Great! Thanks Rachel!
I’m looking forward to it.
Ron
Wouldn’t it be easy to simply add some code to line 990 in wp-db.php?
function prepare( $query, $args) {
Becomes
function prepare( $query, $args = "" ) {
Hi there.
First things first, thanks for the nice plugin !
Now, WP 3.5 has gone public and I need a fix as the update to wp-db broke quite a bunch of functionalities of my current project.
Do you plan to release the update anytime soon ?
Cheers !
p.
I hope to push out the update tomorrow!
That would be perfect !
Thanks for the quick answer ! Keep up the good work !
Thank u Rachel !
I going to make a fix quick in waiting the update tomorow.
B Regards,
I’m also getting the error on line 1348 of cpt_onomy.php.
You can just add “, NULL” to the end of the wpdb::prepare() statement.
Ross
CPT-onomies 1.2 is out and should take care of this. Thanks!
