[MDL-38658] Correct fullpath for dndupload.js inclusion - Moodle Tracker

XML

Word

Printable

Details

Type: Sub-task
Status: Closed
Priority: Critical
Resolution: Fixed
Affects Version/s: 2.3.6, 2.4.3, 2.5
Fix Version/s: 2.3.7, 2.4.4
Component/s: JavaScript
Labels:

Affected Branches:
MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
Fixed Branches:
MOODLE_23_STABLE, MOODLE_24_STABLE
Pull from Repository:
git://github.com/andrewnicols/moodle.git
Pull Master Branch:
~~MDL-38658~~-m
Pull Master Diff URL:
https://github.com/andrewnicols/moodle/commit/MDL-38658-m
Difficulty:
Easy
Testing Instructions:
Hide

Ensure that you have cachejs enabled, and jsrev not set

Open dev tools to the 'Network' tab

Open course page and turn editing on

Check that the course/dndupload.js was loaded through lib/javascript.php

Check that dndupload works
Show
Ensure that you have cachejs enabled, and jsrev not set Open dev tools to the 'Network' tab Open course page and turn editing on Check that the course/dndupload.js was loaded through lib/javascript.php Check that dndupload works

Description

I just discovered that the defintion in course/dnduploadlib.php for inclusion of the JS code (course/dndupload.js) uses a moodle_url, so is not using lib/javascript.php and is not being sent with caching headers.

Attachments

Activity

People

Assignee:: Andrew Lyons

Reporter:: Andrew Lyons

Peer reviewer:: Petr Skoda

Integrator:: Eloy Lafuente (stronk7)

Tester:: Jason Fowler

Participants:: Andrew Lyons, Damyon Wiese, Eloy Lafuente (stronk7), Jason Fowler, Petr Skoda

Component watchers:: David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo

Votes:: 0 Vote for this issue

Watchers:: 2 Start watching this issue

Dates

Created:: 23/Mar/13 1:23 AM

Updated:: 28/Mar/13 4:40 PM

Resolved:: 28/Mar/13 4:40 PM

Fix Release Date:: 13/May/13