Join GitHub today
Advanced Notification sends wrong message when post includes a mention #2871
linc added a commit that referenced this issue
Aug 3, 2015
Solves issue: #2871 Notifications for new discussions with advanced notifications were done wrong: at first the HeadlineFormat.Discussion was set to $Activity['HeadlineFormat'], then it was checked if users were mentioned and were to be informed. If yes, $Activity['HeadlineFormat'] was set to HeadlineFormat.Mention. At last the users with advanced notification were notified but with the last HeadlineFormat. Simply changing the order to first inform the users with advaanced notification before changing the HeadlineFormat solved the problem.