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:Hide
Note to tester: This change is related to
MDL-31655andMDL-31660. To aid your testing, you may want to test all three together. These instructions are additive
Log in to Moodle and navigate to a course
- Choose Edit Settings
Without any changes
- Refresh the page (the page refreshes)
- Hit the back button (you go back in time and space)
- Go back to the Edit Settings Page
- Try closing the browser window (it closes)
- Reopen, Click Save and Display (the form saves)
Now make some changes to various fields
- Go back to the Edit Settings Page
- Make a change to the 'Course full name' - do not move off the field
- Refresh the page (you get a popup) - click to stay
- Hit the back button (you get a popup) - click to stay
- Try closing the browser window/tab (you get a popup) - click to stay
- Click Save and Display (the form saves)
ShowNote to tester: This change is related to MDL-31655 and MDL-31660 . To aid your testing, you may want to test all three together. These instructions are additive Log in to Moodle and navigate to a course Choose Edit Settings Without any changes Refresh the page (the page refreshes) Hit the back button (you go back in time and space) Go back to the Edit Settings Page Try closing the browser window (it closes) Reopen, Click Save and Display (the form saves) Now make some changes to various fields Go back to the Edit Settings Page Make a change to the 'Course full name' - do not move off the field Refresh the page (you get a popup) - click to stay Hit the back button (you get a popup) - click to stay Try closing the browser window/tab (you get a popup) - click to stay Click Save and Display (the form saves) -
Difficulty:Easy
-
Affected Branches:MOODLE_23_STABLE
-
Fixed Branches:MOODLE_23_STABLE
-
Pull from Repository:
-
Pull Master Branch:
MDL-31655-master-3
-
Rank:38228
Description
Editing a text box in a form and then immediatly refreshing / pressing the back button / closing the window without losing focus from the text box will not trigger a dialogue box asking if you want to save changes made to the form.
Issue Links
Activity
I've got a solution in the pipeline for this, but I'm waiting for this week's integration to be released so that I can base it on that.
I've based this change on MDL-31673 as this seemed the safest option. Hopefully you agree!
Looks good to me, took me a bit to understand what was going on and that the 'on blur' event is when an element looses focus and the problem here is that the element isn't loosing focus.
One question unrelated to this change but instead about this feature - we are storing things in 'M.cfg' but I am sure they do not get put back into the config table? Is this the correct place to store such things? What happens if someone reuses the name for a proper config option.
Good question - perhaps we should create a new location for state storage data...
The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.
TIA and ciao
Rebases cleanly onto latest master
Thanks Andrew this has been integrated now
Works as described. Passing.
Your changes are now upstream and will be included in the next minor released scheduled for March 13th (next Monday!).
icao_reverse('arreis olik rebemevon afla letoh ognat');
Closing, ciao
Thanks for reporting that, Adrian.
Andrew: I'm assigning this to you because you worked on the related issue, but if you would rather not work on this, please reassign it to moodle.com.