Route::is() and Route:has()

[FedericoHeichou]

Hello, I suggest to implement Route::is() and Route::has() to check the current route.
Actually I did a workaround:
Route::is('login') became Route::is(app()->getLocale() . '.login')
Route::has('register') became Route::has(app()->getLocale() . '.register')

But this is no nice.
Thanks for your work

[ivanvermeyen]

Hi, I will look into it, seems like a cool idea!
I'm currently doing a similar thing:

Route::is('*.blog.*')

[FedericoHeichou]

Yes, I didn't thought to use "*" lol

[ivanvermeyen]

Just a quick first thought: maybe it's better to include a macro like Route::isLocalized($name, $locale = null) and Route::hasLocalized($name, $locale = null). Else it won't be possible to check for a specific route.

[FedericoHeichou]

Mmh I'm not sure, actually I have for example a blade like this

<div class="card-header">{{ Route::is(app()->getLocale() . '.post.edit') ? __('message.edit') : __('message.new') }} Post</div>

And using this it shows correctly each localization message "edit" or "new" depending of its route

[FedericoHeichou]

But maybe I was missunderstanding and you was talking about how implement it

[ivanvermeyen]

Yeah, so that could become:

<div class="card-header">{{ Route::isLocalized('post.edit') ? __('message.edit') : __('message.new') }} Post</div>

Maybe it is possible to override Route::is() and Route::has(), but I'll have to check for any implications this might have. It does look nicer.

So then if you have a route named en.post.edit, calling Route::is('post.edit') and Route::has('post.edit') would return true. This kinda has the same limitation as the route() helper in this package, that you should avoid using the same name for localized and non-localized routes.

I could also add an optional parameter to check against a specific locale.
If $locale is false, it could do an exact route check (non-localized):

Route::is('post.edit'); // true - checking non-localized and any locale
Route::is('post.edit', $locale = 'en'); // true - checking only 'en' locale
Route::is('post.edit', $locale = false); // false - checking only non-localized

Same idea for Route::has():

Route::has('post.edit'); // true - checking non-localized and any locale
Route::has('post.edit', $locale = 'en'); // true - checking only 'en' locale
Route::has('post.edit', $locale = false); // false - checking only non-localized

[ivanvermeyen]

Partially implemented by #33 but still needs testing.

[AlexisSerneels]

Hi.
After multiple tests in my project, I can't get it to work. It seems the result is always true.
Furthermore, it does not support an array of routes names as parameters like the original Laravel method.

I've been able to successfully get it working using this macro:

Route::macro('isLocalized', function ($patterns, $locale = '*') {

    $names = collect($patterns)->map(function ($name) use ($locale) {
        return $locale . '.' . $name;
    })->all();

    return Route::is($names);
});

This technique supports single and array parameters and has the advantage to use the original Laravel method. You can pass a specific locale if needed.
If it may help. I'm happy to submit a PR if needed.

Cheers,

[ivanvermeyen]

Hi

That seems like a great solution.
PR is always welcome.
I will probably rename the other macro to hasLocalized() (deprecating localizedHas() until the next major version).

I still hope to use a modified version of Route::is() and Route::has() one day, as it really feels like you are just using a default Laravel app. I think that would be cool, but might be tricky. So for now custom macro's will do :)

Thanks!

[sheinfeld]

@ivanvermeyen can you release this changes?

The website is crashing whenever I use an object that is missing at least one localized version...

[ivanvermeyen]

I wrote some tests to verify everything works as expected and implemented a solution based on @AlexisSerneels example.

You can now:

Check if the current route is localized:

Route::isLocalized();

Check if the name of the current localized route matches a specific name in any locale:

Route::isLocalized('route-name');

Check if the name of the current localized route matches a specific name in a specific locale:

Route::isLocalized('route-name', 'en');

Check if the name of the current localized route matches any of the given names in any locale:

Route::isLocalized(['route-one', 'route-two']);

Check if the name of the current localized route matches any of the given names in a specific locale:

Route::isLocalized(['route-one', 'route-two'], 'en');

Check if the name of the current localized route matches the given names and locales:

Route::isLocalized('route-name', ['en', 'nl']);
Route::isLocalized(['route-one', 'route-two'], ['en', 'nl']);

Will tag and release when CI tests have passed 👍

[sheinfeld]

@ivanvermeyen what about a function or check for Route::localizedUrl?

I can't use your original function, because if I try to get a localizedURL of an inexistent route, it will crash my whole website.

Perhaps: Route::hasLocalizedURL?