[ http://issues.apache.org/jira/browse/COCOON-1811?page=comments#action_12372945 ]
Jason Johnston commented on COCOON-1811: ---------------------------------------- Would it perhaps make more sense to modify the behavior of cocoon.load() so that it loads the target script not into the top-level scope but instead into the scope from which it is called? So in the original example the myObject class would only be defined within the scope of the loadScript() function. This seems cleaner to me. > [PATCH] Flow Script: Allow dynamic loading of JavaScript objects even when > scope is locked > ------------------------------------------------------------------------------------------ > > Key: COCOON-1811 > URL: http://issues.apache.org/jira/browse/COCOON-1811 > Project: Cocoon > Type: Improvement > Components: Blocks: Forms > Versions: 2.2-dev (Current SVN), 2.1.9-dev (current SVN) > Reporter: Rob Berens > Priority: Minor > Attachments: FOM_JavaScriptInterpreter.txt > > Currently it is not possible to add variables to the scope of a > FOM_JavaScriptInterpreter, unless the scope is not locked yet or when still > in the main loading process or when loading native java classes. Therefore it > is not possible to dynamically load JavaScript classes like the one below:. > ------------------------------- > function myObject() { // at this point the current > implementation throws the exception > // constructor for myObject > } > myObject.prototype.myMethod = function() { > // implementation of myMethod > } > ------------------------------- > from within a script fragment like this one: > ------------------------------- > function loadScript() { > var scriptURI = "determineScriptURIFromRequest"; > cocoon.load(scriptURI); > } > ------------------------------- > The attached patch solves this by allowing also objects of the type > org.mozilla.javascript.Function to be loaded into a locked scope. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
