Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-6549

Display of AuthorizationException messages seems incorrect

    Details

    • Type: Bug Fix
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0-rc2, 2.0
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KNS
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      Struts AuthorizationExceptionAction populated the exception messages into a request attribute using the key - AuthorizationExceptionAction.class.getName().
      However the same attribute is attempted to be retrieved using the class's simpleName and not the fully qualified name.

      This has to be corrected.

      Also, the exception's Class name is displayed as the error message and not the actual error message.

        Attachments

          Activity

          Hide
          cchandran Chitra Chandran added a comment -

          This piece of code in AuthorizationException.jsp should be changed from

          <c:set var="parameters"
          value="<%=request.getAttribute(\"AuthorizationExceptionAction\")%>" />

          <c:if test="$

          {not empty parameters['message']}

          ">
          <c:set var="message" value="$

          {parameters.message}" />
          <c:if test="${not empty message}">
          <c:set var="exception" value='<%=request.getAttribute("org.apache.struts.action.EXCEPTION")%>'/>
          <c:set var="message" value="${exception['class'].name}" />
          </c:if>
          </c:if>

          to

          <c:set var="parameters"
          value="<%=request.getAttribute(\"org.kuali.rice.kns.web.struts.action.AuthorizationExceptionAction\")%>" />

          <c:if test="${not empty parameters}">
          <c:set var="message" value="${parameters.message}

          " />
          </c:if>

          Show
          cchandran Chitra Chandran added a comment - This piece of code in AuthorizationException.jsp should be changed from <c:set var="parameters" value="<%=request.getAttribute(\"AuthorizationExceptionAction\")%>" /> <c:if test="$ {not empty parameters['message']} "> <c:set var="message" value="$ {parameters.message}" /> <c:if test="${not empty message}"> <c:set var="exception" value='<%=request.getAttribute("org.apache.struts.action.EXCEPTION")%>'/> <c:set var="message" value="${exception['class'].name}" /> </c:if> </c:if> to <c:set var="parameters" value="<%=request.getAttribute(\"org.kuali.rice.kns.web.struts.action.AuthorizationExceptionAction\")%>" /> <c:if test="${not empty parameters}"> <c:set var="message" value="${parameters.message} " /> </c:if>
          Hide
          jcoltrin Jessica Coltrin (Inactive) added a comment -

          Closing since these items are now in the release notes.

          Show
          jcoltrin Jessica Coltrin (Inactive) added a comment - Closing since these items are now in the release notes.

            People

            • Assignee:
              sgibson Scott Gibson (Inactive)
              Reporter:
              cchandran Chitra Chandran
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: