Merge pull request #107 from matchish/fix-queue

Don't use searchable macros
matchish · Mar 26, 2020 · f657318 · f657318
2 parents f2c00ec + 64d5001
commit f657318
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/)
 
 ## [Unreleased]
 
+## [4.0.1] - 2020-03-26
+### Fixed
+-  Prevent unnessasary send `\Laravel\Scout\Jobs\MakeSearchable` to a queue
+
 ## [4.0.0] - 2020-03-12
 ### Added
 -  Scout 8 Support

diff --git a/composer.json b/composer.json
@@ -1,7 +1,7 @@
 {
     "name": "matchish/laravel-scout-elasticsearch",
     "description": "Search among multiple models with ElasticSearch and Laravel Scout",
-    "version": "4.0.0",
+    "version": "4.0.1",
     "keywords": [
         "laravel",
         "scout",

diff --git a/src/Jobs/Stages/PullFromSource.php b/src/Jobs/Stages/PullFromSource.php
@@ -25,8 +25,10 @@ public function __construct(ImportSource $source)
 
     public function handle(): void
     {
-        $results = $this->source->get();
-        $results->filter->shouldBeSearchable()->searchable();
+        $results = $this->source->get()->filter->shouldBeSearchable();
+        if (! $results->isEmpty()) {
+            $results->first()->searchableUsing()->update($results);
+        }
     }
 
     public function estimate(): int