Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 2.3
-
Fix Version/s: 2.3
-
Component/s: Forms Library
-
Testing Instructions:
-
Difficulty:Easy
-
Affected Branches:MOODLE_23_STABLE
-
Fixed Branches:MOODLE_23_STABLE
-
Pull from Repository:
-
Pull Master Branch:
MDL-31660-master-3
Description
As discussed in MDL-31315 we should ideally rename the formslib change detector to something more specific - for example formchangeprotection
In doing this, I also came across a couple of areas to tidy up:
- the unset_changed function isn't required. It was from an early Proof of Concept and somehow made it through review several times
- The listeners only need to be called once, not on every change. This should reduce browser footprints slightly
This change should ideally be tested at the same time as MDL-31655 and MDL-31656 which make modifications to the same functionality