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

DocumentTypePermissionServiceImpl seems to build permission details incorrectly


    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s:
    • Component/s: Development
    • Labels:
    • Similar issues:
      KULRICE-8252Permission check for super user tab seems to be using the wrong namespace
      KULRICE-4295useKimPermission not being checked on canBlanketApprove in DocumentTypePermissionServiceImpl
      KULRICE-7274Creating an Initiate Document permission with an incorrect documentTypeName detail results in exception
      KULRICE-6549Display of AuthorizationException messages seems incorrect
      KULRICE-7182Permission lookup by Attribute Value doesn't seem to work
      KULRICE-3490make assembly of permission details (e.g. in DocumentTypePermissionServiceImpl.buildDocumentTypePermissionDetails(...)) dynamic based on db config
      KULRICE-6513DataObjectMetaDataServiceImpl's areNotesSupported method implementation seems to be incorrect
      KULRICE-4151Permission and responsibility lookups don't seem to take nested roles into account
      KULRICE-10251Maintain KRMS Agenda permission has confusing and unused permission detail
      KULRICE-4302Rice's .classpath seems out of whack...
    • Rice Module:
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required


      I'm not sure if this is a problem or not, but it seems inconsistent. Look at the code below. When the document is not in I or S state, it puts the route status label into the route node detail type. It seems like this is mixing things a little. Shouldn't it be using the current route node name like the buildDocumentTypePermissionDetails() method below it?

      protected AttributeSet buildRouteHeaderIdRoleDocumentTypeDocumentStatusQualifiers(DocumentType documentType, String documentStatus, String routeHeaderId) {
      AttributeSet qualifiers = new AttributeSet();
      qualifiers.put(KimAttributes.DOCUMENT_NUMBER, routeHeaderId);
      if (!StringUtils.isBlank(documentStatus)) {
      qualifiers.put(KEWConstants.DOCUMENT_STATUS_DETAIL, documentStatus);
      if (KEWConstants.ROUTE_HEADER_INITIATED_CD.equals(documentStatus) || KEWConstants.ROUTE_HEADER_SAVED_CD.equals(documentStatus))

      { qualifiers.put(KimAttributes.ROUTE_NODE_NAME, DocumentAuthorizerBase.PRE_ROUTING_ROUTE_NAME); }


      { qualifiers.put(KimAttributes.ROUTE_NODE_NAME, CodeTranslator.getRouteStatusLabel(documentStatus)); }


        Issue Links


          No work has yet been logged on this issue.


            • Assignee:
              William Balderamos (Inactive)
              Jonathan Keller
            • Votes:
              0 Vote for this issue
              0 Start watching this issue


              • Created:

                Structure Helper Panel