search

d6lts / drupal

Fork of Drupal core for Drupal 6 LTS support.
https://www.drupal.org
GNU General Public License v2.0
130 stars 45 forks source link

D6 doesn't have 'bypass node access', use 'administer nodes' instead. #65

Closed amorsent closed 3 years ago

amorsent commented 3 years ago

Per my comments here: https://www.drupal.org/project/d6lts/issues/2966553#comment-14166368

D6 does not have a 'bypass node access' permission, so these lines effectively limit to user 1.

D7 split 'bypass node access' off of 'administer nodes' in #305566 - https://www.drupal.org/node/305566 Commit: https://github.com/drupal/drupal/commit/89b0570d707e7a895a832f877dcdc74b68d26d42

A better fix would be to backport the 'bypass node access' permission, but in lieu of that, we should use the D6 equivalent.

dsnopek commented 3 years ago

Good catch! Looks good, thanks :-)