Bug #32600

Add clear l10n cache when the user clear all cache

Added by Dominique Feyer over 8 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Category:
Localization
Target version:
Start date:
2011-12-15
Due date:
% Done:

100%

TYPO3 Version:
4.7
PHP Version:
5.3
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

Currently the l10n parser cache can only be cleared manually, by adding this in locallang.php:

$GLOBALS['TYPO3_CONF_VARS']['SYS']['lang']['cache']['clear_menu'] = TRUE;

This patch change the default behavior. By default the l10n cache is cleared on "clear all cache", if the clear l10n menu item is enabled, it's not cleared on clear all cache.


Related issues

Related to TYPO3 Core - Bug #32658: Fatal error on clearing all caches in backend Closed 2011-12-18
Duplicated by TYPO3 Core - Bug #29673: Cache in typo3temp/Cache/Data/t3lib_l10n not cleared Closed 2011-09-11

Associated revisions

Revision b3fd4851 (diff)
Added by Dominique Feyer over 8 years ago

[TASK] Add clear l10n cache when the user clear all cache

Currently the l10n parser cache can only be cleared manually, by adding this in locallang.php:

$GLOBALS['TYPO3_CONF_VARS']['SYS']['lang']['cache']['clear_menu'] = TRUE;

This patch change the default behavior. By default the l10n cache is cleared on "clear all cache",
if the clear l10n menu item is enabled, it's not cleared on clear all cache.

Change-Id: I969a76d4a17ed847f39884883a887f4e0cb9c3a3
Fixes: #32600
Releases: 4.7, 4.6
Reviewed-on: http://review.typo3.org/7283
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter

Revision be1d6e45 (diff)
Added by Dominique Feyer about 8 years ago

[TASK] Add clear l10n cache when the user clear all cache

Currently the l10n parser cache can only be cleared manually, by adding this in locallang.php:

$GLOBALS['TYPO3_CONF_VARS']['SYS']['lang']['cache']['clear_menu'] = TRUE;

This patch change the default behavior. By default the l10n cache is cleared on "clear all cache",
if the clear l10n menu item is enabled, it's not cleared on clear all cache.

Change-Id: I61568b93ccec35ccb6d7f2f362c29ddf8b3365cd
Fixes: #32600
Releases: 4.7, 4.6
Reviewed-on: http://review.typo3.org/7358
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
Reviewed-by: Dominique Feyer
Tested-by: Dominique Feyer
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers

History

#1 Updated by Gerrit Code Review over 8 years ago

  • Status changed from New to Under Review

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

#2 Updated by Gerrit Code Review over 8 years ago

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

#3 Updated by Dominique Feyer over 8 years ago

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

#4 Updated by Oliver Hader over 8 years ago

  • Status changed from Resolved to Needs Feedback

#5 Updated by Oliver Hader over 8 years ago

EXT:lang/hooks/clearcache/class.tx_lang_clearallcache.php is missing in master - but I guess it should be class.tx_lang_clearcache.php instead?

#6 Updated by Gerrit Code Review over 8 years ago

  • Status changed from Needs Feedback to Under Review

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/7358

#7 Updated by Steffen Gebert over 8 years ago

Yes Olly, made the same observation just now in master, too!

Can't delete Content Elements :-(

#8 Updated by Gerrit Code Review over 8 years ago

Patch set 2 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/7358

#9 Updated by Gerrit Code Review over 8 years ago

Patch set 3 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/7358

#10 Updated by Xavier Perseguers about 8 years ago

  • Status changed from Under Review to Resolved
  • Target version set to 4.6.5
  • PHP Version set to 5.3
  • Complexity set to easy

#11 Updated by Gerrit Code Review about 8 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/9361

#12 Updated by Oliver Hader almost 7 years ago

  • Category set to Localization
  • Status changed from Under Review to Resolved

#13 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF