Noch mehr exeption werden gefangen und die errors verhalten sich jetzt wie beschrieben

2 files changed, 14 insertions, 10 deletions

diff --git a/modules-available/locationinfo/inc/coursebackend/coursebackend_davinci.inc.php b/modules-available/locationinfo/inc/coursebackend/coursebackend_davinci.inc.php
index 3343d124..523823bf 100644
--- a/modules-available/locationinfo/inc/coursebackend/coursebackend_davinci.inc.php
+++ b/modules-available/locationinfo/inc/coursebackend/coursebackend_davinci.inc.php
@@ -54,7 +54,7 @@ class Coursebackend_Davinci extends CourseBackend
 		} catch (Exception $exception) {
 			$this->error = true;
 			$this->errormsg = "url did not send a xml";
-			$array = [];
+			$array = false;
 		}
 		return $array;
 	}
@@ -95,8 +95,8 @@ class Coursebackend_Davinci extends CourseBackend
 		try {
 			foreach ($roomIds as $sroomId) {
 				$return = $this->fetchArray($sroomId);
-				if ($this->error) {
-					return $schedules;
+				if ($return === false) {
+					return false;
 				}
 				$lessons = $return['Lessons']['Lesson'];
 				$timetable = [];
@@ -121,6 +121,7 @@ class Coursebackend_Davinci extends CourseBackend
 		} catch (Exception $e) {
 			$this->error = true;
 			$this->errormsg = "url returns a wrong xml";
+			return false;
 		}
 
 		return $schedules;
diff --git a/modules-available/locationinfo/inc/coursebackend/coursebackend_hisinone.inc.php b/modules-available/locationinfo/inc/coursebackend/coursebackend_hisinone.inc.php
index fddd3184..ac958b2c 100644
--- a/modules-available/locationinfo/inc/coursebackend/coursebackend_hisinone.inc.php
+++ b/modules-available/locationinfo/inc/coursebackend/coursebackend_hisinone.inc.php
@@ -139,7 +139,7 @@ class CourseBackend_HisInOne extends CourseBackend
 			}
 		} catch (Exception $exception) {
 			if ($this->error) {
-				$id = [];
+				$id = false;
 			} else {
 				$this->error = true;
 				$this->errormsg = "url send a xml in a wrong format";
@@ -190,6 +190,7 @@ class CourseBackend_HisInOne extends CourseBackend
 				} catch (Exception $e) {
 					$this->error = true;
 					$this->errormsg = "url send a xml in a wrong format";
+					return false;
 				}
 			}
 		} else {
@@ -240,21 +241,21 @@ class CourseBackend_HisInOne extends CourseBackend
 	 * Request for a timetable
 	 *
 	 * @param $param int the roomid
-	 * @return string the timetable as json
+	 * @return string the timetable as json or false if there was an error
 	 */
 	public function getJson($param)
 	{
 		//get all eventIDs in a given room
 		$eventIDs = $this->findUnit($param);
 		if ($this->error == true) {
-			return "[]";
+			return false;
 		}
 		//get all information on each event
 		$events = [];
 		foreach ($eventIDs as $each_event) {
 			$events[] = $this->readUnit((int)$each_event);
 			if ($this->error == true) {
-				return "[]";
+				return false;
 			}
 		}
 		$timetable = array();
@@ -314,6 +315,7 @@ class CourseBackend_HisInOne extends CourseBackend
 		} catch (Exception $e) {
 			$this->error = true;
 			$this->errormsg = "url returns a wrong xml";
+			return false;
 		}
 		$timetable = json_encode($timetable);
 		return $timetable;
@@ -334,7 +336,7 @@ class CourseBackend_HisInOne extends CourseBackend
 		return $array;
 	}
 
-	//Request for a timetable with roomids as array it will be empty if there was an error
+	//Request for a timetable with roomids as array it will be boolean false if there was an error
 	public function fetchSchedulesInternal($param)
 	{
 		$tTables = [];
@@ -345,14 +347,14 @@ class CourseBackend_HisInOne extends CourseBackend
 			var_dump($eventIDs);
 			$eventIDs = array_unique($eventIDs);
 			if ($this->error == true) {
-				return $tTables;
+				return false;
 			}
 		}
 		//get all information on each event
 		foreach ($eventIDs as $each_event) {
 			$events[] = $this->readUnit(intval($each_event));
 			if ($this->error == true) {
-				return $tTables;
+				return false;
 			}
 		}
 		$currentWeek = $this->getCurrentWeekDates();
@@ -412,6 +414,7 @@ class CourseBackend_HisInOne extends CourseBackend
 		} catch (Exception $e) {
 			$this->error = true;
 			$this->errormsg = "url returns a wrong xml";
+			return false;
 		}
 		return $tTables;
 	}