Merge pull request #55 from fedeisas/patch-1

Save a COUNT query if all records should be kept
mpociot · Nov 8, 2020 · f3ad4b4 · f3ad4b4
2 parents e21bded + ef5832a
commit f3ad4b4
Showing 1 changed file with 14 additions and 11 deletions.
diff --git a/src/Mpociot/Versionable/VersionableTrait.php b/src/Mpociot/Versionable/VersionableTrait.php
@@ -195,17 +195,20 @@ protected function versionablePostSave()
     private function purgeOldVersions()
     {
         $keep = isset($this->keepOldVersions) ? $this->keepOldVersions : 0;
-        $count = $this->versions()->count();
-
-        if ((int)$keep > 0 && $count > $keep) {
-            $oldVersions = $this->versions()
-                ->latest()
-                ->take($count)
-                ->skip($keep)
-                ->get()
-                ->each(function ($version) {
-                $version->delete();
-            });
+
+        if ((int)$keep > 0) {
+            $count = $this->versions()->count();
+
+            if ($count > $keep) {
+                $oldVersions = $this->versions()
+                    ->latest()
+                    ->take($count)
+                    ->skip($keep)
+                    ->get()
+                    ->each(function ($version) {
+                    $version->delete();
+                });
+            }
         }
     }