- Issue created by @bdunphy
- ๐ฉ๐ชGermany jurgenhaas Gottmadingen
Good catch, thanks for reporting. And yes, declaring this form field as required is the right way to go. Still, I'm surprised that the form validation doesn't catch an empty field since there is the constraint
'#min' => 1,
.I've changed the version to 3.0.x since we always work from the latest release backwards with bug fixes and new features. Do you want to provide an MR for this? If not, I can do that later, too.
- ๐ฎ๐ณIndia Ishani Patel
ishani patel โ made their first commit to this issueโs fork.
- Merge request !506Issue #3523826 by bdunphy, jurgenhaas: Empty Expiration Field Causes WSOD โ (Merged) created by Unnamed author
- ๐ฎ๐ณIndia Ishani Patel
Hello,
I've created MR.
Please check and review.Thank you!
-
jurgenhaas โ
committed b6c5e94e on 3.0.x authored by
ishani patel โ
Issue #3523826 by ishani patel, jurgenhaas, bdunphy: Empty Expiration...
-
jurgenhaas โ
committed b6c5e94e on 3.0.x authored by
ishani patel โ
-
jurgenhaas โ
committed 3ae219c2 on 2.1.x
Issue #3523826 by ishani patel, jurgenhaas, bdunphy: Empty Expiration...
-
jurgenhaas โ
committed 3ae219c2 on 2.1.x
-
jurgenhaas โ
committed 69938a4c on 2.0.x
Issue #3523826 by ishani patel, jurgenhaas, bdunphy: Empty Expiration...
-
jurgenhaas โ
committed 69938a4c on 2.0.x
- ๐ฉ๐ชGermany jurgenhaas Gottmadingen
Thanks, that's exactly it. Committed to 3.0.x and backported to 2.1.x and 2.0.x
- ๐จ๐ฆCanada bdunphy
Thank you! That's fast work and I appreciate the fix. I guess I should learn about creating an MR when I find these issues.