diff options
author | preed%sigkill.com <> | 2002-08-12 07:59:03 +0200 |
---|---|---|
committer | preed%sigkill.com <> | 2002-08-12 07:59:03 +0200 |
commit | 17b301e76d886afd5be8f4e9919afb4446e49405 (patch) | |
tree | fa2d4eca76d68d442880d2a3c34f38f31afa813a /globals.pl | |
parent | b5244171fb8589800372e02d649f4dbf15ee2390 (diff) | |
download | bugzilla-17b301e76d886afd5be8f4e9919afb4446e49405.tar.gz bugzilla-17b301e76d886afd5be8f4e9919afb4446e49405.tar.xz |
Bug 162217: fixed my bustage introduced in bug 160410; yes, I owe the whole Bugzilla team a good bottle of tequila now (ask Baloo for details)
Diffstat (limited to 'globals.pl')
-rw-r--r-- | globals.pl | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/globals.pl b/globals.pl index e54dff245..16cdba73b 100644 --- a/globals.pl +++ b/globals.pl @@ -1440,7 +1440,13 @@ sub Param ($) { # If it's still not defined, we're pimped. die "Can't find param named $value" if (! defined $::param{$value}); - if ($::param_type{$value} eq "m") { + ## Check to make sure the entry in $::param_type is there; if we don't, we + ## get 'use of uninitialized constant' errors (see bug 162217). + ## Interestingly enough, placing this check in the die above causes + ## deaths on some params (the "languages" param?) because they don't have + ## a type? Odd... seems like a bug to me... but what do I know? -jpr + + if (defined $::param_type{$value} && $::param_type{$value} eq "m") { my $valueList = eval($::param{$value}); return $valueList if (!($@) && ref($valueList) eq "ARRAY"); die "Multi-list param '$value' eval() failure ('$@'); data/params is horked"; |