A very small improvement, but all router paths should start with a leading slash.

CommentFileSizeAuthor
#2 2894973-2.patch4.6 KBStijnStroobants
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

StijnStroobants created an issue. See original summary.

StijnStroobants’s picture

StijnStroobants
Primary language Dutch
Location Leuven
CreditAttribution: StijnStroobants at Dropsolid commented
FileSize
2894973-2.patch4.6 KB
StijnStroobants’s picture

StijnStroobants
Primary language Dutch
Location Leuven
CreditAttribution: StijnStroobants at Dropsolid commented
Status: Active » Needs review
jonathanshaw’s picture

jonathanshaw
Primary language English
Location Stroud, UK
CreditAttribution: jonathanshaw commented
Category: Bug report » Task
Priority: Normal » Minor

Status: Needs review » Needs work

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

AdamPS’s picture

AdamPS CreditAttribution: AdamPS at AlbanyWeb commented
Status: Needs work » Needs review
StijnStroobants’s picture

StijnStroobants
Primary language Dutch
Location Leuven
CreditAttribution: StijnStroobants at Dropsolid commented
Status: Needs review » Reviewed & tested by the community

  • 1035807 committed on 8.x-1.x 
    Issue #2894973 by StijnStroobants: All router paths should start with a...
AdamPS’s picture

AdamPS CreditAttribution: AdamPS at AlbanyWeb commented
Status: Reviewed & tested by the community » Fixed

@StijnStroobants You are not supposed to set RTBC for your own patch!

Anyway it's a simple change and the tests ran so I will commit it, thanks.

StijnStroobants’s picture

StijnStroobants
Primary language Dutch
Location Leuven
CreditAttribution: StijnStroobants at Dropsolid commented

@AdamPS Oops indeed, my mistake. :-) Thanks!

Status: Fixed » Closed (fixed)

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