MDL-69758 Web services: core_get_course_contents PHP warnings
authorsam marshall <s.marshall@open.ac.uk>
Mon, 21 Sep 2020 14:24:10 +0000 (15:24 +0100)
committersam marshall <s.marshall@open.ac.uk>
Mon, 21 Sep 2020 14:24:12 +0000 (15:24 +0100)
Calls to core_get_course_contents can cause PHP warnings in the server
log if there are any URL instances in the course with sketchy URLs.
This is because url_export_contents incorrectly returns null instead
of an empty array if there is an error.

mod/url/lib.php

index 815360d..b805029 100644 (file)
@@ -274,7 +274,7 @@ function url_export_contents($cm, $baseurl) {
     $fullurl = str_replace('&amp;', '&', url_get_full_url($urlrecord, $cm, $course));
     $isurl = clean_param($fullurl, PARAM_URL);
     if (empty($isurl)) {
-        return null;
+        return [];
     }
 
     $url = array();