From: Alexander Merkle <alexander.merkle@lauterbach.com>
Date: Tue, 20 Jun 2023 10:07:45 +0000 (+0200)
Subject: Bug 1217882: ensure SQL querys from FlagTypeComment are valid
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=60c3e6495fc17067c04784fcaaff867667b0aef7;p=thirdparty%2Fbugzilla.git

Bug 1217882: ensure SQL querys from FlagTypeComment are valid

when $types is empty the SQL query is invalid
---

diff --git a/extensions/FlagTypeComment/Extension.pm b/extensions/FlagTypeComment/Extension.pm
index e5892fcbe..4a7e1bb11 100644
--- a/extensions/FlagTypeComment/Extension.pm
+++ b/extensions/FlagTypeComment/Extension.pm
@@ -72,7 +72,7 @@ sub _set_ftc_states {
   my $dbh = Bugzilla->dbh;
 
   my $ftc_flags;
-  my $db_result;
+  my $db_result = [];
   if ($file =~ /^admin\//) {
 
     # admin
@@ -121,11 +121,13 @@ sub _set_ftc_states {
 
     my $types = join(',', map { $_->id } @$flag_types);
     my $states = "'" . join("','", FLAGTYPE_COMMENT_STATES) . "'";
-    $db_result = $dbh->selectall_arrayref(
-      "SELECT type_id AS flagtype, on_status AS state, comment AS text
-               FROM flagtype_comments
-              WHERE type_id IN ($types) AND on_status IN ($states)", {Slice => {}}
-    );
+    if ($types) {
+      $db_result = $dbh->selectall_arrayref(
+        "SELECT type_id AS flagtype, on_status AS state, comment AS text
+                 FROM flagtype_comments
+                WHERE type_id IN ($types) AND on_status IN ($states)", {Slice => {}}
+      );
+    }
   }
 
   foreach my $row (@$db_result) {