Moodle
  1. Moodle
  2. MDL-29176

Notice in quiz when trying to move a question to page that doesn't exist

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.1
    • Fix Version/s: 2.1.4, 2.2.1
    • Component/s: Quiz
    • Labels:
    • Testing Instructions:
      Hide

      1. Go to the edit page for a quiz with no student attempts.

      2. Go to the order and paging tab.

      3. Select a question, or don't and enter a value greater than the number of pages into the `Move selected questions to page` text field and submit

      4. Check that you don't get the notice shown in the description - and that the the questions are moved to the page specified.

      Show
      1. Go to the edit page for a quiz with no student attempts. 2. Go to the order and paging tab. 3. Select a question, or don't and enter a value greater than the number of pages into the `Move selected questions to page` text field and submit 4. Check that you don't get the notice shown in the description - and that the the questions are moved to the page specified.
    • Workaround:
      Hide

      Don't type in bogus page numbers

      Show
      Don't type in bogus page numbers
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      If I select a question, and in the `Move selected questions to page` text field enter a number greater than the number of pages (a page that doesn't exist of course) then I get the following:

      Notice: Undefined offset: 5 in mod/quiz/edit.php on line 361

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Sam Hemelryk added a comment -

            Thanks Tim - this has been integrated and tested now

            Show
            Sam Hemelryk added a comment - Thanks Tim - this has been integrated and tested now
            Hide
            Sam Hemelryk added a comment -

            Tested and passed during integration

            Show
            Sam Hemelryk added a comment - Tested and passed during integration
            Hide
            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
            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:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: