Moodle
  1. Moodle
  2. MDL-29535

borked context filter settings after change in PARAM_ALPHAEXT

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1
    • Fix Version/s: 2.0.5, 2.1.2
    • Component/s: Filters
    • Labels:
      None
    • Testing Instructions:
      Hide

      None of the core filters use the code that is affected by this change.

      To test it, you need to create a simple filter plugin that uses http://docs.moodle.org/dev/Filters_2.0#Local_configuration.

      Then test that you can, in fact, edit and save the local configuration for the filter.

      Show
      None of the core filters use the code that is affected by this change. To test it, you need to create a simple filter plugin that uses http://docs.moodle.org/dev/Filters_2.0#Local_configuration . Then test that you can, in fact, edit and save the local configuration for the filter.
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull from Repository:
    • Rank (Obsolete):
      19294

      Description

      no core filters are affected, that is why this was not reported yet I suppose, thanks Eloy for discovering it while working on review of MDL-29504

      most probably my fault

        Issue Links

          Activity

          Hide
          Petr Skoda added a comment - - edited

          to integrators: fix for master is already in MDL-29504, please cherry pick if that gets rejected, thanks

          Show
          Petr Skoda added a comment - - edited to integrators: fix for master is already in MDL-29504 , please cherry pick if that gets rejected, thanks
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated, thanks!

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

          have performed the test : 'ask Tim'

          Show
          Aparup Banerjee added a comment - have performed the test : 'ask Tim'
          Hide
          Aparup Banerjee added a comment -

          Hello worldS, this works for me. Thanks for the test Tim. (more mojitos)

          Show
          Aparup Banerjee added a comment - Hello worldS, this works for me. Thanks for the test Tim. (more mojitos)
          Hide
          Aparup Banerjee added a comment -

          fixes have been rolled merrily up the stream! Thanks everybody!

          Show
          Aparup Banerjee added a comment - fixes have been rolled merrily up the stream! Thanks everybody!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: