Uploaded image for project: '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

          Attachments

            Issue Links

              Activity

              Hide
              andyjdavis Andrew Davis added a comment -

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

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

              Looks good thanks Andrew

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

              Integrated, thanks!

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

              Works Great
              Thanks for fixing this, Andrew.

              Show
              rajeshtaneja Rajesh Taneja added a comment - Works Great Thanks for fixing this, Andrew.
              Hide
              stronk7 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
              stronk7 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:
                    Fix Release Date:
                    9/Jan/12