Notifications do not have sender information in them

drupalnuts - December 6, 2008 - 18:10
Project:Organic groups
Version:6.x-2.x-dev
Component:og.module
Category:bug report
Priority:critical
Assigned:Unassigned
Status:active
Description

In the messaging framework you need to provide a 'sender_account'

$from = check_plain($message['sender_account']->name) . ' <'.$message['sender_account']->mail.'>';

but og on like 604 does from.

'from' => $user,

When a user replies to a message it comes back to the site admin address.

#1

drupalnuts - December 6, 2008 - 18:32

Well making that change in the code does not work for us either.
on line 125 of og_notifications we h ave

        notifications_lite_send($uid, $args['subject'], $args['body'])

We never pass the from param.

#2

emilyf - December 15, 2008 - 16:25

subscribing

#3

scedwar - February 5, 2009 - 13:30

We've noticed this discrepancy too. The sender address is different for plain notifications and og_notifications. This had the frustrating side effect of our SMTP server discarding some messages as spam, and others not. Fortunately we had access to the logs as there was not even an error message in the Drupal logs.

(IMHO notifications is not currently ready for use within og. There are far too many issues. This, and others.)

#4

DanielTheViking - August 19, 2009 - 11:14
Version:6.x-1.0» 6.x-2.x-dev
Priority:critical» normal

Still relevant in 2.x?

#5

drupalnuts - November 1, 2009 - 17:38
Priority:normal» critical

Yes, they still come from the site address rather then from the user who is taking the action.

Same with broadcast.

 
 

Drupal is a registered trademark of Dries Buytaert.