Problems viewing SCOs if javascript aggregation is enabled

BTW, SCOs are made with: Lectora Enterprise Edition v.2007(4297)

Hi Dirk, thanks a lot for chipping in! I've now tried the SCO in latest Safari, Chrome, Firefox and Opera on OS/X and found the following:

• Safari, Chrome, Firefox has the same result
• Opera gives me this instead: Your browser does not support dynamic html. Please download a current version of either Microsoft Internet Explorer or Netscape Navigator and try visiting our site again. Thank You. - more than indicating that this is an old SCO..
• However, all browsers browsers but Opera successfully shows the SCO *if* javascript compressing in Drupal is turned off (administer -> settings -> performance)

..so, have you tried this with js compression on? Does the Adobe Captivate package work then? I'd be happy to send you one of my SCOs to test with, but I don't think I can't attach them here in public as I don't "own" them. If you'd like to test please send me an e-mail at [my username here] @ gmail.com or through the contact form on my profile.

BTW: I'm not the one who have made the SCOs, or the one who use authoring tools, I just get the finished SCOs that people want to show. In other words I can't export them in different versions (this was 1.2, though) :)

Changing category, because I think this actually is an issue with the module. I've experimented a bit and found that the js referenced to in my first screenshot here is never included if javascript aggregation is on. The other files from the module (request.js, rd.js, feedback_popup.js, inject_css.js) are.

The js is included in player.php, line 117:

drupal_add_js('api?id='.$cm->id.$scoidstr.$modestr.$attemptstr, 'core');

I have tried to set 'inline' as an argument instead of 'core', which I believe should solve this, but it doesn't. See second screenshot for the result (now with js aggregation enabled).

However, if you examine both screenshots you will see that it uses & instead of & - could it be that using the entity works if you reference the file through the script tag, but not if you use it inline? If I click the link to the js in the first screenshot I get an error in the browser, but if I change all html entitites (&) to plain &-s I get to see the contents of the file (which in this case is the datamodels/scorm_12.js.php).

Hm, it's actually only adding the reference to the file inline, not the script itself, so no wonder it doesn't work.. :)

So, tried this one too as a replacement of the original line 117:

drupal_set_html_head('<script type="text/javascript" src="/api?id='.$cm->id.$scoidstr.$modestr.$attemptstr.'"></script>');

(allthough I believe it is considered very bad practice..)

I get the desired output (the link to the file, also with js aggregation enabled), but it still doesn't work. Maybe because of the entities, but then again I don't understand why the same reference works when js aggragation is disabled..?

Ah, we are getting there - thanks a lot for taking the time to look at this! :)

After your change to line 117 the js is indeed loaded even with aggregation on, however I still get the same error message as the reference is wrong due to the & entities in the URL, see screenshot.

This line:

<script type="text/javascript" src="/api?id=13&amp;scoid=91&amp;mode=normal&amp;attempt=50?1290598665"></script> should be:
<script type="text/javascript" src="/api?id=13&scoid=91&mode=normal&attempt=50?1290598665"></script>

To fix this I had to change the following lines in player.php accordingly:

line 60: $attemptstr = '&attempt=' . $attempt; should be $attemptstr = '&attempt=' . $attempt;
line 83: $scoidstr = '&scoid='.$sco->id; should be $scoidstr = '&scoid='.$sco->id;
line 85: $modestr = '&mode='.$mode; should be $modestr = '&mode='.$mode;

..and with those additional changes the error is completely gone! I have tested on some other pages with some other SCOs too, and I can't see any immediate side effects.

I've tested this in Firefox, Opera, Safari and Chrome with the same result. I don't quite understand why you do not get the same error, but I'm thinking it must be a browser issue, where your browser reads the & as a &, but mine doesn't..

Here's my player.php attached, where all these changes are done, and everything is working.

..we are experiencing this again, so setting to active, will investigate more and update here when I find something.

..we are experiencing this again, so setting to active, will investigate more and update here when I find something.