Conversation
|
Could you open an issue on the tracker for this? Thanks. |
|
Wouldn't it be better to move these to the cms folder than to have a cms specific modification in the joomla folder? |
|
The fix isn't so much CMS specific than already fixed in the platform. Just in a different way we probably don't wanna do for 2.5.x (these classes are now autoloadable in the platform) |
|
That's what I'm saying I think. Rather than modify the CMS version of the platform why don't we add to cms? I think we should try to keep thing as clean as possible. |
|
Define clean? If you mean 11.4 - well than I should probably not backport all that stuff from the platform ;) Seriously there's no harm in adding this. Yes it will be overwritten by the next full platform sync, but that's fine because it's a better - and fully compatible - fix. |
|
If it helps clarify this patch, libraries/joomla/access/access.php fixes the JComponentHelper dependencies found in the same file at 288 $result = array(JComponentHelper::getParams('com_users')->get('guest_usergroup', and libraries/joomla/installer/installer.php fixes the JDispatcher dependencies found in the same file at 466 $dispatcher = JDispatcher::getInstance(); |
|
I'm closing this as it's not mergeable and 1 year old. Feel free to open if you update it and are still interested in get it merged. Thanks for contributing. |
Add imports necessary for joomla-cms #207