[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:

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
Difficulty:
Easy
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

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 Nicols

Reporter:

Andrew Nicols

Peer reviewer:

Petr Skoda

Integrator:

Eloy Lafuente (stronk7)

Tester:

Jason Fowler

Participants:

Andrew Nicols, Damyon Wiese, Eloy Lafuente (stronk7), Jason Fowler, Petr Skoda

Component watchers:

Jake Dallimore, Jun Pataleta, Ryan Wyllie

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