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:
          19250

          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

            is a regression caused by

            Improvement - An enhancement to an existing Moodle feature. MDL-15897 param cleaning improvements

            • Minor - Minor loss of function where workaround is possible
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Petr Škoda added a comment - - edited

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

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

              Integrated, thanks!

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

              have performed the test : 'ask Tim'

              Show
              Aparup Banerjee added a comment - have performed the test : 'ask Tim'
              Hide
              Permalink
              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
              Permalink
              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: