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

Fix incorrect usage of js_send_uncached

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • 3.0.6, 3.1.2
    • 3.0.4, 3.1.1
    • JavaScript
    • MOODLE_30_STABLE, MOODLE_31_STABLE
    • MOODLE_30_STABLE, MOODLE_31_STABLE
    • MDL-55453-master
    • Hide

      Can use cURL to confirm fix, examples:

      curl -s -D - wwwroot/lib/requirejs.php/-1/core/first.js -o /dev/null
      curl -s -D - wwwroot/lib/javascript.php/-1/lib/requirejs/jquery-private.js -o /dev/null
      

      Should return correct headers like one of these:

      Content-Disposition: inline; filename="requirejs.php"
      Content-Disposition: inline; filename="javascript.php"
      

      Instead of something like:

      Content-Disposition: inline; filename="7984b0a0e139cabadb5afc7756d473fb34d23819"
      

      Show
      Can use cURL to confirm fix, examples: curl -s -D - wwwroot/lib/requirejs.php/-1/core/first.js -o /dev/null curl -s -D - wwwroot/lib/javascript.php/-1/lib/requirejs/jquery-private.js -o /dev/null Should return correct headers like one of these: Content-Disposition: inline; filename="requirejs.php" Content-Disposition: inline; filename="javascript.php" Instead of something like: Content-Disposition: inline; filename="7984b0a0e139cabadb5afc7756d473fb34d23819"

      In the following locations:

      • lib/requirejs.php:144 (At the end of the file)
      • lib/javascript.php:109 (At the end of the file)

      The variable $etag is being passed in, but the function signature does not accept one. In both cases, the $etag parameter should just be removed.

      Not really sure if this actually harms anything or not, just noticed it.

            bushido Mark Nielsen
            bushido Mark Nielsen
            Ryan Wyllie Ryan Wyllie
            Dan Poltawski Dan Poltawski
            Frédéric Massart Frédéric Massart
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.