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.
Looks like the description for 'administer nodes' got applied to 'administer content types'.
Expected:
The Administer content permission has the description "Promote, change ownership, edit revisions, and perform other tasks across all content types."
Actual
The Administer content types permission has the description "Promote, change ownership, edit revisions, and perform other tasks across all content types."
Comment | File | Size | Author |
---|---|---|---|
#12 | after2561325-8patch.png | 11.86 KB | phantomvish |
#12 | before2561325-8patch.png | 10.69 KB | phantomvish |
#8 | administer-nodes-description-2561325-8.patch | 844 bytes | kevin.dutra |
#2 | administer-nodes-description-2561325-2.patch | 733 bytes | kevin.dutra |
Comments
Comment #2
kevin.dutra CreditAttribution: kevin.dutra at Workday, Inc. commentedComment #3
sushantpasteI think the description is at right place as permission is related to "Administer content types". Also instead of removing it entirely I think we need to add something similar description for "Administer content".
Comment #4
kevin.dutra CreditAttribution: kevin.dutra at Workday, Inc. commentedI disagree. Promotion, ownership (author), revisions, etc. are all operations that happen for nodes (content), not content types. Administer content types lets you adjust some of the settings/defaults for some of these things, but the way the description is currently worded, it's talking about performing the actual operations on nodes, which is what Administer content does.
But I do agree that having descriptions on both would help to differentiate.
Comment #5
sushantpaste@kevin.dutra : May be you can add patch for both fields , that will help us to review issue ?
Comment #6
dawehnerShould we have a description instead for 'administer content types'?
Comment #7
ifrikComment #8
kevin.dutra CreditAttribution: kevin.dutra at Workday, Inc. commentedAlrighty, this one includes a first attempt at a new description for 'administer content types'.
Comment #11
DuaelFrComment #12
phantomvish CreditAttribution: phantomvish commentedThe patch works, the permission descriptions are updated as expected.
Comment #13
phantomvish CreditAttribution: phantomvish commentedComment #14
alexpottCommitted 247c9a4 and pushed to 8.0.x. Thanks!