- Status changed to Needs review
over 1 year ago 11:09am 30 January 2023 - Status changed to Needs work
over 1 year ago 11:29am 30 January 2023 - ๐ฌ๐งUnited Kingdom longwave UK
Thanks for rerolling. This still needs a test to exercise the broken behaviour and prove that we have fixed it with the patch.
- ๐บ๐ธUnited States neclimdul Houston, TX
conflicts with ๐ Replace deprecated String.prototype.substr() with String.prototype.substring() Fixed . rerolled in a merge request. not sure what a test would look like so just the reroll atm.
- ๐บ๐ธUnited States pmagunia Philadelphia ๐บ๐ธ
The patch from #52 applied cleanly to Drupal 10.2.5.
- ๐ฎ๐ณIndia yash.rode pune
yash.rode โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia yash.rode pune
Hi, I was trying to reproduce this issue in Drupal Core, where I can click on any button which takes me to the link with a fragment?
I am trying to write the test, but not sure what should be the Drupal page where I can get this error which I can convert to test. - ๐บ๐ธUnited States neclimdul Houston, TX
Not sure of a common real case but I was able recreate this just now like this.
1. Install Drupal site.
2. Ensure registration is allowed.
3. Log out and visit the registration url with a complex url fragment. e.g. https://d10.lndo.site/en/user/register/#badfragment/broken - ๐ฎ๐ณIndia yash.rode pune
Hi I tried the steps above but the issue only occurs when we try to change the URL and not when we visit the URL for the first time?
If that is the expected behaviour how can we write a test for that scenario?