Method call is provided 2 parameters, but the method signature uses 1 parameters

AccessManager.php
$route = $this->routeProvider->getRouteByName($route_name, $parameters);

CommentFileSizeAuthor
#6 interdiff-3102980-4-6.txt833 bytesNathaniel
#6 accessmanager_getroutebyname-3102980-6.patch2.39 KBNathaniel
#4 accessmanager_getrouteby_name-3102980-4.patch1.79 KBHardik_Patel_12
#4 interdiff-3102980-2-4.txt1.17 KBHardik_Patel_12
#2 accessmanager_getroutebyname-3102980-2.patch769 bytesNathaniel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Nathaniel created an issue. See original summary.

Nathaniel’s picture

Nathaniel CreditAttribution: Nathaniel commented
Status: Active » Needs review
FileSize
accessmanager_getroutebyname-3102980-2.patch769 bytes

Status: Needs review » Needs work

The last submitted patch, 2: accessmanager_getroutebyname-3102980-2.patch, failed testing. View results

Hardik_Patel_12’s picture

Hardik_Patel_12 CreditAttribution: Hardik_Patel_12 at QED42 commented
Status: Needs work » Needs review
FileSize
interdiff-3102980-2-4.txt1.17 KB
accessmanager_getrouteby_name-3102980-4.patch1.79 KB

Kindly review a new patch

Status: Needs review » Needs work

The last submitted patch, 4: accessmanager_getrouteby_name-3102980-4.patch, failed testing. View results

Nathaniel’s picture

Nathaniel CreditAttribution: Nathaniel commented
Status: Needs work » Needs review
FileSize
accessmanager_getroutebyname-3102980-6.patch2.39 KB
interdiff-3102980-4-6.txt833 bytes
Hardik_Patel_12’s picture

Hardik_Patel_12 CreditAttribution: Hardik_Patel_12 at QED42 commented

Thank you @Nathaniel. Above patch works fine.

Hardik_Patel_12’s picture

Hardik_Patel_12 CreditAttribution: Hardik_Patel_12 at QED42 commented
Status: Needs review » Reviewed & tested by the community

  • catch committed a7f702a on 9.0.x 
    Issue #3102980 by Nathaniel, Hardik_Patel_12: AccessManager...

  • catch committed fc0b41d on 8.9.x 
    Issue #3102980 by Nathaniel, Hardik_Patel_12: AccessManager...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

The documentation is actually wrong for that method, opened #3110104: Incorrect documentation for RouteProvider::getRouteByName().

Committed a7f702a and pushed to 9.0.x and 8.9.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.