Merge pull request #5 from GiampaoloFalqui/improvements

Adds reverse check of indexOnly to delete not-anymore-indexable documents.

Author: maxiloc

src/EloquentSubscriber.php

@@ -19,8 +19,11 @@ public function saved($model)
 
         /** @var \AlgoliaSearch\Index $index */
         foreach ($this->modelHelper->getIndices($model) as $index) {
+
             if ($this->modelHelper->indexOnly($model, $index->indexName)) {
                 $index->addObject($this->modelHelper->getAlgoliaRecord($model), $this->modelHelper->getKey($model));
+            } else if ($this->modelHelper->wouldBeIndexed($model, $index->indexName)) {
+                $index->deleteObject($this->modelHelper->getObjectId($model));
             }
         }
 
@@ -46,4 +49,4 @@ public function subscribe($events)
         $events->listen('eloquent.saved*', '\AlgoliaSearch\Laravel\EloquentSubscriber@saved');
         $events->listen('eloquent.deleted*', '\AlgoliaSearch\Laravel\EloquentSubscriber@deleted');
     }
-}
+}

src/ModelHelper.php

@@ -43,6 +43,19 @@ public function indexOnly(Model $model, $index_name)
         return !method_exists($model, 'indexOnly') || $model->indexOnly($index_name);
     }
 
+    public function wouldBeIndexed(Model $model, $index_name)
+    {
+        if (! method_exists($model, 'indexOnly')) {
+            return false;
+        }
+
+        $cloned = clone $model;
+
+        $cloned->setRawAttributes($cloned->getOriginal());
+
+        return $cloned->indexOnly($index_name) === true;
+    }
+
     public function getObjectId(Model $model)
     {
         return $model->{$this->getObjectIdKey($model)};
@@ -113,4 +126,4 @@ public function getAlgoliaRecord(Model $model)
     {
         return $model->getAlgoliaRecordDefault();
     }
-}
+}