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

Fatal error when moving question category to upper level

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Questions, Quiz
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      Attempting to move a question category from the course level to a specific quiz (e.g. moving the category <i>up</i> on the screen) causes an error:

      Called to undefined method moodle_page::set_header() in /html/question/contextmove.php on line 196

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            derekw Derek Woolhead added a comment -

            For info: Just during today, I'm looking at this bug as part of the OU's Moodle Monday exercise. If I figure it out by the end of today, I'll attach a patch here. (If it turns out to be too difficult, I'll just leave it - sorry.)

            Show
            derekw Derek Woolhead added a comment - For info: Just during today, I'm looking at this bug as part of the OU's Moodle Monday exercise. If I figure it out by the end of today, I'll attach a patch here. (If it turns out to be too difficult, I'll just leave it - sorry.)
            Hide
            derekw Derek Woolhead added a comment -

            Looks like set_heading() is the correct function, not set_header(). Patch attached.

            Show
            derekw Derek Woolhead added a comment - Looks like set_heading() is the correct function, not set_header(). Patch attached.
            Hide
            timhunt Tim Hunt added a comment -

            Thanks Derek. That is the correct fix. I did a grep for set_heading, and this seems to have been the only place where this mistake was made.

            Show
            timhunt Tim Hunt added a comment - Thanks Derek. That is the correct fix. I did a grep for set_heading, and this seems to have been the only place where this mistake was made.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  24/Nov/10