- Issue created by @apaderno
- 🇮🇹Italy apaderno Brescia, 🇮🇹
I will also verify what Drupal core does in similar cases.
- Merge request !10Issue #3454259: Validate the redirect URL only when "Enable URL redirect" is selected → (Merged) created by apaderno
- Status changed to Needs review
5 months ago 7:13pm 12 June 2024 - 🇮🇹Italy apaderno Brescia, 🇮🇹
I also simplified the code, since
$form_state->hasValue()
is usually not called before calling$form_state->getValue()
. Hello all,
I have applied the MR from here and followed the mentioned steps of this issue, now it's look like the problem is fixed now, moving is to RTBC. I am attaching the screencast also.- Status changed to RTBC
5 months ago 5:48am 13 June 2024 -
apaderno →
committed 62d8bec1 on 2.x
Issue #3454259: Validate the redirect URL only when "Enable URL redirect...
-
apaderno →
committed 62d8bec1 on 2.x
- Status changed to Fixed
5 months ago 8:56am 15 June 2024 - Status changed to Needs work
5 months ago 3:11pm 15 June 2024 - 🇮🇹Italy apaderno Brescia, 🇮🇹
The same change should be done for the Drupal 7 branch.
-
apaderno →
committed e72fbf08 on 7.x-1.x
Issue #3454259: Validate the redirect URL only when "Enable URL redirect...
-
apaderno →
committed e72fbf08 on 7.x-1.x
- Status changed to Fixed
5 months ago 11:00am 18 June 2024 - 🇮🇹Italy apaderno Brescia, 🇮🇹
I committed the changes directly to the project branch because the issue fork branch was not updated and I was not able to update it. (The issue fork UI allows to update the fork only from the default branch, which is not the Drupal 7 branch.)
- 🇮🇹Italy apaderno Brescia, 🇮🇹
apaderno → changed the visibility of the branch 3454259-d7-validate-the-redirect-url-only-when-necessary to hidden.
Automatically closed - issue fixed for 2 weeks with no activity.