? Failure

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
2 Oct 2016

This PR moves the JTable classes from the /libraries/legacy folder to /libraries/cms and undeprecates the JTableContent class. I doubt that there will be any UCM in the forseeable future. When it should one day be implemented, we can still deprecate that class again. But until then...

avatar Hackwar Hackwar - open - 2 Oct 2016
avatar Hackwar Hackwar - change - 2 Oct 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Oct 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 2 Oct 2016
Category Libraries
avatar wilsonge
wilsonge - comment - 2 Oct 2016

Would it not be better to move these classes to their respective components rather than the global libraries folder?

avatar Hackwar
Hackwar - comment - 2 Oct 2016

Yes and no, these classes are used not only in their respective component, but all over the place and moving them to their respective component would require all third party extensions to add that folder to the JTable include paths. In order to not change for the sake of change, I would keep it in the libraries folder.

avatar mbabker
mbabker - comment - 2 Oct 2016

Another way to look at it is what tables would be considered part of the "core" data model? Content's pretty specific but the others are part of the global app.

avatar Hackwar
Hackwar - comment - 2 Oct 2016

I'm not saying that I disagree with you that some of these tables belong to their respective component. It is just that I thought the general consensus was to not change for changes sake. This is a change that I consider to be a nice-to-have-but-not-essential, so that in order to keep the change list shorter I wouldn't touch this.

avatar wilsonge
wilsonge - comment - 3 Oct 2016

For the menu I'd put it in com_menu, for content, com_content, for module com_modules.... I mean I'm not fixed on it. But I'd assume that was the idea behind them going into legacy

avatar wilsonge
wilsonge - comment - 3 Oct 2016

I'm not set on that however. If you both think moving them to libraries is better - let's go with it :)

avatar mbabker
mbabker - comment - 3 Oct 2016

If I wanted to be disruptive I'd move JTableContent to the component and leave the rest in the libraries. But that's being disruptive for minimal gain.

With the current data model in core, the concepts of categories, menus, and modules are all global things and are not isolated to a single component (their management is exposed through a component though). So those table classes are rightfully part of the libraries.

avatar wilsonge wilsonge - change - 3 Oct 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-10-03 12:23:35
Closed_By wilsonge
avatar wilsonge wilsonge - close - 3 Oct 2016
avatar wilsonge wilsonge - merge - 3 Oct 2016

Add a Comment

Login with GitHub to post a comment