diff options
author | Koosha Khajeh Moogahi <koosha.khajeh@gmail.com> | 2012-04-21 14:46:10 +0200 |
---|---|---|
committer | Frédéric Buclin <LpSolit@gmail.com> | 2012-04-21 14:46:10 +0200 |
commit | b66b079bb3ac4c736dd806b73de826faad0a8633 (patch) | |
tree | ca463efac55e1229c004e69f95521d1e3ae54ccc /importxml.pl | |
parent | ff9b4b0dbd044b3a643e5c7668023201022a7c82 (diff) | |
download | bugzilla-b66b079bb3ac4c736dd806b73de826faad0a8633.tar.gz bugzilla-b66b079bb3ac4c736dd806b73de826faad0a8633.tar.xz |
Bug 349015: importxml.pl throws a warning if there is no bug status defined
r/a=LpSolit
Diffstat (limited to 'importxml.pl')
-rwxr-xr-x | importxml.pl | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/importxml.pl b/importxml.pl index f7ce6e66e..9c334efa0 100755 --- a/importxml.pl +++ b/importxml.pl @@ -841,7 +841,6 @@ sub process_bug { my $valid_status = check_field('bug_status', scalar $bug_fields{'bug_status'}, undef, ERR_LEVEL ); - my $is_open = is_open_state($bug_fields{'bug_status'}); my $status = $bug_fields{'bug_status'} || undef; my $resolution = $bug_fields{'resolution'} || undef; @@ -891,7 +890,7 @@ sub process_bug { if ($status) { if($valid_status){ - if($is_open){ + if (is_open_state($status)) { if ($resolution) { $err .= "Resolution set on an open status.\n"; $err .= " Dropping resolution $resolution\n"; @@ -925,7 +924,7 @@ sub process_bug { } } } - else{ # $is_open is false + else { if (!$resolution) { $err .= "Missing Resolution. Setting status to "; if($everconfirmed){ |