You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by gk...@apache.org on 2008/08/01 16:22:33 UTC
svn commit: r681703 -
/cocoon/trunk/blocks/cocoon-flowscript/cocoon-flowscript-impl/src/main/java/org/apache/cocoon/components/flow/javascript/fom/FOM_JavaScriptInterpreter.java
Author: gkossakowski
Date: Fri Aug 1 07:22:32 2008
New Revision: 681703
URL: http://svn.apache.org/viewvc?rev=681703&view=rev
Log:
Don't use deprecated JavaSriptException. It's enough to use RuntimeException as we are wrapping it anyway.
Modified:
cocoon/trunk/blocks/cocoon-flowscript/cocoon-flowscript-impl/src/main/java/org/apache/cocoon/components/flow/javascript/fom/FOM_JavaScriptInterpreter.java
Modified: cocoon/trunk/blocks/cocoon-flowscript/cocoon-flowscript-impl/src/main/java/org/apache/cocoon/components/flow/javascript/fom/FOM_JavaScriptInterpreter.java
URL: http://svn.apache.org/viewvc/cocoon/trunk/blocks/cocoon-flowscript/cocoon-flowscript-impl/src/main/java/org/apache/cocoon/components/flow/javascript/fom/FOM_JavaScriptInterpreter.java?rev=681703&r1=681702&r2=681703&view=diff
==============================================================================
--- cocoon/trunk/blocks/cocoon-flowscript/cocoon-flowscript-impl/src/main/java/org/apache/cocoon/components/flow/javascript/fom/FOM_JavaScriptInterpreter.java (original)
+++ cocoon/trunk/blocks/cocoon-flowscript/cocoon-flowscript-impl/src/main/java/org/apache/cocoon/components/flow/javascript/fom/FOM_JavaScriptInterpreter.java Fri Aug 1 07:22:32 2008
@@ -302,7 +302,7 @@
//java class (used by importClass & importPackage)
if (this.locked && !has(name, start) && !(value instanceof NativeJavaClass) && !(value instanceof Function)) {
// Need to wrap into a runtime exception as Scriptable.put has no throws clause...
- throw new WrappedException (new JavaScriptException("Implicit declaration of global variable '" + name +
+ throw new WrappedException (new RuntimeException("Implicit declaration of global variable '" + name +
"' forbidden. Please ensure all variables are explicitely declared with the 'var' keyword"));
}
this.useSession = true;
@@ -312,7 +312,7 @@
public void put(int index, Scriptable start, Object value) {
// FIXME(SW): do indexed properties have a meaning on the global scope?
if (this.locked && !has(index, start)) {
- throw new WrappedException(new JavaScriptException("Global scope locked. Cannot set value for index " + index));
+ throw new WrappedException(new RuntimeException("Global scope locked. Cannot set value for index " + index));
}
this.useSession = true;
super.put(index, start, value);