Merge pull request #28 from ChemiKyle/fix_log_digest_edge_cases

Fix log digest edge cases

Author: ChemiKyle

get-log-digest.php

@@ -27,7 +27,7 @@
 $results = $module->queryLogs("
 	select MAX(log_id) as max, MIN(log_id) as min
 	where external_module_id = ? and project_id = ?
-	AND timestamp >= ? and timestamp < ?
+	AND timestamp >= ? and timestamp < DATE_ADD(?, INTERVAL 1 DAY)
 ", [$this_module_id, $module->getProjectId(), $start, $end]);
 
 while($row = $results->fetch_assoc()){
@@ -36,6 +36,12 @@
 	$end_log_id = $start_log_id - LOG_WINDOW_SIZE;
 }
 
+// Prevent potential infinite loop if no log entries are found in provided time window
+if (is_null($start_log_id) || is_null($min_log_id)) {
+	echo "No activity found in the provided time window, please expand your search.";
+	exit();
+}
+
 $message_subtrings = $digestLog::createLikeStatements();
 
 // NOTE: probably don't need EM id or project_id