From 8bb44d0d7f2aecf02f787a3838b0c1c4bf87b203 Mon Sep 17 00:00:00 2001
From: Stephan Richter <s.richter@srsoftware.de>
Date: Mon, 18 Aug 2025 01:23:29 +0200
Subject: [PATCH] working on time list

Signed-off-by: Stephan Richter <s.richter@srsoftware.de>
---
 frontend/src/routes/document/List.svelte      |  4 +--
 frontend/src/routes/document/TimeList.svelte  |  6 ++--
 .../de/srsoftware/umbrella/time/SqliteDb.java | 31 +++++++++++++++++--
 .../de/srsoftware/umbrella/time/TimeDb.java   |  5 ++-
 .../srsoftware/umbrella/time/TimeModule.java  | 16 +++++++---
 translations/src/main/resources/de.json       |  2 +-
 web/src/main/resources/web/css/bloodshed.css  |  1 +
 web/src/main/resources/web/css/default.css    |  1 +
 web/src/main/resources/web/css/winter.css     |  1 +
 9 files changed, 52 insertions(+), 15 deletions(-)

diff --git a/frontend/src/routes/document/List.svelte b/frontend/src/routes/document/List.svelte
index bd930c2..5c6d793 100644
--- a/frontend/src/routes/document/List.svelte
+++ b/frontend/src/routes/document/List.svelte
@@ -27,7 +27,7 @@
         }
 
         if (company_id) {
-            for (let comp of companies){
+            for (let comp of Object.keys(companies)){
                 if (comp.id == company_id){
                     load(comp);
                     break;
@@ -80,7 +80,7 @@
 </script>
 
 <fieldset>
-    <legend>{selected_company ? t( 'list_of',selected_company.name) : t('document_list')}</legend>
+    <legend>{selected_company ? t( 'docs_of_company',{company:selected_company.name}) : t('document_list')}</legend>
     {#if error}
     <div class="error">{error}</div>
     {/if}
diff --git a/frontend/src/routes/document/TimeList.svelte b/frontend/src/routes/document/TimeList.svelte
index 160f7ed..e2f8390 100644
--- a/frontend/src/routes/document/TimeList.svelte
+++ b/frontend/src/routes/document/TimeList.svelte
@@ -49,8 +49,8 @@
 <div>
     <h1>Times</h1>
     {#if projects}
-    {#each projects as project,idx1}
-    <button onclick={() => loadTimes(project.id)}>{project.name}</button>
+    {#each Object.entries(projects) as [pid,project]}
+    <button onclick={() => loadTimes(+pid)}>{project.name}</button>
     {/each}
     {/if}
     {#if times}
@@ -63,4 +63,4 @@
     </div>
     {/each}
     {/if}
-</div>
\ No newline at end of file
+</div>
diff --git a/time/src/main/java/de/srsoftware/umbrella/time/SqliteDb.java b/time/src/main/java/de/srsoftware/umbrella/time/SqliteDb.java
index d33736e..a4d0efe 100644
--- a/time/src/main/java/de/srsoftware/umbrella/time/SqliteDb.java
+++ b/time/src/main/java/de/srsoftware/umbrella/time/SqliteDb.java
@@ -1,10 +1,12 @@
 /* © SRSoftware 2025 */
 package de.srsoftware.umbrella.time;
 
+import static de.srsoftware.tools.jdbc.Condition.equal;
 import static de.srsoftware.tools.jdbc.Condition.in;
 import static de.srsoftware.tools.jdbc.Query.SelectQuery.ALL;
 import static de.srsoftware.tools.jdbc.Query.select;
 import static de.srsoftware.umbrella.core.Constants.ID;
+import static de.srsoftware.umbrella.core.Constants.USER_ID;
 import static de.srsoftware.umbrella.time.Constants.*;
 
 import de.srsoftware.umbrella.core.exceptions.UmbrellaException;
@@ -24,7 +26,7 @@ public class SqliteDb implements TimeDb {
 	}
 
 	@Override
-	public Collection<Time> listTimes(Collection<Long> taskIds) throws UmbrellaException {
+	public HashMap<Long,Time> listTimes(Collection<Long> taskIds) throws UmbrellaException {
 		try {
 			var rs = select(ALL).from(TABLE_TASK_TIMES).where(TASK_ID,in(taskIds.toArray())).exec(db);
 			var mapFromTimesToTasks = new HashMap<Long,HashSet<Long>>();
@@ -35,11 +37,34 @@ public class SqliteDb implements TimeDb {
 			}
 			rs.close();
 			rs = select(ALL).from(TABLE_TIMES).where(ID,in(mapFromTimesToTasks.keySet().toArray())).exec(db);
-			var times = new HashSet<Time>();
+			var times = new HashMap<Long,Time>();
 			while (rs.next()) {
 				var time = Time.of(rs);
 				time.taskIds().addAll(mapFromTimesToTasks.get(time.id()));
-				times.add(time);
+				times.put(time.id(),time);
+			}
+			rs.close();
+			return times;
+		} catch (SQLException e) {
+			throw new UmbrellaException("Failed to load times for task list");
+		}
+	}
+
+	@Override
+	public HashMap<Long, Time> listUserTimes(long userId) {
+		try {
+			var rs = select(ALL).from(TABLE_TIMES).where(USER_ID,equal(userId)).exec(db);
+			var times = new HashMap<Long,Time>();
+			while (rs.next()) {
+				var time = Time.of(rs);
+				times.put(time.id(),time);
+			}
+			rs.close();
+			rs = select(ALL).from(TABLE_TASK_TIMES).where(TIME_ID,in(times.keySet().toArray())).exec(db);
+			while (rs.next()){
+				var time = times.get(rs.getLong(TIME_ID));
+				time.taskIds().add(rs.getLong(TASK_ID));
+				System.out.println(time);
 			}
 			rs.close();
 			return times;
diff --git a/time/src/main/java/de/srsoftware/umbrella/time/TimeDb.java b/time/src/main/java/de/srsoftware/umbrella/time/TimeDb.java
index 8e0f919..2ef9715 100644
--- a/time/src/main/java/de/srsoftware/umbrella/time/TimeDb.java
+++ b/time/src/main/java/de/srsoftware/umbrella/time/TimeDb.java
@@ -4,7 +4,10 @@ package de.srsoftware.umbrella.time;
 import de.srsoftware.umbrella.core.exceptions.UmbrellaException;
 import de.srsoftware.umbrella.core.model.Time;
 import java.util.Collection;
+import java.util.HashMap;
 
 public interface TimeDb {
-	Collection<Time> listTimes(Collection<Long> taskIds) throws UmbrellaException;
+	HashMap<Long,Time> listTimes(Collection<Long> taskIds) throws UmbrellaException;
+
+	HashMap<Long,Time>  listUserTimes(long userId);
 }
diff --git a/time/src/main/java/de/srsoftware/umbrella/time/TimeModule.java b/time/src/main/java/de/srsoftware/umbrella/time/TimeModule.java
index 5b4151f..005e401 100644
--- a/time/src/main/java/de/srsoftware/umbrella/time/TimeModule.java
+++ b/time/src/main/java/de/srsoftware/umbrella/time/TimeModule.java
@@ -5,6 +5,7 @@ import static de.srsoftware.umbrella.core.ConnectionProvider.connect;
 import static de.srsoftware.umbrella.core.Constants.*;
 import static de.srsoftware.umbrella.core.Paths.LIST;
 import static de.srsoftware.umbrella.core.ResponseCode.HTTP_NOT_IMPLEMENTED;
+import static de.srsoftware.umbrella.core.Util.mapValues;
 import static de.srsoftware.umbrella.core.exceptions.UmbrellaException.forbidden;
 import static de.srsoftware.umbrella.core.exceptions.UmbrellaException.missingFieldException;
 import static de.srsoftware.umbrella.time.Constants.*;
@@ -82,9 +83,12 @@ public class TimeModule extends BaseHandler implements TimeService {
 
 	private boolean getUserTimes(UmbrellaUser user, HttpExchange ex) throws IOException {
 		// TODO
+		var taskIds = new HashSet<Long>();
 		Map<Long, Project> projects = projectService().listUserProjects(user.id(), true);
-		//timeDb.
-		return sendEmptyResponse(HTTP_NOT_IMPLEMENTED,ex);
+		for (var pid : projects.keySet()) taskIds.addAll(taskService().listProjectTasks(pid).keySet());
+		var times = timeDb.listTimes(taskIds);
+		times.putAll(timeDb.listUserTimes(user.id()));
+		return sendContent(ex,mapValues(times));
 	}
 
 	/*
@@ -138,9 +142,11 @@ public class TimeModule extends BaseHandler implements TimeService {
 		Map<Long,Task> tasksOfProject = taskService().listProjectTasks(projectId);
 
 		List<Map<String, Object>> times = timeDb.listTimes(tasksOfProject.keySet())
-				.stream().filter(not(Time::isClosed))
-				.sorted(Comparator.comparing(Time::start))
-				.map(time -> {
+				.entrySet().stream()
+				.filter(entry -> !entry.getValue().isClosed())
+				.sorted(Comparator.comparing(entry ->entry.getValue().start()))
+				.map(entry -> {
+					var time = entry.getValue();
 					var map = time.toMap();
 					var timeTasks = time.taskIds().stream().map(tasksOfProject::get).map(Task::toMap).toList();
 					map.put(TASKS,timeTasks);
diff --git a/translations/src/main/resources/de.json b/translations/src/main/resources/de.json
index f00b05a..14910ac 100644
--- a/translations/src/main/resources/de.json
+++ b/translations/src/main/resources/de.json
@@ -51,6 +51,7 @@
   "description": "Beschreibung",
   "detail": "Details",
   "display_closed_tasks": "abgeschlossene Aufgaben anzeigen",
+  "docs_of_company": "Dokumente von {company}",
   "document": "Dokument",
   "document_list": "Dokumente",
   "documents": "Dokumente",
@@ -99,7 +100,6 @@
   "language": "Sprache",
   "last_customer_number": "zuletzt vergebene Kundennummer",
   "list": "Dokumente",
-  "list_of": "Dokumente von {0}",
   "LIST_USERS": "Nutzer auflisten",
   "loading": "lade…",
   "loading_data": "Daten werden geladen…",
diff --git a/web/src/main/resources/web/css/bloodshed.css b/web/src/main/resources/web/css/bloodshed.css
index 8de15f5..cc0212f 100644
--- a/web/src/main/resources/web/css/bloodshed.css
+++ b/web/src/main/resources/web/css/bloodshed.css
@@ -91,6 +91,7 @@ td, tr{
 .position_selector{
     background: rgba(0,0,0,0.7);
     backdrop-filter: blur(3px);
+    z-index: 120;
 }
 .task.cancelled > a {
     text-decoration: line-through;
diff --git a/web/src/main/resources/web/css/default.css b/web/src/main/resources/web/css/default.css
index 0af1d7e..44911d5 100644
--- a/web/src/main/resources/web/css/default.css
+++ b/web/src/main/resources/web/css/default.css
@@ -88,6 +88,7 @@ td, tr{
 .position_selector{
     background: rgba(0,0,0,0.7);
     backdrop-filter: blur(3px);
+    z-index: 120;
 }
 .task.cancelled > a {
     text-decoration: line-through;
diff --git a/web/src/main/resources/web/css/winter.css b/web/src/main/resources/web/css/winter.css
index b68f53a..d7948ad 100644
--- a/web/src/main/resources/web/css/winter.css
+++ b/web/src/main/resources/web/css/winter.css
@@ -88,6 +88,7 @@ td, tr{
 .position_selector{
     background: rgba(0,0,0,0.7);
     backdrop-filter: blur(3px);
+    z-index: 120;
 }
 .task.cancelled > a {
     text-decoration: line-through;