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
    • Rank:
      16112

      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

        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: