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

Display of AuthorizationException messages seems incorrect

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Critical 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
    • Similar issues:
      KULRICE-4594specifying the "Lock Owner Principal Name:" incorrectly displays an error message
      KULRICE-9881Incorrect message displayed when using result set limit
      KULRICE-4805DocumentTypePermissionServiceImpl seems to build permission details incorrectly
      KULRICE-4178Some role qualifiers displaying incorrectly
      KULRICE-7463Validation messaging - info missing in section headers & incorrect # of validation messages
      KULRICE-6513DataObjectMetaDataServiceImpl's areNotesSupported method implementation seems to be incorrect
      KULRICE-4302Rice's .classpath seems out of whack...
      KULRICE-3591KualiLookup.jsp - paging causes the message "There are no returnable items within the results." to show up incorrectly
      KULRICE-8485Messages cached incorrectly in the js after arg evaluation
      KULRICE-3452Extra white area appears in Firefox when tabs display error messages
    • 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.

        Activity

        Hide
        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
        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
        Jessica Coltrin (Inactive) added a comment -

        Closing since these items are now in the release notes.

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

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Structure Helper Panel