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

          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