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

borked context filter settings after change in PARAM_ALPHAEXT

          Details

          • Type: Bug
          • Status: Closed
          • Priority: 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:

            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

              Gliffy Diagrams

                Attachments

                  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
                    skodak 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
                    skodak 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
                    Permalink
                    stronk7 Eloy Lafuente (stronk7) added a comment -

                    Integrated, thanks!

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

                    have performed the test : 'ask Tim'

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

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

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

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

                    Show
                    nebgor 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:
                          Fix Release Date:
                          10/Oct/11