Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
A minor point, but for clarity I suggest changing:
function node_configure() {
// Only show rebuild button if there is 0 or more than 2 rows in node_access table,
// or if there are modules that implement node_grant.
...
to
function node_configure() {
// Only show rebuild button if there are 0 or 2 or more rows in {node_access}
// table, or if there are modules that implement hook_node_grants().
...
Patch to follow.
Comment | File | Size | Author |
---|---|---|---|
#1 | permissions_button_226480_6.x.patch | 1.23 KB | gpk |
Comments
Comment #1
gpk CreditAttribution: gpk commentedSelf explanatory I guess?! Patch for 6.x/7.x.
Comment #2
moshe weitzman CreditAttribution: moshe weitzman commentedComment #3
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks.
Comment #4
gpk CreditAttribution: gpk commentedThanks Dries, now for 6.x..
Comment #5
Gábor HojtsyI am probably not the best to ask with English punctuation, but having a comma after "more" seems inappropriate. Anyway, not to nitpick on this, I committed what went into Drupal 7 to Drupal 6 anyway.
Comment #6
gpk CreditAttribution: gpk commentedThanks Gábor.
Strictly speaking the comma after "0" could probably be omitted as well as the one after "more", but I think I was trying to avoid any possibility of confusion as to what the comment was saying!
Comment #7
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.