Moodle

borked context filter settings after change in PARAM_ALPHAEXT

Details

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

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 - The issue is considered finished, the resolution is correct. Issues which are closed should not be reopened.

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

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: