From 121badd2da4e4f4528d29948dd8ca6d44eddfe8e Mon Sep 17 00:00:00 2001
From: "lpsolit%gmail.com" <>
Date: Tue, 6 Feb 2007 22:57:01 +0000
Subject: Bug 348282: importxml.pl throws Bugzilla::Bug::ValidateTime warnings
 if the time value is not defined and param timetrackinggroup defined - Patch
 by Guzmán Brasó <gbn@hqso.net> r/a=LpSolit
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 importxml.pl | 44 +++++++++++++++++++++++++++-----------------
 1 file changed, 27 insertions(+), 17 deletions(-)

(limited to 'importxml.pl')

diff --git a/importxml.pl b/importxml.pl
index 6dff2af7f..f0fd0d8ec 100755
--- a/importxml.pl
+++ b/importxml.pl
@@ -791,26 +791,36 @@ sub process_bug {
           || undef;
         push( @values, $date );
         push( @query,  "deadline" );
-        eval {
-            Bugzilla::Bug::ValidateTime($bug_fields{'estimated_time'}, "e");
-        };
-        if (!$@){
-            push( @values, $bug_fields{'estimated_time'} );
-            push( @query,  "estimated_time" );
+        if ( defined $bug_fields{'estimated_time'} ) {
+            eval {
+                Bugzilla::Bug::ValidateTime($bug_fields{'estimated_time'}, "e");
+            };
+            if (!$@){
+                push( @values, $bug_fields{'estimated_time'} );
+                push( @query,  "estimated_time" );
+            }
         }
-        eval {
-            Bugzilla::Bug::ValidateTime($bug_fields{'remaining_time'}, "r");
-        };
-        if (!$@){
-            push( @values, $bug_fields{'remaining_time'} );
-            push( @query,  "remaining_time" );
+        if ( defined $bug_fields{'remaining_time'} ) {
+            eval {
+                Bugzilla::Bug::ValidateTime($bug_fields{'remaining_time'}, "r");
+            };
+            if (!$@){
+                push( @values, $bug_fields{'remaining_time'} );
+                push( @query,  "remaining_time" );
+            }
         }
-        eval {
-            Bugzilla::Bug::ValidateTime($bug_fields{'actual_time'}, "a");
-        };
-        if ($@){
+        if ( defined $bug_fields{'actual_time'} ) {
+            eval {
+                Bugzilla::Bug::ValidateTime($bug_fields{'actual_time'}, "a");
+            };
+            if ($@){
+                $bug_fields{'actual_time'} = 0.0;
+                $err .= "Invalid Actual Time. Setting to 0.0\n";
+            }
+        }
+        else {
             $bug_fields{'actual_time'} = 0.0;
-            $err .= "Invalid Actual Time. Setting to 0.0\n";
+            $err .= "Actual time not defined. Setting to 0.0\n";
         }
     }
 
-- 
cgit v1.2.3-24-g4f1b