Problem/Motivation

From what I understand most path handling and matching in D8 now requires a leading forward slash. The current path matching does not match as expected without the leading forward slash.

Proposed resolution

Add these into the defaults.

Remaining tasks

Patch.

User interface changes

Updated default settings.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 2642146-updated-paths.patch757 bytesSam152
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sam152 created an issue. See original summary.

Sam152’s picture

Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented
Status: Active » Needs review
FileSize
2642146-updated-paths.patch757 bytes

Status: Needs review » Needs work

The last submitted patch, 2: 2642146-updated-paths.patch, failed testing.

Sam152’s picture

Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented
Status: Needs work » Needs review
Sam152’s picture

Sam152 CreditAttribution: Sam152 as a volunteer and at PreviousNext commented
Parent issue: » #2642154: [META] Reduce procedural code and write tests.
frjo’s picture

frjo CreditAttribution: frjo commented
Status: Needs review » Fixed

Committed to 8-dev, thanks!

  • frjo committed 8626e96 on 8.x-1.x authored by Sam152 
    Issue #2642146 by Sam152: Default disabled paths should begin with a...

Status: Fixed » Closed (fixed)

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