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;
  }