Moodle

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

Details

  • Type: Bug Bug
  • Status: Closed 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:
  • Pull Master Diff URL:

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

Issue Links

has been marked as being related by

Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-26302 Move button on the quiz 'Order and Paging' tab does not work

  • Minor - Minor loss of function where workaround is possible
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
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
Permalink
Sam Hemelryk added a comment -

Tested and passed during integration

Show
Sam Hemelryk added a comment - Tested and passed during integration
Hide
Permalink
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

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: