diff options
author | Max Kanat-Alexander <mkanat@bugzilla.org> | 2011-02-14 12:16:24 -0800 |
---|---|---|
committer | Max Kanat-Alexander <mkanat@bugzilla.org> | 2011-02-14 12:16:24 -0800 |
commit | ff3ae632c32d6d004de167b572e34e4f9a3542e1 (patch) | |
tree | 2b833754e3e1fb2ff272dde56ccdde2dafd0940b | |
parent | 142ebe8513a2796381104548f2869bd14c214793 (diff) | |
download | bugs-ff3ae632c32d6d004de167b572e34e4f9a3542e1.tar bugs-ff3ae632c32d6d004de167b572e34e4f9a3542e1.tar.gz bugs-ff3ae632c32d6d004de167b572e34e4f9a3542e1.tar.bz2 bugs-ff3ae632c32d6d004de167b572e34e4f9a3542e1.tar.xz bugs-ff3ae632c32d6d004de167b572e34e4f9a3542e1.zip |
Bug 633055: Make Bug.legal_values explicitly throw an error if you pass "undef"
for the "field" parameter
r=dkl, a=mkanat
-rw-r--r-- | Bugzilla/WebService/Bug.pm | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/Bugzilla/WebService/Bug.pm b/Bugzilla/WebService/Bug.pm index 124dd6b25..86c6ef5da 100644 --- a/Bugzilla/WebService/Bug.pm +++ b/Bugzilla/WebService/Bug.pm @@ -569,6 +569,10 @@ sub create { sub legal_values { my ($self, $params) = @_; + + defined $params->{field} + or ThrowCodeError('param_required', { param => 'field' }); + my $field = Bugzilla::Bug::FIELD_MAP->{$params->{field}} || $params->{field}; |