- heddn Nicaragua
Another issue with the current
getUriAsDisplayableString
approach is thatroute:entity.group_content.canonical;group_content=135&group=3
gets truncated tontity.group_content.canonical;group_content=135&group=3
. That is an outright bug. - Status changed to Needs review
over 1 year ago 4:46pm 28 March 2023 - heddn Nicaragua
Here we add some test coverage and rebase this as a patch on 10.1.x.
- heddn Nicaragua
Why did we remove the
Text-only links.
test cases? Because there is dedicated test coverage, it wasn't needed and the regex parsing was causing errors on those test cases. The last submitted patch, 20: 3115188-20.patch, failed testing. View results →
- Status changed to RTBC
over 1 year ago 7:50pm 1 April 2023 - 🇺🇸United States smustgrave
Ran the test locally without the fix and it failed as expected
Behat\Mink\Exception\ExpectationException : The string ""route:entity.node.canonical;node=1"" was not found anywhere in the HTML response of the current page.
So removing the tag.
The change in question seems good though.
- Status changed to Needs work
over 1 year ago 11:59pm 3 April 2023 - 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
+++ b/core/modules/link/tests/src/Functional/LinkFieldTest.php @@ -160,11 +162,6 @@ protected function doTestURLValidation() { - // Text-only links. - '<nolink>' => '<nolink>', - 'route:<nolink>' => '<nolink>', - '<none>' => '<none>',
why are we removing these?
Can we also get a red/green set of patches here demonstrating the new test failures?
Thanks
I think we also want a change notice for this detailing the official support for route:
Removing
getUriAsDisplayableString
makes sense.It fixes the situation for me too.
+1 for patch in #20.
- 🇫🇷France mably
Just been hit by the problem today.
Spent a bunch of time to understand what was going on and finally found this issue.
Hoping to have it fixed one day 🙏 😉
- Status changed to Needs review
about 1 year ago 7:23pm 15 June 2023 - last update
about 1 year ago 29,450 pass - last update
about 1 year ago 29,448 pass, 2 fail - heddn Nicaragua
The last submitted patch, 29: 3115188-29-test-only.patch, failed testing. View results →
- last update
about 1 year ago 29,800 pass, 2 fail The last submitted patch, 31: 3115188-31.patch, failed testing. View results →