If you link to something like user/autocomplete you get the following error

<

t('The link must correspond to a valid path on the site.')
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Issue tags: +Needs tests
FileSize
929 bytes

just the validation bit.

dawehner’s picture

Issue tags: +WSCCI, +Routing

.

tim.plunkett’s picture

Issue tags: +MenuSystemRevamp

Tagging

dawehner’s picture

Status: Active » Needs review

Nevertheless a review can already made.

Status: Needs review » Needs work

The last submitted patch, shortcut-2064611-1.patch, failed testing.

dawehner’s picture

dawehner’s picture

Priority: Normal » Major
Status: Needs work » Needs review
FileSize
2.55 KB

I consider this at least as major.

dawehner’s picture

FileSize
2.26 KB

Way simpler approach by using getRoutesByPattern

Status: Needs review » Needs work

The last submitted patch, shortcut-2064611-7.patch, failed testing.

dawehner’s picture

Status: Needs work » Needs review
FileSize
727 bytes
2.26 KB

Ups.

jibran’s picture

Status: Needs review » Reviewed & tested by the community

I think this is cleaner approach and we have tests as well so let's get this in.

catch’s picture

#10: shortcut-2064611-10.patch queued for re-testing.

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs tests, +MenuSystemRevamp, +WSCCI, +Routing

The last submitted patch, shortcut-2064611-10.patch, failed testing.

dawehner’s picture

Status: Needs work » Reviewed & tested by the community
Issue tags: -Needs tests
FileSize
2.18 KB

Rerole.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

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