Minor This fixes some invalid HTML.
On approximately line #1457 of of mod/scorm/locallib.php are the lines:
$link = 'a='.$scorm->id.'&scoid='.$sco->id.'¤torg='.$currentorg.$modestr.'&attempt='.$attempt;
|
$result->toc .= '<a title="'.$link.'">'.$statusicon.' '.format_string($sco->title).' '.$score.'</a>';
|
Because ampersand introduces an entity (like >
, ampersand must be escaped in HTML. It should itself represented as
an entity as it is two lines prior to the problem line. The attached patch does so as follows:
$link = 'a='.$scorm->id.'&scoid='.$sco->id.'&currentorg='.$currentorg.$modestr.'&attempt='.$attempt;
|
- has a non-specific relationship to
-
MDL-29159 Regression: HTML of currently launched/active SCO not rendering correctly in the TOC.
-
- Closed
-