Bug #39103

Fix Clear Language Cache Configuration

Added by Marcus Schwemer about 3 years ago. Updated over 2 years ago.

Status:Resolved Start date:2012-07-19
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:- Spent time: -
Target version:-
TYPO3 Version:6.0 Is Regression:
PHP Version:5.3 Sprint Focus:
Complexity:

Description

Hi,

there are several issues with the configuration of clearing the language cache.

Wrong option naming in UserTSconfig

In "class.tx_lang_clearcachemenu.php" the UserTS option "options.clearCache.clearRTECache" is taken into account. This should be renamed to "options.clearCache.clearLangCache".

"Clear cache" Icon next to the Username (Thunderstorm Icon) is not shown, if clearLangCache (aka clearRTECache) is the only option to delete a cache.

I will dig into this. I assume that &$cacheActions are not initialized.

Configuration is inconsistent.

IMO the configuration of clearing the language cache is inconsistent to the rest of cache clearing. Why?

1) Clearing the the language cache is excluded from "Clear all caches" if $GLOBALS['TYPO3_CONF_VARS']['SYS']['lang']['cache']['clear_menu'] is set
2) In order to enable clearing the language cache for normal editors, one has to set two values
  • localconf.php: $GLOBALS['TYPO3_CONF_VARS']['SYS']['lang']['cache']['clear_menu'] = TRUE
  • UserTSConfig: options.clearCache.clearLangCache

I am in favor of dropping $GLOBALS['TYPO3_CONF_VARS']['SYS']['lang']['cache']['clear_menu'].
1) Language Cache Clearing should be included in "Clear all caches"
2) Other clear cache actions (pages / all) do not require two config options set.

I will come up with a patch

Bests
Marcus

master_39103_v01.diff Magnifier (2.7 kB) Marcus Schwemer, 2012-07-20 00:34


Related issues

precedes TSconfig - Task #47749: Update documentation of options.clearCache.clearRTECache New 2013-04-30

Associated revisions

Revision b1caf3b1
Added by Marcus Schwemer over 2 years ago

[!!!][BUGFIX] Clear Language Cache Configuration broken

Rename BE user TS option
options.clearCache.clearRTECache
to
options.clearCache.clearLangCache
to determine whether the language cache clear item should be shown.

Fixes: #39103
Releases: 6.1

Change-Id: I701dc9a879de4f3681f3fb441545eaa6b3e08e3e
Reviewed-on: https://review.typo3.org/12912
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer
Reviewed-by: Stefan Neufeind
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

History

#1 Updated by Marcus Schwemer about 3 years ago

Hi,

here is the patch. :-)

All three mentioned issues work now.

Bests
Marcus

#2 Updated by Gerrit Code Review about 3 years ago

  • Status changed from New to Under Review

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12912

#3 Updated by Gerrit Code Review about 3 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12912

#4 Updated by Gerrit Code Review about 3 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12912

#5 Updated by Gerrit Code Review about 3 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12912

#6 Updated by Gerrit Code Review over 2 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12912

#7 Updated by Gerrit Code Review over 2 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12912

#8 Updated by Gerrit Code Review over 2 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12912

#9 Updated by Gerrit Code Review over 2 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/12912

#10 Updated by Marcus Schwemer over 2 years ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF