Moodle
  1. Moodle
  2. MDL-37394

Launch Failure for AICC Direct URL

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4.2
    • Component/s: SCORM
    • Labels:
    • Rank:
      47017

      Description

      Content loaded using the new AICC Direct URL option fails to load.

      STEPS TO REPRODUCE
      1. Enable direct AICC URL
      2. Create new SCORM activity and use this url

      http://3rdparty.skillwsa.com/olsa/ro?customerId=sp3rdparty&course_id=mo_olpu_a02_dt_enus&access_mode=aicc&HACP=1&olsa_cvs_value=mpKKpv4jOqu6yh0

      The TOC in the SCORM player us not correctly displayed and the course fails to load generating a JavaScript error

        Issue Links

          Activity

          Hide
          Martin Holden added a comment -

          I believe this issue is related to the TOC generation issue

          Show
          Martin Holden added a comment - I believe this issue is related to the TOC generation issue
          Hide
          Martin Holden added a comment -

          AICC URL Launch JavaScript error and NO TOC

          Show
          Martin Holden added a comment - AICC URL Launch JavaScript error and NO TOC
          Hide
          Martin Holden added a comment -

          Same URL used in Moodle 2.3.2 without issue

          Show
          Martin Holden added a comment - Same URL used in Moodle 2.3.2 without issue
          Hide
          Dan Marsden added a comment -

          bouncing up for integration - note this is for 2.4 and master only - it is not an issue in 2.3

          Show
          Dan Marsden added a comment - bouncing up for integration - note this is for 2.4 and master only - it is not an issue in 2.3
          Hide
          Eloy Lafuente (stronk7) added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          Sam Hemelryk added a comment -

          Thanks Dan, this has been integrated now

          Show
          Sam Hemelryk added a comment - Thanks Dan, this has been integrated now
          Hide
          Ankit Agarwal added a comment -

          Hi Dan,
          Am not sure I understand the testing instructions properly

          the content should be loaded in the TOC - (note that the TOC is useless when using directurl so will be empty

          Does that mean something should be loaded in the TOC or not?

          at present TOC is empty for me.

          Thanks

          Show
          Ankit Agarwal added a comment - Hi Dan, Am not sure I understand the testing instructions properly the content should be loaded in the TOC - (note that the TOC is useless when using directurl so will be empty Does that mean something should be loaded in the TOC or not? at present TOC is empty for me. Thanks
          Hide
          Dan Marsden added a comment -

          hey - TOC empty is either "correct behaviour" or a different bug - I'm thinking of disabling the TOC completely when using Direct URL which makes it closer to "Correct behaviour" - see MDL-37484

          Show
          Dan Marsden added a comment - hey - TOC empty is either "correct behaviour" or a different bug - I'm thinking of disabling the TOC completely when using Direct URL which makes it closer to "Correct behaviour" - see MDL-37484
          Hide
          Ankit Agarwal added a comment -

          Thanks Dan for explaining.
          Passing as content is loading
          Thanks

          Show
          Ankit Agarwal added a comment - Thanks Dan for explaining. Passing as content is loading Thanks
          Hide
          Dan Poltawski added a comment -

          Hurray! We did it! Thanks to all the reporters, testers, user and watchers for a bumper week of Moodling!

          Show
          Dan Poltawski added a comment - Hurray! We did it! Thanks to all the reporters, testers, user and watchers for a bumper week of Moodling!
          Hide
          Shamim Rezaie added a comment - - edited

          Hi

          The 3 lines that were added to player.js to fix this issue are not present in the latest 2.4.1.
          Is that because the problem is sorted out in another way or maybe because of some mistake?

          Show
          Shamim Rezaie added a comment - - edited Hi The 3 lines that were added to player.js to fix this issue are not present in the latest 2.4.1. Is that because the problem is sorted out in another way or maybe because of some mistake?
          Hide
          Shamim Rezaie added a comment -

          please ignore my last comment as it's totally rubbish
          sorry

          Show
          Shamim Rezaie added a comment - please ignore my last comment as it's totally rubbish sorry

            People

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

              Dates

              • Created:
                Updated:
                Resolved: