? ? Pending

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
9 Feb 2019

Summary of Changes

This PR updates the joomla/console package based on the last architectural update and adapts the classes within the CMS environment to those changes. This PR also includes other minor updates to the service definition of the console application class.

Changes include:

  • Update application constructor signature to match new structure
  • Update all command classes to new structure
  • Update console application service to use the defined session.cli service versus the inlined session service and to inject via constructor the dispatcher and container dependencies

Testing Instructions

Apply this patch to a local installation and run composer install to pull the updated Composer dependencies. Run php cli/joomla.php <command> to validate console runs, keeping #23845 in mind and if necessary updating your environment.

avatar mbabker mbabker - open - 9 Feb 2019
avatar mbabker mbabker - change - 9 Feb 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Feb 2019
Category External Library Composer Change Libraries
avatar mbabker mbabker - change - 9 Feb 2019
Labels Added: ? ?
avatar mbabker
mbabker - comment - 19 Feb 2019

Not quite sure why I spent time on proper CLI support for an application that can't even properly support its non-HTML outputs.

avatar mbabker mbabker - close - 19 Feb 2019
avatar mbabker mbabker - change - 19 Feb 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-02-19 01:55:28
Closed_By mbabker
avatar zero-24
zero-24 - comment - 19 Feb 2019

Can you please re-open this one. I havn't have time to look into all PRs but it seams i have missed this. Can you give this an seccond chance?

avatar mbabker mbabker - change - 19 Feb 2019
Status Closed New
Closed_Date 2019-02-19 01:55:28
Closed_By mbabker
avatar mbabker mbabker - change - 19 Feb 2019
Status New Pending
avatar mbabker mbabker - reopen - 19 Feb 2019
avatar zero-24
zero-24 - comment - 19 Feb 2019

Thanks Michael this is now on my personal to-do list. It might take some time but it will not be forgotten.

avatar dneukirchen
dneukirchen - comment - 20 Feb 2019

I have tested this item successfully on e326695

I can confirm that this PR fixed the issue (needed to disable debug plugin as described). Thx michael.


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

bildschirmfoto 2019-02-20 um 11 03 27

avatar dneukirchen
dneukirchen - comment - 20 Feb 2019

I have tested this item successfully on e326695

I can confirm that this PR fixed the issue (needed to disable debug plugin as described). Thx michael.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23847.
avatar dneukirchen dneukirchen - test_item - 20 Feb 2019 - Tested successfully
avatar alikon
alikon - comment - 20 Feb 2019

I have tested this item successfully on e326695


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

avatar alikon alikon - test_item - 20 Feb 2019 - Tested successfully
avatar HLeithner HLeithner - change - 20 Feb 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-02-20 13:49:19
Closed_By HLeithner
avatar HLeithner HLeithner - close - 20 Feb 2019
avatar HLeithner HLeithner - merge - 20 Feb 2019
avatar HLeithner
HLeithner - comment - 20 Feb 2019

thx

avatar zero-24
zero-24 - comment - 20 Feb 2019

Thanks for testing and merging guys ? you just removed one item from my to-do list ?

Add a Comment

Login with GitHub to post a comment