From 9b19d01d0c4830292410f28e757d9b8c20653faa Mon Sep 17 00:00:00 2001
From: jenz <unlozehq@gmail.com>
Date: Sun, 18 Jun 2023 20:34:45 +0200
Subject: [PATCH] moved DB connection validity to onmapstart instead of inside
 timer

---
 .../scripting/jenz_ban_detector.sp            | 31 +++++++++----------
 1 file changed, 15 insertions(+), 16 deletions(-)

diff --git a/jenz_ban_detector/scripting/jenz_ban_detector.sp b/jenz_ban_detector/scripting/jenz_ban_detector.sp
index 9f18a77c..48979ff2 100644
--- a/jenz_ban_detector/scripting/jenz_ban_detector.sp
+++ b/jenz_ban_detector/scripting/jenz_ban_detector.sp
@@ -239,28 +239,27 @@ public void sql_select_sb_bans(Database db, DBResultSet results, const char[] er
 }
 
 public Action start_checks(Handle hTimer)
+{
+    for (int i = 0; i < MaxClients; i++)
+    {
+        if (IsValidClient(i) && validate_state[i] == 0)
+        {
+            validate_state[i] = -1;
+            SQL_addEntry(i);
+        }
+    }
+}
+
+public void OnMapStart()
 {
     if (!g_dDatabase)
     {
         Database.Connect(SQL_OnDatabaseConnect, "jenz_ban_detector");
     }
-    else
+
+    if (!g_hDatabase_sourceban)
     {
-        if (!g_hDatabase_sourceban)
-        {
-            Database.Connect(SQL_OnDatabaseConnect_sb, "sourcebans");
-        }
-        else
-        {
-            for (int i = 0; i < MaxClients; i++)
-            {
-                if (IsValidClient(i) && validate_state[i] == 0)
-                {
-                    validate_state[i] = -1;
-                    SQL_addEntry(i);
-                }
-            }
-        }
+        Database.Connect(SQL_OnDatabaseConnect_sb, "sourcebans");
     }
 }