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

JavaScript nodeName returns in lower case now

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • 1.8
    • 1.8
    • Libraries
    • None
    • FireFox 2.0.0.3
    • MOODLE_18_STABLE
    • MOODLE_18_STABLE

      Best example is clicking on the Hide/Show icon for blocks. When I do this I get a "el has no properties" for this line:

      while(el.nodeName != 'BODY') {

      This is in the findParentNode() method in the lib/javascript-static.js file.

      I found that the el.nodeName returns the node name in lower case instead of all caps. I made a quick fix for this for the findParentNode() (see attached), but perhaps more investigations are needed for a more permanent and long term solution (EG: what other browsers return for .nodeName and other code that needs to be updated, etc).

            bushido Mark Nielsen (Inactive)
            bushido Mark Nielsen (Inactive)
            Nobody Nobody (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved:

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