Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-22154

Flash detection is made on every page load with AJAX call to environment.php script (meant be done only once per session)

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 1.9.8
    • 1.9.9
    • General, Performance
    • None
    • All
    • MOODLE_19_STABLE
    • MOODLE_19_STABLE

    Description

      Flash detection is done with AJAX call to login/environment.php script. It is meant to be done only once per login session, but because of incorrect usage of HTML entity, it is done on every page load.

      This can be fixed simply by replacing '&' entity with single '&' in AJAX GET URL:

       
      Index: lib/weblib.php
      ===================================================================
      RCS file: /cvsroot/moodle/moodle/lib/weblib.php,v
      retrieving revision 1.970.2.167
      diff -r1.970.2.167 weblib.php
      2524c2524
      <            "  YAHOO.util.Connect.asyncRequest('GET','".$CFG->httpswwwroot."/login/environment.php?sesskey=".sesskey()."&amp;flashversion='+flashversion.major+'.'+flashversion.minor+'.'+flashversion.release);\n".
      ---
      >            "  YAHOO.util.Connect.asyncRequest('GET','".$CFG->httpswwwroot."/login/environment.php?sesskey=".sesskey()."&flashversion='+flashversion.major+'.'+flashversion.minor+'.'+flashversion.release);\n".

      Attachments

        Activity

          People

            skodak Petr Skoda
            pasi.hakkinen Pasi Häkkinen
            Nobody Nobody
            Adrian Greeve, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo, Matteo Scaramuccia, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              8/Jun/10