Fixed Trying to send email to user without email Server Error

[Jayt]

Affected version

2.0.2

Got four of these errors yesterday.

Server error log

• Exception: Email to [unknown] failed setup: Trying to send email to user without email (ID: 130)
• src/XF/Mail/Mail.php:86
• Generated by: Twisted
• Feb 3, 2018 at 11:36 AM

Stack trace
#0 src/XF/Service/Conversation/Notifier.php(113): XF\Mail\Mail->setToUser(Object(XFRM\XF\Entity\User))
#1 src/XF/Service/Conversation/Notifier.php(65): XF\Service\Conversation\Notifier->_sendNotifications('reply', Array, Object(XF\Entity\ConversationMessage))
#2 src/XF/Service/Conversation/Replier.php(172): XF\Service\Conversation\Notifier->notifyReply(Object(XF\Entity\ConversationMessage))
#3 src/XF/Service/Conversation/Replier.php(151): XF\Service\Conversation\Replier->sendNotifications()
#4 src/XF/Service/ValidateAndSavableTrait.php(40): XF\Service\Conversation\Replier->_save()
#5 src/XF/Pub/Controller/Conversation.php(486): XF\Service\Conversation\Replier->save()
#6 src/XF/Mvc/Dispatcher.php(249): XF\Pub\Controller\Conversation->actionAddReply(Object(XF\Mvc\ParameterBag))
#7 src/XF/Mvc/Dispatcher.php(88): XF\Mvc\Dispatcher->dispatchClass('XF:Conversation', 'AddReply', 'json', Object(XF\Mvc\ParameterBag), 'conversations', Object(XF\Pub\Controller\Conversation), NULL)
#8 src/XF/Mvc/Dispatcher.php(41): XF\Mvc\Dispatcher->dispatchLoop(Object(XF\Mvc\RouteMatch))
#9 src/XF/App.php(1889): XF\Mvc\Dispatcher->run()
#10 src/XF.php(328): XF\App->run()
#11 index.php(13): XF::runApp('XF\\Pub\\App')
#12 {main}

 

[Fethi.dz]

I upgraded my forum today from1.5.16 to 2.0.2 and I got the same error above.

 

[Chris D]

This is fixed for the next release, we'll no longer attempt to notify users of conversations who have no email address.