"I don't want to be notified of changes that I make myself" as Default for all User...

git-svn-id: http://svn.redmine.org/redmine/trunk@15288 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/models/user_preference.rb b/app/models/user_preference.rb
index 461b63840097e73bae79f0c22b7c2e05b0804a1a..0fdbfb50809cce1b7f75ca83b8fe9e08864f28e5 100644 (file)
--- a/app/models/user_preference.rb
+++ b/app/models/user_preference.rb
@@ -28,6 +28,9 @@ class UserPreference < ActiveRecord::Base
     if new_record? && !(attributes && attributes.key?(:hide_mail))
       self.hide_mail = Setting.default_users_hide_mail?
     end
+    if new_record? && !(attributes && attributes.key?(:no_self_notified))
+      self.no_self_notified = true
+    end
     self.others ||= {}
   end
 

diff --git a/test/fixtures/user_preferences.yml b/test/fixtures/user_preferences.yml
index 04abb4343ee15e530c201010e5ad287f20ee18e7..14243e0bbc35fe2293522d05e898a7e0d610cd2a 100644 (file)
--- a/test/fixtures/user_preferences.yml
+++ b/test/fixtures/user_preferences.yml
@@ -2,6 +2,7 @@
 user_preferences_001: 
   others: |
     --- 
+    :no_self_notified: false
     :my_page_layout: 
       left: 
       - latestnews
@@ -17,6 +18,15 @@ user_preferences_001:
 user_preferences_002: 
   others: |
     --- 
+    :no_self_notified: false
+  
+  id: 2
+  user_id: 2
+  hide_mail: true
+user_preferences_003: 
+  others: |
+    --- 
+    :no_self_notified: false
     :my_page_layout: 
       left: 
       - latestnews
@@ -26,6 +36,6 @@ user_preferences_002:
       top: 
       - calendar
   
-  id: 2
+  id: 3
   user_id: 3
   hide_mail: false

diff --git a/test/functional/issues_controller_test.rb b/test/functional/issues_controller_test.rb
index 2dd0f4618b466f3471e742d1101d1bc0301ff18f..5cb9f87308476a6c5ffaf16676b61bf8c1ff494e 100644 (file)
--- a/test/functional/issues_controller_test.rb
+++ b/test/functional/issues_controller_test.rb
@@ -19,7 +19,7 @@ require File.expand_path('../../test_helper', __FILE__)
 
 class IssuesControllerTest < ActionController::TestCase
   fixtures :projects,
-           :users, :email_addresses,
+           :users, :email_addresses, :user_preferences,
            :roles,
            :members,
            :member_roles,

diff --git a/test/functional/issues_custom_fields_visibility_test.rb b/test/functional/issues_custom_fields_visibility_test.rb
index dc169b7c1688e9b081d51197f3afdd24b6d60c94..51b057f4cb6277738b328ebd08ee740d39deb1a0 100644 (file)
--- a/test/functional/issues_custom_fields_visibility_test.rb
+++ b/test/functional/issues_custom_fields_visibility_test.rb
@@ -20,7 +20,7 @@ require File.expand_path('../../test_helper', __FILE__)
 class IssuesCustomFieldsVisibilityTest < ActionController::TestCase
   tests IssuesController
   fixtures :projects,
-           :users, :email_addresses,
+           :users, :email_addresses, :user_preferences,
            :roles,
            :members,
            :member_roles,

diff --git a/test/functional/messages_controller_test.rb b/test/functional/messages_controller_test.rb
index 87bba5fcf19bc02cb0d473af4a871afa2224b27e..36b0f1dd502d52b67ebdcf2a65cbc47219147104 100644 (file)
--- a/test/functional/messages_controller_test.rb
+++ b/test/functional/messages_controller_test.rb
@@ -18,7 +18,7 @@
 require File.expand_path('../../test_helper', __FILE__)
 
 class MessagesControllerTest < ActionController::TestCase
-  fixtures :projects, :users, :email_addresses, :members, :member_roles, :roles, :boards, :messages, :enabled_modules
+  fixtures :projects, :users, :email_addresses, :user_preferences, :members, :member_roles, :roles, :boards, :messages, :enabled_modules
 
   def setup
     User.current = nil

diff --git a/test/unit/issue_test.rb b/test/unit/issue_test.rb
index 3daca7184c30e2a1290e8ad5946eae3e3a374d32..a07b18ff5b502e73ddf21fda340acd65ed481b6f 100644 (file)
--- a/test/unit/issue_test.rb
+++ b/test/unit/issue_test.rb
@@ -18,7 +18,7 @@
 require File.expand_path('../../test_helper', __FILE__)
 
 class IssueTest < ActiveSupport::TestCase
-  fixtures :projects, :users, :email_addresses, :members, :member_roles, :roles,
+  fixtures :projects, :users, :email_addresses, :user_preferences, :members, :member_roles, :roles,
            :groups_users,
            :trackers, :projects_trackers,
            :enabled_modules,

diff --git a/test/unit/mail_handler_test.rb b/test/unit/mail_handler_test.rb
index 1cba22f808b3df726f18c1a3601d9ea28e3b1864..f08f2a6cdfd0cf23560de0faf2abaf2808240c2e 100644 (file)
--- a/test/unit/mail_handler_test.rb
+++ b/test/unit/mail_handler_test.rb
@@ -22,7 +22,7 @@ require File.expand_path('../../test_helper', __FILE__)
 class MailHandlerTest < ActiveSupport::TestCase
   fixtures :users, :projects, :enabled_modules, :roles,
            :members, :member_roles, :users,
-           :email_addresses,
+           :email_addresses, :user_preferences,
            :issues, :issue_statuses,
            :workflows, :trackers, :projects_trackers,
            :versions, :enumerations, :issue_categories,

diff --git a/test/unit/user_preference_test.rb b/test/unit/user_preference_test.rb
index a2f3c7f1f76a3a3b495d5842b8f971bdc19f457f..7303d178f182c2716950ab230a82cbe8968cec4f 100644 (file)
--- a/test/unit/user_preference_test.rb
+++ b/test/unit/user_preference_test.rb
@@ -32,6 +32,11 @@ class UserPreferenceTest < ActiveSupport::TestCase
     end
   end
 
+  def test_no_self_notified_should_default_to_true
+    preference = UserPreference.new
+    assert_equal true, preference.no_self_notified
+  end
+
   def test_create
     user = User.new(:firstname => "new", :lastname => "user", :mail => "newuser@somenet.foo")
     user.login = "newuser"
@@ -67,11 +72,6 @@ class UserPreferenceTest < ActiveSupport::TestCase
     assert_kind_of Hash, up.others
   end
 
-  def test_others_should_be_blank_after_initialization
-    pref = User.new.pref
-    assert_equal({}, pref.others)
-  end
-
   def test_reading_value_from_nil_others_hash
     up = UserPreference.new(:user => User.new)
     up.others = nil