Merge pull request #779 from otherpaco/cleanup-cache

clear the backup-size cache (fixes #776)
spatie · Oct 29, 2018 · 0aaed02 · 0aaed02
2 parents fc5aa95 + 95e166d
commit 0aaed02
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/BackupDestination/BackupDestination.php b/src/BackupDestination/BackupDestination.php
@@ -155,4 +155,11 @@ public function newestBackupIsOlderThan(Carbon $date): bool
 
         return $newestBackup->date()->gt($date);
     }
+
+    public function fresh(): self
+    {
+        $this->backupCollectionCache = null;
+
+        return $this;
+    }
 }
diff --git a/src/Tasks/Cleanup/CleanupJob.php b/src/Tasks/Cleanup/CleanupJob.php
@@ -42,7 +42,7 @@ public function run()
                 $this->strategy->deleteOldBackups($backupDestination->backups());
                 $this->sendNotification(new CleanupWasSuccessful($backupDestination));
 
-                $usedStorage = Format::humanReadableSize($backupDestination->usedStorage());
+                $usedStorage = Format::humanReadableSize($backupDestination->fresh()->usedStorage());
                 consoleOutput()->info("Used storage after cleanup: {$usedStorage}.");
             } catch (Exception $exception) {
                 consoleOutput()->error("Cleanup failed because: {$exception->getMessage()}.");