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

trivial invalid html (typo) in mod_scorm causes countdown error

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.5.8, 2.6.5, 2.7, 2.8
    • 2.6.6, 2.7.2
    • SCORM

    Description

      This is a trivial html error/type, where </p> was entered as <p>.
      In loadSCO.php, the redirect has a timer showing how many seconds until redirect, which prints essentially this:

          <p>Loading ... <span id='countdown'>XX</span> seconds<p>
      

      The "/" missing from the close tag causes a Javascript error in IE in the two lines that follow the type:

                  var e = document.getElementById("countdown");
                  var cSeconds = parseInt(e.innerHTML);
      

      FYI to Dan and Vignesh - I have several more issues coming your way next week, including some significant regressions. I have fixes in place, I just need to get them cleaned up, committed to Github, tests written where appropriate, etc.

      Attachments

        Activity

          People

            raymor Ray Morris
            raymor Ray Morris
            Dan Marsden Dan Marsden
            Marina Glancy Marina Glancy
            Rajesh Taneja Rajesh Taneja
            Dan Marsden, Matteo Scaramuccia, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie
            Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              8/Sep/14