From e12459baa2c946ab91c637c4367b919bf215c794 Mon Sep 17 00:00:00 2001 From: Ankit Kumar Agarwal Date: Thu, 15 Dec 2011 11:33:18 +0530 Subject: [PATCH] MDL-30652 Scorm adding proper comments --- mod/scorm/report/basic/report.php | 4 +--- mod/scorm/report/interactions/report.php | 2 +- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/mod/scorm/report/basic/report.php b/mod/scorm/report/basic/report.php index d2f9f6840ea..032f0dd7f77 100644 --- a/mod/scorm/report/basic/report.php +++ b/mod/scorm/report/basic/report.php @@ -155,11 +155,9 @@ class scorm_basic_report extends scorm_default_report { $table->sortable(true); $table->collapsible(true); + // This is done to prevent redundant data, when a user has multiple attempts $table->column_suppress('picture'); $table->column_suppress('fullname'); - // I wonder why it is doing all this suppress malarkey? - // However, if it was suppressing idnumber field before, I guess - // it needs to suppress all the user identity fields now foreach ($extrafields as $field) { $table->column_suppress($field); } diff --git a/mod/scorm/report/interactions/report.php b/mod/scorm/report/interactions/report.php index 12e61abdeb9..414728dc5af 100644 --- a/mod/scorm/report/interactions/report.php +++ b/mod/scorm/report/interactions/report.php @@ -211,9 +211,9 @@ class scorm_interactions_report extends scorm_default_report { $table->sortable(true); $table->collapsible(true); + // This is done to prevent redundant data, when a user has multiple attempts $table->column_suppress('picture'); $table->column_suppress('fullname'); - // This is done to prevent redundant data, when a user has multiple attempts foreach ($extrafields as $field) { $table->column_suppress($field); }