Bug #57064

New option clearCache.system does not show up

Added by Francois Suter over 7 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Must have
Category:
Backend User Interface
Target version:
Start date:
2014-03-19
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
5.3
Tags:
Complexity:
easy
Is Regression:
No
Sprint Focus:

Description

With #54991 a new system cache was introduced, along with a new TSconfig property, called options.clearCache.system. However setting this option to true does not work if the user does not also have either options.clearCache.pages or options.clearCache.all set to true too.

This is because the \TYPO3\CMS\Backend\Toolbar\ClearCacheToolbarItem does not add the "system" options to the list of values to check for granting access.


Related issues

Related to TYPO3 Core - Story #54991: Improve caching framework by introducing groupsClosedBenni Mack2014-01-15

Actions
Related to TYPO3 Core - Bug #57703: ClearCacheToolbarItem must implement ClearCacheActionsHookInterfaceClosed2014-04-07

Actions
#1

Updated by Gerrit Code Review over 7 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/28518

#2

Updated by Francois Suter over 7 years ago

Steps to reproduce:

  • take an non-admin user and options.clearCache.system = 1 to its TSconfig, taking care that there are no other "clearCache" options set
  • switch to that user => the clear cache menu does not show up
  • apply the patch => the menu shows up
#3

Updated by Gerrit Code Review over 7 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/28518

#4

Updated by Gerrit Code Review over 7 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/28518

#5

Updated by Gerrit Code Review over 7 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/28518

#6

Updated by Francois Suter over 7 years ago

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

Updated by Riccardo De Contardi almost 4 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF