Replace deprecated database function calls with proper API calls, e.g. db_select().

CommentFileSizeAuthor
remove_deprecated_dbselect.patch768 bytesjalpesh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jalpesh created an issue. See original summary.

Status: Needs review » Needs work

The last submitted patch, remove_deprecated_dbselect.patch, failed testing.

The last submitted patch, remove_deprecated_dbselect.patch, failed testing.

The last submitted patch, remove_deprecated_dbselect.patch, failed testing.

aks22’s picture

aks22 CreditAttribution: aks22 as a volunteer and at Cybage Software Pvt Ltd. commented
Status: Needs work » Reviewed & tested by the community

Patch looks good and successfully applied on local dev instance. changing to RTBC.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Reviewed & tested by the community » Needs work

That file has other deprecated calls, doesn't seem very useful to just update that.

Also, the clas doesn't seem to be used at all right now..

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Title: Remove deprecated usage of db_select from codebase » Replace deprecated usage of database functions from codebase
Issue summary: View changes

Agreed, it'd be better to replace all of the deprecated database functions in one go.

subson’s picture

subson CreditAttribution: subson commented
Status: Needs work » Closed (duplicate)
Related issues: +#2882036: Replace all calls to db_(), which is deprecated

Closing this in favor of another issue - #2882036: Replace all calls to db_(), which is deprecated.