Closed (fixed)
Project:
Mass Contact
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Anonymous (not verified)
Created:
4 Aug 2008 at 12:53 UTC
Updated:
18 Mar 2021 at 00:16 UTC
Jump to comment: Most recent
Comments
Comment #1
oadaeh commentedThe only time that error shows up is when validating the sender's e-mail address, using the valid_email_address() function. Verify your sender e-mail address is valid, based on that function call (the code is in the link), and also that the field doesn't contain any stray white space characters.
Comment #2
compuguru commentedI have a similar issue, but it only happens when I have the default values and I don't let users change them. Using print_r the $form_state['values']['name'] and $form_state['values']['email'] are blank for some reason, so it doesn't validate. Still trying to figure out why though.
Comment #3
compuguru commentedI did some more research and found that this is probably causing the issue: #227966: Use default values to #disabled form fields The values for the name and email aren't passed though because they are disabled.
Comment #4
oadaeh commentedOkay. I have found and confirmed this bug. Thank you compuguru, for going the extra mile and finding the problem. I now have a work around in place on my development system, however, I also have several other code changes in progress that aren't ready for submission, and I don't want to undo them, so it may be a couple of days before I can get the changes into CVS, but I will do my best to get them there ASAP.
Comment #5
compuguru commentedHi - just wondering if you have had a chance to get it updated yet?
Comment #6
oadaeh commentedI have not completed the changes. I have been dealing with other things (server failures, network outages, etc.) and have not had time to work on this module. To complicate matters, I had some problems with other modules, making it impossible for me to test whether the functionality even works or not.
Comment #7
Anonymous (not verified) commentedsubscribing
Comment #8
oadaeh commentedThis should now be fixed in the 6.x-1.x-dev development snapshot. I'm going to add a couple of other things and then tag it 6.x-1.0-beta2. Look for the downloads soon. Sorry for the long delay.
Comment #9
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.