[4.2] Make sure we are not using downloadSources from another version#39446
[4.2] Make sure we are not using downloadSources from another version#39446roland-d merged 4 commits intojoomla:4.2-devfrom
Conversation
|
I have tested this item ✅ successfully on 86d7db1 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39446. |
|
In addition to the testing instructions, I have tested with success that the update works as well as without this PR when the update URL exists so the 404 redirect doesn't happen. You can use the following download URL for this test: https://test5.richard-fath.de/next_minor_list.xml |
|
Thanks so one more test here so this can be fixed :) |
|
Before I confirm as successful (it was).
|
No. But the links in the next_minor_extension.xml currently don’t work because of wrong package names.
Yes, these will be changed after that PR had been tested so they have the right package names and the live update will work, too.
No. Updating from 4.2.x to 4.3.x-dev shall and will be possible when all is fixed. |
Co-authored-by: Quy <quy@nomonkeybiz.com>
|
I have tested this item ✅ successfully on 80d6776 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39446. |
Its broken since the alpha2 commit: e1812a9 it should have been -dev only. Or the URL on the update server had to be updated too. Works both ways but not how it is now. But as Richard said the update server stay the same and we will deploy an workaround for existing sites. Can you confirm this issue and the patch? Than we do have two tests and can deploy the workaround. |
|
I have tested this item ✅ successfully on 80d6776 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39446. |
|
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39446. |
|
@zero-24 @richard67 Thanks for clarification. I had this issue on 2 sites yesterday but didn't check that version was 4.0.4 after update and grumbled and grumbled while debugging and repairing... For others: You can use Upload & Update meanwhile. |
|
Workaround for the update server prepared here: joomla/update.joomla.org#278 (comment) |
|
Thanks @ReLater took me some debugging too but lucky me i found it :) |
@zero24 This fixes only the issue with the downgrade to 4.0.4 for sites which don’t have yet the patch of this PR here, but the wrong URL and package name still needs to be fixed, right? Unfortunately it‘s already late and I need sleep and gotta work tomorrow, so I can’t test that before tomorrow night. |
|
you are right but with that PR the correct message 'download failed" will be shown. The version.php or the version and Download URLs within the xmls needs to be fixed too. |
|
Thank you |
Pull Request for Issue joomla/update.joomla.org#277
Summary of Changes
Make sure we are not using downloadSources from another version
Testing Instructions
Install 4.2.6
set this at you update server: "https://update.joomla.org/core/nightlies/next_minor_list.xml"
Actual result BEFORE applying this Pull Request
Downgrade to 4.0.4
Expected result AFTER applying this Pull Request
Error message "download failed"
Link to documentations
Please select:
Analyse
There are two few issues which in combination cause this issue.
Said downloadSources object from 4.0.4 will be carried over as its not overwritten by the 4.3.0-dev object as it does not exists there.
Now the seccond issue comes into play as the original download URL is 404 joomla tries an alternative via the downloadsource tag. But that tag is set to 4.0.4 URLs which are valid and cause the issue.
Whith the changes here we make sure that this special tag is not carried over to another version when not overriden.
TBH this code itself is far from ideal and should be rewritten for many reasons, but this here should fix the issue as it is for now.
cc @richard67 @obuisard