fix(files_reminders): Always respect json output option

Signed-off-by: Christopher Ng <chrng8@gmail.com>
3 years ago · 6d2292fe1d
parent 0d3297db28
commit 6d2292fe1d
1 changed files with 5 additions and 4 deletions
--- a/apps/files_reminders/lib/Command/ListCommand.php
+++ b/apps/files_reminders/lib/Command/ListCommand.php
@ -77,10 +77,6 @@ class ListCommand extends Base {
 		}

 		$reminders = $this->reminderService->getAll($user ?? null);
-		if (empty($reminders)) {
-			$io->text('No reminders');
-			return 0;
-		}

 		$outputOption = $input->getOption('output');
 		switch ($outputOption) {
@ -97,6 +93,11 @@ class ListCommand extends Base {
 				);
 				return 0;
 			default:
+				if (empty($reminders)) {
+					$io->text('No reminders');
+					return 0;
+				}
+
 				$io->table(
 					['User Id', 'File Id', 'Path', 'Due Date', 'Updated At', 'Created At', 'Notified'],
 					array_map(