User tests: Successful: Unsuccessful:
Namespace content site route helper.
Open an article on the front.
Article is displayed.
Article is displayed.
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_categories com_content Front End com_contact com_search Libraries Modules Plugins |
Labels |
Added:
?
|
I was also thinking about the container. Problem is that mostly legacy code is using it where boot is not called before.
Conflicts fixed
Labels |
Removed:
J4 Issue
|
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-06-14 19:35:05 |
Closed_By | ⇒ | wilsonge |
Thanks!
I still feel that this is wrong because 3PD extensions have no way of adding entries into the extensions mapper file. This file should be something that we use to allow other extensions to function who extended non-namespaced classes. But as you say it gets the namespacing done for now and can be improved in the future.
Can you create an issue with your comments so it doesn't get forgotten
Done
Conflicts. But I want to sit on this one for a bit. I really really don't like mapping this class globally. But I need to have a think about what a better solution actually looks like - presumably something in the extensions container? :/