extract checks to separate function

Author: rcerljenko

src/Tasks/Cleanup/Strategies/DefaultStrategy.php

@@ -95,19 +95,33 @@ protected function removeBackupsOlderThan(Carbon $endDate, BackupCollection $bac
 
     protected function removeOldBackupsUntilUsingLessThanMaximumStorage(BackupCollection $backups)
     {
-        if (! $oldest = $backups->oldest()) {
+        if (! $this->shouldRemoveOldestBackup($backups)) {
             return;
         }
 
+        $backups->oldest()->delete();
+
+        $backups = $backups->filter(fn (Backup $backup) => $backup->exists());
+
+        $this->removeOldBackupsUntilUsingLessThanMaximumStorage($backups);
+    }
+
+    protected function shouldRemoveOldestBackup(BackupCollection $backups): bool
+    {
+        if (! $backups->oldest()) {
+            return false;
+        }
+
         $maximumSize = $this->config->get('backup.cleanup.default_strategy.delete_oldest_backups_when_using_more_megabytes_than');
 
-        if ($maximumSize === null || ($backups->size() + $this->newestBackup->sizeInBytes()) <= $maximumSize * 1024 * 1024) {
-            return;
+        if ($maximumSize === null) {
+            return false;
         }
-        $oldest->delete();
 
-        $backups = $backups->filter(fn (Backup $backup) => $backup->exists());
+        if (($backups->size() + $this->newestBackup->sizeInBytes()) <= $maximumSize * 1024 * 1024) {
+            return false;
+        }
 
-        $this->removeOldBackupsUntilUsingLessThanMaximumStorage($backups);
+        return true;
     }
 }