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

Short form not expanded for errors on submission

XMLWordPrintable

    • MOODLE_310_STABLE, MOODLE_39_STABLE
    • MOODLE_310_STABLE, MOODLE_39_STABLE
    • MDL-70987-master
    • Hide

      Test the fix for client-side validation in forms with randomised ids:

      1. Place 'testform.php' in your web root and access it
      2. Open browser dev tools -> console and make sure 'persist logs' is enabled (or equivalent). Leave the console open.
      3. Click on "Collapse me" to collapse the fieldset
      4. Click on Submit
        1. Confirm that the section was opened
        2. Confirm that there is an error immediately below the field
        3. Confirm that the field error was focused
        4. Confirm that you don't see any JS errors in the console
      5. Submit the form again, watching for a browser post. Verify nothing is posted.
      6. Reload the page to clear the validation error
      7. Click in the text field
      8. Press the [tab] key
        1. Confirm that there is an error immediately below the field

       Regression test 1: Confirming this works for forms without randomised ids (the vast majority of cases in Moodle):

      1. Edit 'testform.php' and change:

        ['data-random-ids' => 1]

        to:

        ['data-random-ids' => 0]

        thus disabling the randomised ids.

      2. Repeat all of the previous tests

      Regression test 2: Confirming client side validation works when multiple forms are used and when randomisation of ids is enabled

      N.B. This test is based on the test in MDL-65217, but adds client side validation test steps.

      1. Place 'testmultipleforms.php' in your web root and access it
      2. Open browser dev tools -> console and make sure 'persist logs' is enabled (or equivalent). Leave the console open.
      3. Click on "Collapse me" to collapse the fieldset for both forms
      4. Click on Submit
        1. Confirm that the section was opened in the form that you submitted
        2. Confirm that there is an error immediately below the field
        3. Confirm that the field error was focused
        4. Confirm that you don't see any JS errors in the console
      5. Submit the form again, watching for a browser post.
        1. Confirm that there is no entryin the NEtwork tab to suggest that the form was submitted
      Show
      Test the fix for client-side validation in forms with randomised ids: Place 'testform.php' in your web root and access it Open browser dev tools -> console and make sure 'persist logs' is enabled (or equivalent). Leave the console open. Click on "Collapse me" to collapse the fieldset Click on Submit Confirm that the section was opened Confirm that there is an error immediately below the field Confirm that the field error was focused Confirm that you don't see any JS errors in the console Submit the form again, watching for a browser post. Verify nothing is posted. Reload the page to clear the validation error Click in the text field Press the [tab] key Confirm that there is an error immediately below the field  Regression test 1: Confirming this works for forms without randomised ids (the vast majority of cases in Moodle): Edit 'testform.php' and change: ['data-random-ids' => 1] to: ['data-random-ids' => 0] thus disabling the randomised ids. Repeat all of the previous tests Regression test 2: Confirming client side validation works when multiple forms are used and when randomisation of ids is enabled N.B. This test is based on the test in MDL-65217 , but adds client side validation test steps. Place 'testmultipleforms.php' in your web root and access it Open browser dev tools -> console and make sure 'persist logs' is enabled (or equivalent). Leave the console open. Click on "Collapse me" to collapse the fieldset for both forms Click on Submit Confirm that the section was opened in the form that you submitted Confirm that there is an error immediately below the field Confirm that the field error was focused Confirm that you don't see any JS errors in the console Submit the form again, watching for a browser post. Confirm that there is no entryin the NEtwork tab to suggest that the form was submitted
    • Internationals - 3.11 Sprint 7

      This is a regressionf rom MDL-66106. At the moment the elementid is passed with a value of the formid.
      $this houdl be $elem for the errorid calculation

        1. Screenshot_1.jpg
          Screenshot_1.jpg
          61 kB
        2. Screenshot_2.jpg
          Screenshot_2.jpg
          75 kB
        3. Screenshot_3.jpg
          Screenshot_3.jpg
          75 kB
        4. testform.php
          1.0 kB
        5. testmultipleforms.php
          0.8 kB

            dobedobedoh Andrew Lyons
            dobedobedoh Andrew Lyons
            Jake Dallimore Jake Dallimore
            Jun Pataleta Jun Pataleta
            Janelle Barcega Janelle Barcega
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 3 hours, 20 minutes
                3h 20m

                  Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.