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

Integrate KR-NS: View Note / Attachment into KRAD views

    Details

    • Type: Task Task
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.5
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Similar issues:
      KULRICE-13089Integrate the new KR-NS: Export Records functionality into KRAD
      KULRICE-8154Location module uses KNS namespace 'KR-NS'
      KULRICE-12471Need to add support in code to use the KR-KRAD namespace permissions instead of the KNS namespace permissions
      KULRICE-11552When a KRAD view is loading in a lightbox, the page blocking does not block the view
      KULRICE-5884Fix KRAD integration test failures
      KULRICE-11382Create Automated Functional Tests for KRAD Labs - View Fluid Container/View Fixed Container
      KULRICE-5280KRAD - KS Test View: need Messaging component
      KULRICE-7663Dialog Test View issue with Bread Crumbs
      KULRICE-5278KRAD: Tab view should be permanent
      KULRICE-10260Analyze performance in targeted KRAD views
    • Sprint:
      Core 2.5.0-m6 Sprint 2, Core 2.5.0-m7 Sprint 1, Core 2.5.0-rc Sprint 1
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required
    • Include in Release Notes?:
      Yes
    • Story Points:
      3

      Description

      It appears that even though KR-NS: View Note / Attachment was put in KRAD, the methods are never called. The methods in KRAD should call this template (no KR-KRAD conversion yet) and the new methods should be called to determine whether a user can view the notes and attachments tab.

      There are two scenarios here that the KNS handled.

      1. The permission should be checked for the section, regardless of the attachmentTypeCode, to see if the user can view any notes/attachments.
      2. The permission should be checked for the "download" button, taking into consideration the attachmentTypeCode, to see if the user can download the attachment.

      We will need to do a code review and AFT to test this.

        Issue Links

          Activity

          Hide
          Steve Edgar (Inactive) added a comment -

          Additional AFT was created yesterday. SQL was modified for additional KIM permissions. Patch file was created on uploaded to Fisheye. Upon approval of code review, can commit.

          Show
          Steve Edgar (Inactive) added a comment - Additional AFT was created yesterday. SQL was modified for additional KIM permissions. Patch file was created on uploaded to Fisheye. Upon approval of code review, can commit.
          Hide
          Steve Edgar (Inactive) added a comment -

          Review was completed yesterday, and the commit was made. There were several issues found, and corrected by Kristina.

          Show
          Steve Edgar (Inactive) added a comment - Review was completed yesterday, and the commit was made. There were several issues found, and corrected by Kristina.
          Hide
          Steve Edgar (Inactive) added a comment -

          AFTs failing on server (but succeed locally).

          Show
          Steve Edgar (Inactive) added a comment - AFTs failing on server (but succeed locally).
          Hide
          Kristina Taylor (Inactive) added a comment -

          Don't worry about that for now since they passed locally. It looks more like an integration problem, for which QA will file a separate bug.

          Show
          Kristina Taylor (Inactive) added a comment - Don't worry about that for now since they passed locally. It looks more like an integration problem, for which QA will file a separate bug.
          Hide
          Steve Edgar (Inactive) added a comment -

          > Don't worry about that for now since they passed locally.
          > It looks more like an integration problem, for which QA will file a separate bug.

          Setting this case back to Resolved.

          Show
          Steve Edgar (Inactive) added a comment - > Don't worry about that for now since they passed locally. > It looks more like an integration problem, for which QA will file a separate bug. Setting this case back to Resolved.

            People

            • Assignee:
              Steve Edgar (Inactive)
              Reporter:
              Kristina Taylor (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Agile

                  Structure Helper Panel