Feeling Lucky
[#46549] - [6.1] Refactor SiteApplication language detection
- Pending
- Medium
- Build: 6.1-dev
- # 46549
- Diff
- joomdonation:site-application-detect-language
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
This PR improves the initialiseApp() method in SiteApplication by extracting the language-detection logic into a separate method. The logic remains unchanged, but the code is cleaner, reduces nesting, and is easier to maintain.
Testing Instructions
- Uses Joomla 6.1
- Apply patch
- Navigate to random pages from frontend of the site and make sure it is still working as before.
Actual result BEFORE applying this Pull Request
Works
Expected result AFTER applying this Pull Request
Works, with cleaner, easier to maintain code
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
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Title |
|
||||||
richard67
- | Title |
|
||||||