Merge pull request #345 from Flowpack/bugfix/response-is-an-object

BUGFIX: Trying to use response from exception as string

Author: daniellienert

Classes/Command/NodeIndexCommandController.php

@@ -320,7 +320,7 @@ public function cleanupCommand(): void
                 $this->logger->info('Nothing to remove.', LogEnvironment::fromMethodName(__METHOD__));
             }
         } catch (ApiException $exception) {
-            $response = json_decode($exception->getResponse());
+            $response = json_decode($exception->getResponse()->getBody()->getContents());
             if ($response->error instanceof \stdClass) {
                 $this->logger->info(sprintf('Nothing removed. ElasticSearch responded with status %s, saying "%s: %s"', $response->status, $response->error->type, $response->error->reason), LogEnvironment::fromMethodName(__METHOD__));
             } else {

Classes/Indexer/NodeIndexer.php

@@ -219,7 +219,8 @@ public function indexNode(NodeInterface $node, $targetWorkspaceName = null)
                 $this->logger->debug(sprintf('NodeIndexer (%s) - Property "%s" not indexed because no configuration found, node type %s.', $documentIdentifier, $propertyName, $node->getNodeType()->getName()), LogEnvironment::fromMethodName(__METHOD__));
             });
 
-            $document = new ElasticSearchDocument($mappingType,
+            $document = new ElasticSearchDocument(
+                $mappingType,
                 $nodePropertiesToBeStoredInIndex,
                 $documentIdentifier
             );