Moodle
  1. Moodle
  2. MDL-26522

The behavior or the user selection menu at gradebook's user report doesn't work as expected

    Details

    • Testing Instructions:
      Hide

      Go into a course as either a teacher or admin. Go to the gradebook then to the user report.

      Initially a report with no grades just showing some overall aggregate information is displayed. What exactly depends on your config and isnt important for this bug.

      Select "all users". The user reports for all students will be displayed. The navigation drop down should still display the "all users" option.

      Select a specific user. Just that user's user report will be displayed. The navigation drop down should still have that user selected.

      Show
      Go into a course as either a teacher or admin. Go to the gradebook then to the user report. Initially a report with no grades just showing some overall aggregate information is displayed. What exactly depends on your config and isnt important for this bug. Select "all users". The user reports for all students will be displayed. The navigation drop down should still display the "all users" option. Select a specific user. Just that user's user report will be displayed. The navigation drop down should still have that user selected.
    • Difficulty:
      Moderate
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:
      MDL-26522_user_report_menu

      Description

      Detected @ MDLQA-707

      It seems that the user-selection menu in the grader report (available as teacher) isn't working ok, specially when related to show the "all users" option. Also, it should (IMO) display the currently selected value always, instead of defaulting to "Choose" all the time.

      Ciao

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Andrew Davis added a comment -

            Added master and 2.1 fixes. Will add 2.0 after peer review. having some git troubles presently.

            Show
            Andrew Davis added a comment - Added master and 2.1 fixes. Will add 2.0 after peer review. having some git troubles presently.
            Hide
            Sam Hemelryk added a comment -

            Looks good thanks Andrew

            Show
            Sam Hemelryk added a comment - Looks good thanks Andrew
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Integrated, thanks!

            Show
            Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
            Hide
            Rajesh Taneja added a comment -

            Works Great
            Thanks for fixing this, Andrew.

            Show
            Rajesh Taneja added a comment - Works Great Thanks for fixing this, Andrew.
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Yes, you did it!

            Now your code is part of the best weeklies released ever, many thanks!

            Closing, ciao

            Show
            Eloy Lafuente (stronk7) added a comment - Yes, you did it! Now your code is part of the best weeklies released ever, many thanks! Closing, ciao

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: