From: jocuri%softhome.net <>
Date: Sat, 3 Jul 2004 07:26:28 +0000 (+0000)
Subject: Patch for bug 248001: Conversion of boolean conditions in SQL statements for better... 
X-Git-Tag: bugzilla-2.18rc1~31
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=00cc8677e25a98e81d70757cc02991e63ee3b975;p=thirdparty%2Fbugzilla.git

Patch for bug 248001: Conversion of boolean conditions in SQL statements for better DB independence; patch by David Lawrence <dkl@redhat.com>; r=joel, vladd; a=justdave.
---

diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm
index bef512e0f7..91cd0d8c82 100755
--- a/Bugzilla/Bug.pm
+++ b/Bugzilla/Bug.pm
@@ -342,11 +342,11 @@ sub groups {
              " LEFT JOIN user_group_map" .
              " ON user_group_map.group_id = groups.id" .
              " AND user_id = $::userid" .
-             " AND NOT isbless" .
+             " AND isbless = 0" .
              " LEFT JOIN group_control_map" .
              " ON group_control_map.group_id = groups.id" .
              " AND group_control_map.product_id = " . $self->{'product_id'} .
-             " WHERE isbuggroup");
+             " WHERE isbuggroup = 1");
 
     while (&::MoreSQLData()) {
         my ($groupid, $name, $description, $ison, $ingroup, $isactive,
diff --git a/buglist.cgi b/buglist.cgi
index 98b7ff116b..2607628d02 100755
--- a/buglist.cgi
+++ b/buglist.cgi
@@ -246,9 +246,9 @@ sub GetGroupsByUserId {
     SendSQL("
         SELECT DISTINCT  groups.id, name, description, isactive
                    FROM  groups, user_group_map
-                  WHERE  user_id = $userid AND NOT isbless
+                  WHERE  user_id = $userid AND isbless = 0
                     AND  user_group_map.group_id = groups.id
-                    AND  isbuggroup
+                    AND  isbuggroup = 1
                ORDER BY  description ");
 
     my @groups;
diff --git a/editgroups.cgi b/editgroups.cgi
index a0af0be122..4390b5d446 100755
--- a/editgroups.cgi
+++ b/editgroups.cgi
@@ -274,7 +274,7 @@ if ($action eq 'changeform') {
              " LEFT JOIN group_group_map as B" .
              " ON B.member_id = groups.id" .
              " AND B.grantor_id = $group_id" .
-             " AND B.isbless" .
+             " AND B.isbless = 1" .
              " WHERE groups.id != $group_id ORDER by name");
 
     while (MoreSQLData()) {
diff --git a/request.cgi b/request.cgi
index 43be698563..e330c2c832 100755
--- a/request.cgi
+++ b/request.cgi
@@ -81,8 +81,8 @@ sub queue {
     # should not have access.
     "            COUNT(DISTINCT ugmap.group_id) AS cntuseringroups, 
                 COUNT(DISTINCT bgmap.group_id) AS cntbugingroups, 
-                ((COUNT(DISTINCT ccmap.who) AND cclist_accessible) 
-                  OR ((bugs.reporter = $::userid) AND bugs.reporter_accessible) 
+                ((COUNT(DISTINCT ccmap.who) AND cclist_accessible = 1) 
+                  OR ((bugs.reporter = $::userid) AND bugs.reporter_accessible = 1) 
                   OR bugs.assigned_to = $::userid ) AS canseeanyway 
     " . 
     # Use the flags and flagtypes tables for information about the flags,