laravel · safaeean · May 18, 2025 · shaedrich · May 20, 2025
diff --git a/src/Illuminate/Pagination/AbstractPaginator.php b/src/Illuminate/Pagination/AbstractPaginator.php
@@ -185,19 +185,25 @@ public function url($page)
            $page = 1;
        }

+        // Only add the page parameter if it's greater than 1.
+        // This prevents unnecessary ?page=1 in the URL and improves SEO by avoiding duplicate content.
+        $parameters = $page > 1 ? [$this->pageName => $page] : [];
+
        // If we have any extra query string key / value pairs that need to be added
        // onto the URL, we will put them in query string form and then attach it
        // to the URL. This allows for extra information like sortings storage.
-        $parameters = [$this->pageName => $page];
-
        if (count($this->query) > 0) {
            $parameters = array_merge($this->query, $parameters);
        }

+        // Build the query string and attach it to the path
+        $query = Arr::query($parameters);
+
        return $this->path()
-                        .(str_contains($this->path(), '?') ? '&' : '?')
-                        .Arr::query($parameters)
-                        .$this->buildFragment();
+            . ($query === ''
+                ? ''
+                : (str_contains($this->path(), '?') ? '&' : '?') . $query)
+            . $this->buildFragment();
-        // Build the query string and attach it to the path
-        $query = Arr::query($parameters);
-
-        return $this->path()
-                        .(str_contains($this->path(), '?') ? '&' : '?')
-                        .Arr::query($parameters)
-                        .$this->buildFragment();
-            . ($query === ''
-                ? ''
-                : (str_contains($this->path(), '?') ? '&' : '?') . $query)
-            . $this->buildFragment();
+        return Uri::of($this->path())->withQuery($query)->withFragment($this->buildFragment);
-        // Build the query string and attach it to the path
-        $query = Arr::query($parameters);
-
-        return $this->path()
-                        .(str_contains($this->path(), '?') ? '&' : '?')
-                        .Arr::query($parameters)
-                        .$this->buildFragment();
-            . ($query === ''
-                ? ''
-                : (str_contains($this->path(), '?') ? '&' : '?') . $query)
-            . $this->buildFragment();
+        return Uri::of($this->path())->withQuery($query)->withFragment($this->buildFragment);
    }

    /**