[#17615] - Rename MVC classes and add type as prefix for the MVC classes
- Fixed in Code Base
- 22 Aug 2017
- Medium
- Build: 3.9-dev
- # 17615
- Diff
- Digital-Peak:mvc-factory
User tests: Successful: Unsuccessful:
Pull Request for Issue #16941.
Summary of Changes
Renames the namesapce and the existing MVC factory classe names to upper case MVC. Additionally it fixes an issue where the factory is not passed down the hierarchy from the models and controllers which do extend from the base model/controller.
The important change is that the MVC classes in the component needs to end with the type to be inline with the rest of the namespaced classes. See #17614 for more details. For example the Articles model which has now the name Joomla\Component\Administrator\Model\Articles will become then
Joomla\Component\Administrator\Model\ArticlesModel.
If this one gets merged, then I will adapt the extensions in the 4.0-dev branch accordingly.
Testing Instructions
- Open the back end
- Create an article
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 MVC factories, then it needs to be updated.
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
Merged.
You don't actually have the type in this PR. So this is just a renaming of the classes and stuff. But I'm good with that
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-08-22 11:52:24 |
| Closed_By | ⇒ | wilsonge |
Needs to be synced up since I merged staging up to the 3.9 branch and also moved the MVC factory classes to the right directory.