Closed (fixed)
Project:
Views Send
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
1 Mar 2012 at 00:46 UTC
Updated:
6 Sep 2012 at 20:08 UTC
Jump to comment: Most recent file
When an email field fails to validate for the recipient email, it would help if there was some way to indicate *which* selected rows fail validation.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 1462126-show-bad-addresses.patch | 1.76 KB | djdevin |
| #2 | Display_emails_in_error_message-1462126-2.patch | 1.53 KB | bigjim |
Comments
Comment #1
hansfn commentedSure. It just haven't been prioritized. I'm assigning this to me, but feel free to submit (another patch).
Comment #2
bigjim commentedThe attached patch concatenates the the string "Bad Addresses:" along with a comma delimited list of the e-mails that failed to pass the valid_email_address() function.
Comment #3
hansfn commentedThx for the patch. I'll review it this week - promise ;-)
Comment #4
hansfn commentedComment #5
hansfn commentedFixed in commit 0bd7bff.
Comment #7
djdevinRolled for D6, can you commit this?
Comment #8
hansfn commentedIs there any reason you didn't roll what I actually committed - ref comment 5 - which was an improved version of the patch from bigjim?
PS! Thx for contribution.
Comment #9
djdevinSorry, I didn't even look! I'll re-roll.
Comment #10
hansfn commentedOK, I have ported this to D6 - see commit d1644cf. (There is no way to get the actual row number on D6.)