Actions
Patch #42445
closed
Remove extra condition in User#notify_about?
Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Code cleanup/refactoring
Target version:
-
Start date:
Due date:
% Done:
0%
Estimated time:
Description
In User#notify_about?, the case of mail_notification.blank? is never. Because User#set_mail_notification (before_create) sets Setting.default_notification_option if blank.
So we can remove its condition. This patch removes it.
User#notify_about?source:tags/6.0.4/app/models/user.rb#L818User#set_mail_notificationsource:tags/6.0.4/app/models/user.rb#L161- before_create source:tags/6.0.4/app/models/user.rb#L133
Files
Updated by 
Updated by
Actions