Project

General

Profile

Actions

Bug #83008

closed

Epic #90674: Backend UI not reflecting permissions

Edit Icon shown in list view despite user not having write permission for table

Added by Moritz Ahl over 6 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
-
Start date:
2017-11-15
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
9
PHP Version:
7.1
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

The list view shows an "edit" icon for records in a table even when the current user does not have write permissions for that table (for example, domain records). When clicked, a spinner shows but the view is never loaded.

So, there should be no icon if the user does not have write permission for that table.


Files

belayout_listing.png (20 KB) belayout_listing.png Riccardo De Contardi, 2017-11-15 23:35
attempt_modify.png (19.1 KB) attempt_modify.png Riccardo De Contardi, 2017-11-15 23:35
attempt_delete.png (28.9 KB) attempt_delete.png Riccardo De Contardi, 2017-11-15 23:35
attempt_hide.png (31.4 KB) attempt_hide.png Riccardo De Contardi, 2017-11-15 23:40

Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #23598: AJAX-menu for content-elements only checks page-record-permissions not content-element-permissionsClosed2010-09-24

Actions
Related to TYPO3 Core - Bug #89307: List Module shows edit action icon even if User has no edit permissionClosed2019-09-30

Actions
Actions #1

Updated by Georg Ringer over 6 years ago

  • Status changed from New to Needs Feedback

Thanks for creating this issue. Can you please check 8.7 as well and if valid add a Screenshot. That would be great!

Updated by Riccardo De Contardi over 6 years ago

This still affects 9.0.0-dev (latest master) as far as I can see.

Steps to reproduce

1) create a usergroup that can list backend_layouts but cannot modify them
2) assign a user
3) create some backend layouts in a sysfolder that the group can access

Results: As you can see, the "edit", "hide" and "delete" button are still visible

- if you try to modify the record, the result is a grey infinite spinning page:

- if you try to hide the record the icon will be replaced with the spin, but an error message is shown

- if you try to delete the record, an error message is shown:

Maybe it would be better to hide or disable the icons, but the error messages are correct.

Actions #3

Updated by Georg Ringer over 6 years ago

  • Status changed from Needs Feedback to Accepted

thanks for the detailed feedback!

Actions #4

Updated by Georg Ringer over 6 years ago

  • TYPO3 Version changed from 7 to 9
Actions #5

Updated by Riccardo De Contardi over 5 years ago

  • Related to Bug #23598: AJAX-menu for content-elements only checks page-record-permissions not content-element-permissions added
Actions #6

Updated by Riccardo De Contardi about 4 years ago

  • Parent task set to #90674
Actions #7

Updated by Riccardo De Contardi about 4 years ago

  • Related to Bug #89307: List Module shows edit action icon even if User has no edit permission added
Actions #8

Updated by Gerrit Code Review about 4 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63696

Actions #9

Updated by Gerrit Code Review about 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63696

Actions #10

Updated by Gerrit Code Review about 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63696

Actions #11

Updated by Gerrit Code Review about 4 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63696

Actions #12

Updated by Gerrit Code Review about 4 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63696

Actions #13

Updated by Gerrit Code Review about 4 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63696

Actions #14

Updated by Gerrit Code Review about 4 years ago

Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63750

Actions #15

Updated by Gerrit Code Review about 4 years ago

Patch set 2 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63750

Actions #16

Updated by Christian Eßl about 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #17

Updated by Benni Mack about 4 years ago

  • Status changed from Resolved to Closed
Actions #18

Updated by Moritz Ahl about 4 years ago

Thanks a lot for fixing!

Actions

Also available in: Atom PDF