From 23f5fc1f34ccc3a271440c3be30d23722c94b1f1 Mon Sep 17 00:00:00 2001
From: Alexander Bias <alexander.bias@lernlink.de>
Date: Mon, 17 Jun 2024 21:02:17 +0200
Subject: [PATCH] Bugfix: Remove debug warning in enrol_semco_enrol_user
 webservice endpoint.

---
 CHANGES.md   |  1 +
 locallib.php | 16 +++++++++-------
 2 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/CHANGES.md b/CHANGES.md
index e144e7e..9a3d96e 100644
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -6,6 +6,7 @@ Changes
 
 ### Unreleased
 
+* 2024-06-01 - Bugfix: Remove debug warning in enrol_semco_enrol_user webservice endpoint.
 * 2024-06-01 - Upgrade: Migrate the enrol_semco_before_standard_top_of_body_html() function to the new hook callback on Moodle 4.4.
 * 2024-06-01 - Release: Let codechecker ignore some sniffs in the language pack.
 
diff --git a/locallib.php b/locallib.php
index 297d15c..e478bc4 100644
--- a/locallib.php
+++ b/locallib.php
@@ -125,7 +125,7 @@ function enrol_semco_detect_enrolment_overlap($courseid, $userid, $timestart, $t
         // OR which do not have an end date and start before this one ends
         // OR which do not have a start date
         // OR which end before this one ends.
-        $overlapstartexistssql = 'SELECT ue.id
+        $overlapendexistssql = 'SELECT ue.id
                 FROM {user_enrolments} ue
                 JOIN {enrol} e ON ue.enrolid = e.id AND e.courseid = :courseid
                 WHERE e.enrol = :enrol AND
@@ -134,7 +134,7 @@ function enrol_semco_detect_enrolment_overlap($courseid, $userid, $timestart, $t
                         OR ue.timeend = 0 AND ue.timestart <= :timeend1
                         OR ue.timestart = 0
                         OR ue.timeend > 0 AND ue.timeend <= :timeend2)';
-        $overlapstartparams = ['courseid' => $courseid,
+        $overlapendparams = ['courseid' => $courseid,
                 'userid' => $userid,
                 'enrol' => 'semco',
                 'timeend1' => $timeend,
@@ -142,10 +142,10 @@ function enrol_semco_detect_enrolment_overlap($courseid, $userid, $timestart, $t
                 'timestart' => $timestart,
         ];
         if ($ignoreoriginalid != null) {
-            $overlapstartexistssql .= ' AND e.id != :ignoreid';
-            $overlapstartparams['ignoreid'] = $ignoreoriginalid;
+            $overlapendexistssql .= ' AND e.id != :ignoreid';
+            $overlapendparams['ignoreid'] = $ignoreoriginalid;
         }
-        $overlapstartexists = $DB->record_exists_sql($overlapstartexistssql, $overlapstartparams);
+        $overlapendexists = $DB->record_exists_sql($overlapendexistssql, $overlapendparams);
     }
 
     // If we have a given end date and a given start date.
@@ -182,8 +182,10 @@ function enrol_semco_detect_enrolment_overlap($courseid, $userid, $timestart, $t
     }
 
     // If any overlap exists.
-    if ($overlapunlimitedexists == true || $overlapstartexists == true || $overlapstartexists == true ||
-            $overlapbothexists == true) {
+    if ((isset($overlapunlimitedexists) && $overlapunlimitedexists == true) ||
+            (isset($overlapstartexists) && $overlapstartexists == true) ||
+            (isset($overlapendexists) && $overlapendexists == true) ||
+            (isset($overlapbothexists) && $overlapbothexists == true)) {
         return true;
 
         // Otherwise.