Moodle
  1. Moodle
  2. MDL-27958

Filter: Manage filter page output error

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0.3, 2.1
    • Fix Version/s: 2.0.4
    • Component/s: Filters
    • Labels:
    • Rank:
      17600

      Description

      Coding problem: this page does not set $PAGE->context properly.

      line 341 of /lib/pagelib.php: call to debugging()
      line 599 of /lib/pagelib.php: call to moodle_page->magic_get_context()
      line 1236 of /lib/weblib.php: call to moodle_page->__get()
      line 3581 of /lib/accesslib.php: call to format_string()
      line 52 of /filter/manage.php: call to print_context_name()

        Issue Links

          Activity

          Hide
          Anthony Borrow added a comment -

          I've added Sam as a watcher since he worked on the changes in MDL-23613. Peace - Anthony

          Show
          Anthony Borrow added a comment - I've added Sam as a watcher since he worked on the changes in MDL-23613 . Peace - Anthony
          Hide
          Anthony Borrow added a comment -

          so flicked

          Show
          Anthony Borrow added a comment - so flicked
          Hide
          Anthony Borrow added a comment -

          I'm linking to the MDLQA-982 but I do not consider MDL-27958 to block it since it does not prevent the behavior but only throws an error

          Show
          Anthony Borrow added a comment - I'm linking to the MDLQA-982 but I do not consider MDL-27958 to block it since it does not prevent the behavior but only throws an error
          Hide
          Sam Hemelryk added a comment -

          Thanks Anthony - had a quick check and is a simple case of adding a set_context call.
          Cheers
          Sam

          Show
          Sam Hemelryk added a comment - Thanks Anthony - had a quick check and is a simple case of adding a set_context call. Cheers Sam
          Hide
          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.

          Show
          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.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated, thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Passing test without further action. Will be tested by MDLQA-982 once this meets upstream.

          Show
          Eloy Lafuente (stronk7) added a comment - Passing test without further action. Will be tested by MDLQA-982 once this meets upstream.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Your awesome code is now part of the next Moodle release, congrats!

          Show
          Eloy Lafuente (stronk7) added a comment - Your awesome code is now part of the next Moodle release, congrats!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: