User tests: Successful: Unsuccessful:
Pull Request for Issue # .
The System – Language Filter plugin currently calls LanguageHelper::getInstalledLanguages(0) multiple times, each time for one language in the loop. The method itself contains bunch of logic, including file system operator - which in general, slow.
This PR stores the result in a local variable so the method is called only once, reducing unnecessary filesystem work and slightly improving performance.
Works
Works, with small performance improvement.
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
| Status | New | ⇒ | Pending |
| Category | ⇒ | Front End Plugins |
| Title |
|
||||||
I have tested this item ✅ successfully on 0dca53f
I have tested this successfully - thanks @joomdonation - Everything in the System Language Filter plugin seems to work as before.
| Status | Pending | ⇒ | Ready to Commit |
RTC. Thanks @ceford and @exlemor for testing.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-02-11 13:27:12 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
RTC
Performance
PR-6.1-dev
|
||
thanks
I have tested this item ✅ successfully on 0dca53f
I don't see, or expect to see, the performance improvement - but nothing was broken.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46812.