From b5d2e0dd1d887e2db2976b98598f543417ab0507 Mon Sep 17 00:00:00 2001
From: "justdave%syndicomm.com" <>
Date: Sat, 8 Mar 2003 16:29:01 +0000
Subject: Bug 196101: use ANSI-compliant SQL for group checks in sanitycheck

---
 sanitycheck.cgi | 26 ++++++++++++--------------
 1 file changed, 12 insertions(+), 14 deletions(-)

(limited to 'sanitycheck.cgi')

diff --git a/sanitycheck.cgi b/sanitycheck.cgi
index f5b8be210..a28858f43 100755
--- a/sanitycheck.cgi
+++ b/sanitycheck.cgi
@@ -664,24 +664,22 @@ if ($c) {
 }
 
 Status("Checking for bugs with groups violating their product's group controls");
-BugCheck("bugs, groups, bug_group_map
-         LEFT JOIN group_control_map
-         ON group_control_map.product_id = bugs.product_id
-         AND group_control_map.group_id = bug_group_map.group_id
-         WHERE bugs.bug_id = bug_group_map.bug_id
-         AND bug_group_map.group_id = groups.id
-         AND groups.isactive != 0
+BugCheck("bugs
+         INNER JOIN bug_group_map ON bugs.bug_id = bug_group_map.bug_id
+         INNER JOIN groups ON bug_group_map.group_id = groups.id
+         LEFT JOIN group_control_map ON bugs.product_id = group_control_map.product_id
+                                     AND bug_group_map.group_id = group_control_map.group_id
+         WHERE groups.isactive != 0
          AND ((group_control_map.membercontrol = " . CONTROLMAPNA . ")
          OR (group_control_map.membercontrol IS NULL))",
          "Have groups not permitted for their products");
 
-BugCheck("bugs, groups, group_control_map
-         LEFT JOIN bug_group_map
-         ON group_control_map.group_id = bug_group_map.group_id
-         AND bugs.bug_id = bug_group_map.bug_id
-         WHERE group_control_map.product_id = bugs.product_id
-         AND bug_group_map.group_id = groups.id
-         AND groups.isactive != 0
+BugCheck("bugs
+         INNER JOIN bug_group_map ON bugs.bug_id = bug_group_map.bug_id
+         INNER JOIN groups ON bug_group_map.group_id = groups.id
+         LEFT JOIN group_control_map ON bugs.product_id = group_control_map.product_id
+                                     AND bug_group_map.group_id = group_control_map.group_id
+         WHERE groups.isactive != 0
          AND group_control_map.membercontrol = " . CONTROLMAPMANDATORY . "
          AND bug_group_map.group_id IS NULL",
          "Are missing groups required for their products");
-- 
cgit v1.2.3-24-g4f1b