Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-37071 Improve Linting of core JS in Moodle
  3. MDL-37526

Fix JSHint issues with moodle-core-notification

XMLWordPrintable

    • Icon: Sub-task Sub-task
    • Resolution: Fixed
    • Icon: Minor Minor
    • 2.5
    • 2.4.1
    • JavaScript
    • MOODLE_24_STABLE
    • MOODLE_25_STABLE
    • Hide

      Moodle notification dialogues are used in various places, though currently mostly for exception displays. The new activity chooser also uses them and this is the easiest place to test them.

      • Open a course page
      • Open your JS console
      • Turn editing on
      • Open the Activity chooser
      • Confirm that it works as expected
      • Edit /enrol/ajax.php
      • Add something which has no chance of being parsed by a json parser (e.g. echo "{];"; die
      • Navigate to Course administration -> Users -> Enrolled users
        • Confirm that an error was shown in a dialogue
        • Confirm that no errors were thrown in the console
      Show
      Moodle notification dialogues are used in various places, though currently mostly for exception displays. The new activity chooser also uses them and this is the easiest place to test them. Open a course page Open your JS console Turn editing on Open the Activity chooser Confirm that it works as expected Edit /enrol/ajax.php Add something which has no chance of being parsed by a json parser (e.g. echo "{];"; die Navigate to Course administration -> Users -> Enrolled users Confirm that an error was shown in a dialogue Confirm that no errors were thrown in the console

      Fix JSHint issues with moodle-core-notification

            dobedobedoh Andrew Lyons
            dobedobedoh Andrew Lyons
            Damyon Wiese Damyon Wiese
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Rajesh Taneja Rajesh Taneja
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved:

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