IOMAD: Logic not quite right on cert regeneration.

iomad · Sep 22, 2023 · d5d195c · d5d195c
1 parent 0970d22
commit d5d195c
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 7 deletions.
diff --git a/local/iomad_track/db/install.php b/local/iomad_track/db/install.php
@@ -133,6 +133,8 @@ function xmldb_local_iomad_track_record_certificates($courseid, $userid, $tracki
     // Get context
     $context = context_course::instance($courseid);
 
+mtrace ("generating certificate with onlyvisible = $onlyvisible</br>");
+die;
     // Get user
     $user = $DB->get_record('user', array('id' => $userid), '*', MUST_EXIST);
 

diff --git a/local/report_completion/index.php b/local/report_completion/index.php
@@ -318,7 +318,7 @@
                     // Re-generate the certificate.
                     if ($trackrec = $DB->get_record('local_iomad_track', array('id' => $key))) {
                         local_iomad_track_delete_entry($key);
-                        xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, true);
+                        xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, false);
                     }
                 }
             }
@@ -362,7 +362,7 @@
 
                         // Re-generate the certificate.
                         local_iomad_track_delete_entry($key);
-                        xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, true);
+                        xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, false);
                     }
                 }
             }
@@ -376,7 +376,7 @@
         if ($action == 'redocert' && !empty($redocertificate)) {
             if ($trackrec = $DB->get_record('local_iomad_track', array('id' => $redocertificate))) {
                 local_iomad_track_delete_entry($redocertificate);
-                if (xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, true)) {
+                if (xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, false)) {
                     redirect(new moodle_url('/local/report_completion/index.php', $params),
                              get_string($action . "_successful", 'local_report_users'),
                              null,

diff --git a/local/report_users/newentry.php b/local/report_users/newentry.php
@@ -107,7 +107,7 @@
     $trackid = $DB->insert_record('local_iomad_track', $newentry);
 
     // Create a certificate, if required.
-    xmldb_local_iomad_track_record_certificates($newentry->courseid, $newentry->userid, $trackid, false, true);
+    xmldb_local_iomad_track_record_certificates($newentry->courseid, $newentry->userid, $trackid, false, false);
 
     // Return success.
     redirect($returnurl,

diff --git a/local/report_users/userdisplay.php b/local/report_users/userdisplay.php
@@ -206,7 +206,7 @@
                     // Re-generate the certificate.
                     if ($trackrec = $DB->get_record('local_iomad_track', array('id' => $key))) {
                         local_iomad_track_delete_entry($key);
-                        xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, true);
+                        xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, false);
                     }
                 }
             }
@@ -250,7 +250,7 @@
 
                         // Re-generate the certificate.
                         local_iomad_track_delete_entry($key);
-                        xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, true);
+                        xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, false);
                     }
                 }
             }
@@ -272,7 +272,7 @@
         if ($action == 'redocert' && !empty($redocertificate)) {
             if ($trackrec = $DB->get_record('local_iomad_track', array('id' => $redocertificate))) {
                 local_iomad_track_delete_entry($redocertificate);
-                if (xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, true)) {
+                if (xmldb_local_iomad_track_record_certificates($trackrec->courseid, $trackrec->userid, $trackrec->id, false, false)) {
                     redirect(new moodle_url('/local/report_users/userdisplay.php', array('userid' => $userid)),
                              get_string($action . "_successful", 'local_report_users'),
                              null,