From 31e903ebeb167a47fe15ca829f450138ce69483b Mon Sep 17 00:00:00 2001
From: Jef Roosens <roosensjef@gmail.com>
Date: Thu, 19 May 2022 22:43:38 +0200
Subject: [PATCH] feat(server): partial implementation of BuildLog API filter

---
 CHANGELOG.md      |  6 ++++++
 src/db/logs.v     | 32 ++++++++++++++++++++++++++++----
 src/models/logs.v | 13 +++++++++++++
 3 files changed, 47 insertions(+), 4 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 8c336fe4..170c97c6 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -10,6 +10,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 
 * Database migrations
+* Query parameters for GitRepo API to filter responses
+* Respective CLI flags for new GitRepo API parameters
+
+### Changed
+
+* Refactor of main types into `models` module
 
 ## [0.3.0-alpha.2](https://git.rustybever.be/vieter/vieter/src/tag/0.3.0-alpha.2)
 
diff --git a/src/db/logs.v b/src/db/logs.v
index 129ec4e1..49f905ba 100644
--- a/src/db/logs.v
+++ b/src/db/logs.v
@@ -1,13 +1,37 @@
 module db
 
-import models { BuildLog }
+import models { BuildLog, BuildLogFilter }
+import time
 
 // get_build_logs returns all BuildLog's in the database.
-pub fn (db &VieterDb) get_build_logs() []BuildLog {
-	res := sql db.conn {
-		select from BuildLog order by id
+pub fn (db &VieterDb) get_build_logs(filter BuildLogFilter) []BuildLog {
+	mut where_parts := []string{}
+
+	if filter.repo != 0 {
+		where_parts << 'repo_id == $filter.repo'
 	}
 
+	if filter.before != time.Time{} {
+		where_parts << 'start_time < $filter.before.unix_time()'
+	}
+
+	if filter.after != time.Time{} {
+		where_parts << 'start_time < $filter.after.unix_time()'
+	}
+
+	where_str := if where_parts.len > 0 {
+		' where ' + where_parts.map('($it)').join(' and ')
+	} else {
+		''
+	}
+
+	query := 'select from BuildLog' + where_str
+	res := db.conn.exec(query)
+
+/* 	res := sql db.conn { */
+/* 		select from BuildLog where filter.repo == 0 || repo_id == filter.repo order by id */
+/* 	} */
+
 	return res
 }
 
diff --git a/src/models/logs.v b/src/models/logs.v
index 173336f4..1bfc7a69 100644
--- a/src/models/logs.v
+++ b/src/models/logs.v
@@ -26,3 +26,16 @@ pub fn (bl &BuildLog) str() string {
 
 	return str
 }
+
+[params]
+pub struct BuildLogFilter {
+pub mut:
+	limit                u64 = 25
+	offset               u64
+	repo                 int
+	before               time.Time
+	after                time.Time
+	exit_codes_whitelist []u8
+	exit_codes_blacklist []u8
+	arch                 string
+}