From 2dd89e1a4c23e756ca245bcb245932ada6f53a4d Mon Sep 17 00:00:00 2001
From: Ryan Rathsam <ryanrath@buffalo.edu>
Date: Tue, 7 May 2019 10:02:31 -0400
Subject: [PATCH] Updated SQL construction for `tablesExist`

So it turns out that PHP prepared statement variable binding doesn't play well
with SQL `IN` statements. Just moving things around a bit so that instead of
supplying the `IN` clause w/ a bind parameter we instead just populate it in the
base query string.
---
 bin/acl-config | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/bin/acl-config b/bin/acl-config
index d3349fa93b..b98ac015e0 100755
--- a/bin/acl-config
+++ b/bin/acl-config
@@ -2127,12 +2127,6 @@ function verifyDatabase()
 function tablesExist(iDatabase $db, array $tables, $schema, $all = true)
 {
     $handle = $db->handle();
-    $query = <<<SQL
-SELECT *
-FROM information_schema.tables
-WHERE table_schema = :table_schema
-AND table_name in (:table_list);
-SQL;
 
     $tables = array_reduce(
         $tables,
@@ -2145,10 +2139,18 @@ SQL;
     );
 
     $tableList = implode(',', $tables);
+
+    $query = <<<SQL
+SELECT *
+FROM information_schema.tables
+WHERE table_schema = :table_schema
+AND table_name in ($tableList);
+SQL;
+
     $params = array(
-        ':table_schema' => $schema,
-        ':table_list' => $tableList
+        ':table_schema' => $schema
     );
+
     $results = $db->query(
         $query,
         $params