Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#17935] - [4.0] Rename the MVC classes to fit the new suffix change

? Pending

User tests: Successful: 0 Unsuccessful: 0

avatar laoneo
laoneo
11 Sep 2017

As changed in #17681, every MVC class in a component needs to end with it's corresponding type. So the articles model gets the name ArticlesModel. This pr changes all the core component classes to that change. Additionally it includes the neccessary controller changes intorduced in pr #17624.

avatar joomla-cms-bot joomla-cms-bot - change - 11 Sep 2017
Category ⇒ Administration com_admin com_associations
avatar laoneo laoneo - open - 11 Sep 2017
avatar laoneo laoneo - change - 11 Sep 2017
Status New ⇒ Pending
avatar laoneo laoneo - change - 18 Sep 2017
Labels Added: ?
avatar dgt41
dgt41 - comment - 20 Sep 2017

@wilsonge can yo merge this? we need it for the Media manager sprint

avatar wilsonge
wilsonge - comment - 20 Sep 2017

Not without taking the time to merge up the corresponding staging commit - otherwise everything will be broken ;) Aim is to do it tonight as I told Allon yesterday

avatar laoneo
laoneo - comment - 20 Sep 2017

This commit is included in the pr as well.

avatar laoneo
laoneo - comment - 20 Sep 2017

2f54ca3

avatar wilsonge wilsonge - change - 20 Sep 2017
Status Pending ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2017-09-20 11:51:30
Closed_By ⇒ wilsonge
avatar wilsonge wilsonge - close - 20 Sep 2017
avatar wilsonge wilsonge - merge - 20 Sep 2017
avatar infograf768
infograf768 - comment - 22 Sep 2017

Looks this has broken the managers when changing status of an item.
For example articles
`An error has occurred.

404 View not found [name, type, prefix]: articlescontroller, html, `
avatar laoneo
laoneo - comment - 22 Sep 2017

@infograf768 #18063 should fix the issue.

Add a Comment

Login with GitHub to post a comment