MDL-33822 SCORM Interactions report - fix formatting of empty cells.

This commit is contained in:
Gilles-Philippe Leblanc 2012-07-06 09:41:19 +12:00 committed by Dan Marsden
parent d71b79a406
commit f007c032be

View File

@ -185,6 +185,7 @@ class scorm_interactions_report extends scorm_default_report {
$countsql .= $from.$where;
$attempts = $DB->get_records_sql($select.$from.$where, $params);
$questioncount = get_scorm_question_count($scorm->id);
$nbmaincolumns = count($columns);
for($id = 0; $id < $questioncount; $id++) {
if ($displayoptions['qtext']) {
$columns[] = 'question' . $id;
@ -476,9 +477,7 @@ class scorm_interactions_report extends scorm_default_report {
$row[] = $score;
}
// interaction data
$i=0;
$element='cmi.interactions_'.$i.'.id';
while(isset($trackdata->$element)) {
for ($i=0; $i < $questioncount; $i++) {
if ($displayoptions['qtext']) {
$element='cmi.interactions_'.$i.'.id';
if (isset($trackdata->$element)) {
@ -513,8 +512,6 @@ class scorm_interactions_report extends scorm_default_report {
$row[] = '&nbsp;';
}
}
$i++;
$element = 'cmi.interactions_'.$i.'.id';
}
//---end of interaction data*/
} else {
@ -525,6 +522,10 @@ class scorm_interactions_report extends scorm_default_report {
} else {
$row[] = $strstatus;
}
// complete the empty cells
for ($i=0; $i < count($columns) - $nbmaincolumns; $i++) {
$row[] = '&nbsp;';
}
}
}
}