Make code checker happy

Author: mdjnelson

classes/element_helper.php

@@ -687,6 +687,11 @@ public static function get_grade_item_info($gradeitemid, $gradeformat, $userid)
         );
     }
 
+    /**
+     * Helper function to return all the date formats.
+     *
+     * @return array the list of date formats
+     */
     public static function get_date_formats(): array {
         // Hard-code date so users can see the difference between short dates with and without the leading zero.
         // Eg. 06/07/18 vs 6/07/18.

classes/report_table.php

@@ -24,7 +24,7 @@
 
 namespace mod_customcert;
 
-use \customcertelement_expiry\element as expiry_element;
+use customcertelement_expiry\element as expiry_element;
 
 defined('MOODLE_INTERNAL') || die;
 

classes/template.php

@@ -266,11 +266,11 @@ public function delete_element($elementid) {
      * Generate the PDF for the template.
      *
      * @param bool $preview true if it is a preview, false otherwise
-     * @param int $userid the id of the user whose certificate we want to view
+     * @param int|null $userid the id of the user whose certificate we want to view
      * @param bool $return Do we want to return the contents of the PDF?
      * @return string|void Can return the PDF in string format if specified.
      */
-    public function generate_pdf(bool $preview = false, int $userid = null, bool $return = false) {
+    public function generate_pdf(bool $preview = false, ?int $userid = null, bool $return = false) {
         global $CFG, $DB, $USER;
 
         if (empty($userid)) {

element/expiry/classes/element.php

@@ -24,7 +24,7 @@
 
 namespace customcertelement_expiry;
 
-use \mod_customcert\element_helper;
+use mod_customcert\element_helper;
 
 /**
  * The customcert element expiry's core interaction API.

tests/email_certificate_task_test.php

@@ -483,10 +483,10 @@ public function test_email_certificates_students_havent_met_required_criteria():
                         ],
                     ],
                     'showc' => [
-                        false
+                        false,
                     ],
-                ]
-            )
+                ],
+            ),
         ]);
 
         // Create template object.

tests/generator/lib.php

@@ -40,7 +40,7 @@ class mod_customcert_generator extends testing_module_generator {
      * @param array|null $options
      * @return stdClass
      */
-    public function create_instance($record = null, array $options = null) {
+    public function create_instance($record = null, ?array $options = null) {
         $record = (object)(array)$record;
 
         $defaultsettings = [