User tests: Successful: Unsuccessful:
Pull Request resolves # .
I improved error reporting when the http code of the updateserver is outside the normal range or when the xml could not be parsed. Instead of breaking out (in http code issue case) it will continue with the rest of the update sites. And it will return clear messages telling you what is wrong.
It will also clearly show in the logs when something is wrong (if you enable "Log almost everything"). Where it was not clear before which updatesite was at fault.
Why this PR? Because over time i've seen people reporting issues updating and they only see a 500 error. They don't know what to do. This way people will know whats going on. I had it before and it cost me a ton of time to figure it out.
To make it easier to install the extensions i created two file packages to test it with.
These install an empty html file in your root/media folder. After testing just uninstall them and the html file will be removed again. Download here: https://share.tlwebdesign.nl/s/test-installation-package
Please select:
Documentation link for guide.joomla.org:
No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Administration com_joomlaupdate Language & Strings JavaScript Repository NPM Change |
@brianteeman yes you're right give me couple minutes to fix my screw up! 😳
| Labels |
Added:
Language Change
NPM Resource Changed
PR-5.4-dev
|
||
@brianteeman i cleaned it up now.
| Title |
|
||||||
surely as this contains a new function it is by very definition a new feature and not a bug fix and therefore should be made agains 6.2
Discussed in today’s Maintainers Meeting: since an HTTP 500 Internal Server Error is a bug and Joomla 5.4 is a long-term support release, we have agreed that this PR can be merged into the 5.4-dev branch.
Before merging, the alphabetical sorting issue must be fixed and all the code changes must be well tested.
It's very hard to review the changes with all the (incorrect) formatting changes