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

Remaining Authorization and Presentation Controller Work

    Details

    • Type: Task
    • Status: Closed
    • Priority: Critical
    • Resolution: Complete
    • Affects Version/s: 2.2
    • Fix Version/s: 2.5
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KNS, KRAD
    • KRAD Feature Area:
      UIF MVC
    • Application Requirement:
      Rice
    • Sprint:
      Core 2.5.0-m7 Sprint 1
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Include in Release Notes?:
      Yes
    • Story Points:
      5

      Description

      Remaining Authorization and Presentation Controller Work. This includes some permissions in inquiry, lookup, maintenance, and document frameworks not implemented.

      Classes to compare:

      • DocumentAuthorizerBase / DocumentViewAuthorizerBase
      • MaintenanceDocumentAuthorizerBase / MaintenanceViewAuthorizerBase
      • TransactionalDocumentAuthorizerBase / TransactionalDocumentAuthorizerBase
      • DocumentPresentationControllerBase / DocumentPresentationControllerBase
      • MaintenanceDocumentPresentationControllerBase / MaintenanceViewPresentationControllerBase
      • TransactionalDocumentPresentationControllerBase / TransactionalDocumentViewPresentationControllerBase

        Attachments

          Issue Links

            Activity

            Hide
            christieheitkamp Christie Heitkamp (Inactive) added a comment - - edited

            Jerry Neal Please provide more information about the impact to user experience.

            Show
            christieheitkamp Christie Heitkamp (Inactive) added a comment - - edited Jerry Neal Please provide more information about the impact to user experience.
            Hide
            jkneal Jerry Neal (Inactive) added a comment -

            We are making this an analysis tasks to figure out what the gaps are.

            Recommendation is:

            • Go through all KNS/KIM permissions and verify they are implemented in KRAD
            • Compare KNS and KRAD authorizer and presentation controller and verify capabilities are the same
            Show
            jkneal Jerry Neal (Inactive) added a comment - We are making this an analysis tasks to figure out what the gaps are. Recommendation is: Go through all KNS/KIM permissions and verify they are implemented in KRAD Compare KNS and KRAD authorizer and presentation controller and verify capabilities are the same
            Hide
            kbtaylor Kristina Taylor (Inactive) added a comment - - edited

            For 2.5, we will not be making any more database changes in this area, so any permissions should stay in the KNS namespace.

            That being said, we have already opened up KULRICE-12471 to convert these in 2.6. I will report additional analysis findings here.

            Show
            kbtaylor Kristina Taylor (Inactive) added a comment - - edited For 2.5, we will not be making any more database changes in this area, so any permissions should stay in the KNS namespace. That being said, we have already opened up KULRICE-12471 to convert these in 2.6. I will report additional analysis findings here.
            Hide
            kbtaylor Kristina Taylor (Inactive) added a comment - - edited

            Ongoing Analysis:

            • KIM Documentation
              • The KNS2KRAD guide's entry on Inquire Into Records needs to be updated (see KULRICE-11438)
            • Document*AuthorizerBase
              • Changes for KULRICE-7864 need to be integrated into canEdit, canCancel
              • The signature for canDeleteNoteAttachment has changed, is this correct and is it documented?
              • Missing canViewNoteAttachment(Document document, String attachmentTypeCode, Person user), but really, is this even supposed to be in here? Shouldn't it be a native KRAD permission like canViewLine? (see KULRICE-13090)
            • TransactionalDocument*AuthorizerBase
              • The change from "Use Transactional Document" to "Use View" needs to be additionally documented in the KNS2KRAD guide (see KULRICE-10772)
            • Document*PresentationControllerBase
            • TransactionalDocument*PresentationControllerBase
              • The change from "Use Transactional Document" to "Use View" needs to be additionally documented in the KNS2KRAD guide (see KULRICE-10772)
            Show
            kbtaylor Kristina Taylor (Inactive) added a comment - - edited Ongoing Analysis: KIM Documentation The KNS2KRAD guide's entry on Inquire Into Records needs to be updated (see KULRICE-11438 ) Document*AuthorizerBase Changes for KULRICE-7864 need to be integrated into canEdit , canCancel The signature for canDeleteNoteAttachment has changed, is this correct and is it documented? Missing canViewNoteAttachment(Document document, String attachmentTypeCode, Person user) , but really, is this even supposed to be in here? Shouldn't it be a native KRAD permission like canViewLine ? (see KULRICE-13090 ) TransactionalDocument*AuthorizerBase The change from "Use Transactional Document" to "Use View" needs to be additionally documented in the KNS2KRAD guide (see KULRICE-10772 ) Document*PresentationControllerBase Integrate changes from KULRICE-8762 TransactionalDocument*PresentationControllerBase The change from "Use Transactional Document" to "Use View" needs to be additionally documented in the KNS2KRAD guide (see KULRICE-10772 )
            Hide
            kbtaylor Kristina Taylor (Inactive) added a comment - - edited

            All confirmed issues have had issues opened against them. We have two remaining questions:

            • The signature for canDeleteNoteAttachment has changed, is this correct and is it documented?
            • Missing canViewNoteAttachment(Document document, String attachmentTypeCode, Person user), but really, is this even supposed to be in here? Shouldn't it be a native KRAD permission like canViewLine? (see KULRICE-13090)

            Jerry Neal, do you know the answers to these?

            Show
            kbtaylor Kristina Taylor (Inactive) added a comment - - edited All confirmed issues have had issues opened against them. We have two remaining questions: The signature for canDeleteNoteAttachment has changed, is this correct and is it documented? Missing canViewNoteAttachment(Document document, String attachmentTypeCode, Person user) , but really, is this even supposed to be in here? Shouldn't it be a native KRAD permission like canViewLine? (see KULRICE-13090 ) Jerry Neal , do you know the answers to these?
            Hide
            kbtaylor Kristina Taylor (Inactive) added a comment - - edited

            Confirmed that both of these should stay as they are now in KRAD as only a minor conversion is necessary. We should probably revisit canViewNoteAttachment at a later date (KULRICE-13164).

            Show
            kbtaylor Kristina Taylor (Inactive) added a comment - - edited Confirmed that both of these should stay as they are now in KRAD as only a minor conversion is necessary. We should probably revisit canViewNoteAttachment at a later date ( KULRICE-13164 ).

              People

              • Assignee:
                kbtaylor Kristina Taylor (Inactive)
                Reporter:
                jkneal Jerry Neal (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: