Allow skipping empty versions if null

Author: cannycookie

src/Diff.php

@@ -68,7 +68,7 @@ public function toSideBySideHtml(array $differOptions = [], array $renderOptions
 
     protected function getContents(bool $stripTags = false): array
     {
-        $newContents = $this->newVersion->contents;
+        $newContents = $this->newVersion->contents ?? [];
 
         // if the version strategy is DIFF, we need to merge the contents of all versions
         // from v1 to v2, v2 to v3, ..., vn-1 to vn.
@@ -85,12 +85,12 @@ protected function getContents(bool $stripTags = false): array
 
             $oldContents = Arr::only($oldContents, array_keys($newContents));
         } else {
-            $oldContents = $this->oldVersion->contents;
+            $oldContents = $this->oldVersion->contents ?? [];
         }
 
         if ($stripTags) {
-            $oldContents = array_map(fn ($item) => strip_tags($item), $oldContents);
-            $newContents = array_map(fn ($item) => strip_tags($item), $newContents);
+            $oldContents = array_map(fn ($item) => strip_tags($item), $oldContents ?? []);
+            $newContents = array_map(fn ($item) => strip_tags($item), $newContents ?? []);
         }
 
         return [$oldContents, $newContents];

src/Versionable.php

@@ -20,6 +20,7 @@ trait Versionable
     // You can add these properties to you versionable model
     // protected $versionable = [];
     // protected $dontVersionable = ['*'];
+    // protected bool $skipEmptyVersions = false; // Skip creating versions when all versionable attributes are empty
 
     // You can define this variable in class, that used this trait to change Model(table) for versions
     // Model MUST extend \Visualbuilder\Versionable\Version
@@ -244,6 +245,23 @@ public function shouldBeVersioning(): bool
 
         $versionableAttributes = $this->getVersionableAttributes($this->getVersionStrategy());
 
+        // Check if we should skip empty versions
+        if (property_exists($this, 'skipEmptyVersions') && $this->skipEmptyVersions) {
+            // Check if all versionable attributes are empty
+            $hasContent = false;
+            foreach ($versionableAttributes as $key => $value) {
+                if (! empty($value)) {
+                    $hasContent = true;
+                    break;
+                }
+            }
+
+            // Don't create version if no content
+            if (! $hasContent) {
+                return false;
+            }
+        }
+
         return $this->versions()->count() === 0 || Arr::hasAny($this->getDirty(), array_keys($versionableAttributes));
     }