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>';
}
Activity
Show
Sam Hemelryk
added a comment - Thanks Petr - this has been integrated now
Hide
Eloy Lafuente (stronk7)
added a comment -
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 
Show
Eloy Lafuente (stronk7)
added a comment - 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.