-
-
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.3] Send an event before an update site is downloaded #44516
Conversation
I have tested this item ✅ successfully on 3bd616f This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44516. |
I'v updated the testing instructions, to actually see if the event also works. |
The event needs validation, if you set a url this should be validated, at least if it is a string hand has a proper schema set. Similar to the headers argument, needs to be an array of strings (with key?). |
All of this should be done done in the parent class and is not in the scope of this pr. |
I have not tested this item. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44516. |
I have tested this item ✅ successfully on aa23300 To be more precise in case this is important: BEFORE the PATCH, there was a whole long list of error lines: For the extension DPCalendar Free version 7.0.2 is available, but it requires at least PHP version 8.3 while your system only has 8.2.26 what is odd is the version I installed was 9.2.1 but the messages stop at 7.0.2 AFTER the PATCH, the messages start with: Update: Could not open update site #2 "Accredited Joomla! Translations", URL: https://update.joomla.org/language/translationlist_5.xmldemo=1 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44516. |
The DPCalendar warnings are not related to this pr. So we are RTC. This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44516. |
Thx |
Summary of Changes
This pr adds the same logic for installer plugins to be able to modify the update site url and headers as when the extension is downloaded. A use case for this pr is to add some authentication before fetching the update site url like a bearer token, the download id or similar. For extension developers it has an additional benefit to install a plugin and modify update sites for specific clients without the extra step and a modification of the update site url in the database.
Testing Instructions
and replace the line 47 with:
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Update: Could not open update site 3 "Joomla! Update Component", URL: https://update.joomla.org/core/extensions/com_joomlaupdate.xmldemo=1
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: Add docu for new update event Manual#340
No documentation changes for manual.joomla.org needed