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

Groups in the roster table should be sorted by title (And use the BaseGroup comparator)

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: OPEN
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 11.5 [Tentative], 12.0, 19.0
    • Fix Version/s: None
    • Component/s: Roster
    • Labels:
      None
    • Test Plan:
      Hide

      I don't have an exact test plan but you'd really just want to see the groups in the table are sorted alphabetically by title.

      Create at least 2-3 groups in Sakai without any sorted names.

      Like for instance "Group 1", "1 Group" "A Group"

      Assign some users in a course to some groups.
      Look at the users in the roster table. You should see a predictable (alphabetical) sort order for the group names.

      Show
      I don't have an exact test plan but you'd really just want to see the groups in the table are sorted alphabetically by title. Create at least 2-3 groups in Sakai without any sorted names. Like for instance "Group 1", "1 Group" "A Group" Assign some users in a course to some groups. Look at the users in the roster table. You should see a predictable (alphabetical) sort order for the group names.

      Description

      The groups within the roster table are returned unsorted, these should be sorted like the other groups.

      In the roster code (SakaiProxyImpl.java), the call to getRosterSite -> getViewableSiteGroups sorts the Groups in the drop down. But the call to getMembership does not do any sorting so groups in the table are unsorted. There's also a comment in there that the code should be fixed to use KNL-1305

              // TODO: change this to BaseGroup's comparator when KNL-1305 has been
              // fixed
      

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  a.fish@lancaster.ac.uk Adrian Fish
                  Reporter:
                  jonespm Matthew Jones
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:

                    Git Source Code