fix: query is for query

theus77 · Feb 1, 2024 · 7183dea · 7183dea
1 parent c9d0142
commit 7183dea
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/EMS/core-bundle/src/Command/Revision/DeleteCommand.php b/EMS/core-bundle/src/Command/Revision/DeleteCommand.php
@@ -74,8 +74,12 @@ protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $this->io->title('EMSCO - Revision - Delete');
 
+        $queryJson = $this->getOptionStringNull(self::OPTION_QUERY);
         if (self::MODE_BY_QUERY === $this->mode) {
-            return $this->deleteByQuery();
+            return $this->deleteByQuery($queryJson);
+        }
+        if (null !== $queryJson) {
+            throw new \RuntimeException(\sprintf('The %s option is forbidden in %s mode', self::OPTION_QUERY, $this->mode));
         }
 
         $this->io->note(\sprintf('Selected "%s" contentType(s)', \implode(',', $this->contentTypeNames)));
@@ -101,9 +105,8 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         return parent::EXECUTE_SUCCESS;
     }
 
-    private function deleteByQuery(): int
+    private function deleteByQuery(?string $queryJson): int
     {
-        $queryJson = $this->getOptionStringNull(self::OPTION_QUERY);
         if (null === $queryJson) {
             throw new \RuntimeException(\sprintf('The %s option is required in %s mode', self::OPTION_QUERY, $this->mode));
         }