? Success

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
28 Mar 2016

Pull Request for Issue #9626

Summary of Changes

Clear the cache after Update. This does not cover OPCACHE as i don't know if i can place it at the script file too ;)

Testing Instructions

  • install 3.5.0
  • enable caching
  • produce some entrys for the cache (navigate a bit in the frontend)
  • notice that there are entrys to the cache
  • update using this Joomla Core update URL: http://jah-tz.de/downloads/extensions/dev/core_update.xml
  • notice that the Joomla cache is cleaned after the update.
avatar zero-24 zero-24 - open - 28 Mar 2016
avatar zero-24 zero-24 - change - 28 Mar 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Mar 2016
Labels Added: ?
avatar andrepereiradasilva
andrepereiradasilva - comment - 28 Mar 2016

nice. I will test when i have time.

avatar richard67 richard67 - test_item - 28 Mar 2016 - Tested successfully
avatar richard67
richard67 - comment - 28 Mar 2016

I have tested this item :white_check_mark: successfully on 3bdfa51

Tested both updating a 3.4.8 and a 3.5.0 with give custom URL.

Cache in both cases on, conservative caching with cache handler = file, page cache plugin = enabled with used browser cache = yes, cache was filled before update.

After update, cache was cleared in both cases.

I am not sure if this will solve all issues related to caching after update, because some of the issues may come from browsers still having js files cached.

But this is not related to this PR here.


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

avatar richard67
richard67 - comment - 28 Mar 2016

Ah I just see Andre had the same thoughts about cached js.


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

avatar zero-24
zero-24 - comment - 28 Mar 2016

Yes there is a feature that is called MD5CHECKSUM File. But This should be another PR. ;)

avatar richard67
richard67 - comment - 28 Mar 2016

@zero-24 Maybe you should give your PR the "Easy test" flag? Could help to find more testers, and is really easy to test.


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 28 Mar 2016

@zero-24 is this suppose to clean Site (frontend) and Administrator (Backend) cache?

Is the test i made it's not clearing all cache.

avatar Fedik
Fedik - comment - 28 Mar 2016

BTW, is there any way to clean all js/css admin template in browser cache too?

Joomla! already use $doc->addStyleSheetVersion() in default template.

avatar richard67
richard67 - comment - 28 Mar 2016

@andrepereiradasilva Some of the admin cache remained also in my test, like fof cache, but I think this is normal, it also happens when you manually clear the cache. Frontend cache was completely cleared.

avatar andrepereiradasilva
andrepereiradasilva - comment - 28 Mar 2016

@andrepereiradasilva Some of the admin cache remained also in my test, like fof cache, but I think this is normal, it also happens when you manually clear the cache. Page cache was completely cleared.

Mine the admin has cleared (some where recreated on load so they appeared on admin cache), but the site wasn't cleared. That's why i asked.

avatar brianteeman brianteeman - change - 28 Mar 2016
Category Cache
avatar mbabker
mbabker - comment - 28 Mar 2016

Few thoughts since this has been hijacked, and I'm just going to end up repeating everything I've said before here (sorry if you've heard it before):

  • Not everything requires a query string version, don't mandate that everything has to have it (don't think that's ever been suggested but getting ahead of the curve here)
  • If media loaded through $doc->add***Version() isn't correctly getting version strings changed post update, there's a bug in that process somewhere that needs to be addressed
  • JHtml already has a version string mechanism with MD5SUM files in directories, don't proxy it to the doc's version methods unless you intend on deprecating the MD5SUM functionality (this is also the only way right now to do version strings with JHtml loaded assets unless you're hijacking the JDocument head and manipulating it before it's rendered)
  • Asset files like this aren't loaded through Joomla at all, so unless there's a global "tell browser to bust cache for everything on this domain" header, not sure what can be done at that level (especially since I'm fairly sure things like last modified time are sent with the response headers and browsers should be smart enough to bust cache if it detects that changed)
avatar andrepereiradasilva
andrepereiradasilva - comment - 28 Mar 2016

ok, now regarding this PR, as said before, in my test the joomla frontend cache was not cleared.

avatar joomla-cms-bot
joomla-cms-bot - comment - 28 Mar 2016

This PR has received new commits.

CC: @richard67


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

avatar zero-24
zero-24 - comment - 28 Mar 2016

@andrepereiradasilva hmm can you please double check? For me it works.

This is how it looks bevor the update:
bevor_update

and this after the update:
after_update

maybe you use the wrong update package?

avatar richard67 richard67 - test_item - 28 Mar 2016 - Tested successfully
avatar richard67
richard67 - comment - 28 Mar 2016

I have tested this item :white_check_mark: successfully on 9b20904


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

avatar andrepereiradasilva andrepereiradasilva - test_item - 28 Mar 2016 - Tested successfully
avatar andrepereiradasilva
andrepereiradasilva - comment - 28 Mar 2016

I have tested this item :white_check_mark: successfully on 9b20904

sorry it must be something on my side.

It works.


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

avatar brianteeman brianteeman - change - 28 Mar 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 28 Mar 2016

Ryc


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

avatar joomla-cms-bot joomla-cms-bot - change - 28 Mar 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 28 Mar 2016
Labels Removed: ?
avatar brianteeman
brianteeman - comment - 28 Mar 2016

Rtc


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

avatar joomla-cms-bot joomla-cms-bot - change - 28 Mar 2016
Labels Added: ?
avatar wilsonge wilsonge - change - 29 Mar 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-03-29 08:27:17
Closed_By wilsonge
avatar wilsonge wilsonge - close - 29 Mar 2016
avatar wilsonge wilsonge - merge - 29 Mar 2016
avatar joomla-cms-bot joomla-cms-bot - close - 29 Mar 2016
avatar wilsonge wilsonge - reference | d64f62b - 29 Mar 16
avatar wilsonge wilsonge - merge - 29 Mar 2016
avatar wilsonge wilsonge - close - 29 Mar 2016
avatar joomla-cms-bot joomla-cms-bot - change - 29 Mar 2016
Labels Removed: ?
avatar wilsonge wilsonge - change - 29 Mar 2016
Milestone Added:
avatar zero-24 zero-24 - head_ref_deleted - 29 Mar 2016
avatar Fedik
Fedik - comment - 29 Mar 2016

hm, thought ... how it affect 3d extensions that has a large amount of data in the cache, on heavily loaded site?

avatar andrepereiradasilva
andrepereiradasilva - comment - 29 Mar 2016

i think @Fedik has a good point here.

avatar Fedik
Fedik - comment - 29 Mar 2016

this change can crash them all,
maybe better would be specify which cache group(s) need to clean up?

avatar andrepereiradasilva
andrepereiradasilva - comment - 29 Mar 2016

probably, and correct if i'm wrong, the most important for the update process in clean the administrator cache.

Add a Comment

Login with GitHub to post a comment