MDL-30652 Scorm adding proper comments
authorAnkit Kumar Agarwal <ankit@moodle.com>
Thu, 15 Dec 2011 06:03:18 +0000 (11:33 +0530)
committerAnkit Kumar Agarwal <ankit@moodle.com>
Tue, 3 Jan 2012 05:08:07 +0000 (10:38 +0530)
mod/scorm/report/basic/report.php
mod/scorm/report/interactions/report.php

index d2f9f68..032f0dd 100644 (file)
@@ -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);
                 }
index 12e61ab..414728d 100644 (file)
@@ -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);
                 }