? Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
19 Aug 2017

Summary of Changes

This are the controller changes to give the component MVC classes more meaningful names. Controllers will have now Controller as suffix to their class name. So Joomla\Component\Banners\Administrator\Controller\Banners becomes Joomla\Component\Banners\Administrator\Controller\BannersController.

Additionally the default controller which is called just Controller gets the name DisplayController, to better reflect whet the class does.

Testing Instructions

  • Create a banner in the back end
  • Create an article in the back end

Expected result

All is working as expected.

Actual result

All is working as expected.

Documentation Changes Required

If there is already documentation written for the namespaced controllers, then it needs to be updated.

avatar joomla-cms-bot joomla-cms-bot - change - 19 Aug 2017
Category Administration com_banners Libraries
avatar laoneo laoneo - open - 19 Aug 2017
avatar laoneo laoneo - change - 19 Aug 2017
Status New Pending
avatar laoneo laoneo - change - 22 Aug 2017
Title
Add Controller suffix to extension controllers
[4.0] Add Controller suffix to extension controllers
avatar laoneo laoneo - edited - 22 Aug 2017
avatar laoneo laoneo - change - 22 Aug 2017
Labels Added: ?
avatar laoneo laoneo - change - 22 Aug 2017
The description was changed
avatar laoneo laoneo - edited - 22 Aug 2017
avatar laoneo laoneo - close - 23 Aug 2017
avatar laoneo laoneo - change - 23 Aug 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-08-23 05:54:37
Closed_By laoneo
avatar laoneo
laoneo - comment - 23 Aug 2017

Accidentally deleted branch. Thanks reflog!!

avatar laoneo laoneo - change - 23 Aug 2017
Status Closed New
Closed_Date 2017-08-23 05:54:37
Closed_By laoneo
avatar laoneo laoneo - change - 23 Aug 2017
Status New Pending
avatar laoneo laoneo - reopen - 23 Aug 2017
avatar wilsonge wilsonge - change - 24 Aug 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-08-24 10:51:37
Closed_By wilsonge
avatar wilsonge wilsonge - close - 24 Aug 2017
avatar wilsonge wilsonge - merge - 24 Aug 2017

Add a Comment

Login with GitHub to post a comment