Merge pull request #118 from fmarquesto/count-cache-support-nullable

Count cache support nullable

src/Behaviours/CountCache/CountCache.php

@@ -44,7 +44,11 @@ public function update(): void
             $originalRelatedModel = $config->emptyRelatedModel($this->model)->find($this->model->getOriginal($foreignKey));
 
             $this->updateCacheValue($originalRelatedModel, $config, -1);
-            $this->updateCacheValue($config->relatedModel($this->model), $config, 1);
+
+            // if the relation is null, then we don't need to do anything else.
+            if($this->model->{$foreignKey}) {
+                $this->updateCacheValue($config->relatedModel($this->model), $config, 1);
+            }
         });
     }
 

tests/Acceptance/CountCacheTest.php

@@ -43,4 +43,18 @@ public function testItCanHandleModelRestoration()
 
         $this->assertEquals(1, $post->fresh()->commentCount);
     }
+
+    public function testItCanHandleNullableRelation()
+    {
+        $user1 = User::factory()->create();
+        $posts = Post::factory()->count(2)->for($user1)->create();
+
+        $this->assertEquals(2, $user1->fresh()->postCount);
+
+        $firstPost = $posts->first()->fresh();
+        $firstPost->userId = null;
+        $firstPost->save();
+
+        $this->assertEquals(1, $user1->fresh()->postCount);
+    }
 }