Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-45564 SCORM Navigation fixes
  3. MDL-42835

JS error when using scorm attached to MDL-42812

    XMLWordPrintable

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.6, 2.7, 2.8
    • Fix Version/s: 2.6.4, 2.7.1
    • Component/s: SCORM
    • Labels:
    • Testing Instructions:
      Hide

      1. Add the scorm package attached to MDL-42812 and set "Show Navigation" to: "Under content"
      2. Use the navigation buttons at the bottom of the player to navigate through the SCO's
      3. Make sure no error appears on the javascript console. The navigation buttons stop working if an error has occurred. Previously pressing the up button '^' and navigating further caused the error.

      Show
      1. Add the scorm package attached to MDL-42812 and set "Show Navigation" to: "Under content" 2. Use the navigation buttons at the bottom of the player to navigate through the SCO's 3. Make sure no error appears on the javascript console. The navigation buttons stop working if an error has occurred. Previously pressing the up button '^' and navigating further caused the error.
    • Affected Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE, MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      While testing MDL-42812 I found I was getting the following JS error when trying to click the next page button near the bottom of the page.

      Uncaught TypeError: Cannot read property 'children' of undefined

      scorm_next module.js:417
      scorm_launch_next_sco module.js:525
      scorm_buttons.(anonymous function).Y.Button.on.click module.js:724
      Y.Subscriber._notify yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:26572
      Y.Subscriber.notify yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:26601
      Y.CustomEvent._notify yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:26264
      Y.CustomEvent._procSubs yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:26390
      CEProto.fireComplex yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:40576
      Y.CustomEvent._fire yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:26337
      ET.fire yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:27492
      (anonymous function) yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:46715
      delegate.notifySub yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:34800
      Y.Subscriber.notify yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:26601
      Y.CustomEvent._notify yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:26264
      Y.CustomEvent._procSubs yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:26390
      Y.CustomEvent.fireSimple yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:26357
      Y.CustomEvent._fire yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:26339
      Y.CustomEvent.fire yui_combo.php?rollup/3.13.0/yui-moodlesimple.js:26308
      cewrapper.fn

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              damyon Damyon Wiese
              Reporter:
              samhemelryk Sam Hemelryk
              Peer reviewer:
              Andrew Nicols
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Eloy Lafuente (stronk7)
              Participants:
              Component watchers:
              Damyon Wiese, Dan Marsden, Matteo Scaramuccia, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              3 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                14/Jul/14