A comma-separated string of emails longer than 128 characters gets quietly truncated on save because of #1360466: Remove maxlength HTML attribute from form elements where #maxlength is >= 128. I am experimenting with changing emails textfield to textarea and will post a patch here if it turns out to be Ok.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | content_moderation_n-3088150-4.patch | 4.9 KB | jhedstrom |
Comments
Comment #2
texas-bronius commentedWell, making the simple change
to
seems to work just fine for storage, but it would be remiss of me to provide a patch without also updating the text field for help text and to handle expected combos of command and/or line breaks, and the module's existing Tests of getMail() to run it through the paces, and I won't get to that any time soon.
This handles both combos of commas and newlines (or linebreaks for that matter):
becomes
Comment #3
texas-bronius commentedOk fine here's a patch as described in #2, sans tests. Would someone like to give this a go and also update the Tests to be account for this change?
Comment #4
jhedstromThanks for finding this issue and fixing it! This adds tests.
Comment #6
jhedstrom