-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[5.4] HTTP: Remove legacy code for Response calls #45825
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
Conversation
I have tested this item ✅ successfully on 368bf17 I did all of the tests mentioned and much more, but I could not test these 2:
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45825. |
I have tested this item ✅ successfully on 890b8c9 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45825. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45825. |
✅ Final test before merge: Named tests, plus additional info:
Not tested:
|
Thank you @Hackwar for your contribution. Thank you @richard67 and @exlemor for testing. |
Summary of Changes
The Joomla HTTP framework package provides a magic method to access code, body and headers of a Response object. This way of accessing this data is considered legacy and will not work with the version 4 of the framework package anymore. This PR thus converts all of this to the correct calls to the respective methods. This change is fully backwards compatible and makes the code forward compatible to Joomla 6+. The only thing changing in Joomla 6.0 now will be how to get an HTTP object. This will be changed in the PR to update the dependencies in 6.0.
Testing Instructions
Alternatively, do a codereview.
In all cases, everything should work the same between the current code and after applying the changes.
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