Uploaded image for project: 'Tapestry 5'
  1. Tapestry 5
  2. TAP5-1251

Spring integration should not assume the external context is successfully initialized

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 5.1.0.5
    • Fix Version/s: 5.3
    • Component/s: tapestry-spring
    • Labels:
      None

      Description

      In SpringModuleDef , the following code assumes that the Spring context is successfully initialized:

      ConfigurableWebApplicationContext context = (ConfigurableWebApplicationContext) servletContext.getAttribute(WebApplicationContext.ROOT_WEB_APPLICATION_CONTEXT_ATTRIBUTE);
      

      but the javadoc of that attribute states that "If the startup of the root context fails, this attribute can contain an exception or error as value. Use WebApplicationContextUtils for convenient lookup of the root WebApplicationContext. "

      The resulting error is

      Error Message: javax.servlet.ServletException: Filter [tapestryFilter]: could not be initialized
      Error Code: 500
      Target Servlet: /business/Main.jsp
      Error Stack:
      java.lang.ClassCastException: org.springframework.beans.factory.BeanCreationException incompatible with org.springframework.web.context.ConfigurableWebApplicationContext
           at org.apache.tapestry5.internal.spring.SpringModuleDef.locateExternalContext(SpringModuleDef.java:61)
           at org.apache.tapestry5.internal.spring.SpringModuleDef.<init>(SpringModuleDef.java:83)
           at org.apache.tapestry5.spring.TapestrySpringFilter.provideExtraModuleDefs(TapestrySpringFilter.java:34)
           at org.apache.tapestry5.TapestryFilter.init(TapestryFilter.java:76) 
      

      The lookup should be ajusted to use WebApplicationContextUtils.

        Attachments

          Activity

            People

            • Assignee:
              hlship Howard M. Lewis Ship
              Reporter:
              rombert Robert Munteanu
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: