Moodle

SCORM content issues in IE6 when zlib.output_compression enabled

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.9.5
  • Fix Version/s: 1.8.12, 1.9.8
  • Component/s: SCORM
  • Labels:
    None
  • Environment:
    Apache 2, PHP5, zlib.output_compression enabled
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_18_STABLE, MOODLE_19_STABLE

Description

Several sites having problems loading SCORM content with zlib compression was enabled.

Issue Links

This issue will help resolve:
MDL-16185 Scorm API not always found when SCO is first launched Critical Closed

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Dakota Duff added a comment -

Our tests show that adding the zlib.output_compression check from filelib.php to the tops of /mod/scorm/api.php and /mod/scorm/player.php fixes this.

//IE compatibiltiy HACK!
if (ini_get('zlib.output_compression')) { ini_set('zlib.output_compression', 'Off'); }

Show
Dakota Duff added a comment - Our tests show that adding the zlib.output_compression check from filelib.php to the tops of /mod/scorm/api.php and /mod/scorm/player.php fixes this. //IE compatibiltiy HACK! if (ini_get('zlib.output_compression')) { ini_set('zlib.output_compression', 'Off'); }
Hide
Permalink
Dan Marsden added a comment -

Hi Dakota,

I'm not too keen adding this as a global change to SCORM as it only affects specific Browsers. - any chance you could add a browser check, test and post a new patch?

thanks,

Dan

Show
Dan Marsden added a comment - Hi Dakota, I'm not too keen adding this as a global change to SCORM as it only affects specific Browsers. - any chance you could add a browser check, test and post a new patch? thanks, Dan
Hide
Permalink
Dakota Duff added a comment -

Sure! This should only affect Internet Explorer.

if (strpos($_SERVER['HTTP_USER_AGENT'], 'MSIE') !== false && ini_get('zlib.output_compression') == 'On') {
    ini_set('zlib.output_compression', 'Off');
}
Show
Dakota Duff added a comment - Sure! This should only affect Internet Explorer. 
if (strpos($_SERVER['HTTP_USER_AGENT'], 'MSIE') !== false && ini_get('zlib.output_compression') == 'On') {
    ini_set('zlib.output_compression', 'Off');
}
Hide
Permalink
Dan Marsden added a comment -

thanks Dakota - that looks better, I've only seen this issue with IE 6, and thought it was resolved in newer versions of IE? - is that consistent with your testing?

Show
Dan Marsden added a comment - thanks Dakota - that looks better, I've only seen this issue with IE 6, and thought it was resolved in newer versions of IE? - is that consistent with your testing?
Hide
Permalink
Dan Marsden added a comment -

..in which case can we use:

strpos($_SERVER['HTTP_USER_AGENT'], 'MSIE 6.')

Show
Dan Marsden added a comment - ..in which case can we use: strpos($_SERVER['HTTP_USER_AGENT'], 'MSIE 6.')
Hide
Permalink
Dan Marsden added a comment -

fix now in 1.8Stable, 1.9Stable and HEAD

note - I have used the IE6 variation of this check - I have only seen this bug occur with IE 6 - I'm happy to modify it if other people can confirm this is a problem in newer versions (or older versions) of IE

thanks for the report!

Show
Dan Marsden added a comment - fix now in 1.8Stable, 1.9Stable and HEAD note - I have used the IE6 variation of this check - I have only seen this bug occur with IE 6 - I'm happy to modify it if other people can confirm this is a problem in newer versions (or older versions) of IE thanks for the report!

People

Vote (3)
Watch (4)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Moodle. Try JIRA - bug tracking software for your team.