Moodle
  1. Moodle
  2. MDL-28999

missing yui require in SCORM

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.0, 2.1.1, 2.2
        • Fix Version/s: 2.0.5, 2.1.2
        • Component/s: SCORM
        • Labels:
        • Testing Instructions:
          Hide

          Requires Firefox and Web Developer toolbar to verify.

          Create a new SCORM using this:
          http://moodle.org/mod/data/view.php?d=50&rid=1655
          set Display package == "new window"
          enter SCORM - Web developer toolbar has 3 icons in top right that are either tick or cross - the 3rd one is for JS errors.(ignore the other 2 for the purpose of this test.)

          click on the 3rd icon - shows a cross and the error "Yahoo.util.Connect is undefined"

          ... if it shows a tick - we're all good!

          I disabled my local cache to test this as well.

          Show
          Requires Firefox and Web Developer toolbar to verify. Create a new SCORM using this: http://moodle.org/mod/data/view.php?d=50&rid=1655 set Display package == "new window" enter SCORM - Web developer toolbar has 3 icons in top right that are either tick or cross - the 3rd one is for JS errors.(ignore the other 2 for the purpose of this test.) click on the 3rd icon - shows a cross and the error "Yahoo.util.Connect is undefined" ... if it shows a tick - we're all good! I disabled my local cache to test this as well.
        • Affected Branches:
          MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
        • Fixed Branches:
          MOODLE_20_STABLE, MOODLE_21_STABLE
        • Pull from Repository:
        • Pull Master Branch:
          master_MDL-28999
        • Pull Master Diff URL:
        • Rank:
          18691

          Description

          Error: YAHOO.util.Connect is undefined

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Eloy Lafuente (stronk7) added a comment -

            Integrated, thanks!

            Show
            Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
            Hide
            Permalink
            Rajesh Taneja added a comment -

            Works Great.
            Thanks for fixing this Dan

            Show
            Rajesh Taneja added a comment - Works Great. Thanks for fixing this Dan
            Hide
            Permalink
            Eloy Lafuente (stronk7) added a comment -

            Many thanks for the hard work, this has been sent upstream and is available in all the git and cvs repositories.

            Show
            Eloy Lafuente (stronk7) added a comment - Many thanks for the hard work, this has been sent upstream and is available in all the git and cvs repositories.

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: