diff options
author | Frédéric Buclin <LpSolit@gmail.com> | 2010-05-14 15:14:54 +0200 |
---|---|---|
committer | Frédéric Buclin <LpSolit@gmail.com> | 2010-05-14 15:14:54 +0200 |
commit | 1f1d6a5bcd5b77e8fbb8d827a614cdc1e3323c4d (patch) | |
tree | e81070c857145057099933ebf4b1038186ef90a1 /request.cgi | |
parent | Bug 559398 - "Old incomplete data warning showing on activity view due to cus... (diff) | |
download | bugzilla-1f1d6a5bcd5b77e8fbb8d827a614cdc1e3323c4d.tar.gz bugzilla-1f1d6a5bcd5b77e8fbb8d827a614cdc1e3323c4d.tar.bz2 bugzilla-1f1d6a5bcd5b77e8fbb8d827a614cdc1e3323c4d.zip |
Bug 565881: 'flag_status_invalid' and 'request_queue_group_invalid' should be user errors, not code errors
a=LpSolit
Diffstat (limited to 'request.cgi')
-rwxr-xr-x | request.cgi | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/request.cgi b/request.cgi index e1137b3f2..2dd12c49b 100755 --- a/request.cgi +++ b/request.cgi @@ -332,8 +332,7 @@ sub validateStatus { return if !defined $status; grep($status eq $_, qw(? +- + - all)) - || ThrowCodeError("flag_status_invalid", - { status => $status }); + || ThrowUserError("flag_status_invalid", { status => $status }); trick_taint($status); return $status; } @@ -343,8 +342,7 @@ sub validateGroup { return if !defined $group; grep($group eq $_, qw(requester requestee category type)) - || ThrowCodeError("request_queue_group_invalid", - { group => $group }); + || ThrowUserError("request_queue_group_invalid", { group => $group }); trick_taint($group); return $group; } |