Replaced Command::(SUCCESS|FAILURE) with self::(SUCCESS|FAILURE)

alongosz · alongosz · commit 51f3ced8fe20 · 2024-08-19T15:52:38.000+02:00
diff --git a/src/bundle/Core/Command/CheckURLsCommand.php b/src/bundle/Core/Command/CheckURLsCommand.php
@@ -106,7 +106,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         }
         $progress->finish();
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     private function getTotalCount(): int
diff --git a/src/bundle/Core/Command/CleanupVersionsCommand.php b/src/bundle/Core/Command/CleanupVersionsCommand.php
@@ -148,7 +148,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         if ($contentIdsCount === 0) {
             $output->writeln('<info>There is no content matching the given Criteria.</info>');
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $output->writeln(sprintf(
@@ -229,7 +229,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             $contentIdsCount
         ));
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     /**
diff --git a/src/bundle/Core/Command/CopySubtreeCommand.php b/src/bundle/Core/Command/CopySubtreeCommand.php
@@ -152,7 +152,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         );
 
         if (!$input->getOption('no-interaction') && !$questionHelper->ask($input, $output, $question)) {
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $this->locationService->copySubtree(
@@ -164,7 +164,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             '<info>Finished</info>'
         );
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     /**
diff --git a/src/bundle/Core/Command/DebugConfigResolverCommand.php b/src/bundle/Core/Command/DebugConfigResolverCommand.php
@@ -97,7 +97,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         if ($input->getOption('json')) {
             $output->write(json_encode($parameterData));
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $output->writeln('<comment>SiteAccess name:</comment> ' . $this->siteAccess->name);
@@ -112,6 +112,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             )
         );
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 }
diff --git a/src/bundle/Core/Command/DeleteContentTranslationCommand.php b/src/bundle/Core/Command/DeleteContentTranslationCommand.php
@@ -127,7 +127,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 $this->repository->rollback();
                 $this->output->writeln('Reverting and aborting.');
 
-                return Command::SUCCESS;
+                return self::SUCCESS;
             }
 
             // Delete Translation
@@ -144,7 +144,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             throw $e;
         }
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     /**
diff --git a/src/bundle/Core/Command/ExpireUserPasswordsCommand.php b/src/bundle/Core/Command/ExpireUserPasswordsCommand.php
@@ -148,7 +148,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         if ($totalCount === 0) {
             $output->writeln('<info>There are no users matching given criteria</info>');
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $output->writeln(sprintf(
@@ -213,7 +213,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             );
         }
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     /**
diff --git a/src/bundle/Core/Command/NormalizeImagesPathsCommand.php b/src/bundle/Core/Command/NormalizeImagesPathsCommand.php
@@ -113,11 +113,11 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         if ($imagePathsToNormalizeCount === 0) {
             $io->success('No paths to normalize.');
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         if (!$io->confirm('Do you want to continue?')) {
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $io->writeln('Normalizing image paths. Please wait...');
@@ -136,7 +136,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $io->progressFinish();
         $io->success('Done!');
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     /**
diff --git a/src/bundle/Core/Command/RegenerateUrlAliasesCommand.php b/src/bundle/Core/Command/RegenerateUrlAliasesCommand.php
@@ -126,7 +126,7 @@ static function (Repository $repository): int {
         if ($locationsCount === 0) {
             $output->writeln('<info>No location was found. Exiting.</info>');
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         if (!$input->getOption('no-interaction')) {
@@ -140,10 +140,10 @@ static function (Repository $repository): int {
                 false
             );
             if (!$helper->ask($input, $output, $question)) {
-                return Command::SUCCESS;
+                return self::SUCCESS;
             }
         } elseif (!$input->getOption('force')) {
-            return Command::FAILURE;
+            return self::FAILURE;
         }
 
         $this->regenerateSystemUrlAliases($output, $locationsCount, $locationIds, $iterationCount);
@@ -157,7 +157,7 @@ static function (Repository $repository): int {
         $output->writeln("<info>Done. Deleted {$corruptedAliasesCount} entries.</info>");
         $output->writeln('<comment>Make sure to clear HTTP cache.</comment>');
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     /**
diff --git a/src/bundle/Core/Command/ReindexCommand.php b/src/bundle/Core/Command/ReindexCommand.php
@@ -222,7 +222,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 EOT);
 
                 if (!$io->confirm('Continue?', true)) {
-                    return Command::SUCCESS;
+                    return self::SUCCESS;
                 }
             }
 
@@ -232,7 +232,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             return $this->indexIncrementally($input, $output, $iterationCount, $commit);
         }
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     /**
diff --git a/src/bundle/Core/Command/ResizeOriginalImagesCommand.php b/src/bundle/Core/Command/ResizeOriginalImagesCommand.php
@@ -157,7 +157,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 )
             );
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         try {
@@ -170,7 +170,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 )
             );
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $query = new Query();
@@ -195,13 +195,13 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 )
             );
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $helper = $this->getHelper('question');
         $question = new ConfirmationQuestion('<question>The changes you are going to perform cannot be undone. Remember to do a proper backup before. Would you like to continue?</question> ', false);
         if (!$helper->ask($input, $output, $question)) {
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $progressBar = new ProgressBar($output, $totalCount);
@@ -228,7 +228,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             )
         );
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     /**
diff --git a/src/bundle/Core/Command/UpdateTimestampsToUTCCommand.php b/src/bundle/Core/Command/UpdateTimestampsToUTCCommand.php
@@ -153,17 +153,17 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 sprintf('The selected mode is not supported. Use one of the following modes: %s', implode(', ', array_keys(self::MODES)))
             );
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $from = $input->getOption('from');
         $to = $input->getOption('to');
 
         if ($from && !$this->validateDateTimeString($from, $output)) {
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
         if ($to && !$this->validateDateTimeString($to, $output)) {
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
         if ($from) {
             $this->from = $this->dateStringToTimestamp($from);
@@ -195,7 +195,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             if ($count == 0) {
                 $output->writeln('Nothing to process, exiting.');
 
-                return Command::SUCCESS;
+                return self::SUCCESS;
             }
 
             $helper = $this->getHelper('question');
@@ -207,7 +207,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             if (!$helper->ask($input, $output, $question)) {
                 $output->writeln('');
 
-                return Command::SUCCESS;
+                return self::SUCCESS;
             }
 
             $progressBar = $this->getProgressBar($count, $output);
@@ -256,7 +256,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             ]);
         }
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     /**
diff --git a/src/bundle/IO/Command/MigrateFilesCommand.php b/src/bundle/IO/Command/MigrateFilesCommand.php
@@ -98,14 +98,14 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         if ($input->getOption('list-io-handlers')) {
             $this->outputConfiguredHandlers($output);
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $bulkCount = (int)$input->getOption('bulk-count');
         if ($bulkCount < 1) {
             $output->writeln('The value for --bulk-count must be a positive integer.');
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $output->writeln($this->getProcessedHelp());
@@ -124,7 +124,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         }
 
         if (!$this->validateHandlerOptions($fromHandlers, $toHandlers, $output)) {
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         $output->writeln([
@@ -159,7 +159,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         if ($totalCount === 0) {
             $output->writeln('Nothing to process.');
 
-            return Command::SUCCESS;
+            return self::SUCCESS;
         }
 
         if (!$input->getOption('no-interaction')) {
@@ -172,7 +172,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             if (!$helper->ask($input, $output, $question)) {
                 $output->writeln('Aborting.');
 
-                return Command::SUCCESS;
+                return self::SUCCESS;
             }
         }
 
@@ -183,7 +183,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             $output
         );
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     /**
diff --git a/src/bundle/RepositoryInstaller/Command/InstallPlatformCommand.php b/src/bundle/RepositoryInstaller/Command/InstallPlatformCommand.php
@@ -88,7 +88,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         if (!empty($schemaManager->listTables())) {
             $io = new SymfonyStyle($input, $output);
             if (!$io->confirm('Running this command will delete data in all Ibexa generated tables. Continue?')) {
-                return Command::SUCCESS;
+                return self::SUCCESS;
             }
         }
 
@@ -114,7 +114,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             $this->indexData($output, $siteaccess);
         }
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 
     private function checkPermissions()
diff --git a/src/bundle/RepositoryInstaller/Command/ValidatePasswordHashesCommand.php b/src/bundle/RepositoryInstaller/Command/ValidatePasswordHashesCommand.php
@@ -46,6 +46,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             $output->writeln('OK - <info>All users have supported password hash types</info>');
         }
 
-        return Command::SUCCESS;
+        return self::SUCCESS;
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`106`	`106`	`}`
`107`	`107`	`$progress->finish();`
`108`	`108`
`109`		`- return Command::SUCCESS;`
	`109`	`+ return self::SUCCESS;`
`110`	`110`	`}`
`111`	`111`
`112`	`112`	`private function getTotalCount(): int`
Original file line number	Diff line number	Diff line change
`@@ -148,7 +148,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`148`	`148`	`if ($contentIdsCount === 0) {`
`149`	`149`	`$output->writeln('<info>There is no content matching the given Criteria.</info>');`
`150`	`150`
`151`		`- return Command::SUCCESS;`
	`151`	`+ return self::SUCCESS;`
`152`	`152`	`}`
`153`	`153`
`154`	`154`	`$output->writeln(sprintf(`
`@@ -229,7 +229,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`229`	`229`	`$contentIdsCount`
`230`	`230`	`));`
`231`	`231`
`232`		`- return Command::SUCCESS;`
	`232`	`+ return self::SUCCESS;`
`233`	`233`	`}`
`234`	`234`
`235`	`235`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -152,7 +152,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`152`	`152`	`);`
`153`	`153`
`154`	`154`	`if (!$input->getOption('no-interaction') && !$questionHelper->ask($input, $output, $question)) {`
`155`		`- return Command::SUCCESS;`
	`155`	`+ return self::SUCCESS;`
`156`	`156`	`}`
`157`	`157`
`158`	`158`	`$this->locationService->copySubtree(`
`@@ -164,7 +164,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`164`	`164`	`'<info>Finished</info>'`
`165`	`165`	`);`
`166`	`166`
`167`		`- return Command::SUCCESS;`
	`167`	`+ return self::SUCCESS;`
`168`	`168`	`}`
`169`	`169`
`170`	`170`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`97`	`97`	`if ($input->getOption('json')) {`
`98`	`98`	`$output->write(json_encode($parameterData));`
`99`	`99`
`100`		`- return Command::SUCCESS;`
	`100`	`+ return self::SUCCESS;`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`$output->writeln('<comment>SiteAccess name:</comment> ' . $this->siteAccess->name);`
`@@ -112,6 +112,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`112`	`112`	`)`
`113`	`113`	`);`
`114`	`114`
`115`		`- return Command::SUCCESS;`
	`115`	`+ return self::SUCCESS;`
`116`	`116`	`}`
`117`	`117`	`}`
Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`127`	`127`	`$this->repository->rollback();`
`128`	`128`	`$this->output->writeln('Reverting and aborting.');`
`129`	`129`
`130`		`- return Command::SUCCESS;`
	`130`	`+ return self::SUCCESS;`
`131`	`131`	`}`
`132`	`132`
`133`	`133`	`// Delete Translation`
`@@ -144,7 +144,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`144`	`144`	`throw $e;`
`145`	`145`	`}`
`146`	`146`
`147`		`- return Command::SUCCESS;`
	`147`	`+ return self::SUCCESS;`
`148`	`148`	`}`
`149`	`149`
`150`	`150`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -113,11 +113,11 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`113`	`113`	`if ($imagePathsToNormalizeCount === 0) {`
`114`	`114`	`$io->success('No paths to normalize.');`
`115`	`115`
`116`		`- return Command::SUCCESS;`
	`116`	`+ return self::SUCCESS;`
`117`	`117`	`}`
`118`	`118`
`119`	`119`	`if (!$io->confirm('Do you want to continue?')) {`
`120`		`- return Command::SUCCESS;`
	`120`	`+ return self::SUCCESS;`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`$io->writeln('Normalizing image paths. Please wait...');`
`@@ -136,7 +136,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`136`	`136`	`$io->progressFinish();`
`137`	`137`	`$io->success('Done!');`
`138`	`138`
`139`		`- return Command::SUCCESS;`
	`139`	`+ return self::SUCCESS;`
`140`	`140`	`}`
`141`	`141`
`142`	`142`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -126,7 +126,7 @@ static function (Repository $repository): int {`
`126`	`126`	`if ($locationsCount === 0) {`
`127`	`127`	`$output->writeln('<info>No location was found. Exiting.</info>');`
`128`	`128`
`129`		`- return Command::SUCCESS;`
	`129`	`+ return self::SUCCESS;`
`130`	`130`	`}`
`131`	`131`
`132`	`132`	`if (!$input->getOption('no-interaction')) {`
`@@ -140,10 +140,10 @@ static function (Repository $repository): int {`
`140`	`140`	`false`
`141`	`141`	`);`
`142`	`142`	`if (!$helper->ask($input, $output, $question)) {`
`143`		`- return Command::SUCCESS;`
	`143`	`+ return self::SUCCESS;`
`144`	`144`	`}`
`145`	`145`	`} elseif (!$input->getOption('force')) {`
`146`		`- return Command::FAILURE;`
	`146`	`+ return self::FAILURE;`
`147`	`147`	`}`
`148`	`148`
`149`	`149`	`$this->regenerateSystemUrlAliases($output, $locationsCount, $locationIds, $iterationCount);`
`@@ -157,7 +157,7 @@ static function (Repository $repository): int {`
`157`	`157`	`$output->writeln("<info>Done. Deleted {$corruptedAliasesCount} entries.</info>");`
`158`	`158`	`$output->writeln('<comment>Make sure to clear HTTP cache.</comment>');`
`159`	`159`
`160`		`- return Command::SUCCESS;`
	`160`	`+ return self::SUCCESS;`
`161`	`161`	`}`
`162`	`162`
`163`	`163`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -222,7 +222,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`222`	`222`	`EOT);`
`223`	`223`
`224`	`224`	`if (!$io->confirm('Continue?', true)) {`
`225`		`- return Command::SUCCESS;`
	`225`	`+ return self::SUCCESS;`
`226`	`226`	`}`
`227`	`227`	`}`
`228`	`228`
`@@ -232,7 +232,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`232`	`232`	`return $this->indexIncrementally($input, $output, $iterationCount, $commit);`
`233`	`233`	`}`
`234`	`234`
`235`		`- return Command::SUCCESS;`
	`235`	`+ return self::SUCCESS;`
`236`	`236`	`}`
`237`	`237`
`238`	`238`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -157,7 +157,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`157`	`157`	`)`
`158`	`158`	`);`
`159`	`159`
`160`		`- return Command::SUCCESS;`
	`160`	`+ return self::SUCCESS;`
`161`	`161`	`}`
`162`	`162`
`163`	`163`	`try {`
`@@ -170,7 +170,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`170`	`170`	`)`
`171`	`171`	`);`
`172`	`172`
`173`		`- return Command::SUCCESS;`
	`173`	`+ return self::SUCCESS;`
`174`	`174`	`}`
`175`	`175`
`176`	`176`	`$query = new Query();`
`@@ -195,13 +195,13 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`195`	`195`	`)`
`196`	`196`	`);`
`197`	`197`
`198`		`- return Command::SUCCESS;`
	`198`	`+ return self::SUCCESS;`
`199`	`199`	`}`
`200`	`200`
`201`	`201`	`$helper = $this->getHelper('question');`
`202`	`202`	`$question = new ConfirmationQuestion('<question>The changes you are going to perform cannot be undone. Remember to do a proper backup before. Would you like to continue?</question> ', false);`
`203`	`203`	`if (!$helper->ask($input, $output, $question)) {`
`204`		`- return Command::SUCCESS;`
	`204`	`+ return self::SUCCESS;`
`205`	`205`	`}`
`206`	`206`
`207`	`207`	`$progressBar = new ProgressBar($output, $totalCount);`
`@@ -228,7 +228,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int`
`228`	`228`	`)`
`229`	`229`	`);`
`230`	`230`
`231`		`- return Command::SUCCESS;`
	`231`	`+ return self::SUCCESS;`
`232`	`232`	`}`
`233`	`233`
`234`	`234`	`/**`