core html helper clasess extends core helper classes
Permalink 43 43 Browser Info EnvironmentThe three core output objects should be extending the "non-core" HeaderOutputObject.
Status: Minor
Still Valid:
This bug is valid a newer version of concrete5. View Current BugThe real fix will be more complicated I fear so I reverted it for now.
But currently you have helpers/html.
Code (my code and other code) does Loader::helper('html');
In the super-complex and super-extensible world, you would have, at the top of helpers/html:
Loader::model('output/css'); Loader::model('output/javascript'); Loader::model('output/inline');
Each one of those would have:
Loader::model('output/parent'); // or whatever
So, html helper gets loaded. Before it instantiates anything, the others are loaded. If you consider 4 new files as overkill and too complex, they could be collapsed into fewer.
https://github.com/concrete5/concrete5/pull/1293...