Updated: Comment 0

Problem/Motivation

The PathProcessorAlias always replaced the path with an alias, even the options says on url() says that with 'alias' => TRUE.

Proposed resolution

Remaining tasks

User interface changes

API changes

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Status: Active » Needs review
Issue tags: +PHPUnit
FileSize
3.74 KB

Added tag and a patch.

ParisLiakos’s picture

yar, extra tests for free:)

can we add some groups on the unit test really quickly?

dawehner’s picture

The only group which really seem to make sense is Drupal

jibran’s picture

Status: Needs review » Reviewed & tested by the community

As #2 is addressed so RTBC.

longwave’s picture

This actually caused a bug in rel="shortlink" URLs that do not have test coverage, I fixed it the same way but added different tests in #2087043: shortlink is not using node/nid or taxonomy/term/tid - I think we should add those tests here as well.

longwave’s picture

This is #3 plus tests from #2087043: shortlink is not using node/nid or taxonomy/term/tid, if this is committed that can be closed along with #2017087: Term shortlink link relation is broken

dawehner’s picture

Nice!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 144ab03 and pushed to 8.x. Thanks!

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