You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dj...@apache.org on 2005/07/13 07:02:39 UTC
svn commit: r216113 - in
/geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty:
JettyPOJOWebServiceHolder.java JettyServletHolder.java
Author: djencks
Date: Tue Jul 12 22:02:37 2005
New Revision: 216113
URL: http://svn.apache.org/viewcvs?rev=216113&view=rev
Log:
Further fix for GERONIMO-645. POJO ws need to participate in the load-on-startup ordering
Modified:
geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyPOJOWebServiceHolder.java
geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyServletHolder.java
Modified: geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyPOJOWebServiceHolder.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyPOJOWebServiceHolder.java?rev=216113&r1=216112&r2=216113&view=diff
==============================================================================
--- geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyPOJOWebServiceHolder.java (original)
+++ geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyPOJOWebServiceHolder.java Tue Jul 12 22:02:37 2005
@@ -65,6 +65,7 @@
Set servletMappings,
Map webRoleRefPermissions,
StoredObject storedWebServiceContainer,
+ ServletHolder previous, //dependency for startup ordering
JettyServletRegistration context) throws Exception {
super(context == null ? null : context.getServletHandler(), servletName, POJOWebServiceServlet.class.getName(), null);
//context will be null only for use as "default servlet info holder" in deployer.
@@ -105,38 +106,6 @@
super.handle(request, response);
}
- public static final GBeanInfo GBEAN_INFO;
-
- static {
- GBeanInfoBuilder infoBuilder = new GBeanInfoBuilder(JettyPOJOWebServiceHolder.class, NameFactory.SERVLET_WEB_SERVICE_TEMPLATE);
- //todo replace with interface
- infoBuilder.addInterface(ServletHolder.class);
-
- infoBuilder.addAttribute("pojoClassName", String.class, true);
- infoBuilder.addAttribute("servletName", String.class, true);
- infoBuilder.addAttribute("initParams", Map.class, true);
- infoBuilder.addAttribute("loadOnStartup", Integer.class, true);
- infoBuilder.addAttribute("servletMappings", Set.class, true);
- infoBuilder.addAttribute("webRoleRefPermissions", Map.class, true);
- infoBuilder.addAttribute("webServiceContainer", StoredObject.class, true);
- infoBuilder.addReference("JettyServletRegistration", JettyServletRegistration.class);
-
- infoBuilder.setConstructor(new String[]{"pojoClassName",
- "servletName",
- "initParams",
- "loadOnStartup",
- "servletMappings",
- "webRoleRefPermissions",
- "webServiceContainer",
- "JettyServletRegistration"});
-
- GBEAN_INFO = infoBuilder.getBeanInfo();
- }
-
- public static GBeanInfo getGBeanInfo() {
- return GBEAN_INFO;
- }
-
public void doStart() throws Exception {
if (context != null) {
Class pojoClass = context.getWebClassLoader().loadClass(pojoClassName);
@@ -169,6 +138,40 @@
}
public void doFail() {
+ }
+
+ public static final GBeanInfo GBEAN_INFO;
+
+ static {
+ GBeanInfoBuilder infoBuilder = new GBeanInfoBuilder(JettyPOJOWebServiceHolder.class, NameFactory.SERVLET_WEB_SERVICE_TEMPLATE);
+ //todo replace with interface
+ infoBuilder.addInterface(ServletHolder.class);
+
+ infoBuilder.addAttribute("pojoClassName", String.class, true);
+ infoBuilder.addAttribute("servletName", String.class, true);
+ infoBuilder.addAttribute("initParams", Map.class, true);
+ infoBuilder.addAttribute("loadOnStartup", Integer.class, true);
+ infoBuilder.addAttribute("servletMappings", Set.class, true);
+ infoBuilder.addAttribute("webRoleRefPermissions", Map.class, true);
+ infoBuilder.addAttribute("webServiceContainer", StoredObject.class, true);
+ infoBuilder.addReference("Previous", ServletHolder.class, NameFactory.SERVLET);
+ infoBuilder.addReference("JettyServletRegistration", JettyServletRegistration.class);
+
+ infoBuilder.setConstructor(new String[]{"pojoClassName",
+ "servletName",
+ "initParams",
+ "loadOnStartup",
+ "servletMappings",
+ "webRoleRefPermissions",
+ "webServiceContainer",
+ "Previous",
+ "JettyServletRegistration"});
+
+ GBEAN_INFO = infoBuilder.getBeanInfo();
+ }
+
+ public static GBeanInfo getGBeanInfo() {
+ return GBEAN_INFO;
}
Modified: geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyServletHolder.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyServletHolder.java?rev=216113&r1=216112&r2=216113&view=diff
==============================================================================
--- geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyServletHolder.java (original)
+++ geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyServletHolder.java Tue Jul 12 22:02:37 2005
@@ -45,11 +45,9 @@
*/
public class JettyServletHolder extends ServletHolder {
private static final ThreadLocal currentServletName = new ThreadLocal();
- private final JettyServletHolder previous;
//todo consider interface instead of this constructor for endpoint use.
public JettyServletHolder() {
- this.previous = null;
}
public JettyServletHolder(String servletName,
@@ -59,13 +57,11 @@
Integer loadOnStartup,
Set servletMappings,
Map webRoleRefPermissions,
- JettyServletHolder previous,
+ ServletHolder previous, //dependency for startup ordering
JettyServletRegistration context) throws Exception {
super(context == null? null: context.getServletHandler(), servletName, servletClassName, jspFile);
//context will be null only for use as "default servlet info holder" in deployer.
- this.previous = previous;
-
if (context != null) {
putAll(initParams);
if (loadOnStartup != null) {
@@ -84,10 +80,6 @@
return getName();
}
- public JettyServletHolder getPrevious() {
- return previous;
- }
-
/**
* Service a request with this servlet. Set the ThreadLocal to hold the
* current JettyServletHolder.
@@ -129,7 +121,7 @@
infoBuilder.addAttribute("servletMappings", Set.class, true);
infoBuilder.addAttribute("webRoleRefPermissions", Map.class, true);
- infoBuilder.addReference("Previous", JettyServletHolder.class, NameFactory.DEFAULT_SERVLET);
+ infoBuilder.addReference("Previous", ServletHolder.class, NameFactory.SERVLET);
infoBuilder.addReference("JettyServletRegistration", JettyServletRegistration.class, NameFactory.WEB_MODULE);
infoBuilder.setConstructor(new String[] {"servletName",