Would like to extend the db_rewrite_sql statement and add an identifier to that args variable that uniquely identifies the sql statement. This is useful for anyone who wants to use hook_db_rewrite_sql to only alter that particular statement.

CommentFileSizeAuthor
#4 googlenews-n1181648-4.patch519 bytesdamienmckenna
#2 googlenews-alter-query_1181648_2.patch483 bytesfgm
#1 extend-db_rewrite_sql.patch765 byteslittlecow

Comments

littlecow’s picture

littlecow commented
StatusFileSize
new extend-db_rewrite_sql.patch765 bytes

Attached is a patch that adds this feature.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Title: Modify db_rewrite_sql Statement For Extensibility » Modify Googlenews query for extensibility
Version: 6.x-1.4 » 7.x-1.x-dev
StatusFileSize
new googlenews-alter-query_1181648_2.patch483 bytes

This is still very much a need for 7.x, and even simpler than on D6. Adjusting title accordingly.

Suggested patch attached ; will need to be backported to 6.x if accepted.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Active » Needs review

Forgot to set status.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new googlenews-n1181648-4.patch519 bytes

Rerolled.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed. FYI I changed the tag to "googlenews_sitemap".

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Fixed » Patch (to be ported)

Needs to be backported to D6.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Issue summary: View changes
Status: Patch (to be ported) » Fixed

The D6 version is no longer supported. Sorry folks.

Status: Fixed » Closed (fixed)

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