Fix for bug 216008: Time Tracking: default values cause change bug

errors. Check if the time values haven't actually changed by using
integer comparison (instead of string comparison). r=jouni, a218=justdave.

diff --git a/process_bug.cgi b/process_bug.cgi
index b69b3da271626ff568d37a5eadf720e53191c5a6..d759aa8dbd731fd50bdaf78e4eb870fd17f6324a 100755 (executable)
--- a/process_bug.cgi
+++ b/process_bug.cgi
@@ -362,14 +362,20 @@ sub CheckCanChangeField {
             }
         }
     }
-    
+
     # Return true if they haven't changed this field at all.
     if ($oldvalue eq $newvalue) {
         return 1;
     }    
     elsif (trim($oldvalue) eq trim($newvalue)) {
         return 1;
+    # numeric fields need to be compared using == 
+    } elsif (($field eq "estimated_time" || $field eq "remaining_time") &&
+             $oldvalue == $newvalue) {
+        return 1;
     }
+        
+    
     
     # A resolution change is always accompanied by a status change. So, we 
     # always OK resolution changes; if they really can't do this, we will 
@@ -1158,6 +1164,7 @@ foreach my $id (@idlist) {
         }
         $i++;
     }
+
     $oldhash{'product'} = get_product_name($oldhash{'product_id'});
     if (!CanEditProductId($oldhash{'product_id'})) {
         ThrowUserError("product_edit_denied",