Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-30364

Minor regressions from the report plugin

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2, 2.3
    • Fix Version/s: 2.2.1
    • Component/s: Navigation
    • Labels:
    • Testing Instructions:
      Hide
      1. Log in as an admin
      2. Browse to a course with participants and activity
      3. Browse to the course participants page
      4. Select a user
      5. Expand there course profile in the navigation, open reports and click `Grade`
      Show
      Log in as an admin Browse to a course with participants and activity Browse to the course participants page Select a user Expand there course profile in the navigation, open reports and click `Grade`
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w51_MDL-30364_m23_report

      Description

      There are a couple of minor regressions from the report plugin that I spotted today.

      1. The grade link in a users course profile within navigation under reports is missing the users id from it's parameters.
      2. The course/user.php file no longer sets PAGE->url

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            skodak Petr Skoda added a comment -

            thanks a lot for the report and sorry for the regression!

            Show
            skodak Petr Skoda added a comment - thanks a lot for the report and sorry for the regression!
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Petr - integrated now

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Petr - integrated now
            Hide
            ankit_frenz Ankit Agarwal added a comment -

            Everything working great here!
            Passing
            Thanks

            Show
            ankit_frenz Ankit Agarwal added a comment - Everything working great here! Passing Thanks
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks!

            Now... disconnect, relax and enjoy the next days, yay!

            Closing...ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks! Now... disconnect, relax and enjoy the next days, yay! Closing...ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  9/Jan/12