From d5b4df9d5649032bf4fd3e36dc773922e7b57825 Mon Sep 17 00:00:00 2001
From: Jamie Banks <jamie@jdbnet.co.uk>
Date: Wed, 10 Jun 2026 23:20:40 +0100
Subject: [PATCH] feat: add folder filtering support to job reports and update
 UI filters

---
 internal/db/queries.go     | 20 ++++++++++++++++----
 internal/web/server.go     | 21 ++++++++++++++++++++-
 web/templates/history.html | 26 ++++++++++++++++++++++----
 3 files changed, 58 insertions(+), 9 deletions(-)

diff --git a/internal/db/queries.go b/internal/db/queries.go
index 3b9d50a..d272e90 100644
--- a/internal/db/queries.go
+++ b/internal/db/queries.go
@@ -2,6 +2,7 @@ package db
 
 import (
 	"database/sql"
+	"strings"
 )
 
 // Watch Folders
@@ -151,21 +152,32 @@ func AddJobReport(j Job, status string, encodedSize int64, sizeSaved int64, proc
 	return err
 }
 
-func GetJobReports(limit, offset int, statusFilter string) ([]JobReport, int, error) {
+func GetJobReports(limit, offset int, statusFilter string, folderFilter string) ([]JobReport, int, error) {
 	var total int
 	queryArgs := []interface{}{}
 	countQuery := `SELECT COUNT(*) FROM job_reports`
 	selectQuery := `SELECT id, file_path, media_type, status, original_size, encoded_size, size_saved, processing_time, target_resolution, ffmpeg_flags, error_message, created_at FROM job_reports`
-	whereClause := ""
+	
+	whereClauses := []string{}
 
 	if statusFilter != "" && statusFilter != "all" {
 		if statusFilter == "exclude_skipped" {
-			whereClause = ` WHERE status != 'skipped'`
+			whereClauses = append(whereClauses, `status != 'skipped'`)
 		} else {
-			whereClause = ` WHERE status = ?`
+			whereClauses = append(whereClauses, `status = ?`)
 			queryArgs = append(queryArgs, statusFilter)
 		}
 	}
+	
+	if folderFilter != "" && folderFilter != "all" {
+		whereClauses = append(whereClauses, `(file_path LIKE ? OR file_path = ?)`)
+		queryArgs = append(queryArgs, folderFilter+"/%", folderFilter)
+	}
+
+	whereClause := ""
+	if len(whereClauses) > 0 {
+		whereClause = " WHERE " + strings.Join(whereClauses, " AND ")
+	}
 
 	if err := DB.QueryRow(countQuery+whereClause, queryArgs...).Scan(&total); err != nil {
 		return nil, 0, err
diff --git a/internal/web/server.go b/internal/web/server.go
index 7946ad4..230fa5c 100644
--- a/internal/web/server.go
+++ b/internal/web/server.go
@@ -281,6 +281,7 @@ func (s *Server) handleHistory(w http.ResponseWriter, r *http.Request) {
 	limitStr := r.URL.Query().Get("limit")
 	pageStr := r.URL.Query().Get("page")
 	statusFilter := r.URL.Query().Get("status")
+	folderFilterIDStr := r.URL.Query().Get("folder")
 
 	limit := 50
 	if l, err := strconv.Atoi(limitStr); err == nil && l > 0 {
@@ -294,7 +295,21 @@ func (s *Server) handleHistory(w http.ResponseWriter, r *http.Request) {
 
 	offset := (page - 1) * limit
 
-	reports, total, err := db.GetJobReports(limit, offset, statusFilter)
+	folders, _ := db.GetWatchFolders()
+	
+	folderFilterPath := ""
+	folderFilterID := 0
+	if fID, err := strconv.Atoi(folderFilterIDStr); err == nil && fID > 0 {
+		folderFilterID = fID
+		for _, f := range folders {
+			if f.ID == fID {
+				folderFilterPath = f.FolderPath
+				break
+			}
+		}
+	}
+
+	reports, total, err := db.GetJobReports(limit, offset, statusFilter, folderFilterPath)
 	if err != nil {
 		http.Error(w, err.Error(), http.StatusInternalServerError)
 		return
@@ -316,6 +331,8 @@ func (s *Server) handleHistory(w http.ResponseWriter, r *http.Request) {
 		PrevPage     int
 		NextPage     int
 		FilterStatus string
+		FilterFolder int
+		Folders      []db.WatchFolder
 		Limit        int
 	}{
 		AuthEnabled:  s.cfg.Auth.Username != "",
@@ -328,6 +345,8 @@ func (s *Server) handleHistory(w http.ResponseWriter, r *http.Request) {
 		PrevPage:     page - 1,
 		NextPage:     page + 1,
 		FilterStatus: statusFilter,
+		FilterFolder: folderFilterID,
+		Folders:      folders,
 		Limit:        limit,
 	}
 	
diff --git a/web/templates/history.html b/web/templates/history.html
index 8d330a2..b833266 100644
--- a/web/templates/history.html
+++ b/web/templates/history.html
@@ -5,12 +5,21 @@
         <div style="display: flex; align-items: center; gap: 1rem; flex-wrap: wrap;">
             <div class="filter-bar">
                 <label for="status-filter" style="font-size: 0.875rem; color: var(--text-secondary);">Status:</label>
-                <select id="status-filter" class="filter-select" onchange="window.location.href='?status=' + this.value">
+                <select id="status-filter" class="filter-select" onchange="updateFilter('status', this.value)">
                     <option value="all" {{if eq .FilterStatus "all"}}selected{{end}}>All</option>
                     <option value="exclude_skipped" {{if eq .FilterStatus "exclude_skipped"}}selected{{end}}>Exclude Skipped</option>
                     <option value="success" {{if eq .FilterStatus "success"}}selected{{end}}>Success Only</option>
                     <option value="failed" {{if eq .FilterStatus "failed"}}selected{{end}}>Failed Only</option>
                 </select>
+                
+                <label for="folder-filter" style="font-size: 0.875rem; color: var(--text-secondary); margin-left: 0.5rem;">Folder:</label>
+                <select id="folder-filter" class="filter-select" onchange="updateFilter('folder', this.value)">
+                    <option value="0" {{if eq .FilterFolder 0}}selected{{end}}>All Folders</option>
+                    {{$filterFolder := .FilterFolder}}
+                    {{range .Folders}}
+                    <option value="{{.ID}}" {{if eq $filterFolder .ID}}selected{{end}}>{{.FolderPath}}</option>
+                    {{end}}
+                </select>
             </div>
             <input type="text" id="history-search" class="form-control" placeholder="Search page..." onkeyup="filterHistory()" style="max-width: 200px;">
             <span style="color: var(--text-secondary); font-size: 0.875rem; white-space: nowrap;">Total Records: <span id="record-count">{{.Total}}</span></span>
@@ -64,7 +73,7 @@
     <div style="display: flex; justify-content: space-between; align-items: center; margin-top: 1rem;">
         <div>
             <label for="limit-select" style="color: var(--text-secondary); font-size: 0.875rem; margin-right: 0.5rem;">Rows per page:</label>
-            <select id="limit-select" onchange="window.location.href='?limit=' + this.value + '&status={{.FilterStatus}}'" style="background: var(--surface); color: var(--text-primary); border: 1px solid var(--border); border-radius: 4px; padding: 0.25rem 0.5rem;">
+            <select id="limit-select" onchange="updateFilter('limit', this.value)" style="background: var(--surface); color: var(--text-primary); border: 1px solid var(--border); border-radius: 4px; padding: 0.25rem 0.5rem;">
                 <option value="10" {{if eq .Limit 10}}selected{{end}}>10</option>
                 <option value="50" {{if eq .Limit 50}}selected{{end}}>50</option>
                 <option value="100" {{if eq .Limit 100}}selected{{end}}>100</option>
@@ -72,7 +81,7 @@
         </div>
         <div style="display: flex; gap: 0.5rem; align-items: center;">
             {{if .HasPrev}}
-            <a href="?page={{.PrevPage}}&limit={{.Limit}}&status={{.FilterStatus}}" class="btn btn-sm btn-outline" style="text-decoration: none;">Previous</a>
+            <a href="?page={{.PrevPage}}&limit={{.Limit}}&status={{.FilterStatus}}&folder={{.FilterFolder}}" class="btn btn-sm btn-outline" style="text-decoration: none;">Previous</a>
             {{else}}
             <button class="btn btn-sm btn-outline" disabled>Previous</button>
             {{end}}
@@ -80,7 +89,7 @@
             <span style="color: var(--text-secondary); font-size: 0.875rem;">Page {{.CurrentPage}} of {{.TotalPages}}</span>
 
             {{if .HasNext}}
-            <a href="?page={{.NextPage}}&limit={{.Limit}}&status={{.FilterStatus}}" class="btn btn-sm btn-outline" style="text-decoration: none;">Next</a>
+            <a href="?page={{.NextPage}}&limit={{.Limit}}&status={{.FilterStatus}}&folder={{.FilterFolder}}" class="btn btn-sm btn-outline" style="text-decoration: none;">Next</a>
             {{else}}
             <button class="btn btn-sm btn-outline" disabled>Next</button>
             {{end}}
@@ -89,6 +98,15 @@
 </div>
 
 <script>
+function updateFilter(key, value) {
+    const url = new URL(window.location.href);
+    url.searchParams.set(key, value);
+    if (key !== 'limit' && key !== 'page') {
+        url.searchParams.set('page', '1');
+    }
+    window.location.href = url.toString();
+}
+
 function filterHistory() {
     var input, filter, table, tr, td, i, txtValue;
     input = document.getElementById("history-search");