GH-620 linting fixes

classes/task/cancel_expired_attempts.php

@@ -34,6 +34,8 @@
 use mod_adaptivequiz\local\attempt\attempt_state;
 use stdClass;
 
+defined('MOODLE_INTERNAL') || die();
+
 global $CFG;
 require_once("$CFG->dirroot/mod/adaptivequiz/locallib.php");
 
@@ -155,6 +157,7 @@ public function execute() {
      * @param int $timecreated Timestamp when the attempt was created
      * @param ?int $quizmaxtime Maximum time specified by the quiz
      * @param int $defaultmaxtime Default maximum time
+     * @param int $now
      * @return bool
      */
     public function exceeds_maxtime(int $timecreated, ?int $quizmaxtime, int $defaultmaxtime, int $now): bool {

settings.php

@@ -124,7 +124,7 @@
         'local_catquiz/maximum_attempt_duration_hours',
         get_string('maxattemptduration', 'local_catquiz'),
         get_string('maxattemptduration_desc', 'local_catquiz'),
-        24, // Default value
-        PARAM_INT // Expect integer type
-    )); 
+        24, // Default value.
+        PARAM_INT // Expect integer type.
+    ));
 }

tests/task/cancel_expired_attempts_test.php

@@ -21,9 +21,6 @@
  * @author David Szkiba <[email protected]>
  * @copyright  2023 Georg Maißer <[email protected]>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- * @autor David Szkiba <[email protected]>
- * @copirignt 2023 Georg Maißer <[email protected]>
- * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
 namespace local_catquiz;
@@ -47,10 +44,21 @@ final class cancel_expired_attempts_test extends basic_testcase {
     /**
      * Test if the newton raphson multi stable function returns expected outputs.
      *
-     * @dataProvider attempt_expiration_is_detected_correctly_provider
+     * @param int $attemptcreated
+     * @param ?int $quizmaxtime
+     * @param int $defaultmaxtime
+     * @param int $now
+     * @param bool $expected
      * @return void
+     * @dataProvider attempt_expiration_is_detected_correctly_provider
      */
-    public function test_attempt_expiration_is_detected_correctly(int $attemptcreated, ?int $quizmaxtime, int $defaultmaxtime, int $now, bool $expected) {
+    public function test_attempt_expiration_is_detected_correctly(
+        int $attemptcreated,
+        ?int $quizmaxtime,
+        int $defaultmaxtime,
+        int $now,
+        bool $expected
+    ): void {
         $task = new cancel_expired_attempts();
         $shouldbecompleted = $task->exceeds_maxtime($attemptcreated, $quizmaxtime, $defaultmaxtime, $now);
         $this->assertEquals($expected, $shouldbecompleted);