Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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
Related Issues
Comment | File | Size | Author |
---|---|---|---|
#6 | 2096135-path-processor-alias-6.patch | 6.37 KB | longwave |
#3 | interdiff.txt | 540 bytes | dawehner |
#3 | path_processor-2096135-3.patch | 3.76 KB | dawehner |
#1 | path_alias-2096135-1.patch | 3.74 KB | dawehner |
Comments
Comment #1
dawehnerAdded tag and a patch.
Comment #2
ParisLiakos CreditAttribution: ParisLiakos commentedyar, extra tests for free:)
can we add some groups on the unit test really quickly?
Comment #3
dawehnerThe only group which really seem to make sense is Drupal
Comment #4
jibranAs #2 is addressed so RTBC.
Comment #5
longwaveThis 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.
Comment #6
longwaveThis 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
Comment #7
dawehnerNice!
Comment #8
alexpottCommitted 144ab03 and pushed to 8.x. Thanks!