Bug #59966

EM should not clear all system caches on extension configuration change

Added by Markus Klein over 5 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
Caching
Target version:
Start date:
2014-06-28
Due date:
% Done:

100%

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

Description

If you only change the configuration of an extension, the EM clears the system caches.
This is overkill, since a configuration change will never have any influence on the classes cache for instance.


Related issues

Related to TYPO3 Core - Bug #59845: Full system crash after installing extension on a live system Closed 2014-06-24
Related to TYPO3 Core - Feature #65279: Clear Cache and Opcode automatically after Extension Update Closed 2015-02-24

Associated revisions

Revision 2dd84dd2 (diff)
Added by Markus Klein almost 5 years ago

[!!!][TASK] Do not clear caches after extension config change

This change removes the cache flush after extension configuration changes.
Furthermore we avoid duplicate cache flushing on extension installation.

Resolves: #59966
Releases: master
Change-Id: I6b214ac76fb7305c40f0ce7a33b895323979cbe6
Reviewed-on: http://review.typo3.org/31180
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>
Reviewed-by: Stefan Froemken <>
Tested-by: Stefan Froemken <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

History

#1 Updated by Gerrit Code Review over 5 years ago

  • Status changed from Accepted to Under Review

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/31180

#2 Updated by Gerrit Code Review about 5 years ago

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

#3 Updated by Gerrit Code Review almost 5 years ago

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

#4 Updated by Markus Klein almost 5 years ago

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

#5 Updated by Herbert Sojnik about 4 years ago

Dear Markus,

shouldn't this patch already be included within 6.2.14 and 6.2.15?

Best regards,
Herbert

#6 Updated by Markus Klein about 4 years ago

Hi Herbert!

No, I reported this for 6.2, but the fix is a major breaking change, hence it was included only for CMS 7.

Cheers,
Markus

#7 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF