diff options
| author | Byron Jones <bjones@mozilla.com> | 2012-04-12 07:33:50 +0200 |
|---|---|---|
| committer | Byron Jones <bjones@mozilla.com> | 2012-04-12 07:33:50 +0200 |
| commit | 6620c43ae99c3f79fb1450b5f11e9b3c57057429 (patch) | |
| tree | 19a67fb2ed2c2643896b6a623ff159720c0aee92 /extensions/GuidedBugEntry/Extension.pm | |
| parent | 8b3e405968a4611ff901000bdcb2ce4e6d4fd997 (diff) | |
| download | bugzilla-6620c43ae99c3f79fb1450b5f11e9b3c57057429.tar.gz bugzilla-6620c43ae99c3f79fb1450b5f11e9b3c57057429.tar.xz | |
Bug 744490: fix "uninitialized value" warning in GuidedBugEntry
Diffstat (limited to 'extensions/GuidedBugEntry/Extension.pm')
| -rw-r--r-- | extensions/GuidedBugEntry/Extension.pm | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/extensions/GuidedBugEntry/Extension.pm b/extensions/GuidedBugEntry/Extension.pm index 4fbc20611..796c1ad3b 100644 --- a/extensions/GuidedBugEntry/Extension.pm +++ b/extensions/GuidedBugEntry/Extension.pm @@ -67,10 +67,12 @@ sub enter_bug_start { # we use the __default__ format to bypass the guided entry # it isn't understood upstream, so remove it once a product # has been selected. - $cgi->delete('format') - if $cgi->param('format') - && $cgi->param('format') eq "__default__" - && $cgi->param('product') ne ''; + if ( + ($cgi->param('format') && $cgi->param('format') eq "__default__") + && ($cgi->param('product') && $cgi->param('product') ne '') + ) { + $cgi->delete('format') + } } sub _init_vars { |