Closed (fixed)
Project:
CCK comment reference
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Aug 2008 at 04:27 UTC
Updated:
18 Sep 2008 at 23:52 UTC
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.
Comments
Comment #1
swentel commentedHmm good point, I'll fix that this week!
Comment #2
swentel commentedfixed in D5 & D6 branch, thanks for the report!
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.