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

No warning after editing textbox and not saving.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Forms Library
    • Labels:
    • Testing Instructions:
      Hide

      Note 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)
      Show
      Note 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

      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.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              salvetore Michael de Raadt added a comment -

              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.

              Show
              salvetore Michael de Raadt added a comment - 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.
              Hide
              dobedobedoh Andrew Nicols added a comment -

              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.

              Show
              dobedobedoh Andrew Nicols added a comment - 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.
              Hide
              dobedobedoh Andrew Nicols added a comment -

              I've based this change on MDL-31673 as this seemed the safest option. Hopefully you agree!

              Show
              dobedobedoh Andrew Nicols added a comment - I've based this change on MDL-31673 as this seemed the safest option. Hopefully you agree!
              Hide
              dobedobedoh Andrew Nicols added a comment -

              This patchset probably all want testing together to save the tester some sanity:

              Show
              dobedobedoh Andrew Nicols added a comment - This patchset probably all want testing together to save the tester some sanity: MDL-31655 MDL-31656 MDL-31673
              Hide
              poltawski Dan Poltawski added a comment -

              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.

              Show
              poltawski Dan Poltawski added a comment - 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.
              Hide
              dobedobedoh Andrew Nicols added a comment -

              Good question - perhaps we should create a new location for state storage data...

              Show
              dobedobedoh Andrew Nicols added a comment - Good question - perhaps we should create a new location for state storage data...
              Hide
              dobedobedoh Andrew Nicols added a comment -

              The content of this change hasn't differed greatly - just renamed the variables and functions to fit into the changes made in MDL-31660.

              Waiting for MDL-31660 to go through peer review and will submit this for Integration Review at the same time as MDL-31660 and MDL-31740

              Show
              dobedobedoh Andrew Nicols added a comment - The content of this change hasn't differed greatly - just renamed the variables and functions to fit into the changes made in MDL-31660 . Waiting for MDL-31660 to go through peer review and will submit this for Integration Review at the same time as MDL-31660 and MDL-31740
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              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

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - 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
              Hide
              dobedobedoh Andrew Nicols added a comment -

              Rebases cleanly onto latest master

              Show
              dobedobedoh Andrew Nicols added a comment - Rebases cleanly onto latest master
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Thanks Andrew this has been integrated now

              Show
              samhemelryk Sam Hemelryk added a comment - Thanks Andrew this has been integrated now
              Hide
              andyjdavis Andrew Davis added a comment -

              Works as described. Passing.

              Show
              andyjdavis Andrew Davis added a comment - Works as described. Passing.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              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

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

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    25/Jun/12