failure to send notification email for comments on messages
Submitted by tartan on Thu, 04/10/2008 - 20:38.
| Project: | ProjectPier |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed - duplicate |
Jump to:
Description
PP v 0.8.0.2
Apache/2.0.58
PHP 5.2.6RC3-pl0-gentoo (cli) (built: Apr 8 2008 21:33:21)
Email notifications of new messages work fine. Comments on those messages, however, result in a failure in cache/log.php, even when the particular message has subscribers.
Session "default" started at 2008-04-10T20:11:04+0000
#1 ERROR: Error: Undefined variable: recipients in '/var/www/localhost/htdocs/project/application/models/notifier/Notifier.class.php' on line 139 (error code: 8)
Time since start: 1.43069601059 seconds
I had the same issue. Looked around a bit, and noticed a typo:
on file
base_dir/application/models/notifier/Notifier.class.php
around line 139
it says:
return self::sendEmail($recipients,
It _should_ say:
return self::sendEmail($recepients,
best,
mickey
This solved it! So it's a spelling error, that isn't seen in other files.
A patch for that was provided here: http://projectpier.org/node/721#comment-1405
It was committed to the trunk of the SVN repository, so will be included in the next release.
Sorry about the inconvenience!
Tim
And actually, it should say "recipients" and not "recepients". The problem is that the original code contained both spelling, some corrections were made but some were missed.
The patch fixes all the spelling for "recipient".
Thanks, Tim. I noticed that most other places said "recipient", but this routine had "recepient". Can't wait until the next release.