Use scoped instances of instead of singletons

AlexVanderbist · web-flow · commit 0d0075b9c56e · 2022-04-07T22:15:43.000+02:00
See https://laravel.com/docs/8.x/container#binding-scoped. These are essentially the same thing as `singleton` instances, only difference being that Laravel will make sure that the `scoped` instances get refreshed every request (in Octane) and every job (Horizon).
diff --git a/src/ActivitylogServiceProvider.php b/src/ActivitylogServiceProvider.php
@@ -29,11 +29,11 @@ public function registeringPackage()
     {
         $this->app->bind(ActivityLogger::class);
 
-        $this->app->singleton(LogBatch::class);
+        $this->app->scoped(LogBatch::class);
 
-        $this->app->singleton(CauserResolver::class);
+        $this->app->scoped(CauserResolver::class);
 
-        $this->app->singleton(ActivityLogStatus::class);
+        $this->app->scoped(ActivityLogStatus::class);
     }
 
     public static function determineActivityModel(): string

Original file line number	Diff line number	Diff line change
`@@ -29,11 +29,11 @@ public function registeringPackage()`
`29`	`29`	`{`
`30`	`30`	`$this->app->bind(ActivityLogger::class);`
`31`	`31`
`32`		`- $this->app->singleton(LogBatch::class);`
	`32`	`+ $this->app->scoped(LogBatch::class);`
`33`	`33`
`34`		`- $this->app->singleton(CauserResolver::class);`
	`34`	`+ $this->app->scoped(CauserResolver::class);`
`35`	`35`
`36`		`- $this->app->singleton(ActivityLogStatus::class);`
	`36`	`+ $this->app->scoped(ActivityLogStatus::class);`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`public static function determineActivityModel(): string`