Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
After saving a scheduled email handler with a positive amount of days, and then edit it again, the number of days doesn't show the right value.
Comment | File | Size | Author |
---|---|---|---|
#4 | interdiff-2924247-2-4.txt | 1.15 KB | Nitebreed |
#4 | scheduled_mail-wrong_default_value_for_days-2924247-4.patch | 808 bytes | Nitebreed |
| |||
#2 | scheduled_mail-wrong_default_value_for_days-2924247-2.patch | 949 bytes | Nitebreed |
|
Comments
Comment #2
NitebreedPatch attached
Comment #3
Lendude#2 works but I would change the $days_options array to match the data that is stored for the handler. In #2 we add the + twice.
so do:
Or update the webform.handler.scheduled_email schema for the days key to store a string and not an integer (which is what is currently getting stored if you want to include the + in the value). But a schema change would break BC so we'd need an upgrade path for that, seems like a hassle.
Comment #4
NitebreedYes you're right, since it is an integer we do't need the '+' in the key. Changing the schema to a string isn't a solution, since we also do calculations on this value.
Comment #6
jrockowitz CreditAttribution: jrockowitz as a volunteer and at The Big Blue House commentedThanks for catching that.