Moodle
  1. Moodle
  2. MDL-41996

Displaying a broken link when a student hasn't attempted the quiz

    Details

    • Testing Instructions:
      Hide

      Setup new quiz with 1 question.
      Enrol 2 students.
      Attempt with one student and finish.
      Review attempts and show 'enrolled users who have not attempted the quiz'
      "Review attempt" link will not be shown.
      Review attempts and show 'enrolled users who have attempted the quiz'
      "Review attempt" link will be shown.

      Show
      Setup new quiz with 1 question. Enrol 2 students. Attempt with one student and finish. Review attempts and show 'enrolled users who have not attempted the quiz' "Review attempt" link will not be shown. Review attempts and show 'enrolled users who have attempted the quiz' "Review attempt" link will be shown.
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE, MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE, MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      mdl41996-master

      Description

      When you look in the quiz overview report for a student that hasn't completed the quiz, there is a link to review attempt when they haven't attempted it!

      Error: Can not find data record in database table quiz_attempts.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Tim Hunt added a comment -

            How did no-one notice that before?

            Please can you edit the testing instructions to be testing instructions, and not steps to reproduce.

            Show
            Tim Hunt added a comment - How did no-one notice that before? Please can you edit the testing instructions to be testing instructions, and not steps to reproduce.
            Hide
            Eloy Lafuente (stronk7) added a comment -

            I had some problems with your patches above for 25_STABLE and 24_STABLE, so I just have cherry-picked the master fix to those branches.

            23_STABLE is out of support since some months ago… so this won't land there.

            Integrated (24, 25 & master), thanks!

            Show
            Eloy Lafuente (stronk7) added a comment - I had some problems with your patches above for 25_STABLE and 24_STABLE, so I just have cherry-picked the master fix to those branches. 23_STABLE is out of support since some months ago… so this won't land there. Integrated (24, 25 & master), thanks!
            Hide
            Adrian Greeve added a comment -

            Tested on the 2.4, 2.5 and master integration branches.
            No broken links in the quiz reports anymore.
            Test passed.

            Show
            Adrian Greeve added a comment - Tested on the 2.4, 2.5 and master integration branches. No broken links in the quiz reports anymore. Test passed.
            Hide
            Dan Poltawski added a comment -

            You did it!

            Thanks for your contribution, this change has been merged upstream and is now available on your local git mirror and on download sites shortly.

            Show
            Dan Poltawski added a comment - You did it! Thanks for your contribution, this change has been merged upstream and is now available on your local git mirror and on download sites shortly.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: