better middleware

src/CaptureReferrerMiddleware.php

@@ -16,36 +16,34 @@ class CaptureReferrerMiddleware
      */
     public function handle(Request $request, Closure $next): Response
     {
-        $sources = $this->getReferrerBySource($request);
+        $sources = Referrer::getSources();
 
-        if ($sources->count()) {
-            Referrer::put($sources);
-        }
-
-        return $next($request);
-    }
-
-    public function getReferrerBySource(Request $request): ReferrerSources
-    {
-        $items = Referrer::getSources();
-
-        $sources = Referrer::getSourcesFromConfig();
+        $updated = false;
 
-        foreach ($sources as $sourceName) {
+        foreach (Referrer::getSourcesFromConfig() as $sourceName) {
             $strategy = Referrer::getStrategy($sourceName);
             $source = $sourceName::fromRequest($request);
 
-            if (
-                $source->isNotEmpty() &&
-                ! $items->has($source)
-            ) {
-                $items->put(
-                    source: $source,
-                    strategy: $strategy
-                );
+            if ($source->isEmpty()) {
+                continue;
             }
+
+            if ($sources->has($source)) {
+                continue;
+            }
+
+            $sources->put(
+                source: $source,
+                strategy: $strategy
+            );
+
+            $updated = true;
         }
 
-        return $items;
+        if ($updated) {
+            Referrer::put($sources);
+        }
+
+        return $next($request);
     }
 }