[locationinfo] Revert HisInOne calendar workaround

The hack introduced in 87686da817dfd349c9f89a01065f799408e29f10 to work
around HisInOne returning incomplete data in ical files is obsolete, now
that HisInOne seems to have fixed this bug.

1 files changed, 3 insertions, 4 deletions

diff --git a/modules-available/locationinfo/inc/icalcoursebackend.inc.php b/modules-available/locationinfo/inc/icalcoursebackend.inc.php
index c9113ec6..493d0008 100644
--- a/modules-available/locationinfo/inc/icalcoursebackend.inc.php
+++ b/modules-available/locationinfo/inc/icalcoursebackend.inc.php
@@ -38,17 +38,16 @@ abstract class ICalCourseBackend extends CourseBackend
 	 *
 	 * @param string $roomId The ID of the room for which iCal data is to be downloaded.
 	 * @param ?int $deadline Optional deadline timestamp for the download operation.
-	 * @param ?string $appendToUrl Optional appendage to the URL used for the download.
 	 *
 	 * @return ?ICalEvent[] An array of iCal events if successful, or null on failure.
 	 */
-	protected function downloadIcal(string $roomId, ?int $deadline, ?string $appendToUrl = null, ?bool $limitRange = true): ?array
+	protected function downloadIcal(string $roomId, ?int $deadline): ?array
 	{
 		if (!$this->isOK())
 			return null;
 
 		try {
-			$ical = new ICalParser($limitRange ? ['filterDaysBefore' => 7, 'filterDaysAfter' => 7] : []);
+			$ical = new ICalParser(['filterDaysBefore' => 7, 'filterDaysAfter' => 7]);
 		} catch (Exception $e) {
 			$this->addError('Error instantiating ICalParser: ' . $e->getMessage(), true);
 			return null;
@@ -70,7 +69,7 @@ abstract class ICalCourseBackend extends CourseBackend
 			CURLOPT_FOLLOWLOCATION => true,
 			CURLOPT_SSL_VERIFYHOST => $this->verifyHostname ? 2 : 0,
 			CURLOPT_SSL_VERIFYPEER => $this->verifyCert ? 1 : 0,
-			CURLOPT_URL => str_replace('%ID%', $roomId, $this->location . ($appendToUrl ?? '')),
+			CURLOPT_URL => str_replace('%ID%', $roomId, $this->location),
 			CURLOPT_TIMEOUT => min(20, $timeout),
 			CURLOPT_CONNECTTIMEOUT => 4,
 		];