Posted by lirazsiri on June 19, 2011 at 9:13pm
2 followers
Jump to:
| Project: | Subscriptions |
| Version: | 6.x-1.4 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (won't fix) |
Issue Summary
When a non-logged in user posts a comment, the subscription notification message sets comment_name as variable_get('anonymous') (I.e., 'Anonymous' or 'Guest') instead of using the name provided by the commenter.
If Joe isn't logged in but provides his name when he submits a comment, it's much more useful for subscriptions to send out a notification identifying the author of the comment as Joe rather than as Anonymous.
See patch.
| Attachment | Size |
|---|---|
| subscriptions_anonname.patch | 886 bytes |
Comments
#1
I agree, but in that case we need a new variable, something like !comment_name_is_verified, which is 0 or 1.
#2
No follow-up.