From 864ee35126ff8053ee46d559f9f4dd53dd871ffd Mon Sep 17 00:00:00 2001
From: "lpsolit%gmail.com" <>
Date: Thu, 20 Nov 2008 01:30:27 +0000
Subject: [PATCH] =?utf8?q?Bug=20465606:=20If=20a=20user=20doesn't=20have?=
 =?utf8?q?=20privs=20to=20change=20the=20target=20milestone,=20it=20gets?=
 =?utf8?q?=20set=20back=20to=20default=20-=20Patch=20by=20Fr=C3=83=C2=A9d?=
 =?utf8?q?=C3=83=C2=A9ric=20Buclin=20<LpSolit@gmail.com>=20r/a=3Dmkanat?=
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

---
 Bugzilla/Bug.pm | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm
index d57c0a802a..25c2146ef9 100644
--- a/Bugzilla/Bug.pm
+++ b/Bugzilla/Bug.pm
@@ -1953,13 +1953,13 @@ sub set_product {
         # just die if any of these are invalid.
         $self->set_component($comp_name);
         $self->set_version($vers_name);
-        if ($self->check_can_change_field('target_milestone', 0, 1)) {
-            $self->set_target_milestone($tm_name);
-        }
-        else {
+        if ($product_changed && !$self->check_can_change_field('target_milestone', 0, 1)) {
             # Have to set this directly to bypass the validators.
             $self->{target_milestone} = $product->default_milestone;
         }
+        else {
+            $self->set_target_milestone($tm_name);
+        }
     }
     
     if ($product_changed) {
-- 
2.47.3