Closed (fixed)
Project:
Signup
Version:
5.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
7 Mar 2007 at 23:58 UTC
Updated:
24 Mar 2007 at 20:00 UTC
Jump to comment: Most recent file
Hello
I've installed the HEAD version of the signup module while running Drupal 5.1
It lets me add signups and I can see a list of who has signed up, but I am getting the following two error messages.
1. When someone does signup for something, after they press submit they get:
* warning: mail() [function.mail]: Failed to connect to mailserver at "localhost" port 25, verify your "SMTP" and "smtp_port" setting in php.ini or use ini_set() in C:\www\drupal\includes\common.inc on line 1970.
* warning: mail() [function.mail]: Failed to connect to mailserver at "localhost" port 25, verify your "SMTP" and "smtp_port" setting in php.ini or use ini_set() in C:\www\drupal\includes\common.inc on line 1970.
(it always shows up twice) and no confirmation emails are sent.
2. Whenever I as admin try to edit an event I get:
: Object of class stdClass could not be converted to string in C:\www\drupal\modules\signup\signup.module on line 957.
I did a search but couldn't find anything on this issue - is it something I'm likely to be able to sort out, or should I wait for the official 5.x release?
Thanks in advence for your help
Tom
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | signup_forms_5x.patch.txt | 1.37 KB | dww |
| #2 | signup_forms_php5_error.patch.txt | 694 bytes | dww |
Comments
Comment #1
add1sun commentedWell, issue number one has to do with your mailserver setup and isn't specific to signup. I can however replicate issue number 2. I get the same error. Tried it on an event content type as well as a non-event. The error doesn't seem to affect the function at all since you can still edit, change dates, change signup settings, etc and it isn't effected by signups open/closed.
Using D5.1 and the latest HEAD signup (signup.module,v 1.79 2007/03/03 07:14:57)
Comment #2
dwwre: #1 -- yeah, your site just isn't configured for email. search elsewhere for help with that.
re: #2 -- please try this patch. php5 is more picky than php4 about such things. ;)
Comment #3
add1sun commentednope, still getting the error with the patch on PHP 5.2.0. :( At least you're right that it is a PHP 5 error, since 4.3.11 worked no problem.
Comment #4
dwwactually, after doing some debugging, reading the docs, and getting some wisdom from eaton, i now realize that the existing
signup_forms()implementation is very broken. we should only be doing this re-mapping magic if we see a signup_user_cancel_form. attached patch should fix the problems, and then resolve the php5 warning. needs heavy testing, especially the "signups" tab on a signup-enabled node, since it's screwing with some deep FAPI plumbing.Comment #5
add1sun commentedYessir, that patch does the trick. Hammered the signups tab pretty well on a number of nodes, turning things off and on and cancelling, etc. Worked fine on both PHP 4 and 5.
Comment #6
dwwcommitted to HEAD. thanks for the help testing!
Comment #7
Tom_Mason commentedFantastic, works perfectly now. Thanks for your help.
Tom
Comment #8
dww"backported" (really, same patch committed) to DRUPAL-5 branch, too. thanks to add1sun for reminding me. ;)
Comment #9
(not verified) commented