Fix some phpcs issues phpcbf could not fix automatically

Author: dlmsr

addcourse.php

@@ -14,6 +14,13 @@
 // You should have received a copy of the GNU General Public License
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
+/**
+ * Link another course for evaluation with this Moodle course
+ *
+ * @package    block_evasys_sync
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
 require_once('../../config.php');
 $id = required_param('id', PARAM_INT);
 
@@ -30,7 +37,7 @@
     echo get_string('forbidden', 'block_evasys_sync');
     echo $OUTPUT->footer();
     return;
-} else if(!array_key_exists('lsf_unification', core_plugin_manager::instance()->get_plugins_of_type('local'))){
+} else if (!array_key_exists('lsf_unification', core_plugin_manager::instance()->get_plugins_of_type('local'))) {
     echo $OUTPUT->header();
     throw new \moodle_exception('nohisconnection_error', 'block_evasys_sync');
     echo $OUTPUT->footer();
@@ -76,7 +83,9 @@
     // Once retrieve the teachers course list in order to search for its values later.
     $coursesofteacher = get_teachers_course_list($USER->username, false, true);
 
-    // Add all courses that were already mapped prior to the current change (even if the logged in user does not own these courses herself).
+    // Add all courses that were already mapped prior to the current
+    // change (even if the logged in user does not own these courses
+    // herself).
     foreach ($preexistingmappings as $veranstid) {
         $iscourseofteacher = !empty($coursesofteacher[$veranstid]);
         if (!$iscourseofteacher && !is_siteadmin()) {

alterstandardtime.php

@@ -14,6 +14,11 @@
 // You should have received a copy of the GNU General Public License
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
+/**
+ * @package block_evasys_sync
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
 require_once('../../config.php');
 
 require_login();

block_evasys_sync.php

@@ -14,7 +14,10 @@
 // You should have received a copy of the GNU General Public License
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
-defined('MOODLE_INTERNAL') || die();
+/**
+ * @package block_evasys_sync
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
 
 class block_evasys_sync extends block_base {
 
@@ -252,4 +255,3 @@ public function has_config() {
         return true;
     }
 }
-

coursesearch.php

@@ -104,7 +104,7 @@
         }
         $queuedtasks = \core\task\manager::get_adhoc_tasks(evasys_bulk_task::class);
         $tasksofcurrentmodule = array_filter($queuedtasks, fn($task) => $task->get_custom_data()->categoryid === $id);
-        if(empty($tasksofcurrentmodule)){
+        if (empty($tasksofcurrentmodule)) {
             $task = new evasys_bulk_task();
             $data = new stdClass();
             $data->courses = (array) $courses;

managecategory_manual.php

@@ -52,7 +52,7 @@
     $courses = required_param_array('ids', PARAM_INT);
     $queuedtasks = \core\task\manager::get_adhoc_tasks(evasys_bulk_task::class);
     $tasksofcurrentmodule = array_filter($queuedtasks, fn($task) => $task->get_custom_data()->categoryid === $id);
-    if(empty($tasksofcurrentmodule)){
+    if (empty($tasksofcurrentmodule)) {
         $task = new evasys_bulk_task();
         $data = new stdClass();
         $data->courses = $courses;

managecategory_remaining.php

@@ -66,7 +66,7 @@
     }
     $queuedtasks = \core\task\manager::get_adhoc_tasks(evasys_bulk_task::class);
     $tasksofcurrentmodule = array_filter($queuedtasks, fn($task) => $task->get_custom_data()->categoryid === $id);
-    if(empty($tasksofcurrentmodule)){
+    if (empty($tasksofcurrentmodule)) {
         $task = new evasys_bulk_task();
         $data = new stdClass();
         $data->courses = $courses;
@@ -122,7 +122,7 @@
 
 $mform->display();
 
-if(!empty($tasksofcurrentmodule)){
+if (!empty($tasksofcurrentmodule)) {
     $category = \core_course_category::get($id);
     \core\notification::warning(get_string("running_crontask", "block_evasys_sync", $category->name));
 }

settings.php

@@ -14,6 +14,11 @@
 // You should have received a copy of the GNU General Public License
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
+/**
+ * @package block_evasys_sync
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
 defined('MOODLE_INTERNAL') || die;
 
 if ($hassiteconfig) { // Needs this condition or there is error on login page.

sync.php

@@ -14,6 +14,11 @@
 // You should have received a copy of the GNU General Public License
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
+/**
+ * @package block_evasys_sync
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
 use block_evasys_sync\course_evaluation_allocation;
 use block_evasys_sync\date_decoder;
 
@@ -36,7 +41,7 @@
 if (!optional_param('activate_standard', false, PARAM_BOOL)) {
 
     if (optional_param('datedisabled', false, PARAM_BOOL)) {
-        // We already have an evaluation request for this course: fetch the start- and enddates
+        // We already have an evaluation request for this course: fetch the start- and enddates.
         $sql = 'SELECT v.id, v.starttime, v.endtime, max(v.timemodified) as time FROM {' . \block_evasys_sync\dbtables::EVAL_VERANSTS . '} v INNER JOIN ' .
             '{' . \block_evasys_sync\dbtables::EVAL_COURSES . '} c on v.evalid=c.evalid WHERE c.courseid = :courseid GROUP BY v.id, v.starttime ORDER BY time DESC';
         $record = $DB->get_record_sql($sql, ['courseid' => $courseid]);

version.php

@@ -14,6 +14,13 @@
 // You should have received a copy of the GNU General Public License
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
+/**
+ * Version information for this plugin.
+ *
+ * @package block_evasys_sync
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
 defined('MOODLE_INTERNAL') || die();
 
 $plugin->component = 'block_evasys_sync';