From be0b7b6a2c2d082f0f07134323d73cf638cdc8de Mon Sep 17 00:00:00 2001 From: Jonas Date: Tue, 11 Jun 2024 16:07:29 +0200 Subject: [PATCH] feat(reset): Add `--all` option to `text:reset` occ command Allows to reset all editing sessions at once. Signed-off-by: Jonas --- lib/Command/ResetDocument.php | 56 ++++++++++++++++++++++++----------- 1 file changed, 39 insertions(+), 17 deletions(-) diff --git a/lib/Command/ResetDocument.php b/lib/Command/ResetDocument.php index 0864ccb68ec..4702db69abb 100644 --- a/lib/Command/ResetDocument.php +++ b/lib/Command/ResetDocument.php @@ -23,6 +23,7 @@ namespace OCA\Text\Command; +use OCA\Text\Db\Document; use OCA\Text\Exception\DocumentHasUnsavedChangesException; use OCA\Text\Service\DocumentService; use Symfony\Component\Console\Command\Command; @@ -44,9 +45,15 @@ protected function configure(): void { ->setDescription('Reset a text document session to the current file content') ->addArgument( 'file-id', - InputArgument::REQUIRED, + InputArgument::OPTIONAL, 'File id of the document to reset' ) + ->addOption( + 'all', + 'a', + null, + 'Reset all document sessions' + ) ->addOption( 'force', 'f', @@ -56,30 +63,45 @@ protected function configure(): void { ; } - /** - * @param InputInterface $input - * @param OutputInterface $output - * @return int - */ protected function execute(InputInterface $input, OutputInterface $output): int { $fileId = $input->getArgument('file-id'); + $all = $input->getOption('all'); $fullReset = $input->getOption('force'); - if ($fullReset) { - $output->writeln('Force-reset the document session for file ' . $fileId); - $this->documentService->resetDocument($fileId, true); + if (!$fileId && !$all) { + $output->writeln('Either --all option or file-id argument is required.'); + return 1; + } + if ($fileId && $all) { + $output->writeln('The --all option and file id argument are exclusionary.'); + return 1; + } - return 0; + if ($all) { + $fileIds = array_map(static function (Document $document) { + return $document->getId(); + }, $this->documentService->getAll()); + } else { + $fileIds = [$fileId]; } - $output->writeln('Reset the document session for file ' . $fileId); - try { - $this->documentService->resetDocument($fileId); - } catch (DocumentHasUnsavedChangesException) { - $output->writeln('Not resetting due to unsaved changes'); - return 1; + $rc = 0; + foreach ($fileIds as $id) { + if ($fullReset) { + $output->writeln('Force-reset the document session for file ' . $id); + $this->documentService->resetDocument($id, true); + continue; + } + + $output->writeln('Reset the document session for file ' . $id); + try { + $this->documentService->resetDocument($id); + } catch (DocumentHasUnsavedChangesException) { + $output->writeln('Not resetting due to unsaved changes'); + $rc = 1; + } } - return 0; + return $rc; } }