Author: mrdon Date: Thu Jul 13 18:09:55 2006 New Revision: 421778 URL: http://svn.apache.org/viewvc?rev=421778&view=rev Log: More cleanups - Javadocs, empty packages, unused methods, unused imports, accessors, etc WW-1349
Removed: struts/struts2/trunk/core/src/main/java/org/apache/struts2/spring/interceptor/ Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/interceptor/debugging/DebuggingInterceptor.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/jsf/FacesSetupInterceptor.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusFilter.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusLifecycleListener.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusObjectFactory.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusThreadLocal.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusUtils.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletApplicationMap.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletRequestMap.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletSessionMap.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/context/ServletContextHolderListener.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/dispatcher/Jsr168Dispatcher.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/result/PortletVelocityResult.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/util/PortletUrlHelper.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/spring/StrutsSpringObjectFactory.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/spring/lifecycle/SpringExternalReferenceResolverSetupListener.java Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/interceptor/debugging/DebuggingInterceptor.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/interceptor/debugging/DebuggingInterceptor.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/interceptor/debugging/DebuggingInterceptor.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/interceptor/debugging/DebuggingInterceptor.java Thu Jul 13 18:09:55 2006 @@ -214,7 +214,7 @@ ActionContext ctx = ActionContext.getContext(); writer.startNode(DEBUG_PARAM); serializeIt(ctx.getParameters(), "parameters", writer, - new ArrayList()); + new ArrayList<Object>()); writer.startNode("context"); String key; Map ctxMap = ctx.getContextMap(); @@ -229,15 +229,15 @@ } } if (print) { - serializeIt(ctxMap.get(key), key, writer, new ArrayList()); + serializeIt(ctxMap.get(key), key, writer, new ArrayList<Object>()); } } writer.endNode(); - serializeIt(ctx.getSession(), "request", writer, new ArrayList()); - serializeIt(ctx.getSession(), "session", writer, new ArrayList()); + serializeIt(ctx.getSession(), "request", writer, new ArrayList<Object>()); + serializeIt(ctx.getSession(), "session", writer, new ArrayList<Object>()); OgnlValueStack stack = (OgnlValueStack) ctx.get(ActionContext.VALUE_STACK); - serializeIt(stack.getRoot(), "valueStack", writer, new ArrayList()); + serializeIt(stack.getRoot(), "valueStack", writer, new ArrayList<Object>()); writer.endNode(); } @@ -256,7 +256,7 @@ * of this function (to prevent looping on circular references). */ protected void serializeIt(Object bean, String name, - PrettyPrintWriter writer, List stack) { + PrettyPrintWriter writer, List<Object> stack) { writer.flush(); // Check stack for this object if ((bean != null) && (stack.contains(bean))) { Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/jsf/FacesSetupInterceptor.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/jsf/FacesSetupInterceptor.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/jsf/FacesSetupInterceptor.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/jsf/FacesSetupInterceptor.java Thu Jul 13 18:09:55 2006 @@ -41,16 +41,13 @@ private static final long serialVersionUID = -621512342655103941L; private String lifecycleId = LifecycleFactory.DEFAULT_LIFECYCLE; - private FacesContextFactory facesContextFactory; - private Lifecycle lifecycle; /** * Sets the lifecycle id * - * @param id - * The id + * @param id The id */ public void setLifecycleId(String id) { this.lifecycleId = id; Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusFilter.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusFilter.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusFilter.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusFilter.java Thu Jul 13 18:09:55 2006 @@ -28,20 +28,34 @@ import java.util.Collections; /** + * Creates a plexus container for the application, session, and request */ public class PlexusFilter implements Filter { private static final Log log = LogFactory.getLog(PlexusObjectFactory.class); private static final String CHILD_CONTAINER_NAME = "request"; - public static boolean loaded = false; + private static boolean loaded = false; private ServletContext ctx; + /** + * @return Returns if the container is loaded. + */ + public static boolean isLoaded() { + return loaded; + } + + /* (non-Javadoc) + * @see javax.servlet.Filter#init(javax.servlet.FilterConfig) + */ public void init(FilterConfig filterConfig) throws ServletException { ctx = filterConfig.getServletContext(); loaded = true; } + /* (non-Javadoc) + * @see javax.servlet.Filter#doFilter(javax.servlet.ServletRequest, javax.servlet.ServletResponse, javax.servlet.FilterChain) + */ public void doFilter(ServletRequest req, ServletResponse res, FilterChain chain) throws IOException, ServletException { PlexusContainer child = null; try { @@ -83,6 +97,9 @@ } } + /* (non-Javadoc) + * @see javax.servlet.Filter#destroy() + */ public void destroy() { } } Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusLifecycleListener.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusLifecycleListener.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusLifecycleListener.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusLifecycleListener.java Thu Jul 13 18:09:55 2006 @@ -31,14 +31,24 @@ import java.util.Collections; /** - * + * Manages the Plexus lifecycle for the servlet and session contexts */ public class PlexusLifecycleListener implements ServletContextListener, HttpSessionListener { private static final Log log = LogFactory.getLog(PlexusObjectFactory.class); - public static boolean loaded = false; + private static boolean loaded = false; public static final String KEY = "struts.plexus.container"; + /** + * @return Returns if the container is loaded. + */ + public static boolean isLoaded() { + return loaded; + } + + /* (non-Javadoc) + * @see javax.servlet.ServletContextListener#contextInitialized(javax.servlet.ServletContextEvent) + */ public void contextInitialized(ServletContextEvent servletContextEvent) { loaded = true; @@ -55,6 +65,9 @@ } } + /* (non-Javadoc) + * @see javax.servlet.ServletContextListener#contextDestroyed(javax.servlet.ServletContextEvent) + */ public void contextDestroyed(ServletContextEvent servletContextEvent) { try { ServletContext ctx = servletContextEvent.getServletContext(); @@ -65,6 +78,9 @@ } } + /* (non-Javadoc) + * @see javax.servlet.http.HttpSessionListener#sessionCreated(javax.servlet.http.HttpSessionEvent) + */ public void sessionCreated(HttpSessionEvent httpSessionEvent) { try { HttpSession session = httpSessionEvent.getSession(); @@ -80,6 +96,9 @@ } } + /* (non-Javadoc) + * @see javax.servlet.http.HttpSessionListener#sessionDestroyed(javax.servlet.http.HttpSessionEvent) + */ public void sessionDestroyed(HttpSessionEvent httpSessionEvent) { try { HttpSession session = httpSessionEvent.getSession(); Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusObjectFactory.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusObjectFactory.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusObjectFactory.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusObjectFactory.java Thu Jul 13 18:09:55 2006 @@ -70,18 +70,13 @@ private static final String PLEXUS_COMPONENT_TYPE = "plexus.component.type"; - // ---------------------------------------------------------------------- - // Privates - // ---------------------------------------------------------------------- - private PlexusContainer base; - // ---------------------------------------------------------------------- - // ObjectFactory overrides - // ---------------------------------------------------------------------- - + /* (non-Javadoc) + * @see org.apache.struts2.util.ObjectFactoryInitializable#init(javax.servlet.ServletContext) + */ public void init(ServletContext servletContext) { - if (!PlexusLifecycleListener.loaded || !PlexusFilter.loaded) { + if (!PlexusLifecycleListener.isLoaded() || !PlexusFilter.isLoaded()) { // uh oh! looks like the lifecycle listener wasn't installed. Let's inform the user String message = "********** FATAL ERROR STARTING UP PLEXUS-STRUTS INTEGRATION **********\n" + "Looks like the Plexus listener was not configured for your web app! \n" + @@ -113,6 +108,9 @@ base = (PlexusContainer) servletContext.getAttribute(PlexusLifecycleListener.KEY); } + /* (non-Javadoc) + * @see com.opensymphony.xwork2.ObjectFactory#buildAction(java.lang.String, java.lang.String, com.opensymphony.xwork2.config.entities.ActionConfig, java.util.Map) + */ public Object buildAction(String actionName, String namespace, ActionConfig config, Map extraContext) throws Exception { if (extraContext == null) { @@ -124,6 +122,9 @@ return super.buildAction(actionName, namespace, config, extraContext); } + /* (non-Javadoc) + * @see com.opensymphony.xwork2.ObjectFactory#buildInterceptor(com.opensymphony.xwork2.config.entities.InterceptorConfig, java.util.Map) + */ public Interceptor buildInterceptor(InterceptorConfig interceptorConfig, Map interceptorRefParams) throws ConfigurationException { String interceptorClassName = interceptorConfig.getClassName(); @@ -167,6 +168,9 @@ throw new ConfigurationException(message, cause); } + /* (non-Javadoc) + * @see com.opensymphony.xwork2.ObjectFactory#buildResult(com.opensymphony.xwork2.config.entities.ResultConfig, java.util.Map) + */ public Result buildResult(ResultConfig resultConfig, Map extraContext) throws Exception { if (extraContext == null) { @@ -178,6 +182,9 @@ return super.buildResult(resultConfig, extraContext); } + /* (non-Javadoc) + * @see com.opensymphony.xwork2.ObjectFactory#buildValidator(java.lang.String, java.util.Map, java.util.Map) + */ public Validator buildValidator(String className, Map params, Map extraContext) throws Exception { Map context = new HashMap(); @@ -188,6 +195,9 @@ return validator; } + /* (non-Javadoc) + * @see com.opensymphony.xwork2.ObjectFactory#buildBean(java.lang.Class, java.util.Map) + */ public Object buildBean(Class clazz, Map extraContext) throws Exception { try { @@ -206,6 +216,9 @@ } } + /* (non-Javadoc) + * @see com.opensymphony.xwork2.ObjectFactory#getClassInstance(java.lang.String) + */ public Class getClassInstance(String className) throws ClassNotFoundException { PlexusContainer pc = PlexusThreadLocal.getPlexusContainer(); @@ -242,21 +255,28 @@ } } - private Object lookup(String role) - throws Exception { - return lookup(role, null, null); - } - + /** + * Looks up an object + * + * @param role The role name + * @param extraContext The extra context + * @return The object + * @throws Exception If the lookup fails + */ private Object lookup(String role, Map extraContext) throws Exception { return lookup(role, null, extraContext); } - private Object lookup(String role, String roleHint) - throws Exception { - return lookup(role, roleHint, null); - } - + /** + * Looks up an object + * + * @param role The role name + * @param roleHint The role hint + * @param extraContext The extra context + * @return The object + * @throws Exception If the lookup fails + */ private Object lookup(String role, String roleHint, Map extraContext) throws Exception { PlexusContainer pc = PlexusThreadLocal.getPlexusContainer(); Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusThreadLocal.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusThreadLocal.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusThreadLocal.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusThreadLocal.java Thu Jul 13 18:09:55 2006 @@ -20,15 +20,22 @@ import org.codehaus.plexus.PlexusContainer; /** + * Stores and retrieves the plexus container in the thread */ public class PlexusThreadLocal { - static ThreadLocal ptl = new ThreadLocal(); + private static ThreadLocal<PlexusContainer> ptl = new ThreadLocal<PlexusContainer>(); + /** + * @param pc The plexus container for this thread + */ public static void setPlexusContainer(PlexusContainer pc) { ptl.set(pc); } + /** + * @return The plexus container for this thread + */ public static PlexusContainer getPlexusContainer() { - return (PlexusContainer) ptl.get(); + return ptl.get(); } } Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusUtils.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusUtils.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusUtils.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/plexus/PlexusUtils.java Thu Jul 13 18:09:55 2006 @@ -27,10 +27,18 @@ import java.io.ByteArrayInputStream; /** + * Utility methods for dealing with Plexus */ public class PlexusUtils { private static final Log log = LogFactory.getLog(PlexusObjectFactory.class); + /** + * Configures the container with the configuration file + * + * @param pc The plexus container + * @param file The file path + * @throws PlexusConfigurationResourceException If the plexus configuration can't be loaded + */ public static void configure(PlexusContainer pc, String file) throws PlexusConfigurationResourceException { InputStream is = Thread.currentThread().getContextClassLoader().getResourceAsStream(file); if (is == null) { Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletApplicationMap.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletApplicationMap.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletApplicationMap.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletApplicationMap.java Thu Jul 13 18:09:55 2006 @@ -33,9 +33,9 @@ */ public class PortletApplicationMap extends AbstractMap implements Serializable { - PortletContext context; + private PortletContext context; - Set entries; + private Set<Object> entries; /** * Creates a new map object given the [EMAIL PROTECTED] PortletContext}. @@ -69,7 +69,7 @@ */ public Set entrySet() { if (entries == null) { - entries = new HashSet(); + entries = new HashSet<Object>(); // Add portlet context attributes Enumeration enumeration = context.getAttributeNames(); Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletRequestMap.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletRequestMap.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletRequestMap.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletRequestMap.java Thu Jul 13 18:09:55 2006 @@ -36,8 +36,7 @@ private static final Log LOG = LogFactory.getLog(PortletRequestMap.class); - private Set entries = null; - + private Set<Object> entries = null; private PortletRequest request = null; /** @@ -79,7 +78,7 @@ */ public Set entrySet() { if (entries == null) { - entries = new HashSet(); + entries = new HashSet<Object>(); Enumeration enumeration = request.getAttributeNames(); Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletSessionMap.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletSessionMap.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletSessionMap.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/PortletSessionMap.java Thu Jul 13 18:09:55 2006 @@ -40,8 +40,7 @@ private static final Log LOG = LogFactory.getLog(PortletSessionMap.class); private PortletSession session = null; - - private Set entries = null; + private Set<Object> entries = null; /** * Creates a new session map given a portlet request. @@ -67,7 +66,7 @@ public Set entrySet() { synchronized (session) { if (entries == null) { - entries = new HashSet(); + entries = new HashSet<Object>(); Enumeration enumeration = session.getAttributeNames(); Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/context/ServletContextHolderListener.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/context/ServletContextHolderListener.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/context/ServletContextHolderListener.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/context/ServletContextHolderListener.java Thu Jul 13 18:09:55 2006 @@ -33,6 +33,9 @@ private static ServletContext context = null; + /** + * @return The current servlet context + */ public static ServletContext getServletContext() { return context; } Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/dispatcher/Jsr168Dispatcher.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/dispatcher/Jsr168Dispatcher.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/dispatcher/Jsr168Dispatcher.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/dispatcher/Jsr168Dispatcher.java Thu Jul 13 18:09:55 2006 @@ -161,9 +161,9 @@ private ActionProxyFactory factory = null; - private Map modeMap = new HashMap(3); + private Map<PortletMode,String> modeMap = new HashMap<PortletMode,String>(3); - private Map actionMap = new HashMap(3); + private Map<PortletMode,ActionMapping> actionMap = new HashMap<PortletMode,ActionMapping>(3); private String portletNamespace = null; @@ -352,7 +352,7 @@ // TODO Must put http request/response objects into map for use with // ServletActionContext - HashMap extraContext = new HashMap(); + HashMap<String,Object> extraContext = new HashMap<String,Object>(); extraContext.put(ActionContext.PARAMETERS, parameterMap); extraContext.put(ActionContext.SESSION, sessionMap); extraContext.put(ActionContext.APPLICATION, applicationMap); @@ -411,7 +411,6 @@ HashMap extraContext = createContextMap(requestMap, parameterMap, sessionMap, applicationMap, request, response, getPortletConfig(), phase); - PortletMode mode = request.getPortletMode(); String actionName = mapping.getName(); String namespace = mapping.getNamespace(); try { Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/result/PortletVelocityResult.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/result/PortletVelocityResult.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/result/PortletVelocityResult.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/result/PortletVelocityResult.java Thu Jul 13 18:09:55 2006 @@ -94,6 +94,9 @@ private static final Log log = LogFactory .getLog(PortletVelocityResult.class); + /* (non-Javadoc) + * @see org.apache.struts2.dispatcher.StrutsResultSupport#doExecute(java.lang.String, com.opensymphony.xwork2.ActionInvocation) + */ public void doExecute(String location, ActionInvocation invocation) throws Exception { if (PortletActionContext.isEvent()) { @@ -104,8 +107,10 @@ } /** - * @param location - * @param invocation + * Executes the result + * + * @param location The location string + * @param invocation The action invocation */ private void executeActionResult(String location, ActionInvocation invocation) { @@ -268,7 +273,7 @@ } /** - * + * Prepares the servlet action context for this request */ private void prepareServletActionContext() throws PortletException, IOException { Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/util/PortletUrlHelper.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/util/PortletUrlHelper.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/util/PortletUrlHelper.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/portlet/util/PortletUrlHelper.java Thu Jul 13 18:09:55 2006 @@ -17,7 +17,6 @@ */ package org.apache.struts2.portlet.util; -import java.io.UnsupportedEncodingException; import java.net.URLEncoder; import java.util.HashMap; import java.util.Iterator; @@ -30,12 +29,10 @@ import javax.portlet.RenderRequest; import javax.portlet.RenderResponse; import javax.portlet.WindowState; -import javax.servlet.jsp.JspException; import org.apache.commons.lang.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; - import org.apache.struts2.portlet.PortletActionConstants; import org.apache.struts2.portlet.context.PortletActionContext; @@ -49,18 +46,6 @@ */ public class PortletUrlHelper { private static final Log LOG = LogFactory.getLog(PortletUrlHelper.class); - - /** - * Default HTTP port (80). - */ - private static final int DEFAULT_HTTP_PORT = 80; - - /** - * Default HTTPS port (443). - */ - private static final int DEFAULT_HTTPS_PORT = 443; - - private static final String AMP = "&"; /** * Create a portlet URL with for the specified action and namespace. Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/spring/StrutsSpringObjectFactory.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/spring/StrutsSpringObjectFactory.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/spring/StrutsSpringObjectFactory.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/spring/StrutsSpringObjectFactory.java Thu Jul 13 18:09:55 2006 @@ -41,6 +41,9 @@ public class StrutsSpringObjectFactory extends SpringObjectFactory implements ObjectFactoryInitializable { private static final Log log = LogFactory.getLog(StrutsSpringObjectFactory.class); + /* (non-Javadoc) + * @see org.apache.struts2.util.ObjectFactoryInitializable#init(javax.servlet.ServletContext) + */ public void init(ServletContext servletContext) { log.info("Initializing Struts-Spring integration..."); Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/spring/lifecycle/SpringExternalReferenceResolverSetupListener.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/spring/lifecycle/SpringExternalReferenceResolverSetupListener.java?rev=421778&r1=421777&r2=421778&view=diff ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/spring/lifecycle/SpringExternalReferenceResolverSetupListener.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/spring/lifecycle/SpringExternalReferenceResolverSetupListener.java Thu Jul 13 18:09:55 2006 @@ -48,26 +48,45 @@ private Map<ServletContext,Listener> listeners = new HashMap<ServletContext,Listener>(); + /* (non-Javadoc) + * @see javax.servlet.ServletContextListener#contextDestroyed(javax.servlet.ServletContextEvent) + */ public synchronized void contextDestroyed(ServletContextEvent event) { Listener l = listeners.get(event.getServletContext()); Dispatcher.removeDispatcherListener(l); listeners.remove(event.getServletContext()); } + /* (non-Javadoc) + * @see javax.servlet.ServletContextListener#contextInitialized(javax.servlet.ServletContextEvent) + */ public synchronized void contextInitialized(ServletContextEvent event) { Listener l = new Listener(event.getServletContext()); Dispatcher.addDispatcherListener(l); listeners.put(event.getServletContext(), l); } + /** + * Handles initializing and cleaning up the dispatcher + * @author brownd + * + */ private class Listener implements DispatcherListener { private ServletContext servletContext; + /** + * Constructs the listener + * + * @param ctx The servlet context + */ public Listener(ServletContext ctx) { this.servletContext = ctx; } + /* (non-Javadoc) + * @see org.apache.struts2.dispatcher.DispatcherListener#dispatcherInitialized(org.apache.struts2.dispatcher.Dispatcher) + */ public void dispatcherInitialized(Dispatcher du) { ApplicationContext appContext = WebApplicationContextUtils .getWebApplicationContext(servletContext); @@ -87,6 +106,9 @@ } + /* (non-Javadoc) + * @see org.apache.struts2.dispatcher.DispatcherListener#dispatcherDestroyed(org.apache.struts2.dispatcher.Dispatcher) + */ public void dispatcherDestroyed(Dispatcher du) { } }