`core/modules/path/path.api.php` documentation has become out of sync this the implementation of the hooks it documents.
The documentation references PathInterface class for the structure of the $path array.
The following patch updates this to be AliasStorageInterface

CommentFileSizeAuthor
#5 2941945-4.patch1.43 KBchiranjeeb2410
#2 drupal-path_documentation-2941945-1-d8.patch1.51 KBL-four
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

L-four created an issue. See original summary.

L-four’s picture

L-four CreditAttribution: L-four commented
FileSize
drupal-path_documentation-2941945-1-d8.patch1.51 KB

Status: Needs review » Needs work

The last submitted patch, 2: drupal-path_documentation-2941945-1-d8.patch, failed testing. View results

chiranjeeb2410’s picture

chiranjeeb2410 CreditAttribution: chiranjeeb2410 at Google Summer of Code commented
Title: path.api.php references non-existent PathInterface updated to AliasStorageInterface » path.api.php references to a non-existent PathInterface class and updated to AliasStorageInterface
chiranjeeb2410’s picture

chiranjeeb2410 CreditAttribution: chiranjeeb2410 at Google Summer of Code commented
Status: Needs work » Needs review
FileSize
2941945-4.patch1.43 KB

Patch uploaded against 8.5.x. Hope to get green.
Review please.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer commented
Status: Needs review » Reviewed & tested by the community

  • catch committed 4f1df3f on 8.6.x 
    git commit -m 'Issue #2941945 by L-four, chiranjeeb2410: path.api.php...

  • catch committed 3401657 on 8.5.x 
    git commit -m 'Issue #2941945 by L-four, chiranjeeb2410: path.api.php...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Version: 8.6.x-dev » 8.5.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.6.x and cherry-picked to 8.5.x. Thanks!

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer commented

Looks lik you pasted the wrong command in the commit message @catch.

Status: Fixed » Closed (fixed)

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