Merge remote-tracking branch 'origin/module/stock' into module/stock

Signed-off-by: Stephan Richter <s.richter@srsoftware.de>

core/src/main/java/de/srsoftware/umbrella/core/Constants.java

@@ -96,6 +96,7 @@ public class Constants {
 	public static final String PARENT_TASK_ID     = "parent_task_id";
 	public static final String PASS               = "pass";
 	public static final String PASSWORD           = "password";
+	public static final String PATH               = "path";
 	public static final String PERMISSION         = "permission";
 	public static final String POST               = "POST";
 	public static final String PRIORITY           = "priority";

frontend/src/App.svelte

@@ -99,6 +99,8 @@
         <Route path="/project/:id/view"            component={ViewPrj} />
         <Route path="/search"                      component={Search} />
         <Route path="/stock"                       component={Stock} />
+        <Route path="/stock/location/:location_id" component={Stock} />
+        <Route path="/stock/:owner/:owner_id/item/:item_id" component={Stock} />
         <Route path="/tags"                        component={TagList} />
         <Route path="/tags/use/:tag"               component={TagUses} />
         <Route path="/task"                        component={TaskList} />

frontend/src/routes/project/Kanban.svelte

@@ -47,7 +47,7 @@
     function updateUrl(){
         let url = window.location.origin + window.location.pathname;
         if (filter_input) url += '?filter=' + encodeURI(filter_input);
-        window.history.replaceState(history.state, '', url);
+        window.history.replaceState(window.history.state, '', url);
     }
 
     async function create(name,user_id,state){

frontend/src/routes/stock/Index.svelte

@@ -1,6 +1,6 @@
 <script>
     import { onMount }      from 'svelte';
-    import { api }          from '../../urls.svelte';
+    import { api, drop, get, patch } from '../../urls.svelte';
     import { error, yikes } from '../../warn.svelte';
     import { t }            from '../../translations.svelte';
 
@@ -17,10 +17,18 @@
     let location        = $state(null);
     let draggedItem     = $state(null)
     let draggedLocation = $state(null)
+    let { item_id, location_id, owner, owner_id } = $props();
 
     $effect(() => {
         // This effect runs whenever `location` changes
-        if (location !== null) item = null;
+        if (location !== null) {
+            item = null;
+            setLocationUrl();
+        }
+    });
+
+    $effect(() => {
+        if (item !== null) setItemUrl();
     });
     let properties = $state(null);
     let top_level = $state(null);
@@ -28,22 +36,13 @@
     async function deleteLocation(loc){
         if (!confirm(t('confirm_delete',{element:loc.name}))) return;
         const url = api(`stock/location/${loc.id}`);
-        const res = await fetch(url,{
+        const res = await drop(url);
-            credentials: 'include',
-            method: 'DELETE',
-        });
         if (res.ok){
             yikes();
             unlistLocation(loc);
         } else error(res);
     }
 
-    function unlistLocation(loc){
-        for (var owner of top_level){
-            if (owner.locations && dropNestedLocation(owner.locations,loc)) break;
-        }
-    }
-
     function drag_item(item){
         draggedLocation = null;
         draggedItem = item;
@@ -68,11 +67,7 @@
     async function move_dragged_to(new_loc){
         const data = draggedItem ? { item : draggedItem.id, target: new_loc.id } : { parent_location_id: new_loc.id }
         const url = api(draggedItem ? 'stock/move_item' : `stock/location/${draggedLocation.id}`);
-        const res = await fetch(url,{
+        const res = await patch(url,data);
-            credentials : 'include',
-            method      : 'PATCH',
-            body        : JSON.stringify(data)
-        });
         if (res.ok){
             yikes();
             location        = new_loc;
@@ -86,8 +81,8 @@
 
     async function loadLocation(){
         if (!location) return null;
-        const url = api(`stock/location/${location.id}`)
+        const url = api(`stock/location/${location.id}`);
-        const res = await fetch(url,{credentials:'include'});
+        const res = await get(url);
         if (res.ok){
             yikes();
             return res.json();
@@ -97,9 +92,24 @@
         }
     }
 
+    async function loadPath(){
+        if (!location_id) return;
+        const url = api(`stock/location/${location_id}`);
+        const res = await get(url);
+        if (res.ok){
+            yikes();
+            var path = res.json();
+
+        } else {
+            error(res);
+            return null;
+        }
+
+    }
+
     async function loadProperties(){
         const url = api('stock/properties')
-        const res = await fetch(url,{credentials:'include'});
+        const res = await get(url);
         if (res.ok){
             var json = await res.json();
             var dict = {}
@@ -112,7 +122,7 @@
 
     async function loadUserLocations(){
         const url = api('stock/locations/of_user')
-        const res = await fetch(url,{credentials:'include'});
+        const res = await get(url);
         if (res.ok){
             top_level = await res.json();
             yikes();
@@ -121,6 +131,7 @@
 
     function load(){
         loadUserLocations();
+        loadPath();
         loadProperties();
     }
 
@@ -140,11 +151,7 @@
         const data = {};
         data[field] = newValue;
         const url = api(`stock/location/${location.id}`);
-        const res = await fetch(url,{
+        const res = await patch(url,data);
-            credentials: 'include',
-            method:'PATCH',
-            body:JSON.stringify(data)
-        });
         if (res.ok){
             yikes();
             return true;
@@ -154,10 +161,34 @@
         }
     }
 
+    function setItemUrl(){
+        var owner = `/${item.owner.type}/${item.owner.id}`
+        var code = `/item/${item.owner_number}`
+        let url = window.location.origin + '/stock' + owner + code;
+        window.history.replaceState(window.history.state, '', url);
+    }
+
+    function setLocationUrl(){
+       let url = window.location.origin + '/stock/location/' + location.id;
+       window.history.replaceState(window.history.state, '', url);
+    }
+
+    function unlistLocation(loc){
+        for (var owner of top_level){
+            if (owner.locations && dropNestedLocation(owner.locations,loc)) break;
+        }
+    }
+
     onMount(load);
 </script>
 
 <h2>{t('Stock')}</h2>
+<span class="warn">
+    {#if location_id}location id: {location_id}{/if}
+    {#if owner}owner type: {owner},{/if}
+    {#if owner_id}owner id: {owner_id},{/if}
+    {#if item_id}item id: {item_id}{/if}
+</span>
 <div class="grid3">
     <div class="locations">
         {#if top_level}

frontend/src/routes/stock/Locations.svelte

@@ -1,5 +1,5 @@
 <script>
-    import { api } from '../../urls.svelte';
+    import { api, get } from '../../urls.svelte';
     import { error, yikes } from '../../warn.svelte';
     import { t }   from '../../translations.svelte';
 
@@ -41,7 +41,7 @@
             delete location.locations;
         } else {
             const url = api(`stock/locations/below/${location.id}`);
-            const res = await fetch(url,{credentials:'include'});
+            const res = await get(url);
             if (res.ok){
                 yikes();
                 location.locations = await res.json();
@@ -67,11 +67,7 @@
             parent: parent
         }
         const url = api('stock/location');
-        const res = await fetch(url,{
+        const res = await post(url,data);
-            credentials: 'include',
-            method: 'POST',
-            body: JSON.stringify(data)
-        });
         if (res.ok){
             yikes;
             const saved = await res.json();

stock/src/main/java/de/srsoftware/umbrella/stock/SqliteDb.java

@@ -286,6 +286,30 @@ public class SqliteDb extends BaseDb implements StockDb {
 		}
 	}
 
+	@Override
+	public Map<String, Object> pathToLocation(Location target) {
+		var root = new HashMap<String,Object>();
+		var location = loadLocation(target.id());
+		root.put(NAME,location.name());
+		root.put(ID,location.id());
+		try {
+			while (!is0(location.parent())) {
+				var rs = select(ALL).from(TABLE_LOCATIONS).where(ID, equal(location.parent())).exec(db);
+				var parent = DbLocation.of(rs);
+				rs.close();
+				var current = new HashMap<String,Object>();
+				current.put(NAME,parent.name());
+				current.put(ID,parent.id());
+				current.put(PATH,root);
+				root = current;
+				location = parent;
+			}
+		} catch (SQLException e){
+			throw databaseException("Failed to load path to location {0}",target);
+		}
+		return root;
+	}
+
 	private void replaceItemsTable() throws SQLException {
 		db.prepareStatement(format("DROP TABLE {0}",TABLE_ITEMS)).execute();
 		db.prepareStatement(format("ALTER TABLE {0} RENAME TO {1}","items_temp",TABLE_ITEMS)).execute();

stock/src/main/java/de/srsoftware/umbrella/stock/StockDb.java

@@ -5,6 +5,7 @@ import de.srsoftware.umbrella.core.api.Owner;
 import de.srsoftware.umbrella.core.model.*;
 import de.srsoftware.umbrella.core.model.Location;
 import java.util.Collection;
+import java.util.Map;
 
 public interface StockDb {
 	Property                addNewProperty(long itemId, String name, Object value, String unit);
@@ -16,9 +17,9 @@ public interface StockDb {
 	Collection<DbLocation>  listUserLocations(UmbrellaUser userId);
 	Item                 	loadItem(long id);
 	DbLocation          	loadLocation(long locationId);
+	long					nextItemNumberFor(Owner owner);
+	Map<String,Object>  	pathToLocation(Location location);
 	DbLocation 				save(DbLocation location);
 	Item     				save(Item item);
 	Property   				setProperty(long itemId, long existingPropId, Object value);
-
-	long nextItemNumberFor(Owner owner);
 }

stock/src/main/java/de/srsoftware/umbrella/stock/StockModule.java

@@ -165,8 +165,8 @@ public class StockModule extends BaseHandler implements StockService {
 			case USER -> List.of(owner.id());
 			case null, default -> throw unprocessable("Unprocessable owner type: {0}",owner.type());
 		};
-
+		var pathToLocation = stockDb.pathToLocation(location);
-		return sendContent(ex,Map.of(ITEMS,items,USERS,userIds));
+		return sendContent(ex,Map.of(ITEMS,items,USERS,userIds,PATH,pathToLocation));
 	}
 
 	private boolean getLocations(Path path, UmbrellaUser user, HttpExchange ex) throws IOException {