BUGFIX: correctly test for settings

test for the existence of the `host`, `protocol`, `token_auth` and `idSite` settings
Flowpack · Jun 30, 2020 · e5cb6ac · e5cb6ac
1 parent 97efb90
commit e5cb6ac
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Classes/Service/Reporting.php b/Classes/Service/Reporting.php
@@ -92,7 +92,7 @@ public function callAPI(
         bool $useCache = true,
         string $sitename = ''
     ): ?array {
-        if (!empty($this->settings['host']) && !empty($this->settings['token_auth'] && !empty($this->settings['token_auth']))) {
+        if (!empty($this->settings['host']) && !empty($this->settings['protocol']) && !empty($this->settings['token_auth']) && !empty($this->settings['idSite'])) {
             $apiCallUrl = $this->buildApiCallUrl($sitename, array_merge($arguments, ['method' => $methodName]));
             return $this->request($apiCallUrl, $useCache);
         }
@@ -114,7 +114,7 @@ public function getNodeStatistics(
         array $arguments = [],
         $useCache = true
     ): ?AbstractDataResult {
-        if (!empty($this->settings['host']) && !empty($this->settings['token_auth'] && !empty($this->settings['token_auth']))) {
+        if (!empty($this->settings['host']) && !empty($this->settings['protocol']) && !empty($this->settings['token_auth']) && !empty($this->settings['idSite'])) {
             $contextProperties = $node->getContext()->getProperties();
             $contextProperties['workspaceName'] = 'live';