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.If a node type's 'Create new revision' is enabled and the $user does NOT have 'administer nodes' permission, she cannot edit the nodes of this type without creating a new revision, regardless the fact that she is granted the 'delete revisions' permission. IMO if the $user is allowed to delete revisions, then she should even be allowed to edit the nodes without creating a new revision.
I was even thinking about taking the 'revert revisions' permission into account. If the $user is only allowed to revert revisions, but not delete them, then she is not allowed to hide her own edits from the history, so granting her the 'revert revisions' by itself should not allow her to edit a node without creating a new revision. OTOH if she is granted the 'delete revisions', but not the 'revert revisions' permission, then she can hide the fact that there was a revision before her own one, anyway.
In short, it is kind of usability improvement: allow the $user to edit a node without creating a new revision if she can hide the fact that there was any previous revisions before her own one (by deleting the old ones). The attached patch does this.
| Comment | File | Size | Author |
|---|---|---|---|
| allow_unrevisioned_edit_with_delete_revisions.patch | 817 bytes | Boobaa | |
Comments
Comment #1
BoobaaOops, forgot to correct the title when I was thinking about the thing again before submitting. Sorry.
Comment #2
ff1 CreditAttribution: ff1 commentedBumping to D8...
Comment #3
kscheirerallow_unrevisioned_edit_with_delete_revisions.patch queued for re-testing.
Comment #5
xjm(Merging "node system" and "node.module" components for 8.x; disregard.)