You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2012/06/15 11:25:08 UTC
svn commit: r1350549 -
/openejb/trunk/openejb/tomee/tomee-webapp/src/main/java/org/apache/tomee/webapp/servlet/ConsoleServlet.java
Author: rmannibucau
Date: Fri Jun 15 09:25:07 2012
New Revision: 1350549
URL: http://svn.apache.org/viewvc?rev=1350549&view=rev
Log:
reactivating the scripter in console servlet
Modified:
openejb/trunk/openejb/tomee/tomee-webapp/src/main/java/org/apache/tomee/webapp/servlet/ConsoleServlet.java
Modified: openejb/trunk/openejb/tomee/tomee-webapp/src/main/java/org/apache/tomee/webapp/servlet/ConsoleServlet.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/tomee/tomee-webapp/src/main/java/org/apache/tomee/webapp/servlet/ConsoleServlet.java?rev=1350549&r1=1350548&r2=1350549&view=diff
==============================================================================
--- openejb/trunk/openejb/tomee/tomee-webapp/src/main/java/org/apache/tomee/webapp/servlet/ConsoleServlet.java (original)
+++ openejb/trunk/openejb/tomee/tomee-webapp/src/main/java/org/apache/tomee/webapp/servlet/ConsoleServlet.java Fri Jun 15 09:25:07 2012
@@ -19,6 +19,7 @@ package org.apache.tomee.webapp.servlet;
import org.apache.openejb.util.OpenEJBScripter;
import org.apache.tomee.webapp.JsonExecutor;
+import org.apache.tomee.webapp.TomeeException;
import org.apache.tomee.webapp.listener.UserSessionListener;
import javax.script.*;
@@ -43,16 +44,11 @@ public class ConsoleServlet extends Http
return; //nothing to do
}
- final HttpSession session = req.getSession();
-
String engineName = req.getParameter("engineName");
if (engineName == null || "".equals(engineName.trim())) {
engineName = "js";
}
- final ScriptEngineManager manager = new ScriptEngineManager();
- final ScriptEngine engine = manager.getEngineByName(engineName);
-
//new context for the execution of this script
final ScriptContext newContext = new SimpleScriptContext();
@@ -80,7 +76,11 @@ public class ConsoleServlet extends Http
});
//note that "engine" does not know "bindings". It only knows the current context.
- engine.eval(scriptCode, newContext);
+ final Object result = SCRIPTER.evaluate(engineName, req.getParameter("scriptCode"));
+ // TODO: don't throw an exception here
+ // doing it just to print the result but should be done in a 'normal' way
+ // i like the idea of the error popup but in the "result popup" way
+ throw new TomeeException(new RuntimeException(result.toString())); // todo: json
}
});
}