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

Some dialogues (exceptions, ajaxexception, confirm, etc) are no longer shown when instantiated

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • 2.6.1
    • 2.6
    • JavaScript
    • MOODLE_26_STABLE
    • MOODLE_26_STABLE
    • MDL-43325-master
    • Hide

      Run full behat suite

      • Edit help_ajax.php
      • At some point after config.php has been included, throw a new moodle_exception
      • Open the login page in the browser
      • Click on the help icon
        • Confirm that the ajaxException dialogue was displayed
      • Remove the moodle_exception
      • echo out some random junk at the end of the file
      • Click on the help icon again
        • Confirm that an exception dialogue was displayed
      • Edit enrol/yui/rolemanager/rolemanager.js
        • Find the visible : true line, and remove it
      • Login to moodle as an admin/manager/teacher
      • Navigate to a course's enrol page
      • Remove a role from a user
        • Confirm that you get a confirmation dialogue
      • Navigate to a course and turn editing on
      • Open the activity chooser
        • Confirm that it was shown as expected
      • Cancel the activity chooser
      • Reopen it
        • Confirm that it was shown again
      • Add a new assignment instance
      • As a student, upload a PDF to it
        • Confirm that the filepicker was not displayed until you requested it
      • As a tutor, Launch the PDF editor
        • Confirm that no new dialogues are visible initially
      • Click on the comment search button
        • Confirm that the dialogue is now visible

      We have few examples of M.core.alert in core, and these are very difficult to force a display of so skipping testing for these.

      Show
      Run full behat suite Edit help_ajax.php At some point after config.php has been included, throw a new moodle_exception Open the login page in the browser Click on the help icon Confirm that the ajaxException dialogue was displayed Remove the moodle_exception echo out some random junk at the end of the file Click on the help icon again Confirm that an exception dialogue was displayed Edit enrol/yui/rolemanager/rolemanager.js Find the visible : true line, and remove it Login to moodle as an admin/manager/teacher Navigate to a course's enrol page Remove a role from a user Confirm that you get a confirmation dialogue Navigate to a course and turn editing on Open the activity chooser Confirm that it was shown as expected Cancel the activity chooser Reopen it Confirm that it was shown again Add a new assignment instance As a student, upload a PDF to it Confirm that the filepicker was not displayed until you requested it As a tutor, Launch the PDF editor Confirm that no new dialogues are visible initially Click on the comment search button Confirm that the dialogue is now visible We have few examples of M.core.alert in core, and these are very difficult to force a display of so skipping testing for these.
    • 3
    • FRONTEND Sprint 7

      They used to be... and should still be.
      We were previously ignoring the 'visible' field. We now respect it (sanity prevails), but should be changing the default to true for all but the base dialogue.

      Note, although everywhere that we have confirm and alert dialogues in core, we already forcibly call show, or we set visible: true in it's config, but this is still a change from the API which has existed for quite some time so should be fixed.

      Exceptions and ajaxExceptions no longer display at all in many places.

            dobedobedoh Andrew Lyons
            dobedobedoh Andrew Lyons
            Damyon Wiese Damyon Wiese
            Sam Hemelryk Sam Hemelryk
            Sam Hemelryk Sam Hemelryk
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved:

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