I find the documentation of db_rewrite_sql() a bit vague and propose the following changes:
-- add the {node_type} table to the list
-- clarify $primary_field
-- clarify $args as "not currently used"
Same changes also for _db_rewrite_sql(), except the third (don't know whether it applies).
If I understand this correctly, then single-node accesses should use node_access() for checking permissions. If this is correct, then adding a pointer to node_access() (and possibly vice versa) would be helpful, but I don't know how to add this properly. @see doesn't seem to work correctly.
| Comment | File | Size | Author |
|---|---|---|---|
| database.inc_.db_rewrite_sql.doc_.patch | 1.99 KB | salvis |
Comments
Comment #1
dpearcefl commentedDoes this issue exist in current D6?
Comment #2
salvisYes, the issue still exists, but no one cares about it anymore.
Feel free to close it.
Thanks for going through all those old issues!
Comment #3
dpearcefl commentedSo let it be written. So let it be done.