db_rewrite_sql

danielb - August 23, 2008 - 04:27
Project:CCK comment reference
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Hey mate I notice your use of db_rewrite_sql does not specify all the arguments as instructed by the api. By using it without the extra argument it will only truly work on the {node} table (that's why nodereference doesn't use the extra arguments). I had the same problems with my module. No symptoms will appear until you try to install a module like i18n internationalization (language module) and it will try to rewrite your queries on the {comments} table using column 'nid' thinking that it is the {node} table - which will cause errors.

#1

swentel - August 24, 2008 - 12:18

Hmm good point, I'll fix that this week!

#2

swentel - September 4, 2008 - 23:45
Status:active» fixed

fixed in D5 & D6 branch, thanks for the report!

#3

Anonymous (not verified) - September 18, 2008 - 23:52
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.