New module directive this.applicationHelper to allow for automatic loading/unloading of helpers

Description

You can only set one application level view helper. It would be helpful, especially with modules, to be able to set a list or array, just like with Application Helpers.

Activity

Show:
Eric Peterson
November 13, 2017, 3:30 AM
Eric Peterson
November 13, 2017, 3:31 AM

I think it's fine if we want to make a module setting to declare helpers, but we'd need to fix this under the hood anyway.

Luis Majano
November 13, 2017, 3:32 AM

I know, me too. That's what I am saying, maybe we need a convention where modules can just do a list or array of helpers and let ColdBox load and unload them for you.

This can be something like:

Eric Peterson
November 13, 2017, 3:34 AM

Fine with me. We'd want applicationHelpers and viewHelpers

Brad Wood
November 13, 2017, 3:36 AM

I think there needs to be a proper convention for modules to declare helpers. Then the framework (ColdBox, CommandBox) does the work of appending them into the main array (and checking for duplicates, etc) and when the module unloads, the framework should automatically remove them as well. All the boilerplate we have to do right now isn't good at all. A module shouldn't reach into the guts of the framework and inject its bits. It should declare it's "exports" and let the framework pick them up and use them.

Fixed

Assignee

Luis Majano

Reporter

Eric Peterson

Labels

None

Fix versions

Priority

Major
Configure