Skip to content

Commit 82ee57a

Browse files
committed
Extract UrlBuilder to separate package
1 parent 27e81eb commit 82ee57a

File tree

3 files changed

+5
-233
lines changed

3 files changed

+5
-233
lines changed

composer.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
"0.0.0/composer-include-files": "^1.5",
2525
"codezero/browser-locale": "^3.0",
2626
"codezero/laravel-uri-translator": "^1.0",
27+
"codezero/php-url-builder": "^1.0",
2728
"illuminate/support": "^7.0|^8.0|^9.0|^10.0"
2829
},
2930
"require-dev": {

src/LocalizedUrlGenerator.php

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
namespace CodeZero\LocalizedRoutes;
44

55
use CodeZero\LocalizedRoutes\Facades\LocaleConfig;
6+
use CodeZero\UrlBuilder\UrlBuilder;
67
use InvalidArgumentException;
78
use Illuminate\Support\Collection;
89
use Illuminate\Support\Facades\App;
@@ -40,7 +41,7 @@ public function __construct()
4041
public function generateFromRequest(string $locale = null, $parameters = null, bool $absolute = true, bool $keepQuery = true): string
4142
{
4243
$urlBuilder = UrlBuilder::make(Request::fullUrl());
43-
$requestQueryString = $urlBuilder->getQueryStringArray();
44+
$requestQueryString = $urlBuilder->getQuery();
4445

4546
$currentDomain = $urlBuilder->getHost();
4647
$currentLocaleSlug = $urlBuilder->getSlugs()[0] ?? null;
@@ -68,12 +69,12 @@ public function generateFromRequest(string $locale = null, $parameters = null, b
6869
// $queryStringParameters contains "key" => "value" pairs.
6970
list($routePlaceholders, $routeParameters, $queryStringParameters) = $this->extractRouteAndQueryStringParameters($routeUri, $normalizedParameters);
7071

71-
$urlBuilder->setQueryString(
72+
$urlBuilder->setQuery(
7273
$this->determineQueryStringParameters($requestQueryString, $queryStringParameters, $keepQuery)
7374
);
7475

7576
// Merge the route parameters with the query string parameters, if any.
76-
$namedRouteParameters = array_merge($routeParameters, $urlBuilder->getQueryStringArray());
77+
$namedRouteParameters = array_merge($routeParameters, $urlBuilder->getQuery());
7778

7879
// Generate the URL using the route's name, if possible.
7980
if ($url = $this->generateNamedRouteURL($locale, $namedRouteParameters, $absolute)) {

src/UrlBuilder.php

Lines changed: 0 additions & 230 deletions
This file was deleted.

0 commit comments

Comments
 (0)