Laravel optional multiple params routes

@abdullaf If you really need the url to be: localhost/products/1&nike You can still use it that way, it will work with: Route::get('/products/{id?}&{brand?}')

@gych Now doesn't recognize anymore as optional params

@abdullaf In your controller? or you get an error for the route?

@gych When I go to route /products without params. This error appears: The GET method is not supported for route contact-us. Supported methods: POST.

@abdullaf You have to create two routes in your routes file One for /products/{id?}&{brands?} and one for /products

Because if you only use this route /products/{id?}&{brands?} It will expect you to use /products/& if you use it without parameters

If they use the same controller also make sure the default value for the params is null, for example:

public function index($id = null, $brand = null)
{
// REST OF YOUR CODE
}

@gych The controller is like you provided

@abdullaf Ok good and did you add two routes in your routes file One for /products/{id?}&{brands?} and one for /products

Route::get('/products/{id?}&{brand?}')

Route::get('/products}')

@gych For the same function in the controller?

@abdullaf Yes it can use the same function and controller

@gych Thank you! But the same issue, appear in URL like this: localhost/products/& when I didn't provide params

@abdullaf So a url like localhost/products and localhost/products/1&nike work fine now? But you still get an error when you use localhost/products/& , right?

Is it the same error as before?

@gych No when I go just to URL: localhost/products show me this URL: localhost/products&

@abdullaf Can you show me the full code of your controller method?

@gych

public function products($id = null, $brand= null)
    {
        return view('products.index', [
            'id' => $id,
            'brand' => $brand,
        ]);
    }

@abdullaf strange so when you enter localhost/products in your adress bar it auto redirects you to localhost/products& ?

@gych Yes, I did the

php artisan optimize

but same issue

@abdullaf Can you show the code of your blade view products.index

@gych Maybe this is the problem from the view because the second parameter is not coming from the blade it will randomly generated and will be converted to a QR Code and the URL should be like this: localhost/products/nike

@abdullaf Yes I'm also thinking that this is caused by your view

@gych But the params are optional I don't know.

@abdullaf Yes but it might be possible that when there are not parameters you view creates a url like this localhost/products& without the parameters

You should use a statement in your view that when there are no paramters it doesnt add the '&' to the URL But I'm not sure about this without seeing your view file.

@gych Thank you for your time.

@fulufhelo Thank you for your answer, but this one requires a link/button or anything else to implement but I don't need to be in the app because this URL will be used for QR Code just to scan, and I will generate random URLs.

@martinbean Thank you, for your answer I know the "resource", but for me is a bit different because the second parameter will be used to track how many views will come from that specific URL it will be a QR Code but the first parameter is when a user click the button that will go to the details page. Now I need 2 optional parameters when can I use them for different purposes

@abdullaf You have an ID in the URI. That suggests you have an “index” route and a “show” route. I don’t really see what tracking has to do with anything.

Route::get('/products', [ProductController::class, 'index']);
Route::get('/products/{id}', [ProductController::class, 'show']);

You can then use a query string on your index view to filter results, i.e. /products?brand=nike to only show Nike-branded products in the listing:

public function index(Request $request)
{
    $products = Product::query()
        ->when($request->query('brand'), function (Builder $query, string $brand) {
            $query->where('brand', '=', $brand);
        })
        ->paginate();

    return view('products.index', compact('products'));
}

public function show(Product $product)
{
    return view('products.show', compact('product'));
}

There’s no need to try and shoehorn multiple responsibilities into a single controller action.

@martinbean Thank you!