? Language Change ? Maintainers Checked Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
21 Jun 2022

Pull Request for Issue #38110 .

Summary of Changes

added a check if the xml is returned

Testing Instructions

add a $response->body= ""; on line 153 to simulate the case

Actual result BEFORE applying this Pull Request

PHP Warning: Attempt to read property "extension" on bool in /shared/httpd/dev4/joomla-cms/administrator/components/com_installer/src/Model/LanguagesModel.php on line 158

Expected result AFTER applying this Pull Request

a proper error message

avatar alikon alikon - open - 21 Jun 2022
avatar alikon alikon - change - 21 Jun 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Jun 2022
Category Administration com_installer Language & Strings
avatar wilsonge wilsonge - change - 21 Jun 2022
Labels Added: Language Change ? Maintainers Checked
avatar brianteeman brianteeman - test_item - 22 Jun 2022 - Tested successfully
avatar brianteeman
brianteeman - comment - 22 Jun 2022

I have tested this item successfully on 6d0afcf


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38113.

avatar Quy Quy - test_item - 22 Jun 2022 - Tested successfully
avatar Quy
Quy - comment - 22 Jun 2022

I have tested this item successfully on 6d0afcf


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38113.

avatar Quy Quy - change - 22 Jun 2022
Status Pending Ready to Commit
avatar Quy
Quy - comment - 22 Jun 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38113.

avatar Quy Quy - change - 23 Jun 2022
Labels Added: ?
avatar wilsonge wilsonge - change - 23 Jun 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-06-23 17:11:19
Closed_By wilsonge
avatar wilsonge wilsonge - close - 23 Jun 2022
avatar wilsonge wilsonge - merge - 23 Jun 2022
avatar wilsonge
wilsonge - comment - 23 Jun 2022

Thanks!

Add a Comment

Login with GitHub to post a comment