-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[5.2] Cache: Fix for counting the Number of Files correctly #43986
Conversation
This is not correct. When you turn on the cache plugin a pages folder is not created. When you visit a page on the site then the folder is created with an index.html and any other cache files When you delete the cache using the system menu the entire cache folder is deleted. That includes the index.hmtl file. Therefore the number of cached files includes the index.html file |
This pull request has been automatically rebased to 5.2-dev. |
We create the index.html file because we don't want to have the files in the directory listed. So the index.html is not created as cached file. I think the PR is correct, even it doesn't makes a big difference. |
I have tested this item ✅ successfully on 683a10f This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43986. |
I remember the requirement to have index.html file in each folder was removed from long time ago, so I think the right solution if really needed would be not creating that file. Anyway, as you say, it doesn't makes a big difference, I don't really care if this PR is closed or merged. |
We still need an index.html in any folder that would otherwise be distributed as an empty folder. The folder is not created on some systems if it is empty but in this case you are correct as these folders are not distributed so they do not need the index.html file, In other words the correct fix is not to create the file in the first place - not to exclude it from the count |
I have tested this item ✅ successfully on 683a10f This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43986. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43986. |
Issue: The
Maintenance: Clear Cache
page does not display the correct number + size of cached files.This PR corrects the
Number of Files
on theMaintenance: Clear Cache
page.Summary of Changes
When using Joomla's
System - Page Cache
plugin, an/administrator/cache/page
folder withindex.html
file is created. When you visit one page on the front-end, you will cache one page.However, the
Maintenance: Clear Cache
page shows 2 files underNumber of Files
because it also counts theindex.html
file (which is not a cached file). It's confusing to see that number.Testing Instructions
System - Page Cache
pluginMaintenance: Clear Cache
counts 2 files.You'll see just one cached file and the
index.html
.Actual result BEFORE applying this Pull Request
You'll see too many cached files under
Number of Files
(and Size) because it includes theindex.html
:Expected result AFTER applying this Pull Request
You'll see the real number of cached files + size because the
index.html
is not counted as cache anymore:Link to documentations
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed