Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-2649

Verify that our jsps and tag libraries are using quotes properly and work successfully in Tomcat 5.5.27

    Details

    • Type: Task Task
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0
    • Component/s: Application Server
    • Labels:
      None
    • Similar issues:
      KULRICE-3383KualiIncidentReport.jsp doesn't work on Tomcat 5.5.27+/Tomcat 6.0 (and possibly other screens?)
      KULRICE-1225Verify CXF is a valid library to use
      KULRICE-145Change JSPs and Tags to use correct paths
      KULRICE-10505Fix Tomcat 7 jsp compilation errors in WorkgroupTab.jsp & RemoveReplaceDisplay.jsp
      KULRICE-8770easyXDM issues with Firefox and Tomcat
      KULRICE-4569rowDisplay.tag doesn't work properly in Tomcat 5.5.30
      KULRICE-3729JSPs are erroring without tomcat jasper jars
      KULRICE-6377Remove single quotes from binding paths due to JS issues
      KULRICE-5843JSP Errors in Tomcat 7
      KULRICE-2879Make sure our Message Resource properties that are used in KEW are working properly

      Description

      See the following Tomcat bug report for details of what we need to verify

      https://issues.apache.org/bugzilla/show_bug.cgi?id=45015

        Issue Links

          Activity

          Hide
          Jeremy Hanson added a comment -

          In the bugzilla link there is a nice regular expression to find instances of these problems.

          <\w+:[^>]="[^<"]<%=[^%]"|<\w:[^>]+='[^<']<%=[^%]'

          I ran this and there are currently only two places in all jsp and tag files that need to be updated.

          Row 29 of KualiExceptionIncident.jsp
          – should be changed to:
          value='<%=request.getAttribute("org.kuali.rice.kns.web.struts.action.KualiExceptionHandlerAction")%>' />
          and
          Row 670 of rowDisplay.tag
          – should be changed to:
          <c:out value='<%=((String) request.getAttribute("fileName"))%>' /> 

          Show
          Jeremy Hanson added a comment - In the bugzilla link there is a nice regular expression to find instances of these problems. <\w+: [^>] =" [^<"] <%= [^%] "|<\w : [^>] +=' [^<'] <%= [^%] ' I ran this and there are currently only two places in all jsp and tag files that need to be updated. Row 29 of KualiExceptionIncident.jsp – should be changed to: value='<%=request.getAttribute("org.kuali.rice.kns.web.struts.action.KualiExceptionHandlerAction")%>' /> and Row 670 of rowDisplay.tag – should be changed to: <c:out value='<%=((String) request.getAttribute("fileName"))%>' /> 
          Hide
          Eric Westfall added a comment -

          Initial version has been deployed to maven repository and weekly deployments are in place on Friday's at 4 in continuum.

          Show
          Eric Westfall added a comment - Initial version has been deployed to maven repository and weekly deployments are in place on Friday's at 4 in continuum.
          Hide
          Srikanth Mummadi (Inactive) added a comment -

          I am still getting the following error when I try to deploy "rice-0.9.4-SNAPSHOT-server.war" in Tomcat.

          org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'rice' defined in class path resource [org/kuali/rice/config/RiceSpringBeans.xml]: Invocation of init method failed; nested exception is java.io.FileNotFoundException: class path resource [log4j.properties] cannot be resolved to absolute file path because it does not reside in the file system: jar:file:/C:/Program%20Files/Apache%20Software%20Foundation/Tomcat%205.5/webapps/rice-0.9.4-SNAPSHOT-server/WEB-INF/lib/bsf-2.4.0.jar!/log4j.properties

          But I tested rowDisplay.tag in KFS 1092 build, but it throws the following exception

          org.apache.jasper.JasperException: /WEB-INF/tags/kr/rowDisplay.tag(670,26) Attribute value ((String) request.getAttribute("fileName")) is quoted with " which must be escaped when used within the value

          I corrected this file and checked in.
          For KualiExceptionIncident.jsp, I could not generate Exception condition in KFS so could not test/change it.

          Show
          Srikanth Mummadi (Inactive) added a comment - I am still getting the following error when I try to deploy "rice-0.9.4-SNAPSHOT-server.war" in Tomcat. org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'rice' defined in class path resource [org/kuali/rice/config/RiceSpringBeans.xml] : Invocation of init method failed; nested exception is java.io.FileNotFoundException: class path resource [log4j.properties] cannot be resolved to absolute file path because it does not reside in the file system: jar: file:/C:/Program%20Files/Apache%20Software%20Foundation/Tomcat%205.5/webapps/rice-0.9.4-SNAPSHOT-server/WEB-INF/lib/bsf-2.4.0.jar!/log4j.properties But I tested rowDisplay.tag in KFS 1092 build, but it throws the following exception org.apache.jasper.JasperException: /WEB-INF/tags/kr/rowDisplay.tag(670,26) Attribute value ((String) request.getAttribute("fileName")) is quoted with " which must be escaped when used within the value I corrected this file and checked in. For KualiExceptionIncident.jsp, I could not generate Exception condition in KFS so could not test/change it.
          Hide
          Eric Westfall added a comment -

          Thanks Srikanth and Jeremy. I went ahead and fixed this in the incident report as well.

          Show
          Eric Westfall added a comment - Thanks Srikanth and Jeremy. I went ahead and fixed this in the incident report as well.
          Hide
          Eric Westfall added a comment -

          Bulk change of all Rice 1.0 issues to closed after public release.

          Show
          Eric Westfall added a comment - Bulk change of all Rice 1.0 issues to closed after public release.

            People

            • Assignee:
              Srikanth Mummadi (Inactive)
              Reporter:
              Eric Westfall
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Structure Helper Panel