[drupal-devel] [bug] SQL error when node_access is used

[tangent]

Issue status update for http://drupal.org/node/17794

 Project:      Drupal
 Version:      cvs
 Component:    taxonomy.module
 Category:     bug reports
 Priority:     critical
 Assigned to:  Anonymous
 Reported by:  tangent
 Updated by:   tangent
 Status:       patch

Can we get this commited? It's a pretty critical error.


tangent



Previous comments:
------------------------------------------------------------------------

February 23, 2005 - 02:03 : tangent

Attachment: http://drupal.org/files/issues/nodeaccess.patch (1.21 KB)

I am testing the nodeperm_role.module and I experienced an SQL error
when node permissions are changed.
The problem is that the query in question uses "INNER JOIN {term_node}
USING (tid)" but the db_rewrite_sql() function adds another inner join
in front of this one so the USING clause breaks. My patch changes all
the USING statements to ON to prevent this.
Perhaps the documentation for db_rewrite_sql() should be updated to
warn against the use of the USING clause. I don't see any other
instances of it in core but many contrib modules use it.


------------------------------------------------------------------------

February 23, 2005 - 02:15 : chx

Attachment: http://drupal.org/files/issues/nodeaccess_0.patch (1.21 KB)

Handbook updated, thanks for warning. +1 on this.