Unknown column 'n.language'

reopening that bug - cause the explanation dosn't fit my experience.

actually there might be a permission problem - but the bigger problem is, the db_rewrite_sql() !

in case of the comment, there are some params missing:
- primary table
- primary field

per default those are 'n' and 'nid' - but in case of the comments those should read 'c' and 'cid'!

please change the code in moderation.moderation.inc in function moderation_comment_queue() at line 97

from

  $result = pager_query(db_rewrite_sql($query), 50);

to

  $result = pager_query(db_rewrite_sql($query, 'c', 'cid'), 50);

cu
kiwi

well, just have to correct myself...

just found out that when using some node_access modules my presented solution is not really a solution.

so - best would be if you change the sql statement - not the rewrite statement:

  $query = "SELECT c.*, mm.status as moderate FROM {comments} c
           JOIN {node} n ON (c.nid = n.nid)
           LEFT JOIN {moderation_moderation} mm ON c.cid = mm.obj_id
           WHERE mm.obj_type = 'comment'
             AND (mm.status IS NULL OR mm.status=0)
           ORDER BY c.timestamp DESC, c.subject ASC";
  $result = pager_query(db_rewrite_sql($query), 50);

i included the corresponding node - in order for the node_access modules to kick in correctly!

cu
kiwi

Just to note that this fix also worked when I got a similar message:

Unknown column 'n.nid' in 'on clause'

Changing the SQL as suggested in #5 did the job for me. In case it might be of use, a patch file is attached.

It would be good if this change could be checked and committed into the 1.* dev line, though I see that there is now a 2.x branch so if the new version is ready for use soon it may not be needed. A proper fix would be good - my users really like this modules approach to moderation.

if stevehuk's review is sufficient - please commit the change!

Commited to 6-DEV.
Thanks.