Change Project#notified_users to check for the 'all' notification option. #6541

The previous mail_notification? check would always pass since the
notifications where converted to strings and strings are always true.

Also changed Project#recipients to use #notified_users instead of duplicated
code.

Based on contribution by Felix Schäfer.

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4247 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
Eric Davis 2010-10-10 21:42:24 +00:00
parent d2986eb98f
commit 700c302fca
2 changed files with 53 additions and 2 deletions

View File

@ -382,12 +382,13 @@ class Project < ActiveRecord::Base
# Returns the mail adresses of users that should be always notified on project events
def recipients
members.select {|m| m.mail_notification? || m.user.mail_notification == 'all'}.collect {|m| m.user.mail}
notified_users.collect {|user| user.mail}
end
# Returns the users that should be notified on project events
def notified_users
members.select {|m| m.mail_notification? || m.user.mail_notification?}.collect {|m| m.user}
# TODO: User part should be extracted to User#notify_about?
members.select {|m| m.mail_notification? || m.user.mail_notification == 'all'}.collect {|m| m.user}
end
# Returns an array of all custom fields enabled for project issues

View File

@ -964,4 +964,54 @@ class ProjectTest < ActiveSupport::TestCase
end
end
context "#notified_users" do
setup do
@project = Project.generate!
@role = Role.generate!
@user_with_membership_notification = User.generate!(:mail_notification => 'selected')
Member.generate!(:project => @project, :roles => [@role], :principal => @user_with_membership_notification, :mail_notification => true)
@all_events_user = User.generate!(:mail_notification => 'all')
Member.generate!(:project => @project, :roles => [@role], :principal => @all_events_user)
@no_events_user = User.generate!(:mail_notification => 'none')
Member.generate!(:project => @project, :roles => [@role], :principal => @no_events_user)
@only_my_events_user = User.generate!(:mail_notification => 'only_my_events')
Member.generate!(:project => @project, :roles => [@role], :principal => @only_my_events_user)
@only_assigned_user = User.generate!(:mail_notification => 'only_assigned')
Member.generate!(:project => @project, :roles => [@role], :principal => @only_assigned_user)
@only_owned_user = User.generate!(:mail_notification => 'only_owner')
Member.generate!(:project => @project, :roles => [@role], :principal => @only_owned_user)
end
should "include members with a mail notification" do
assert @project.notified_users.include?(@user_with_membership_notification)
end
should "include users with the 'all' notification option" do
assert @project.notified_users.include?(@all_events_user)
end
should "not include users with the 'none' notification option" do
assert !@project.notified_users.include?(@no_events_user)
end
should "not include users with the 'only_my_events' notification option" do
assert !@project.notified_users.include?(@only_my_events_user)
end
should "not include users with the 'only_assigned' notification option" do
assert !@project.notified_users.include?(@only_assigned_user)
end
should "not include users with the 'only_owner' notification option" do
assert !@project.notified_users.include?(@only_owned_user)
end
end
end