Hi, thanks for the great module. I am having a problem though, if multiple email addresses are entered for delivery, I get an error saying that the first email address is not valid. It works fine with one single email address.

Thanks for helping!

Comments

hadsie’s picture

hadsie commented

are you using the | separator to specify a name as well? like:

Scott Hadfield|hadsie@gmail

or just the email address by itself? also, you may want to give the 2.x branch a try as I think this might be fixed in that (I haven't been able to reproduce the issue at least).

esbon’s picture

esbon commented

I am only entering a list of email addresses one below the other. You may want to try our site: http://www.capitalcitychurchdc.org

hadsie’s picture

hadsie commented
Category: bug » support
Priority: Critical » Normal
Status: Active » Postponed (maintainer needs more info)

Ok, I'm not able to reproduce this issue on either version 1x or 2x. I would have guessed that more people would have reported this if it was a problem with the module. I went to your site but couldn't see anywhere to test the node invite. Maybe can you paste the exact text that you entered in the email address textarea. Also, do you get any errors in your mail logs?

esbon’s picture

esbon commented

Thanks for looking into this. If I enter one email address only, the email gets delivered fine, but if I entered two or more, a message appears saying that the first email address is not formatted correctly. Here's the full link to a page a node invite enabled page: http://www.capitalcitychurchdc.org/you-will-love-church

Best Regards

hadsie’s picture

hadsie commented

Ok, I'm able to reproduce the issue on your site.

Can you:

  1. Confirm the exact version and timestamp of the module (paste the full contents of the .info file here)
  2. Give me your PHP version
  3. List all modifications you've done on the form (including pasting any code that alters the form).
esbon’s picture

esbon commented

Thanks for looking into this:

name = "Node Invite"
description = "Provides an invite/acceptance framework like RSVP, but with some added features"
core = 6.x
dependencies[] = token

; Information added by drupal.org packaging script on 2009-01-29
version = "6.x-1.0-beta6"
core = "6.x"
project = "node_invite"
datestamp = "1233258348"

; Information added by drupal.org packaging script on 2009-07-16
version = "6.x-1.0-beta8"
core = "6.x"
project = "node_invite"
datestamp = "1247778398"

PHP Version 5.2.9

At this point there are no modifications to the currently installed module

Best Regards

hadsie’s picture

hadsie commented

ok, so you're only on beta8, not beta11 as specified in the ticket. Please update to the either the beta12 or the 2.x branch and this problem should go away.

Cheers,
Scott

esbon’s picture

esbon commented

beautiful, it now works, however, I forgot how to remove the token help link

hadsie’s picture

hadsie commented

i think i should add an option to hide that or replace it with something. the only way i know is to set the #access on that form field to FALSE. but that would require a hook_form_alter i think.

esbon’s picture

esbon commented

I know why we could not send more than one email at a time. Turns out we were removing the $form['node_invite_tokens'] form element and that's when we started having the problems. There needs to be a way to hide or remove that link

Thanks again for the great module

hadsie’s picture

hadsie commented
Version: 6.x-1.0-beta11 » 6.x-2.x-dev
Status: Postponed (maintainer needs more info) » Fixed

I think I'm going to leave this as is. I'd suggest using a form_alter or overriding the form template and setting:

$form['personal_message']['node_invite_tokens']['#access'] = FALSE;

That will hide the token field.

(Note that this applies to the 2.x branch, for older versions it might just be $form['node_invite_tokens']['#access'] = FALSE;)

esbon’s picture

esbon commented

Thanks, where do I put this code please?

hadsie’s picture

hadsie commented

you can put it in a hook_form_alter function in a custom module or a tpl.php file that overrides the node_invite_send form.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

davidshaw’s picture

davidshaw commented

I have beta12 working successfully on one install but not on another. I kept receiving the same error that the first 'email address is not valid'. I too tried to hide the token help so maybe that is how things broke, but i tried putting it back and still no joy. I checked permissions and for other modules that might conflict... but no joy after hours. After upgrading to the 2.x branch i can now send to multiple recipients again. :)

Dave Shaw

hadsie’s picture

hadsie commented

Thanks Dave,

Yeah I need to make a proper release from the 2.x branch. I believe it's quite stable now.