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

Missing exception classes in autolinker.js

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.8.6, 2.9
    • 2.8.7, 2.9.1
    • Glossary
    • MOODLE_28_STABLE, MOODLE_29_STABLE
    • MOODLE_28_STABLE, MOODLE_29_STABLE
    • MDL-50438-master
    • Hide
      1. Create a glossary with an item and enable it's linking.
      2. Create a module and use the glossary term
      3. Open mod/glossary/showentry_ajax.php in your editor and have it echo a string out (e.g. echo "foo" at the top of the script).
      4. Click on the use of your glossary term
        • Confirm that you were shown an ajax exception explaining that there was an error
      5. Remove your changes and refresh the page
      6. Open a second tab in your browser and logout at this tab
      7. Back in the second tab, click on the use of your glossary term
        • Confirm that you were shown a JavaScript exception
      Show
      Create a glossary with an item and enable it's linking. Create a module and use the glossary term Open mod/glossary/showentry_ajax.php in your editor and have it echo a string out (e.g. echo "foo" at the top of the script). Click on the use of your glossary term Confirm that you were shown an ajax exception explaining that there was an error Remove your changes and refresh the page Open a second tab in your browser and logout at this tab Back in the second tab, click on the use of your glossary term Confirm that you were shown a JavaScript exception

    Description

      Autolinker.js can throw either an exception or ajaxexception, none of these classes are included in autolinker module required files.

      Attachments

        Issue Links

          Activity

            People

              mastnym Martin Mastny
              mastnym Martin Mastny
              Ankit Agarwal Ankit Agarwal
              Andrew Lyons Andrew Lyons
              Mark Nelson Mark Nelson
              Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                6/Jul/15