Closed (fixed)
Project:
Drupal core
Version:
8.4.x-dev
Component:
routing system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 Jan 2017 at 14:55 UTC
Updated:
21 Feb 2017 at 12:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
webflo commentedComment #3
webflo commentedComment #5
webflo commentedComment #6
joelpittet@webflo, I'm quite curious what removed the attributes, this solution could work but maybe better if we find where the attributes are being removed because that may have other negative effects elsewhere in the system?
Comment #8
dawehnerComment #9
webflo commentedComment #10
dawehnerThis is a unit test which renders the link twice. Its a bug in the underlying link generator.
Comment #11
dawehnerRelated bug ... #2847611: \Drupal\Core\Template\TwigExtension::getLink overrides existing classes
Comment #13
webflo commentedThe root cause is in LinkGenerator.
Comment #14
webflo commentedThis is a combination of #10 and the test-only patch from #2
Comment #16
dawehner@jorges and @dawehner debugged the remaining failure. Turns out, we need the original fix from @webflo as well.
Comment #17
dawehnerThis time with less commented out.
Comment #18
webflo commentedLooks good, nice test coverage in the LinkGenerator und TwigExtension.
Comment #19
tim.plunkettNit, s/url/URL
Fix looks good.
Comment #20
alexpottUrl should so be immutable.
Committed and pushed 9c77748 to 8.4.x and 91d653d to 8.3.x. Thanks!
Fixed on commit.