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

Quiz repaginate should remove empty pages at the start of the layout

    Details

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

      Create a quiz with layout:

      • blank page
      • blank page
      • question on a page by itself.

      Repaginate the quiz (using the button on the order and paging tab) to one question-per-page.

      Verify the layout is now

      • question on a page by itself.

      Also, run the unit tests in mod/quiz.

      Show
      Create a quiz with layout: blank page blank page question on a page by itself. Repaginate the quiz (using the button on the order and paging tab) to one question-per-page. Verify the layout is now question on a page by itself. Also, run the unit tests in mod/quiz.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      At the moment, when I follow the testing instructions, the layout stays as

      • blank page
      • blank page
      • question on a page by itself.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Tim, changes look good.
            I'll integrate this during the next integration cycle.

            Cheers
            Sam

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Tim, changes look good. I'll integrate this during the next integration cycle. Cheers Sam
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Tim, integrated now

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Tim, integrated now
            Hide
            rwijaya Rossiani Wijaya added a comment -

            This is working great.

            Thanks for fixing.

            Test passed.

            Show
            rwijaya Rossiani Wijaya added a comment - This is working great. Thanks for fixing. Test passed.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            The git and cvs repositories are happy receiving your very first contribution to Moodle for 2012. Happy new year!

            Closing, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - The git and cvs repositories are happy receiving your very first contribution to Moodle for 2012. Happy new year! 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