implemented GUI update on cloning items

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

backend/src/main/java/de/srsoftware/umbrella/backend/Application.java

@@ -12,6 +12,7 @@ import de.srsoftware.tools.ColorLogger;
 import de.srsoftware.umbrella.bookmarks.BookmarkApi;
 import de.srsoftware.umbrella.company.CompanyModule;
 import de.srsoftware.umbrella.contact.ContactModule;
+import de.srsoftware.umbrella.core.SettingsService;
 import de.srsoftware.umbrella.core.Util;
 import de.srsoftware.umbrella.core.exceptions.UmbrellaException;
 import de.srsoftware.umbrella.documents.DocumentApi;
@@ -88,6 +89,7 @@ public class Application {
 			new WebHandler().bindPath("/").on(server);
 			new WikiModule(config).bindPath("/api/wiki").on(server);
 			new FileModule(config).bindPath("/api/files").on(server);
+			new SettingsService(config).bindPath("/api/settings").on(server);
 		} catch (Exception e) {
 			LOG.log(ERROR,"Startup failed",e);
 			System.exit(-1);

bus/src/main/java/de/srsoftware/umbrella/messagebus/MessageApi.java

@@ -49,11 +49,11 @@ public class MessageApi extends BaseHandler{
 					if (++counter > 300) counter = sendBeacon(addr,stream);
 				} else {
 					var event = eventQueue.removeFirst();
-					//if (event.isIntendedFor(user.get())) {
+					if (event.isIntendedFor(user.get())) {
 						LOG.log(DEBUG, "sending event to {0}", addr);
 						sendEvent(stream, event);
 						counter = 0;
-					//}
+					}
 				}
 			}
 			LOG.log(INFO,"{0} disconnected from event stream.",addr);

bus/src/main/java/de/srsoftware/umbrella/messagebus/MessageBus.java

@@ -12,7 +12,7 @@ public class MessageBus {
 
 	private MessageBus(){}
 
-	public void dispatch(Event event){
+	public void dispatch(Event<?> event){
 		new Thread(() -> { // TODO: use thread pool
 			try {
 				Thread.sleep(100);

bus/src/main/java/de/srsoftware/umbrella/messagebus/events/ItemEvent.java

@@ -0,0 +1,49 @@
+package de.srsoftware.umbrella.messagebus.events;
+
+import de.srsoftware.umbrella.core.ModuleRegistry;
+import de.srsoftware.umbrella.core.api.Owner;
+import de.srsoftware.umbrella.core.constants.Field;
+import de.srsoftware.umbrella.core.model.*;
+
+import java.util.Collection;
+import java.util.List;
+
+import static de.srsoftware.umbrella.core.constants.Field.*;
+import static de.srsoftware.umbrella.core.model.Translatable.t;
+
+public class ItemEvent extends Event<Item>{
+	public ItemEvent(UmbrellaUser initiator, String module, Item item, EventType type) {
+		super(initiator, module, item, type);
+	}
+
+	@Override
+	public Collection<UmbrellaUser> audience() {
+		Owner owner = payload().location().resolve().owner().resolve();
+		if (owner instanceof UmbrellaUser user) return List.of(user);
+		if (owner instanceof Company company) return ModuleRegistry.companyService().getMembers(company.id());
+		return List.of();
+	}
+
+	@Override
+	public Translatable describe() {
+		return switch (eventType()){
+			case CREATE -> describeCreate();
+			case null, default -> null;
+		};
+	}
+
+	private Translatable describeCreate() {
+		var loc  = payload().location().resolve().name();
+		return t("{user} added \"{item}\" to \"{location}\"", USER,initiator().name(), ITEM, payload().name(), LOCATION, loc);
+	}
+
+
+	@Override
+	public Translatable subject() {
+		var loc  = payload().location().resolve().name();
+		return switch (eventType()){
+			case CREATE -> t("A new item has been added to \"{location}\":",LOCATION,loc);
+			case null, default -> null;
+		};
+	}
+}

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

@@ -0,0 +1,119 @@
+/* © SRSoftware 2025 */
+package de.srsoftware.umbrella.core;
+
+import static de.srsoftware.umbrella.core.ModuleRegistry.userService;
+import static de.srsoftware.umbrella.core.constants.Constants.CLASS;
+import static de.srsoftware.umbrella.core.constants.Field.*;
+import static de.srsoftware.umbrella.core.constants.Path.MENU;
+import static de.srsoftware.umbrella.core.constants.Text.*;
+import static de.srsoftware.umbrella.core.constants.Text.USERS;
+import static java.text.MessageFormat.format;
+
+import com.sun.net.httpserver.HttpExchange;
+import de.srsoftware.configuration.Configuration;
+import de.srsoftware.tools.Mappable;
+import de.srsoftware.tools.Path;
+import de.srsoftware.tools.SessionToken;
+import de.srsoftware.umbrella.core.constants.Module;
+import de.srsoftware.umbrella.core.constants.Text;
+import de.srsoftware.umbrella.core.exceptions.UmbrellaException;
+import de.srsoftware.umbrella.core.model.Token;
+import de.srsoftware.umbrella.core.model.UmbrellaUser;
+import java.io.IOException;
+import java.util.*;
+import org.json.JSONObject;
+
+public class SettingsService extends BaseHandler {
+
+	private final Configuration config;
+
+	public SettingsService(Configuration config) {
+		this.config = config;
+	}
+
+	@Override
+	public boolean doGet(Path path, HttpExchange ex) throws IOException {
+		addCors(ex);
+		try {
+			Optional<Token> token = SessionToken.from(ex).map(Token::of);
+			var user = userService().loadUser(token);
+			if (user.isEmpty()) return unauthorized(ex);
+			var head = path.pop();
+			return switch (head) {
+				case MENU -> getMenuSettings(user.get(), ex);
+				case null, default -> super.doGet(path, ex);
+			};
+		} catch (UmbrellaException e) {
+			return send(ex, e);
+		}
+	}
+
+	private record MenuEntry(int pos, String module, String clazz, String title) implements Mappable {
+		public static MenuEntry of(int pos, String module){
+			return new MenuEntry(pos, module, module, module);
+		}
+
+		public static MenuEntry of(int pos, String module, String title){
+			return new MenuEntry(pos,module,module,title);
+		}
+		public static MenuEntry of(int pos, String module, String clazz, String title){
+			return new MenuEntry(pos,module,clazz,title);
+		}
+
+		@Override
+		public Map<String, Object> toMap() {
+			return Map.of(MODULE,module,CLASS,clazz,TITLE,title);
+		}
+	}
+	private boolean getMenuSettings(UmbrellaUser user, HttpExchange ex) throws IOException {
+		Optional<JSONObject> modules = config.get("umbrella.modules");
+		if (modules.isEmpty()) throw UmbrellaException.missingConfig("umbrella.modules");
+
+		var entries = new ArrayList<MenuEntry>();
+		entries.add(MenuEntry.of(1, Module.USER, USERS));
+		entries.add(MenuEntry.of(2, Module.COMPANY, COMPANIES));
+		entries.add(MenuEntry.of(3, Module.PROJECT,Text.PROJECTS));
+		entries.add(MenuEntry.of(4,Module.TASK,Text.TASKS));
+		entries.add(MenuEntry.of(5,Module.TAGS));
+		entries.add(MenuEntry.of(6,Module.DOCUMENT,"doc",Text.DOCUMENTS));
+		entries.add(MenuEntry.of(7,Module.BOOKMARK,"mark",BOOKMARKS));
+		entries.add(MenuEntry.of(8,Module.NOTES,"note",Text.NOTES));
+		entries.add(MenuEntry.of(9,Module.FILES,"file", FILES));
+		entries.add(MenuEntry.of(10,Module.TIME, Text.TIMETRACKING));
+		entries.add(MenuEntry.of(11,Module.WIKI));
+		entries.add(MenuEntry.of(12,Module.CONTACT, CONTACTS));
+		entries.add(MenuEntry.of(13,Module.STOCK));
+		entries.add(MenuEntry.of(14,Module.MESSAGE, MESSAGES));
+		entries.add(MenuEntry.of(15,Module.POLL,Text.POLLS));
+
+		for (var i=0; i<entries.size(); i++){
+			var entry = entries.get(i);
+			var key = format("umbrella.modules.{0}.menuIndex",entry.module);
+			Optional<Integer> val = config.get(key);
+			if (val.isEmpty()) {
+				key = format("umbrella.modules.{0}.menu_index",entry.module);
+				val = config.get(key);
+			}
+			if (val.isPresent()) {
+				var index = val.get();
+				if (index<0) {
+					entries.remove(i);
+					i--;
+					continue;
+				} else {
+					entry = MenuEntry.of(index,entry.module,entry.clazz,entry.title);
+					entries.set(i,entry);
+				}
+			}
+			key = format("umbrella.modules.{0}.baseUrl",entry.module);
+			Optional<String> baseUrl = config.get(key);
+			if (baseUrl.isPresent()) {
+				entry = MenuEntry.of(entry.pos,baseUrl.get(), entry.clazz,entry.title);
+				entries.set(i,entry);
+			}
+		}
+		var list = entries.stream().sorted((a,b) -> a.pos - b.pos)
+				.map(MenuEntry::toMap).toList();
+		return sendContent(ex,list);
+	}
+}

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

@@ -12,7 +12,10 @@ public class Constants {
 		// prevent instantiation
 	}
 
-	public static final String COUNT          = "COUNT(*)";
+	public static final String CLASS                   = "class";
+	public static final String CONFIG_SESSION_DURATION = "umbrella.session.duration";
+	public static final String COUNT                   = "COUNT(*)";
+
 	public static final String FALLBACK_LANG  = "de";
 	public static final String HOME           = "home";
 	public static final String JSONARRAY      = "json array";
@@ -20,8 +23,10 @@ public class Constants {
 	public static final String KEEP_ALIVE     = "keep-alive";
 	public static final String NO_CACHE       = "no-cache";
 	public static final String NONE           = "none";
-	public static final String TABLE_SETTINGS = "settings";
+
+	public static final String TABLE_SETTINGS            = "settings";
 	public static final DateTimeFormatter TIME_FORMATTER = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss");
+
 	public static final String UMBRELLA       = "Umbrella";
 	public static final String UTF8           = UTF_8.displayName();
 

core/src/main/java/de/srsoftware/umbrella/core/constants/Field.java

@@ -86,6 +86,7 @@ public class Field {
 	public static final String LOGIN       = "login";
 
 	public static final String MEMBERS     = "members";
+	public static final String MENU_INDEX  = "menu_index";
 	public static final String MESSAGE_ID  = "message_id";
 	public static final String MIME        = "mime";
 	public static final String MODULE      = "module";

core/src/main/java/de/srsoftware/umbrella/core/constants/Module.java

@@ -4,8 +4,17 @@ package de.srsoftware.umbrella.core.constants;
 public class Module {
 	public static final String BOOKMARK = "bookmark";
 	public static final String COMPANY  = "company";
+	public static final String CONTACT  = "contact";
+	public static final String DOCUMENT = "document";
+	public static final String FILES    = "files";
+	public static final String MESSAGE  = "message";
+	public static final String NOTES    = "notes";
+	public static final String POLL     = "poll";
 	public static final String PROJECT  = "project";
+	public static final String STOCK    = "stock";
+	public static final String TAGS     = "tags";
 	public static final String TASK     = "task";
+	public static final String TIME     = "time";
 	public static final String USER     = "user";
 	public static final String WIKI     = "wiki";
 }

core/src/main/java/de/srsoftware/umbrella/core/constants/Path.java

@@ -5,11 +5,13 @@ public class Path {
 	private Path(){};
 
 	public static final String ADD              = "add";
-    public static final String AVAILABLE        = "available";
-	public static final String CSS              = "css";
-	public static final String COMMON_TEMPLATES = "common_templates";
-	public static final String COMPANY          = "company";
-	public static final String CONNECTED        = "connected";
+	public static final String AVAILABLE        = "available";
+
+	public static final String CSS                     = "css";
+	public static final String CLONE                   = "clone";
+	public static final String COMMON_TEMPLATES        = "common_templates";
+	public static final String COMPANY                 = "company";
+	public static final String CONNECTED               = "connected";
 
 	public static final String ITEM = "item";
 
@@ -22,6 +24,9 @@ public class Path {
 	public static final String LOGIN     = "login";
 
 	public static final String LOGOUT     = "logout";
+
+	public static final String MENU = "menu";
+
 	public static final String PAGE       = "page";
 	public static final String PASSWORD   = "password";
 	public static final String PROJECT    = "project";

core/src/main/java/de/srsoftware/umbrella/core/constants/Text.java

@@ -5,8 +5,9 @@ package de.srsoftware.umbrella.core.constants;
  * This is a collection of messages that appear throughout the project
  */
 public class Text {
-	public static final String BOOKMARK = "bookmark";
-	public static final String BOOLEAN  = "Boolean";
+	public static final String BOOKMARK  = "bookmark";
+	public static final String BOOKMARKS = "bookmarks";
+	public static final String BOOLEAN   = "Boolean";
 
 	public static final String COMPANIES         = "companies";
 	public static final String COMPANY           = "company";
@@ -24,6 +25,8 @@ public class Text {
 
 	public static final String EMAILS_FOR_RECEIVER = "emails for {email}";
 
+	public static final String FILES = "files";
+
 	public static final String INVALID_DB_CODE = "Encountered invalid dbCode: {code}";
 	public static final String ITEM            = "item";
 	public static final String ITEMS           = "items";
@@ -34,13 +37,16 @@ public class Text {
 	public static final String LONG          = "Long";
 
 	public static final String MESSAGE = "message";
+	public static final String MESSAGES = "messages";
 
 	public static final String NOTE         = "note";
+	public static final String NOTES        = "notes";
 	public static final String NOTE_WITH_ID = "note ({id})";
 	public static final String NUMBER       = "number";
 
 	public static final String PATH            = "path";
-	public static final String PROJECT         = "project";
+	public static final String POLLS           = "polls";
+	public static final String PROJECTS        = "projects";
 	public static final String PROJECT_WITH_ID = "project ({id})";
 	public static final String PROPERTIES      = "properties";
 	public static final String PROPERTY        = "property";
@@ -52,18 +58,21 @@ public class Text {
 	public static final String SERVICE_WITH_ID = "service ({id})";
 	public static final String SESSION         = "session";
 	public static final String SETTINGS        = "settings";
+	public static final String STOCK           = "stock";
 	public static final String STRING          = "string";
 
 	public static final String TABLE_WITH_NAME = "table {name}";
 	public static final String TAGS            = "tags";
 	public static final String TASK            = "task";
 	public static final String TASKS           = "tasks";
+	public static final String TIMETRACKING    = "timetracking";
 	public static final String TIME_WITH_ID    = "time ({id})";
 	public static final String TYPE            = "type";
 
 	public static final String UNIT         = "unit";
 	public static final String USER_WITH_ID = "user ({id})";
 
+	public static final String WIKI       = "wiki";
 	public static final String WIKI_PAGE  = "wiki page";
 	public static final String WIKI_PAGES = "wiki pages";
 

documents/src/main/java/de/srsoftware/umbrella/documents/DocumentApi.java

@@ -552,7 +552,7 @@ public class DocumentApi extends BaseHandler implements DocumentService {
 	private boolean postToDocument(HttpExchange ex, de.srsoftware.tools.Path path, UmbrellaUser user, long docId) throws IOException, UmbrellaException {
 		var head = path.pop();
 		return switch (head){
-			case CLONE -> postCloneDoc(docId,ex,user);
+			case Path.CLONE -> postCloneDoc(docId,ex,user);
 			case POSITION -> postDocumentPosition(docId,ex,user);
 			case PATH_SEND -> sendDocument(ex,path,user,docId);
 			case null, default -> super.doPost(path,ex);

frontend/src/Components/MarkdownEditor.svelte

@@ -8,6 +8,7 @@
         onclick  = evt => {},
         onSet    = newVal => {return true;},
         simple   = false,
+        store_id = null,
         type     = 'div',
         value    = $bindable({source:null,rendered:null})
     } = $props();
@@ -15,6 +16,7 @@
     let editing   = $state(false);
     let editValue = $state({source:value.source,rendered:value.rendered});
     let start     = 0;
+    let stored_source = $state(store_id ? localStorage.getItem(store_id) : null);
     let timer     = null;
 
     async function applyEdit(){
@@ -27,6 +29,7 @@
     }
 
     function doSave(){
+        if (store_id) localStorage.removeItem(store_id);
         if (simple){
             onSet(editValue.source);
         } else applyEdit();
@@ -50,6 +53,7 @@
             body       : editValue.source
         });
         editValue.rendered = await resp.text();
+        if (store_id) localStorage.setItem(store_id,editValue.source);
     }
 
     function typed(ev){
@@ -100,13 +104,37 @@
         measured(evt, evt.timeStamp - start);
     }
 
+    function restore(ev){
+        editValue.source = stored_source;
+        stored_source = null;
+        render();
+    }
+
     activeField.subscribe((val) => resetEdit());
     if (simple) startEdit();
 </script>
 
+<style>
+    .markdown{
+        position: relative;
+    }
+    #restore_markdown{
+        position: absolute;
+        right: 50%;
+        top: -10px;
+        background: orange;
+        color: black;
+        padding: 5px;
+        border-radius: 5px;
+    }
+</style>
+
 <div class="markdown {editing?'editing':''}">
 {#if editing}
     <span class="hint">{@html t('markdown_supported')}</span>
+    {#if stored_source}
+    <span id="restore_markdown" onclick={restore} class="hint">{t('unsaved_content')}</span>
+    {/if}
     <textarea bind:value={editValue.source} onkeyup={typed} autofocus={!simple}></textarea>
     <div class="preview">{@html target(editValue.rendered)}</div>
     {#if !simple}
@@ -118,4 +146,4 @@
 {:else}
     <svelte:element this={type} {onclick} {oncontextmenu} class={{editable}} title={t('right_click_to_edit')} >{@html target(value.rendered)}</svelte:element>
 {/if}
-</div>
+</div>

frontend/src/Components/Menu.svelte

@@ -2,27 +2,25 @@
 import { onMount }       from 'svelte';
 import { useTinyRouter } from 'svelte-tiny-router';
 
-import { logout, user }  from '../user.svelte.js';
-import { t }             from '../translations.svelte.js';
+import { api, get }      from '../urls.svelte';
+import { error }         from '../warn.svelte';
+import { logout, user }  from '../user.svelte';
+import { t }             from '../translations.svelte';
 
 import TimeRecorder from './TimeRecorder.svelte';
 
 let key = $state(null);
 const router  = useTinyRouter();
-const modules = $state([]);
+let modules = $state(null);
 let expand = $state(false);
 
 async function fetchModules(){
-    const url  = `${location.protocol}//${location.host.replace('5173','8080')}/legacy/user/modules`;
-    const resp = await fetch(url,{credentials:'include'});
-    if (resp.ok){
-        const arr = await resp.json();
-        for (let entry of arr) {
-            let name = t('module.'+entry.module);
-            if (name) modules.push({name:name,url:entry.url});
-        }
+    let url = api('settings/menu');
+    const res = await get(url);
+    if (res.ok){
+        modules = await res.json();
     } else {
-        console.log('error');
+        error(res);
     }
 }
 
@@ -30,7 +28,11 @@ function onclick(e){
     e.preventDefault();
     expand = false;
     let href = e.target.getAttribute('href');
-    if (href) router.navigate(href);
+    if (href) {
+        if (href.includes('://')) {
+            window.location.href = href;
+        } else router.navigate(href);
+    }
     return false;
 }
 
@@ -55,27 +57,13 @@ onMount(fetchModules);
         <input type="text" bind:value={key} />
         <button type="submit">{t('search')}</button>
     </form>
-	<button class="symbol" onclick={e => expand = !expand}></button>
-    <a href="/user"     {onclick} class="user">{t('users')}</a>
-    <a href="/company"  {onclick} class="company">{t('companies')}</a>
-    <a href="/project"  {onclick} class="project">{t('projects')}</a>
-    <a href="/task"     {onclick} class="task">{t('tasks')}</a>
-    <a href="/tags"     {onclick} class="tags">{t('tags')}</a>
-    <a href="/document" {onclick} class="doc">{t('documents')}</a>
-    <a href="/bookmark" {onclick} class="mark">{t('bookmarks')}</a>
-    <a href="/notes"    {onclick} class="note">{t('notes')}</a>
-    <a href="/files"    {onclick} class="file">{t('files')}</a>
-    <a href="/time"     {onclick} class="time">{t('timetracking')}</a>
-    <a href="/wiki"     {onclick} class="wiki">{t('wiki')}</a>
-    <a href="/contact"  {onclick} class="contact">{t('contacts')}</a>
-    <a href="/stock"    {onclick} class="stock">{t('stock')}</a>
-    <a href="/message"  {onclick} class="message">{t('messages')}</a>
+    <button class="symbol" onclick={e => expand = !expand}> </button>
+    {#each modules as module,i}
+    <a href={module.module.includes('://') ? module.module : '/'+module.module} {onclick} class={module.class}>{@html t(module.title)}</a>
+    {/each}
     {#if user.id == 2}
     <a href="https://svelte.dev/tutorial/svelte/state" target="_blank">{t('tutorial')}</a>
     {/if}
-    {#each modules as module,i}
-    {#if module.name.trim()}<a href={module.url}>{module.name}</a>{/if}
-    {/each}
     {#if user.name }
     <a class="logout" onclick={logout}>{t('logout_user',{user:user.name})}</a>
     {/if}

frontend/src/routes/message/Messages.svelte

@@ -62,7 +62,7 @@
 </script>
 
 <fieldset>
-    <legend>{t('messages')} <button onclick={showSettings}>{t('settings')}</button></legend>
+    <legend>{@html t('messages')} <button onclick={showSettings}>{t('settings')}</button></legend>
     <table>
         <thead>
             <tr>

frontend/src/routes/stock/ItemList.svelte

@@ -1,11 +1,39 @@
 <script>
-    import { api, post }    from '../../urls.svelte';
+    import { onDestroy, onMount } from 'svelte';
+    import { api, eventStream, post }    from '../../urls.svelte';
     import { error, yikes } from '../../warn.svelte';
     import { t }            from '../../translations.svelte';
 
     let { items, selected = $bindable(null), location = null, drag_start = item => console.log({dragging:item}) } = $props();
+    let eventSource      = null;
     let newItem = $state({name:null, code: null});
 
+    function handleCreateEvent(evt){
+        let json = JSON.parse(evt.data);
+        if (json.item.location.id == location.id){
+            items = [...items,json.item];
+            selected = json.item;
+        }
+    }
+
+    function handleEvent(evt,method){
+        console.log(evt,method);
+    }
+
+	function handleDeleteEvent(evt){
+        handleEvent(evt,'delete');
+    }
+
+    function handleUpdateEvent(evt){
+        handleEvent(evt,'update');
+    }
+
+    function load(){
+        try {
+			eventSource = eventStream(handleCreateEvent,handleUpdateEvent,handleDeleteEvent);
+        } catch (ignored) {}
+    }
+
     async function saveNewItem(){
         newItem.location = location;
         const url = api('stock/item');
@@ -17,6 +45,12 @@
             items = [...items, it];
         } else error(res);
     }
+
+    onDestroy(() => {
+		if (eventSource) eventSource.close();
+	});
+    onMount(load);
+
 </script>
 <table>
     <thead>

frontend/src/routes/stock/ItemProps.svelte

@@ -1,7 +1,7 @@
 <script>
     import LineEditor       from '../../Components/LineEditor.svelte';
     import { onMount }      from 'svelte';
-    import { api }          from '../../urls.svelte';
+    import { api, patch, post } from '../../urls.svelte';
     import { error, yikes } from '../../warn.svelte';
     import { t }            from '../../translations.svelte';
 
@@ -15,6 +15,15 @@
         }
     });
 
+    async function doClone(ev){
+        let url = api('stock/clone');
+        let res = await post(url,{id:item.id});
+        if (res.ok){
+            let json = await res.json();
+            yikes(res);
+        } else error(res);
+    }
+
     function byName(a,b){
         return a.name.localeCompare(b.name);
     }
@@ -30,11 +39,7 @@
             },
             add_prop : add_prop
         }
-        const res = await fetch(url,{
-            credentials:'include',
-            method:'POST',
-            body:JSON.stringify(data)
-        });
+        const res = await post(url,data);
         if (res.ok){
             const prop = await res.json();
             const id = prop.id;
@@ -45,18 +50,14 @@
         return false;
     }
 
-    async function patch(key,newVal){
+    async function update(key,newVal){
         const url = api('stock');
         const data = {
             id    : item.id,
             owner : item.owner,
         };
         data[key] = newVal;
-        const res = await fetch(url,{
-            credentials:'include',
-            method:'PATCH',
-            body:JSON.stringify(data)
-        });
+        const res = await patch(url,data);
         if (res.ok){
             yikes();
             return true;
@@ -67,13 +68,19 @@
 </script>
 
 {#if item}
-<LineEditor type="h3" editable={true} value={item.name} onSet={v => patch('name',v)} />
-Code: <LineEditor type="span" editable={true} value={item.code} onSet={v => patch('code',v)} />
+<LineEditor type="h3" editable={true} value={item.name} onSet={v => update('name',v)} />
+<button class="clone symbol" title={t('clone')} onclick={doClone}></button>
 <div>
     {@html item.description.rendered}
 </div>
 <table>
     <tbody>
+        <tr>
+            <td>{t('Code')}:</td>
+            <td>
+                <LineEditor type="span" editable={true} value={item.code} onSet={v => update('code',v)} />
+            </td>
+        </tr>
         {#each item.properties.toSorted(byName) as prop}
         <tr>
             <td>

frontend/src/routes/task/Add.svelte

@@ -17,7 +17,7 @@
     let parent_task      = $state(null);
     let task = $state({
         name           : '',
-        description    : { source : '', rendered : '' },
+        description    : { source:'',rendered:''},
         due_date       : null,
         est_time       : null,
         members        : {},
@@ -42,6 +42,14 @@
         ///  TODO: ?
     }
 
+    async function getCandidates(text){
+        const origin = parent_task ? parent_task.members : project.members;
+        const candidates = Object.values(origin)
+            .filter(member => member.user.name.toLowerCase().includes(text.toLowerCase()))
+            .map(member => [member.user.id,member.user.name]);
+        return Object.fromEntries(candidates);
+    }
+
     async function load(){
         if (parent_task_id) await loadParent();
         if (project_id) loadProject();
@@ -87,16 +95,7 @@
         }
     }
 
-    async function getCandidates(text){
-        const origin = parent_task ? parent_task.members : project.members;
-        const candidates = Object.values(origin)
-            .filter(member => member.user.name.toLowerCase().includes(text.toLowerCase()))
-            .map(member => [member.user.id,member.user.name]);
-        return Object.fromEntries(candidates);
-    }
-
-
-    function onkeydown(e){
+     function onkeydown(e){
         if (e.ctrlKey && e.keyCode === 83) {
             e.preventDefault();
             saveTask();
@@ -111,6 +110,7 @@
             body        : JSON.stringify(task)
         });
         if (resp.ok) {
+            localStorage.removeItem(`task/${task.id}/description`);
             if (!assignee) { // if assignee is set, this form was opened within an external context. hence we don`t want to navigate somewhere else!
                 task = await resp.json();
                 if (task.parent_task_id){
@@ -136,120 +136,74 @@
 
 <fieldset>
     <legend>{t('add_object',{object:t('task')})}</legend>
-    <table {onkeydown}>
-        <tbody>
-            <tr>
-                <th>
-                    {t('name')}
-                </th>
-                <td>
-                    <input bind:value={task.name} autofocus>
-                </td>
-            </tr>
-            {#if project}
-            <tr>
-                <th>
-                    {t('project')}
-                </th>
-                <td>
-                    {project.name}
-                </td>
-            </tr>
-            {/if}
-            {#if parent_task}
-            <tr>
-                <th>
-                    {t('parent_task')}
-                </th>
-                <td>
-                    {parent_task.name}
-                </td>
-            </tr>
-            {/if}
+    <div class="task grid2">
+        <div>{t('name')}</div>
+        <div>
+            <input bind:value={task.name} autofocus>
+        </div>
 
-            <tr>
-                <th>
-                    {t('description')}
-                </th>
-                <td>
-                    <MarkdownEditor bind:value={task.description} simple={true} />
-                </td>
-            </tr>
-            <tr>
-                <th>
-                    {t('tags')}
-                </th>
-                <td>
-                    <Tags module="task" bind:tags={task.tags} />
-                </td>
-            </tr>
-            {#if extendedSettings}
-            <tr>
-                <th>
-                    {t('members')}
-                </th>
-                <td>
-                    <PermissionEditor members={task.members} {addMember} {dropMember} {getCandidates} />
-                </td>
-            </tr>
-            <tr>
-                <th>
-                    {t('estimated_time')}
-                </th>
-                <td>
-                    <input type="number" bind:value={task.est_time} /> {t('hours')}
-                </td>
-            </tr>
-            <tr>
-                <th>
-                    {t('start_date')}
-                </th>
-                <td>
-                    <input type="date" bind:value={task.start_date} />
-                </td>
-            </tr>
-            <tr>
-                <th>
-                    {t('due_date')}
-                </th>
-                <td>
-                    <input type="date" bind:value={task.due_date} />
-                </td>
-            </tr>
-            <tr>
-                <th>
-                    {t('subtasks')}
-                </th>
-                <td>
-                    <label>
-                        <input type="checkbox" bind:checked={task.show_closed} >
-                        {t('display_closed_tasks')}
-                    </label>
-                </td>
-            </tr>
-            <tr>
-                <th>
-                    {t('index_page')}
-                </th>
-                <td>
-                    <label>
-                        <input type="checkbox" bind:checked={task.no_index} >
-                        {t('hide_on_index_page')}
-                    </label>
-                </td>
-            </tr>
-            {:else}
-            <tr>
-                <th>
-                    {t('extended_settings')}
-                </th>
-                <td>
-                    <button onclick={toggleSettings}>{t('show')}</button>
-                </td>
-            </tr>
-            {/if}
-        </tbody>
-    </table>
+        {#if project}
+        <div>{t('project')}</div>
+        <a href="/project/{project.id}/view">{project.name}</a>
+        {/if}
+
+        {#if parent_task}
+        <div>{t('parent_task')}</div>
+        <div>{parent_task.name}</div>
+        {/if}
+
+        <div>{t('description')}</div>
+        <div>
+            <MarkdownEditor bind:value={task.description} simple={true} store_id="task/add_to/{project_id}"/>
+        </div>
+        <div>{t('tags')}</div>
+        <div>
+            <Tags module="task" bind:tags={task.tags} />
+        </div>
+
+        {#if extendedSettings}
+        <div>{t('members')}</div>
+        <div>
+            <PermissionEditor members={task.members} {addMember} {dropMember} {getCandidates} />
+        </div>
+
+        <div>{t('estimated_time')}</div>
+        <div>
+            <input type="number" bind:value={task.est_time} /> {t('hours')}
+        </div>
+
+        <div>{t('start_date')}</div>
+        <div>
+            <input type="date" bind:value={task.start_date} />
+        </div>
+
+        <div>{t('due_date')}</div>
+        <div>
+            <input type="date" bind:value={task.due_date} />
+        </div>
+
+        <div>{t('subtasks')}</div>
+        <div>
+            <label>
+                <input type="checkbox" bind:checked={task.show_closed} >
+                {t('display_closed_tasks')}
+            </label>
+        </div>
+
+        <div>
+            {t('index_page')}
+        </div>
+        <label>
+            <input type="checkbox" bind:checked={task.no_index} >
+            {t('hide_on_index_page')}
+        </label>
+        {:else}
+        <div>{t('extended_settings')}</div>
+        <div>
+            <button onclick={toggleSettings}>{t('show')}</button>
+        </div>
+        {/if}
+    </div>
     <button onclick={saveTask}>{t('save_object',{object:t('task')})}</button>
     {#if on_abort}
     <button onclick={on_abort}>{t('abort')}</button>

frontend/src/routes/task/View.svelte

@@ -241,7 +241,7 @@
     </div>
     {#if task.description}
     <div>{t('description')}</div>
-    <MarkdownEditor bind:value={task.description} editable={true} onSet={val => update({description:val})} />
+    <MarkdownEditor store_id="task/{task.id}/description" bind:value={task.description} editable={true} onSet={val => update({description:val})} />
     {/if}
     {#if !showSettings && task.start_date}
         <div>{t('start_date')}</div>

frontend/src/routes/wiki/AddPage.svelte

@@ -22,6 +22,7 @@
         });
         if (res.ok){
             yikes();
+            localStorage.removeItem('wiki/new/content');
             router.navigate(`/wiki/${title}/view`);
         } else {
             error(res);
@@ -60,7 +61,7 @@
     </label>
     <label>
         {t('content')}
-        <Markdown bind:value={content} simple={true} />
+        <Markdown bind:value={content} simple={true} store_id="wiki/new/content" />
         <button type="submit">{t('save')}</button>
     </label>
 </form>

frontend/src/routes/wiki/Index.svelte

@@ -39,6 +39,6 @@
 
 <h2>{lastLetter = page.charAt(0).toUpperCase()||page.charAt(0).toUpperCase()}</h2>
 {/if}
-<a class="wikilink" href={`/wiki/${page}/view`} {onclick} >{page}</a>
+<a class="wikilink" href={`/wiki/${encodeURIComponent(page)}/view`} {onclick} >{page}</a>
 {/each}
 {/if}

frontend/src/routes/wiki/View.svelte

@@ -188,7 +188,7 @@
     </table>
     {/if}
     {/if}
-    <MarkdownEditor {editable} value={page.content} onSet={s => patch({content:s})} />
+    <MarkdownEditor {editable} value={page.content} onSet={s => patch({content:s})} store_id="wiki/{page.id}/description" />
     <TagList module="wiki" id={page.id} user_list={Object.keys(page.members).map(id => +id)} />
     <div class="notes">
         <h3>{t('notes')}</h3>

stock/build.gradle.kts

@@ -1,6 +1,7 @@
 description = "Umbrella : Stock"
 
 dependencies{
+    implementation(project(":bus"))
     implementation(project(":core"))
     implementation("de.srsoftware:configuration.json:1.0.3")
 }

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

@@ -14,6 +14,7 @@ import static de.srsoftware.umbrella.core.constants.Module.USER;
 import static de.srsoftware.umbrella.core.constants.Path.*;
 import static de.srsoftware.umbrella.core.constants.Path.PROPERTY;
 import static de.srsoftware.umbrella.core.exceptions.UmbrellaException.*;
+import static de.srsoftware.umbrella.messagebus.MessageBus.messageBus;
 import static de.srsoftware.umbrella.stock.Constants.*;
 import static java.lang.System.Logger.Level.WARNING;
 import static java.util.Comparator.comparing;
@@ -26,12 +27,18 @@ import de.srsoftware.umbrella.core.*;
 import de.srsoftware.umbrella.core.api.Owner;
 import de.srsoftware.umbrella.core.api.StockService;
 import de.srsoftware.umbrella.core.constants.Field;
+import de.srsoftware.umbrella.core.constants.Module;
 import de.srsoftware.umbrella.core.constants.Path;
+import de.srsoftware.umbrella.core.constants.Text;
 import de.srsoftware.umbrella.core.exceptions.UmbrellaException;
 import de.srsoftware.umbrella.core.model.*;
 import de.srsoftware.umbrella.core.model.Location;
+
 import java.io.IOException;
 import java.util.*;
+
+import de.srsoftware.umbrella.messagebus.events.Event;
+import de.srsoftware.umbrella.messagebus.events.ItemEvent;
 import org.json.JSONObject;
 
 public class StockModule extends BaseHandler implements StockService {
@@ -190,6 +197,7 @@ public class StockModule extends BaseHandler implements StockService {
 			if (user.isEmpty()) return unauthorized(ex);
 			var head = path.pop();
 			return switch (head) {
+				case Path.CLONE -> postClone(user.get(),ex);
 				case Path.ITEM -> postItem(user.get(), ex);
 				case LIST     -> postItemList(user.get(), path, ex);
 				case Path.LOCATION -> postLocation(user.get(),ex);
@@ -285,7 +293,7 @@ public class StockModule extends BaseHandler implements StockService {
 		var json = json(ex);
 		if (!(json.get(ID) instanceof Number id)) throw missingField(ID);
 		json.remove(ID);
-
+		LOG.log(WARNING,"Missing permission check in StockModule.patchItem()!");
 		var item = stockDb.loadItem(id.longValue());
 		item.patch(json);
 		return sendContent(ex,stockDb.save(item));
@@ -336,6 +344,23 @@ public class StockModule extends BaseHandler implements StockService {
 		return sendContent(ex,location);
 	}
 
+	private boolean postClone(UmbrellaUser user, HttpExchange ex) throws IOException {
+		var json = json(ex);
+		if (!json.has(ID))throw missingField(ID);
+		if (!(json.get(ID) instanceof Number num)) throw invalidField(ID,Text.NUMBER);
+		long itemId = num.longValue();
+		var item     = stockDb.loadItem(itemId);
+		stockDb.loadProperties(item);
+		var location = item.location().resolve();
+		var owner    = location.owner().resolve();
+		if (!assigned(owner,user)) throw forbidden("You are not allowed to add items to \"{location}\"!", Text.LOCATION,location.name());
+		var newItem = new Item(0,owner,0,location,item.code(),item.name(),item.description());
+		for (var property : item.properties()) newItem.properties().add(property);
+		newItem = stockDb.save(newItem);
+		messageBus().dispatch(new ItemEvent(user, Module.STOCK, newItem, Event.EventType.CREATE));
+		return sendContent(ex,newItem);
+	}
+
 	private boolean postItem(UmbrellaUser user, HttpExchange ex) throws IOException {
 		var json = json(ex);
 		if (!json.has(NAME) || !(json.get(NAME) instanceof String name)) throw missingField(NAME);
@@ -345,8 +370,10 @@ public class StockModule extends BaseHandler implements StockService {
 		var location = stockDb.loadLocation(locationData.getLong(ID));
 		var owner = location.owner().resolve();
 		if (!assigned(owner,user)) throw forbidden("You are not allowed to add items to {location}!", Field.LOCATION,location);
-		var newItem = new Item(0,owner,0,location,code,name,description);
-		return sendContent(ex,stockDb.save(newItem));
+		var newItem = stockDb.save(new Item(0,owner,0,location,code,name,description));
+		messageBus().dispatch(new ItemEvent(user, Module.STOCK, newItem, Event.EventType.CREATE));
+		return sendContent(ex,newItem);
+
 	}
 
 	private boolean postItemList(UmbrellaUser user, de.srsoftware.tools.Path path, HttpExchange ex) throws IOException {
@@ -395,6 +422,7 @@ public class StockModule extends BaseHandler implements StockService {
 		if (!(itemData.get(ID) instanceof Number itemId)) throw missingField(ID);
 		if (!(json.get("add_prop") instanceof JSONObject propData)) throw missingField("add_prop");
 		if (!propData.has(VALUE)) throw missingField(VALUE);
+		LOG.log(WARNING,"Missing permission check in StockModule.postProperty()!");
 		var value = propData.get(VALUE);
 		if (value == null) throw missingField(VALUE);
 

translations/src/main/resources/de.json

@@ -210,7 +210,7 @@
   "member": "Mitarbeiter",
   "members": "Mitarbeiter",
   "message": "Nachricht",
-  "messages": "Benachrichtigungen",
+  "messages": "Be­nach­rich­ti­gung­en",
   "miscellaneous_settings": "sonstige Einstellungen",
   "missing_new_item_id": "Alter Artikel-ID ({0}) wurde keine neue ID zugeordnet!",
   "mismatch": "ungleich",
@@ -399,6 +399,7 @@
   "unit_price": "Preis/Einheit",
   "unknown_item_location": "Artikel {0} von {1} {2} ist verknüpft mit unbekanntem Lagerort {3}!",
   "unlink": "Trennen",
+  "unsaved_content": "Hier klicken, um ungespeicherte Änderungen zu laden",
   "update": "aktualisieren",
   "UPDATE_USERS" : "Nutzer aktualisieren",
   "upload_file": "Datei hochladen",

translations/src/main/resources/en.json

@@ -399,6 +399,7 @@
   "unit_price": "price/unit",
   "unknown_item_location": "Item {0} of {1} {2} refers to location {3}, which is unknown!",
   "unlink": "unlink",
+  "unsaved_content": "Click here to load unsaved changes",
   "update": "update",
   "UPDATE_USERS" : "update users",
   "upload_file": "upload file",

user/src/main/java/de/srsoftware/umbrella/user/UserModule.java

@@ -9,6 +9,7 @@ import static de.srsoftware.umbrella.core.ModuleRegistry.postBox;
 import static de.srsoftware.umbrella.core.ResponseCode.*;
 import static de.srsoftware.umbrella.core.ResponseCode.HTTP_SERVER_ERROR;
 import static de.srsoftware.umbrella.core.Util.*;
+import static de.srsoftware.umbrella.core.constants.Constants.CONFIG_SESSION_DURATION;
 import static de.srsoftware.umbrella.core.constants.Field.*;
 import static de.srsoftware.umbrella.core.constants.Field.PASSWORD;
 import static de.srsoftware.umbrella.core.constants.Field.REDIRECT;
@@ -88,9 +89,14 @@ public class UserModule extends BaseHandler implements UserService {
 	public UserModule(Configuration config) throws UmbrellaException {
 		super();
 		var dbFile = config.get(CONFIG_DATABASE).orElseThrow(() -> missingConfig(CONFIG_DATABASE));
+		var sqlite    = new SqliteDB(connect(dbFile));
+
 		// may be splitted in separate db files later
-		logins        = new SqliteDB(connect(dbFile));
-		users         = new SqliteDB(connect(dbFile));
+		logins        = sqlite;
+		users         = sqlite;
+
+		Optional<Number> sessionDuration = config.get(CONFIG_SESSION_DURATION);
+		sessionDuration.ifPresent(users::setSessionDuration);
 		ModuleRegistry.add(this);
 	}
 

user/src/main/java/de/srsoftware/umbrella/user/api/UserDb.java

@@ -41,4 +41,6 @@ public interface UserDb {
 	UmbrellaUser save(DbUser user) throws UmbrellaException;
 
 	Map<Long,DbUser> search(String key);
+
+	UserDb setSessionDuration(Number minutes);
 }

user/src/main/java/de/srsoftware/umbrella/user/sqlite/SqliteDB.java

@@ -35,12 +35,14 @@ import java.security.NoSuchAlgorithmException;
 import java.sql.Connection;
 import java.sql.ResultSet;
 import java.sql.SQLException;
+import java.time.Duration;
 import java.time.Instant;
 import java.time.LocalDateTime;
 import java.util.*;
 
 public class SqliteDB extends BaseDb implements LoginServiceDb, UserDb {
 	private static final System.Logger LOG = System.getLogger(SqliteDB.class.getSimpleName());
+	private static Duration sessionDuration = DEFAULT_SESSION_DURATION;
 
 	public SqliteDB(Connection conn){
 		super(conn);
@@ -481,8 +483,15 @@ CREATE TABLE IF NOT EXISTS {0} (
 		}
 	}
 
+	@Override
+	public UserDb setSessionDuration(Number minutes) {
+		LOG.log(INFO,"Session duration set to {} minutes",minutes);
+		sessionDuration = Duration.ofMinutes(minutes.longValue());
+		return this;
+	}
+
 	public Instant then(){
-		return LocalDateTime.now().plus(DEFAULT_SESSION_DURATION).toInstant(UTC);
+		return LocalDateTime.now().plus(sessionDuration).toInstant(UTC);
 	}
 
 	private ForeignLogin toForeignLogin(ResultSet rs) throws SQLException {

web/src/main/resources/web/css/bloodshed-color.css

@@ -74,6 +74,10 @@ tr:hover .taglist .tag button {
     color: black;
 }
 
+.code{
+	color: orangered;
+}
+
 .error {
   background-color: red;
   color: black;

web/src/main/resources/web/css/bloodshed.css

@@ -142,16 +142,6 @@ td, tr{
   font-weight: normal;
 }
 
-.settings {
-  position: fixed;
-  top: 60px;
-  left: 10px;
-  right: 10px;
-  padding: 10px;
-  border: 1px solid;
-  border-radius: 5px;
-}
-
 .project.list td:not(.actions){
     cursor: pointer;
 }

web/src/main/resources/web/css/default-color.css

@@ -72,6 +72,10 @@ tr:hover .taglist .tag button {
     color: black;
 }
 
+.code{
+	color: chocolate;
+}
+
 .em {
     background: rgba(255, 215, 0, 0.09);
 }

web/src/main/resources/web/css/default.css

@@ -225,16 +225,6 @@ td, tr{
   font-weight: normal;
 }
 
-.settings {
-  position: fixed;
-  top: 60px;
-  left: 10px;
-  right: 10px;
-  padding: 10px;
-  border: 1px solid;
-  border-radius: 5px;
-}
-
 .project.list td:not(.actions){
     cursor: pointer;
 }
@@ -486,6 +476,16 @@ table{
     bottom: 0;
 }
 
+.properties{
+	position: relative;
+}
+
+.properties .clone{
+	position: absolute;
+	right: 10px;
+	top: 40px;
+}
+
 .version > a{
     padding: 5px;
 }

web/src/main/resources/web/css/winter-color.css

@@ -67,6 +67,10 @@ tr:hover .taglist .tag button {
     color: black;
 }
 
+.code{
+	color: black;
+}
+
 .error {
   background-color: red;
   color: black;

web/src/main/resources/web/css/winter.css

@@ -142,16 +142,6 @@ td, tr{
   font-weight: normal;
 }
 
-.settings {
-  position: fixed;
-  top: 60px;
-  left: 10px;
-  right: 10px;
-  padding: 10px;
-  border: 1px solid;
-  border-radius: 5px;
-}
-
 .project.list td:not(.actions){
     cursor: pointer;
 }