Module is not compatible with table prefixes. Many queries used tables without '{..}' brackets. It's very very bad.

Comments

pyromanfo’s picture

pyromanfo commented

Please look at other issues before posting. This has been fixed in CVS for months. CVS works with 4.5.x. There have been at least 2 other issues on this.

axel’s picture

axel commented

Ops, sorry for double. Though in CVS version at least one query still miss prefix. See query before end of the module code under comment: "// Determine which nodes don't belong to a category". But thanx for flash response to bugreport :)

pyromanfo’s picture

pyromanfo commented

Good catch, fixed in CVS.

axel’s picture

axel commented

:) Any case module is good, thanx for your work.

malfunct’s picture

malfunct commented

Missed one other table reference in this fix:

its on line 788 that I see it, I have revision 1.33 from cvs

the actual line: $query = db_query("SELECT n.nid FROM node n LEFT JOIN {term_node} t ON t.nid=n.nid WHERE t.nid IS NULL");

the fixed line: $query = db_query("SELECT n.nid FROM {node} n LEFT JOIN {term_node} t ON t.nid=n.nid WHERE t.nid IS NULL");

malfunct’s picture

malfunct commented

The issue is fixed in the latest CVS version. Sorry it took me so long to try it out.

pyromanfo’s picture

pyromanfo commented

Great, thanks!

Anonymous’s picture

(not verified) commented