Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-33408

Dropbox > Group/Section filter visibility is hard-coded to true; should be based on permissions

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Verified
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 11.3
    • Fix Version/s: 12.0, 19.0
    • Component/s: Drop box
    • Labels:
    • 12 status:
      Resolved
    • Test Plan:
      Hide

      1) Add the Drop Box to a site
      2) As a student/access (or any user) that does not have either 'dropbox.maintain' or 'dropbox.maintain.own.groups' permissions, navigate to the Drop Box tool
      3) Verify the group/section filter drop down does not appear

      Show
      1) Add the Drop Box to a site 2) As a student/access (or any user) that does not have either 'dropbox.maintain' or 'dropbox.maintain.own.groups' permissions, navigate to the Drop Box tool 3) Verify the group/section filter drop down does not appear

      Description

      SAK-14625 introduced a drop down control to filter the list on groups or sections. The boolean value that determines if this control is rendered was hard-coded to true. The correct approach is to conditionally render this control based on if the user has one of two permissions (dropbox.maintain or dropbox.maintain.own.groups). If the user doesn't have either of these permissions, it's not appropriate to display the drop down filter.

        Gliffy Diagrams

          Zeplin

            Attachments

              Issue Links

              depends on

              (Deprecated) Contributed Patch - We use Github and Pull Requests. We no longer use contributed patches (for the most part. ask for exceptions.) SAK-14625 Add group/section filter to DropBox

              • Major - Major loss of function or need for functionality in production in the near future.
              • CLOSED

                Activity

                  People

                  • Assignee:
                    bjones86 Brian Jones
                    Reporter:
                    bjones86 Brian Jones
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    2 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved:

                      Git Integration