#314464: Convert db_placeholders() to DBTNG introduced a second argument for DatabaseConnection::prepareQuery($query, $cache) (patch), but it was never used.

CommentFileSizeAuthor
#2 prepareQuery-2.patch1.59 KBc960657
prepareQuery-1.patch1.53 KBc960657
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Crell’s picture

Crell CreditAttribution: Crell commented
Status: Needs review » Closed (duplicate)

I believe this is also included in #550124: Remove prepared statement caching, which is now asking for benchmarks to see if the cache is even worthwhile. So marking as dupe.

c960657’s picture

c960657 CreditAttribution: c960657 commented
Status: Closed (duplicate) » Needs review
FileSize
prepareQuery-2.patch1.59 KB

#550124: Remove prepared statement caching has been postponed to D8, so we need to fix this bug in D7.

Crell’s picture

Crell CreditAttribution: Crell commented
Status: Needs review » Reviewed & tested by the community

Makes sense. I forget why we added that capability off hand, to be honest. :-)

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed! Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.