Bug 180966 - fix misc warnings

r,a=myk

webtools/bugzilla/Bugzilla/User.pm

@@ -267,7 +267,7 @@ sub match_field {
         next if !defined($vars->{'mform'}->{$field});
 
         # Skip it if this is a --do_not_change-- field
-        next if $dontchange eq $vars->{'form'}->{$field};
+        next if $dontchange && $dontchange eq $vars->{'form'}->{$field};
 
         # We need to move the query to $raw_field, where it will be split up,
         # modified by the search, and put back into $::FORM and $::MFORM

webtools/bugzilla/process_bug.cgi

@@ -109,6 +109,8 @@ if (defined $::FORM{'dup_id'} && $::FORM{'knob'} eq "duplicate") {
 
 ValidateComment($::FORM{'comment'});
 
+$::FORM{'dontchange'} = '' unless exists $::FORM{'dontchange'};
+
 # If the bug(s) being modified have dependencies, validate them
 # and rebuild the list with the validated values.  This is important
 # because there are situations where validation changes the value
@@ -1202,13 +1204,13 @@ foreach my $id (@idlist) {
     SendSQL("select now()");
     $timestamp = FetchOneColumn();
 
+    if (UserInGroup(Param('timetrackinggroup'))) {
         if ($::FORM{'work_time'} > 99999.99) {
             ThrowUserError("value_out_of_range", {field => 'work_time'});
         }
         if (defined $::FORM{'comment'} || defined $::FORM{'work_time'}) {
             if ($::FORM{'work_time'} != 0 && 
                 (!defined $::FORM{'comment'} || $::FORM{'comment'} =~ /^\s*$/)) {
-        
                 ThrowUserError('comment_required');
             } else {
                 AppendComment($id, $::COOKIE{'Bugzilla_login'}, $::FORM{'comment'},
@@ -1220,6 +1222,7 @@ foreach my $id (@idlist) {
                 }
             }
         }
+    }
 
     if (@::legal_keywords) {
         # There are three kinds of "keywordsaction": makeexact, add, delete.

webtools/bugzilla/showdependencytree.cgi

@@ -164,7 +164,7 @@ sub GetBug {
          $bug->{'assignee_email'}) = FetchSQLData();
      }
     
-    $bug->{'open'} = IsOpenedState($bug->{'status'});
+    $bug->{'open'} = $bug->{'exists'} && IsOpenedState($bug->{'status'});
     $bug->{'dependencies'} = [];
     
     return $bug;