check if index exists before deleting it

limenet · limenet · commit 5ed4d824af51 · 2022-12-16T10:53:54.000+01:00
diff --git a/src/Jobs/SetupIndex.php b/src/Jobs/SetupIndex.php
@@ -43,7 +43,7 @@ private function migrate(ElasticaClient $elastica): void
         try {
             $response = $elastica->getClient()->request(sprintf('_alias/%s', $this->indexConfig->getName()));
         } catch (ClientException|ConnectionException|ResponseException) {
-            if (count($index->getAliases()) === 0) {
+            if ($index->exists() && count($index->getAliases()) === 0) {
                 $index->delete();
             }
 
@@ -58,7 +58,9 @@ private function migrate(ElasticaClient $elastica): void
             return;
         }
 
-        $index->delete();
+        if ($index->exists()) {
+            $index->delete();
+        }
     }
 
     private function cleanup(ElasticaClient $elastica): void

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ private function migrate(ElasticaClient $elastica): void`
`43`	`43`	`try {`
`44`	`44`	`$response = $elastica->getClient()->request(sprintf('_alias/%s', $this->indexConfig->getName()));`
`45`	`45`	`} catch (ClientException\|ConnectionException\|ResponseException) {`
`46`		`- if (count($index->getAliases()) === 0) {`
	`46`	`+ if ($index->exists() && count($index->getAliases()) === 0) {`
`47`	`47`	`$index->delete();`
`48`	`48`	`}`
`49`	`49`
`@@ -58,7 +58,9 @@ private function migrate(ElasticaClient $elastica): void`
`58`	`58`	`return;`
`59`	`59`	`}`
`60`	`60`
`61`		`- $index->delete();`
	`61`	`+ if ($index->exists()) {`
	`62`	`+ $index->delete();`
	`63`	`+ }`
`62`	`64`	`}`
`63`	`65`
`64`	`66`	`private function cleanup(ElasticaClient $elastica): void`