From: Jean-Baptiste Barth <jeanbaptiste.barth@gmail.com>
Date: Sun, 26 Sep 2010 13:52:25 +0000 (+0000)
Subject: Fixed: don't reset watchers when changing tracker. #6479
X-Git-Tag: 1.1.0~332
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=2f380a7985929b8b53325569b7c1c5f5230b101c;p=redmine.git

Fixed: don't reset watchers when changing tracker. #6479

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4181 e93f8b46-1217-0410-a6f0-8f06a7374b81
---

diff --git a/app/controllers/issues_controller.rb b/app/controllers/issues_controller.rb
index 385bfa757..a48a9e8ff 100644
--- a/app/controllers/issues_controller.rb
+++ b/app/controllers/issues_controller.rb
@@ -284,6 +284,7 @@ private
   end
 
   # TODO: Refactor, lots of extra code in here
+  # TODO: Changing tracker on an existing issue should not trigger this
   def build_new_issue_from_params
     if params[:id].blank?
       @issue = Issue.new
@@ -302,7 +303,9 @@ private
     end
     if params[:issue].is_a?(Hash)
       @issue.safe_attributes = params[:issue]
-      @issue.watcher_user_ids = params[:issue]['watcher_user_ids'] if User.current.allowed_to?(:add_issue_watchers, @project)
+      if User.current.allowed_to?(:add_issue_watchers, @project) && @issue.new_record?
+        @issue.watcher_user_ids = params[:issue]['watcher_user_ids']
+      end
     end
     @issue.author = User.current
     @issue.start_date ||= Date.today