From 264c8ccbdcbc18794c4fe25b29a23d270b36dfec Mon Sep 17 00:00:00 2001 From: Thomas Williams Date: Thu, 4 Jul 2024 17:07:16 +0100 Subject: [PATCH] Fixed defect with deleting old logs not working with 0 rows --- log.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/log.py b/log.py index 7b39ce6..86ee9c4 100644 --- a/log.py +++ b/log.py @@ -139,9 +139,11 @@ class logsManager: oldestLogQuery = f"SELECT TOP 1 {logTimeColumn} FROM {tableName} ORDER BY {logTimeColumn} ASC" cursor.execute(oldestLogQuery) - oldestLogTime = cursor.fetchone()[0] - if oldestLogTime is not None and oldestLogTime < datetime.now() - timedelta(days=int(logRetentionDays)): + if cursor.rowcount > 0: + oldestLogTime = cursor.fetchone()[0] + + if cursor.rowcount > 0 and oldestLogTime < datetime.now() - timedelta(days=int(logRetentionDays)): deleteQuery = f"DELETE FROM {tableName} WHERE {logTimeColumn} < ?" cursor.execute(deleteQuery, datetime.now() - timedelta(days=int(logRetentionDays)))