Fix code standards

apiclients/easikit/classes/privacy/provider.php

@@ -32,7 +32,7 @@ class provider implements \core_privacy\local\metadata\null_provider {
      *
      * @return  string
      */
-    public static function get_reason() : string {
+    public static function get_reason(): string {
         return 'privacy:metadata';
     }
 }

apiclients/stutalkdirect/classes/privacy/provider.php

@@ -32,7 +32,7 @@ class provider implements \core_privacy\local\metadata\null_provider {
      *
      * @return  string
      */
-    public static function get_reason() : string {
+    public static function get_reason(): string {
         return 'privacy:metadata';
     }
 }

classes/assessment/assign.php

@@ -40,7 +40,7 @@ public function get_all_participants(): array {
      *
      * @return int|null
      */
-    public function get_start_date() : ?int {
+    public function get_start_date(): ?int {
         return $this->moduleinstance->allowsubmissionsfromdate;
     }
 
@@ -49,7 +49,7 @@ public function get_start_date() : ?int {
      *
      * @return int|null
      */
-    public function get_end_date() : ?int {
+    public function get_end_date(): ?int {
         return $this->moduleinstance->duedate;
     }
 }

classes/assessment/quiz.php

@@ -40,7 +40,7 @@ public function get_all_participants(): array {
      *
      * @return int|null
      */
-    public function get_start_date() : ?int {
+    public function get_start_date(): ?int {
         return $this->moduleinstance->timeopen;
     }
 
@@ -49,7 +49,7 @@ public function get_start_date() : ?int {
      *
      * @return int|null
      */
-    public function get_end_date() : ?int {
+    public function get_end_date(): ?int {
         return $this->moduleinstance->timeclose;
     }
 }

classes/assessment/turnitintooltwo.php

@@ -65,7 +65,7 @@ public function get_all_participants(): array {
      *
      * @return int|null
      */
-    public function get_start_date() : ?int {
+    public function get_start_date(): ?int {
         if ($this->partid) {
             // Return the start date of this part.
             return $this->turnitinparts[$this->partid]->dtstart;
@@ -80,7 +80,7 @@ public function get_start_date() : ?int {
      *
      * @return int|null
      */
-    public function get_end_date() : ?int {
+    public function get_end_date(): ?int {
         if ($this->partid) {
             // Return the end date of the part currently pointing to.
             return $this->turnitinparts[$this->partid]->dtdue;

classes/logger.php

@@ -37,7 +37,7 @@ class logger {
      * @return bool|int
      * @throws \dml_exception
      */
-    public static function log (string $message, string $requesturl = null, string $data = null, string $response = null) {
+    public static function log(string $message, string $requesturl = null, string $data = null, string $response = null) {
         global $DB, $USER;
 
         // Create the insert object.
@@ -61,7 +61,7 @@ public static function log (string $message, string $requesturl = null, string $
      * @return bool|int
      * @throws \dml_exception
      */
-    public static function log_request_error (string $message, irequest $request, string $response = null) {
+    public static function log_request_error(string $message, irequest $request, string $response = null) {
         global $DB, $USER;
 
         // Create the insert object.

classes/manager.php

@@ -796,7 +796,7 @@ public function push_submission_log_to_sits(\stdClass $assessmentmapping, int $u
      * @throws \dml_exception
      * @throws \moodle_exception
      */
-    public function get_required_data_for_pushing (\stdClass $assessmentmapping, int $userid): \stdClass {
+    public function get_required_data_for_pushing(\stdClass $assessmentmapping, int $userid): \stdClass {
         global $USER;
 
         // Get SPR_CODE from SITS.
@@ -830,7 +830,7 @@ public function get_required_data_for_pushing (\stdClass $assessmentmapping, int
      * @return array
      * @throws \dml_exception
      */
-    public function get_transfer_logs (int $assessmentmappingid, int $userid, string $type = null): array {
+    public function get_transfer_logs(int $assessmentmappingid, int $userid, string $type = null): array {
         global $DB;
 
         // Initialize params.
@@ -1409,8 +1409,8 @@ public function get_data_for_page_update(int $courseid, int $couresmoduleid = 0)
      * @throws \dml_exception
      */
     private function save_transfer_log(
-        string $type, int $assessmentmappingid, int $userid, mixed $request, array $response, ?int $taskid, int $errorlogid = null)
-    : void {
+        string $type, int $assessmentmappingid, int $userid, mixed $request, array $response, ?int $taskid, int $errorlogid = null
+    ): void {
         global $USER, $DB;
         $insert = new \stdClass();
         $insert->type = $type;

classes/output/pushrecord.php

@@ -137,7 +137,7 @@ public function __construct(\stdClass $student, int $coursemoduleid, \stdClass $
      * @return void
      * @throws \moodle_exception
      */
-    protected function set_grade (int $coursemoduleid, int $studentid): void {
+    protected function set_grade(int $coursemoduleid, int $studentid): void {
         $grade = $this->manager->get_student_grade($coursemoduleid, $studentid);
         if (isset($grade)) {
             $this->marks = $grade;
@@ -149,7 +149,7 @@ protected function set_grade (int $coursemoduleid, int $studentid): void {
      * @param \stdClass $student
      * @return void
      */
-    protected function set_student_info (\stdClass $student): void {
+    protected function set_student_info(\stdClass $student): void {
         $this->userid = $student->id;
         $this->idnumber = $student->idnumber;
         $this->firstname = $student->firstname;
@@ -165,7 +165,7 @@ protected function set_student_info (\stdClass $student): void {
      * @throws \dml_exception
      * @throws \moodle_exception
      */
-    protected function set_submission (int $coursemoduleid, int $studentid): void {
+    protected function set_submission(int $coursemoduleid, int $studentid): void {
 
         // Get submission.
         $submission = submissionfactory::get_submission($coursemoduleid, $studentid);

classes/output/renderer.php

@@ -62,7 +62,7 @@ public function __construct(moodle_page $page, $target) {
      * @return string Rendered HTML
      * @throws \moodle_exception
      */
-    public function render_button(string $id, string $name, string $disabled = '', string $class = '') : string {
+    public function render_button(string $id, string $name, string $disabled = '', string $class = ''): string {
         return $this->output->render_from_template(
             'local_sitsgradepush/button',
             ['id' => $id, 'name' => $name, 'disabled' => $disabled, 'class' => $class]
@@ -78,7 +78,7 @@ public function render_button(string $id, string $name, string $disabled = '', s
      * @return string
      * @throws \moodle_exception
      */
-    public function render_link(string $id, string $name, string $url) : string {
+    public function render_link(string $id, string $name, string $url): string {
         return $this->output->render_from_template('local_sitsgradepush/link', ['id' => $id, 'name' => $name, 'url' => $url]);
     }
 
@@ -90,7 +90,7 @@ public function render_link(string $id, string $name, string $url) : string {
      * @return string Rendered HTML
      * @throws \moodle_exception
      */
-    public function render_marks_transfer_history_page(array $assessmentdata, int $courseid) : string {
+    public function render_marks_transfer_history_page(array $assessmentdata, int $courseid): string {
         // Check if the user has the capability to see the submission log column.
         $showsublogcolumn = has_capability('local/sitsgradepush:showsubmissionlogcolumn', \context_course::instance($courseid));
 
@@ -155,7 +155,7 @@ public function render_marks_transfer_history_page(array $assessmentdata, int $c
      * @throws \dml_exception
      * @throws \moodle_exception
      */
-    public function render_dashboard(array $moduledeliveries, int $courseid) : string {
+    public function render_dashboard(array $moduledeliveries, int $courseid): string {
         // Set default value for the select module delivery dropdown list.
         $options[] = (object) ['value' => 'none', 'name' => 'NONE'];
 
@@ -305,7 +305,7 @@ public function render_existing_activity_page(array $param): bool|string {
      * @param int|null $errortype
      * @return string
      */
-    private function get_label_html(int $errortype = null) : string {
+    private function get_label_html(int $errortype = null): string {
         // This is for old data that does not have the error type.
         if (is_null($errortype)) {
             return '<span class="badge badge-danger">'.errormanager::get_error_label(errormanager::ERROR_UNKNOWN).'</span> ';

classes/privacy/provider.php

@@ -36,7 +36,7 @@ class provider implements
      *
      * @return  string
      */
-    public static function get_reason() : string {
+    public static function get_reason(): string {
         return 'privacy:metadata';
     }
 
@@ -46,7 +46,7 @@ public static function get_reason() : string {
      * @param   collection     $collection The initialised collection to add items to.
      * @return  collection     A listing of user data stored through this system.
      */
-    public static function get_metadata(collection $collection) : collection {
+    public static function get_metadata(collection $collection): collection {
         $collection->add_database_table('local_sitsgradepush_tfr_log', [
             'type' => 'privacy:metadata:local_sitsgradepush_tfr_log:type',
             'userid' => 'privacy:metadata:local_sitsgradepush_tfr_log:userid',

classes/task/pushtask.php

@@ -38,7 +38,7 @@ class pushtask extends scheduled_task {
      * @return string
      * @throws \coding_exception
      */
-    public function get_name() : string {
+    public function get_name(): string {
         return get_string('task:pushtask:name', 'local_sitsgradepush');
     }
 

classes/taskmanager.php

@@ -333,7 +333,7 @@ public static function get_pending_task_in_queue(int $assessmentmappingid): bool
      * @throws \dml_exception
      * @throws \moodle_exception
      */
-    public static function send_email_notification(int $taskid) : void {
+    public static function send_email_notification(int $taskid): void {
         global $DB, $PAGE, $OUTPUT;
 
         // Get the content of the task.