Moodle
  1. Moodle
  2. MDL-20588

SCORM content issues in IE6 when zlib.output_compression enabled

    Details

    • Type: Bug Bug
    • Status: 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
    • Rank:
      31909

      Description

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

        Issue Links

          Activity

          Hide
          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
          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
          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
          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
          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
          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

            • Votes:
              3 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: