[locations] Cleanup Location class

There was a mess of functions which mostly, but not quite, did the same
things. Get rid of a couple of them and fix call sites to use
alternative ones that also fit the job.
While at it, add phpdoc and comments to the remaining functions, trying
to clarify what they were designed for.
Lastly, the return type of functions that retrieve a location id has
been changed from false|int to ?int (nullable types are just nicer).

1 files changed, 5 insertions, 1 deletions

diff --git a/modules-available/dozmod/api.inc.php b/modules-available/dozmod/api.inc.php
index 5a8f6302..11a542ef 100644
--- a/modules-available/dozmod/api.inc.php
+++ b/modules-available/dozmod/api.inc.php
@@ -343,7 +343,11 @@ if (substr($ip, 0, 7) === '::ffff:') {
 
 /* lookup location id(s) */
 $location_ids = Location::getFromIp($ip, true);
-$location_ids = Location::getLocationRootChain($location_ids);
+if ($location_ids === null) {
+	$location_ids = [];
+} else {
+	$location_ids = Location::getLocationRootChain($location_ids);
+}
 
 if ($resource === 'list') {
 	outputLectureXmlForLocation($location_ids);