Add a routeExists method to RouteProvider

public function routeExists($name)
This needs to be typehinted both function and parameter

public function routeExists($name)
Parameter needs typehinting

All instances of that outside the tests.

This adds type declarations and uses casting instead of count() to check whether the array is empty.

If the merge request is to to used, I think the person who created the merge request needs to update its target branch to point to 11.x.

The last submitted patch, 34: drupal-create_routeExists-3029545-34.patch, failed testing. View results →

Fix test.

CR looks good, updated branch numbers.

Think this is a good addition.

The last submitted patch, 36: drupal-create_routeExists-3029545-36.patch, failed testing. View results →

Hitting retest to see if we can display a green run.

The random test failure has already been logged: 🐛 Random test fail in Drupal\Tests\Component\Utility\RandomTest::testRandomMachineNamesUniqueness Needs work

Slight improvement, imo. Not a deal breaker for me.

public function routeExists(string $name): bool {
  return !empty($this->getRoutesByNames([$name]));
}

Unless we anticipate a lot of calls to routeExists with an empty string then let's just assume the supplied string is a potential route and then explicitly declare true if not empty rather than casting.

I'm triaging RTBC issues → .

I read the Issue summary, comments and the change record. I found that all questions have been answered. The CR is complete and has a before and after section which makes it easy for the reader. It is a bit brief for my tastes but it does not require a change.

I looked at the patch and ran the tests.

1. +++ b/core/tests/Drupal/KernelTests/Core/Routing/RouteProviderTest.php
   @@ -655,7 +655,7 @@ public function testRouteCaching() {
   +   * Tests RouteProvider::getRouteByName(), RouteProvider::getRoutesByNames(), and RouteProvider::routeExists().
   

   The summary line is to be 80 characters max. This could be changed by using @covers for the methods being tested.

2. +++ b/core/tests/Drupal/KernelTests/Core/Routing/RouteProviderTest.php
   @@ -667,6 +667,11 @@ public function testRouteByName() {
   +    $this->assertFalse($provider->routeExists(''));
   

   And since a change is being made, this could use a simple comment to help the next person working on this.

I decided to make a patch for the above and at the same time I have added the suggestion in #42.

All green so back to RTBC

The last submitted patch, 43: 3029545-43.patch, failed testing. View results →