? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
6 Apr 2021

Flush both system caches after rebuilding update sites

Last bit missing from already merged #32862

// @richard67

avatar PhilETaylor PhilETaylor - open - 6 Apr 2021
avatar PhilETaylor PhilETaylor - change - 6 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Apr 2021
Category Administration com_installer
avatar HLeithner
HLeithner - comment - 6 Apr 2021

What's the reason to clear frontend cache? shouldn't this only effect backend?

avatar PhilETaylor
PhilETaylor - comment - 6 Apr 2021

With the exception of one place in Joomla 3, wherever

cleanCache('_system', 1)

is, we always clear the other one 0 too. Belt and braces I guess.

avatar PhilETaylor PhilETaylor - change - 6 Apr 2021
Labels Added: ?
avatar PhilETaylor
PhilETaylor - comment - 6 Apr 2021

I have removed the extra cleanCache for you

avatar richard67
richard67 - comment - 6 Apr 2021

Belt and braces I guess.

At least we are not formatting hard disks ;-)

avatar richard67 richard67 - test_item - 6 Apr 2021 - Tested successfully
avatar richard67
richard67 - comment - 6 Apr 2021

I have tested this item successfully on c65d8a3


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33040.

avatar richard67 richard67 - change - 6 Apr 2021
Title
[3] Flush both system caches after rebuilding update sites
[3] Flush administrator system cache after rebuilding update sites
avatar richard67 richard67 - edited - 6 Apr 2021
avatar richard67
richard67 - comment - 6 Apr 2021

Code review ok, and rebuild update sites still works.

avatar HLeithner HLeithner - change - 6 Apr 2021
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-06 21:37:19
Closed_By HLeithner
avatar HLeithner HLeithner - close - 6 Apr 2021
avatar HLeithner HLeithner - merge - 6 Apr 2021
avatar HLeithner
HLeithner - comment - 6 Apr 2021

Thanks

Add a Comment

Login with GitHub to post a comment