maqetta/maqetta — Issue #1639: actionContainer is no longer used
Author: billreed63
Created: 2012-01-26 16:34:12+00:00
State: open
Labels: Feature, Priority High

With the AMD change we have made ```` actionSet.actionsContainer```` is no longer needed, In workbech.js we have the following code that loads the "ActionContaier" file.

````
_loadActionSetContainer: function(actionSet)
{
if (actionSet.actionsContainer)
{
if (typeof actionSet.actionsContainer == "string"/* && item.action instanceof String*/)
{
actionSet.actionsContainer=dojo["require"](actionSet.actionsContainer);
}
actionSet.actions.forEach(function(action) { action.actionLoaded = true; });
}

},
````
Now each action is defined in it's own file not in one big action container file... So the code that loades the Action js is in workbench.js But this code could be modified also to stop checking for action container before loading the class

````
_loadActionClass: function(item)
{
if (typeof item.action == "string"/* && item.action instanceof String*/)
{
if (!item.actionLoaded){
var actionContainerClassName = item.actionsContainer?item.actionsContainer:item.action;
dojo["require"](actionContainerClassName);
}
var actionClass= eval(item.action);
item.action=new actionClass;
item.action.item=item;
}

},

````