confirm pruning

matfish2 · Jul 8, 2023 · ad7be23 · ad7be23
1 parent 7dfb38e
commit ad7be23
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Release Notes for Activity Log
 
+## 1.5.2 - 2023-07-08
+### Improved
+- Add a confirmation message to the prune command with number of days and date
+
 ## 1.5.1 - 2023-07-05
 ### Added
 - Add Payload Search (optional field, enable in Settings page)

diff --git a/src/controllers/console/LogsController.php b/src/controllers/console/LogsController.php
@@ -18,13 +18,19 @@ public function options($actionID): array
 
     public function actionPrune()
     {
-        $cutoff = Carbon::today()->subDays($this->days ?? 30);
+        $days = $this->days;
 
-        $this->stdout('Pruning records before ' . $cutoff->format('d-m-Y') . '...' . PHP_EOL, Console::FG_GREEN);
+        $cutoff = Carbon::today()->subDays($days);
 
-        ActivityLog::deleteAll(['<', 'createdAt', $cutoff->format('Y-m-d')]);
+        if ($this->confirm("Are you sure you want to permanently delete all records before the last {$days} days (" .$cutoff->format('d-m-Y') . ")?")) {
+            $this->stdout('Pruning records before ' . $cutoff->format('d-m-Y') . '...' . PHP_EOL, Console::FG_GREEN);
 
-        $this->stdout('Done!' . PHP_EOL, Console::FG_GREEN);
+            ActivityLog::deleteAll(['<', 'createdAt', $cutoff->format('Y-m-d')]);
+
+            $this->stdout('Done!' . PHP_EOL, Console::FG_GREEN);
+        } else {
+            $this->stdout('Aborted' . PHP_EOL, Console::FG_YELLOW);
+        }
 
         return 1;
     }