Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 2.1.2
-
Fix Version/s: 2.1.3
-
Component/s: Accessibility
-
Testing Instructions:
-
Difficulty:Easy
-
Affected Branches:MOODLE_21_STABLE
-
Fixed Branches:MOODLE_21_STABLE
-
Pull from Repository:
-
Pull Master Branch:w46_
MDL-30308_m22_reqstrict -
Pull Master Diff URL:
Description
type is not specified on lines 981 & 983
...
|
if (debugging('', DEBUG_DEVELOPER)) {
|
$code .= '<script src="'.$this->yui3loader->base.'yui/yui-debug.js"></script>';
|
} else {
|
$code .= '<script src="'.$this->yui3loader->base.'yui/yui-min.js"></script>';
|
}
|
should be...
...
|
if (debugging('', DEBUG_DEVELOPER)) {
|
$code .= '<script type="text/javascript" src="'.$this->yui3loader->base.'yui/yui-debug.js"></script>';
|
} else {
|
$code .= '<script type="text/javascript" src="'.$this->yui3loader->base.'yui/yui-min.js"></script>';
|
}
|
Gliffy Diagrams
Attachments
Activity
- All
- Comments
- Work Log
- History
- Activity
- Links Hierarchy
Thanks for the report and patch!
To integrators: please cherry pick to 2.1 stable.
Thanks Petr - this has been integrated now 
Test passed. Verified in source
Closing as fixed, many thanks for your effort!
Note that the changes related to master (2.2beta) have been already sent upstream. But the stable ones will be part of next weeklies (Wed/Thu) as usual.
Ciao
Thanks for spotting that and suggesting a solution.