-
Bug
-
Resolution: Fixed
-
Minor
-
4.3 regressions, 4.3.5, 4.4.1
In the SCORM database refactor (MDL-46279), two field name changes were missed in the AICC ExitAU processing.
in mod/scorm/aicc.php starting around line 414:
$track = new stdClass(); |
$track->scoid = $sco->id;
|
$track->element = scorm_get_elementid('cmi.core.total_time'); |
$track->value = $scormsession->sessiontime;
|
$atobject = scorm_get_attempt($aiccuser->id, $scormsession->scormid, $attempt);
|
$track->attempt = $atobject->id;
|
$track->timemodified = time();
|
$id = $DB->insert_record('scorm_scoes_value', $track); |
Should be:
$track = new stdClass(); |
$track->scoid = $sco->id;
|
$track->elementid = scorm_get_elementid('cmi.core.total_time'); |
$track->value = $scormsession->sessiontime;
|
$atobject = scorm_get_attempt($aiccuser->id, $scormsession->scormid, $attempt);
|
$track->attemptid = $atobject->id;
|
$track->timemodified = time();
|
$id = $DB->insert_record('scorm_scoes_value', $track); |
Specifically, $track->element should be $track->elementid, and $track->attempt should be $track->attemptid.
I did check, and the values being populated there are correct, it's just the field names that are incorrect.
- is a regression caused by
-
MDL-46279 Refactor SCORM database schema to improve performance
-
- Closed
-
