Bug fix: the downloadquizattempts.php script was broken, and had never worked on pgsql databases either. It does now.

Author: trampgeek

downloadquizattempts.php

@@ -33,6 +33,7 @@
 use context_course;
 use html_writer;
 use moodle_url;
+use html_table;
 
 require_once(__DIR__ . '/../../../config.php');
 require_once($CFG->libdir . '/questionlib.php');

getallattempts.php

@@ -52,6 +52,9 @@ public function add_data($row) {
     }
 }
 
+// A version of Moodle's table_sql that handles the screw up with formatting
+// of code (converting it all to HTML, stripping newlines etc - stupid, stupid, stupid -
+// and which formats the datetime field (a Unix timestamp) in a DB independent manner.
 class UnscrewedSqlTable extends table_sql {
     /**
      * Get, and optionally set, the export class.
@@ -75,6 +78,22 @@ function export_class_instance($exportclass = null) {
         }
         return $this->exportclass;
     }
+
+    /**
+     * Format Unix timestamp into readable date/time
+     * This method will be automatically called for the 'datetime' column
+     *
+     * @param object $row The row of data
+     * @return string Formatted date
+     */
+    public function col_datetime($row) {
+        if (empty($row->datetime)) {
+            return '-';
+        }
+        
+        // Use Moodle's built-in time formatting function
+        return userdate($row->datetime);
+    }
     //phpcs:enable
 }
 
@@ -125,7 +144,7 @@ function export_class_instance($exportclass = null) {
         quest.name as qname,
         slot.maxmark as mark,
         qattsteps.timecreated as timestamp,
-        FROM_UNIXTIME(qattsteps.timecreated,'%Y/%m/%d %H:%i:%s') as datetime,
+        qattsteps.timecreated as datetime,
         qattsteps.fraction,
         qattsteps.state,
         qasd.attemptstepid,