[4.4] Untranslated string for warning message in update:extensions:check CLI

[chrisdavenport]

Pull Request for Issue #43206 .

Summary of Changes

Language file is now loaded before outputting the warning message.

Testing Instructions

Arrange for an extension update site to have an invalid URL (eg. temporarily mangle the URL in the extensions database). Then run php cli/joomla.php update:extensions:check from the command line.

Actual result BEFORE applying this Pull Request

Warning message: [WARNING] JLIB_UPDATER_ERROR_OPEN_UPDATE_SITE

Expected result AFTER applying this Pull Request

Warning message something like
[WARNING] Update: Could not open update site #4 "xxxxxxxxxxx Package Update Site", URL: https://example.com/xxxxx/update.xml

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

[brianteeman]

Not usre this is the correct fix. Everything else in the cli is hardcoded english strings and not translatable

[chrisdavenport]

Not sure what the "official" convention is so you'll need to educate me on that. However, from looking at the code, log messages are untranslated English strings, but messages intended for CLI output, which is the case here, are translated into the default language. This PR ensures that the CLI user gets to see the same message as the web user in the default language.

[brianteeman]

are you sure? Everything here is hardcoded english

[chrisdavenport]

If the convention is that the CLI is always in English, then fine, I've learnt something today. :-)

But, thinking about it, the code I'm modifying here (UpdateAdapter.php) is also used by the web interface (when you click "Check For Updates"), so it needs to be translated. The correct language file is already loaded in the web interface, but isn't for CLI.

Or maybe a better fix would be to load the lang file in cli/joomla.php? Dunno.

[alikon]

i'll do it in the CheckUpdatesCommand

and only the html_entity_decode in the UpdateAdapter

my 0.02 €

[chrisdavenport]

@alikon Okay, done.

[alikon]

I have tested this item ✅ successfully on 223769c

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43208.}

[viocassel]

I have tested this item ✅ successfully on 223769c

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43208.}

[alikon]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43208.}

[MacJoom]

Thank you!