RFC PR-5.1-dev Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
18 Feb 2024

Pull Request for Issue #40693 .

Summary of Changes

if you want an extension to be installable from cli then add this tag <updatefromcli>true</updatefromcli> to the updateserver.xml
for example
image

new command to perform the extension updated
php cli\joomla.php extension:update --eid=extension_id

Testing Instructions

run php cli\joomla.php extension:update --eid=extension_id

Actual result BEFORE applying this Pull Request

N/A

Expected result AFTER applying this Pull Request

run
php cli\joomla.php extension:update --eid=249
image

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

avatar alikon alikon - open - 18 Feb 2024
avatar alikon alikon - change - 18 Feb 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Feb 2024
Category Administration com_installer Libraries
avatar alikon alikon - change - 19 Feb 2024
Labels Added: PR-5.1-dev
avatar alikon alikon - change - 19 Feb 2024
The description was changed
avatar alikon alikon - edited - 19 Feb 2024
0370371 19 Feb 2024 avatar alikon cs
cbc4843 19 Feb 2024 avatar alikon cs
avatar Hackwar
Hackwar - comment - 19 Feb 2024

I would strongly advise against this at this moment. We should first introduce a flag into extension manifests to mark if an extension supports updating via CLI, because otherwise especially the big extensions with their own install scripts will all fail when updating.

avatar alikon
alikon - comment - 19 Feb 2024

doesn't that will fail from backend com_installer as well ?

avatar Hackwar
Hackwar - comment - 19 Feb 2024

No, an extension which does redirects after installation, maybe does ajax calls when the update is finished or expects a web context in generell, will fail with the update when you are in a CLI context.

avatar alikon
alikon - comment - 19 Feb 2024

that's is more like a general joomla issue
not all joomla api performs like a client agnostic
there already are plenty of issue reports about that
cli vs workflows just as a recent example

out of scope of this silly pr

avatar Hackwar
Hackwar - comment - 20 Feb 2024

Considering that this could pretty much destroy a website, I don't consider this something we should just risk. Yes, there are things which don't work right now, but that isn't a reason to introduce yet something else which is broken.

avatar alikon
alikon - comment - 20 Feb 2024

ok then
i'll split this in 2 pr

  • one for the update:extensions:check
  • and one RFC for the extension:update
cac5711 20 Feb 2024 avatar alikon split
4f93115 20 Feb 2024 avatar alikon cs
avatar alikon alikon - change - 20 Feb 2024
Title
[5] [cli] extension Update command
[5] [RFC] [cli] extension Update command
avatar alikon alikon - edited - 20 Feb 2024
avatar alikon
alikon - comment - 20 Feb 2024

the update:extensions:check pr #42844

avatar alikon alikon - change - 21 Feb 2024
The description was changed
avatar alikon alikon - edited - 21 Feb 2024
avatar alikon alikon - change - 21 Feb 2024
Labels Added: RFC
avatar alikon alikon - change - 22 Feb 2024
The description was changed
avatar alikon alikon - edited - 22 Feb 2024
avatar alikon
alikon - comment - 22 Feb 2024

i've introduced a tag <updatefromcli>true</updatefromcli> for the updateserver.xml

avatar alikon alikon - change - 11 Apr 2024
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2024-04-11 19:14:20
Closed_By alikon
avatar alikon
alikon - comment - 11 Apr 2024

no interest

avatar alikon alikon - close - 11 Apr 2024

Add a Comment

Login with GitHub to post a comment