diff options
author | Gervase Markham <gerv@mozilla.org> | 2014-04-16 21:26:49 +0200 |
---|---|---|
committer | Frédéric Buclin <LpSolit@gmail.com> | 2014-04-16 21:26:49 +0200 |
commit | 060fa9003b9177a73f02ba5ea561bb5be306c74a (patch) | |
tree | 35f2ad988f42b917db7fcbe598356fd8b8ca1d06 /Bugzilla/User.pm | |
parent | 6f4b2629b2899e09a98ac42283d3f872dc6c1f29 (diff) | |
download | bugzilla-060fa9003b9177a73f02ba5ea561bb5be306c74a.tar.gz bugzilla-060fa9003b9177a73f02ba5ea561bb5be306c74a.tar.xz |
Bug 995988: can_edit_product() can throw warnings if SUM() returns NULL
r=LpSolit a=justdave
Diffstat (limited to 'Bugzilla/User.pm')
-rw-r--r-- | Bugzilla/User.pm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Bugzilla/User.pm b/Bugzilla/User.pm index 1f0acee3d..5cebe728b 100644 --- a/Bugzilla/User.pm +++ b/Bugzilla/User.pm @@ -906,7 +906,7 @@ sub can_edit_product { CASE WHEN canedit = 1 AND group_id IN ($groups) THEN 1 ELSE 0 END AS cnt_group_member FROM group_control_map WHERE product_id = $prod_id) AS p"); - return ($cnt_can_edit == 0 or $cnt_group_member > 0); + return (!$cnt_can_edit or $cnt_group_member); } else { # For and-groups, a user needs to be in all canedit groups. Therefore |