How to reproduce the issue:

  • Disable the 'administer content types' for a specific role (e.g. webmaster)
  • Login as a user with the webmaster role
  • Navigate to the content manager

The content type field is empty for all content because the 'administer content types' has been disabled for this role. This is not the expected behavior because I want to see which content type a piece of content has as a webmaster.

CommentFileSizeAuthor
#4 2561331-4-complete.patch2.38 KBgeertvd
#4 2561331-4-test.patch985 bytesgeertvd
#3 2561331-3.patch1.42 KBgeertvd
#3 interdiff-2561331-2-3.txt1.59 KBgeertvd
#2 2561331-2.patch2.18 KBgeertvd
Selection_006.png12.14 KBbceyssens

Comments

bceyssens created an issue. See original summary.

geertvd’s picture

geertvd commented
Status: Active » Needs review
Issue tags: +Needs tests
StatusFileSize
new 2561331-2.patch2.18 KB

Something like this fixes it.
I'm not sure if this if this ok to do though, maybe we should be using some other permission here? At least 'access content'?

In any case, this will need tests.

geertvd’s picture

geertvd commented
StatusFileSize
new interdiff-2561331-2-3.txt1.59 KB
new 2561331-3.patch1.42 KB

I guess this is better.
That access method was an accidental copy.

geertvd’s picture

geertvd commented
StatusFileSize
new 2561331-4-test.patch985 bytes
new 2561331-4-complete.patch2.38 KB

And the test.

aspilicious’s picture

aspilicious commented
Status: Needs review » Reviewed & tested by the community

seems fine to me

The last submitted patch, 2: 2561331-2.patch, failed testing.

The last submitted patch, 4: 2561331-4-test.patch, failed testing.

geertvd’s picture

geertvd commented
Version: 8.0.0-beta14 » 8.0.x-dev
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed
Issue tags: -Needs tests

Committed b19760f and pushed to 8.0.x. Thanks!

  • alexpott committed b19760f on 8.0.x 
    Issue #2561331 by geertvd: When 'administer content types' permission is...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

Status: Closed (fixed) » Needs work

The last submitted patch, 4: 2561331-4-complete.patch, failed testing.

geertvd’s picture

geertvd commented
Status: Needs work » Closed (fixed)