The sql statement for insert node_invite doesn't match the schema:
1. uid_invitee failed due to integer type. Is there an intention to only sent registered user email for this case.
2. statement still set status code as null, should be removed from the statement so it can take database default.

I like this module's simple and intuitive ui. that RSVP module is pretty much unuseable for consumer facing web... This module has great potential so let me know if you need help.

Comments

suydam’s picture

suydam commented

I'll be working on this tonight. Glad you like the UI.

suydam’s picture

suydam commented

Ok, I've worked on this for you.

1 - the uid_invitee really should have been named "email_invitee" to be semantically correct. It doesn't store a uid but an email address. I have changed this to make it easier to understand.

2 - I don't see the issue with the status field. It has a default of NULL but is inserted as "NEW" during the insert statement to node_invite. Can you clarify this a bit?

suydam’s picture

suydam commented
Status: Active » Fixed

I think version 0.5 fixes any bugs here. Re-open this or start a new one if I'm wrong. Thanks!

frednwright’s picture

frednwright commented

Brian,

Thanks for your work on this - I really appreciate your time and efforts here.

I wish I had better news, but here is what happened a few minutes ago:

I updated the module - ran update.php, and got a mysql error regarding "email_invitee" when I went to "My Invites" It also wiped out the RSVP pages for invitations that were existing.

I disabled the module, deleted my old invitations, the content type for invitations, tried installing again, ran update.php etc. and still got the same errors.

THEN I did something really dumb! I took a look at your install file and I thought the only table that really mattered was node_invites, so I dropped that off my database thinking I could get a clean install. Well, that was a bad idea. It is still broken because now the module will not install the table node_invites for some reason. I don't know why I thought I should screw around with a mysql table, but that will teach me.

would an uninstaller help remove the node_invite files still floating around? untill then I'm going to have to leave it disabled.

sorry for the problems, and I hope this may help you somehow in the dev.

thanks again,

Fred

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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

suydam’s picture

suydam commented
Status: Closed (fixed) » Fixed

Please updated to 5.x-0.6 or 6.x-1.0

Status: Fixed » Closed (fixed)

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