Bug #6869

Report: Email notifications not going out

Added by Matt Gold about 5 years ago. Updated about 5 years ago.

Priority name:
Category name:
Email Notifications
Target version:
Start date:
Due date:
% Done:


Estimated time:


I received a report that email notifications are not being sent for group forum posts. Can you please check into the matter? No email notifications went out for the last two posts here --


#1 Updated by Boone Gorges about 5 years ago

  • Assignee changed from Boone Gorges to Raymond Hoh

Forum posts were causing fatal errors of the following form:

[Tue Nov 29 14:45:32 2016] [error] [pid 16668] sapi_apache2.c(325): [client] PHP Fatal error: Call to undefined method BP_Reply_By_Email::prepend_rbe_marker_to_content() in /var/www/html/commons/www/wp-content/plugins/bp-reply-by-email/bp-rbe-core.php on line 246

I'm not sure what happened here. I did have to move something around on the production server this morning, during which time it's possible that I removed some changes that'd been staged but not committed on production. Ray, do you have any ideas?

I manually added the missing method, based on previous changesets. Ray, please check `git diff wp-content/plugins/bp-reply-by-email` to make sure that I did the right thing :-D

#2 Updated by Matt Gold about 5 years ago

Hi Boone -- is there any possibility that this is related to the changes Lihua just made to the email server? I thought I saw an email from him about that.

#3 Updated by Boone Gorges about 5 years ago

The only change I recall was to the log file locations. If there were a mistake in the mail configuration, the server would not be sending any emails at all. But it is, so I don't think it's related.

#4 Updated by Raymond Hoh about 5 years ago

  • Status changed from Assigned to Resolved

Odd, 1.10.x branch has the prepend_rbe_marker_to_content() method, but master branch didn't.

It looks like this recent merge commit from 1.10.x to master caused the issue:

I've rectified this by re-applying the missing lines for RBE to master branch as a hotfix:

#5 Updated by Boone Gorges about 5 years ago

Thanks for fixing this, Ray!

Also available in: Atom PDF