? ? Failure

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
18 Sep 2016

Summary of Changes

JApplicationCli is recomposed to inherit from the Framework's Joomla\Application\AbstractCliApplication class now. This allows the CMS app class to use all features from the Framework.

To do so without losing features, several behaviors from the CMS app class chain are moved into traits and those are now used as appropriate.

This also follows through with the noted change to JApplicationCli to make it abstract.

Testing Instructions

For CLI applications, they should continue to function as is with no changes. The changes I did make to the doExecute method signatures just causes the visibility to match the parent definition.

Web applications should work too. Because of the trait implementations their code is reshuffled a bit but nothing changes from a practical standpoint.

Documentation Changes Required

The class inheritance for JApplicationCli has changed, this will break instanceof checks.

3.x:

  • JApplicationCli
  • JApplicationBase
  • Joomla\Application\AbstractApplication

4.0:

  • JApplicationCli
  • Joomla\Application\AbstractCliApplication
  • Joomla\Application\AbstractApplication

Also, as documented, JApplicationCli is now abstract.

avatar mbabker mbabker - open - 18 Sep 2016
avatar mbabker mbabker - change - 18 Sep 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Sep 2016
Labels Added: ? ?
avatar joomla-cms-bot joomla-cms-bot - change - 18 Sep 2016
Category Administration CLI Libraries Unit Tests
avatar wilsonge wilsonge - change - 18 Sep 2016
Labels Added: ?
avatar wilsonge wilsonge - change - 18 Sep 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-09-18 15:09:47
Closed_By wilsonge
avatar zero-24
zero-24 - comment - 18 Sep 2016

@wilsonge i think there is no reason to not add the 4.0 milestone correct?

avatar wilsonge
wilsonge - comment - 18 Sep 2016

Correct. We can add a 4.x milestone

avatar zero-24
zero-24 - comment - 18 Sep 2016

Thanks ?

Add a Comment

Login with GitHub to post a comment