More robust calling of file_get_contents

slub · Jul 24, 2024 · 879a0fd · 879a0fd
1 parent ea78222
commit 879a0fd
Showing 1 changed file with 19 additions and 9 deletions.
diff --git a/Classes/Controller/BibliografieController.php b/Classes/Controller/BibliografieController.php
@@ -225,20 +225,30 @@ public function getApiResults($action='items') : array
 
     public function request($url, $requestCount=0)
     {
-        $json = file_get_contents($url);
-        $data = json_decode($json, true);
-        $this->reIndexArray($data);
-
-        if ($requestCount==0 && count($data)==$this->settings['zotero']['limit']) {
-            // load only one page more
-            $url .= '&start='.$this->settings['zotero']['limit'];
-            $dataNext = $this->request($url, 1);
-            $data = array_merge($data, $dataNext);
+        $data = '',
+
+        if ($this->settings['debug'] == true) {
+            DebugUtility::debug($url, 'Debug: ' . __FILE__ . ' in Line: ' . __LINE__ . ' Function: '. __FUNCTION__);
+        }
+
+        $json = @file_get_contents($url);
+
+        if($json !== FALSE) {
+            $data = json_decode($json, true);
+            $this->reIndexArray($data);
+
+            if ($requestCount==0 && count($data)==$this->settings['zotero']['limit']) {
+                // load only one page more
+                $url .= '&start='.$this->settings['zotero']['limit'];
+                $dataNext = $this->request($url, 1);
+                $data = array_merge($data, $dataNext);
+            }
         }
 
         if ($this->settings['debug'] == true) {
             DebugUtility::debug($data, 'Debug: ' . __FILE__ . ' in Line: ' . __LINE__ . ' Function: '. __FUNCTION__);
         }
+
         return $data;
     }