working on loading item by id

This commit is contained in:
2026-02-11 13:41:45 +01:00
parent 76651b1e46
commit 12ed6d47ec
4 changed files with 33 additions and 5 deletions

View File

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

View File

@@ -17,7 +17,7 @@
let location = $state(null); let location = $state(null);
let draggedItem = $state(null) let draggedItem = $state(null)
let draggedLocation = $state(null) let draggedLocation = $state(null)
let { item_id, location_id, owner, owner_id } = $props(); let { item_id, location_id, owner, owner_id, owner_number } = $props();
let skip_location = false; // disable effect on setting location within loadItem() let skip_location = false; // disable effect on setting location within loadItem()
$effect(() => { $effect(() => {
@@ -98,8 +98,8 @@
} }
async function loadItem(){ async function loadItem(){
if (!item_id) return; if (!item_id && !owner_number) return;
const url = api(`stock/${owner}/${owner_id}/item/${item_id}`); const url = item_id ? api(`stock/item/${item_id}`) : api(`stock/${owner}/${owner_id}/item/${owner_number}`);
const res = await get(url); const res = await get(url);
if (res.ok){ if (res.ok){
yikes(); yikes();
@@ -116,7 +116,7 @@
} }
} }
for (let i of json.items){ for (let i of json.items){
if (i.owner_number == +item_id) item = i; if (i.owner_number == +owner_number) item = i;
} }
} else { } else {
error(res); error(res);

View File

@@ -74,6 +74,15 @@ Code: <LineEditor type="span" editable={true} value={item.code} onSet={v => patc
</div> </div>
<table> <table>
<tbody> <tbody>
<tr>
<td>
{t('ID')}
</td>
<td>
{item.id}
</td>
</tr>
{#each item.properties.toSorted(byName) as prop} {#each item.properties.toSorted(byName) as prop}
<tr> <tr>
<td> <td>

View File

@@ -27,6 +27,7 @@ import de.srsoftware.umbrella.core.api.Owner;
import de.srsoftware.umbrella.core.api.StockService; import de.srsoftware.umbrella.core.api.StockService;
import de.srsoftware.umbrella.core.constants.Field; import de.srsoftware.umbrella.core.constants.Field;
import de.srsoftware.umbrella.core.constants.Path; 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.exceptions.UmbrellaException;
import de.srsoftware.umbrella.core.model.*; import de.srsoftware.umbrella.core.model.*;
import de.srsoftware.umbrella.core.model.Location; import de.srsoftware.umbrella.core.model.Location;
@@ -113,6 +114,7 @@ public class StockModule extends BaseHandler implements StockService {
yield super.doGet(path,ex); yield super.doGet(path,ex);
} }
} }
case Path.ITEM -> getItemById(user.get(),path,ex);
case Path.LOCATION -> { case Path.LOCATION -> {
try { try {
var location = Location.of(Long.parseLong(path.pop())); var location = Location.of(Long.parseLong(path.pop()));
@@ -155,6 +157,22 @@ public class StockModule extends BaseHandler implements StockService {
} }
} }
private boolean getItemById(UmbrellaUser user, de.srsoftware.tools.Path path, HttpExchange ex) throws IOException {
var head = path.pop();
if (head == null) throw missingField(Field.ID);
try {
var itemId = Long.parseLong(head);
var item = stockDb.loadItem(itemId);
var owner = item.location().resolve().owner().resolve();
boolean allowed = owner instanceof UmbrellaUser u && user.equals(u);
allowed = allowed || owner instanceof Company c && companyService().membership(c.id(),user.id());
if (!allowed) throw forbidden("You are not allowed to access item {id}",ID,itemId);
return sendContent(ex,item);
} catch (NumberFormatException e) {
throw invalidField(Field.ID, Text.NUMBER);
}
}
@Override @Override
public boolean doPatch(de.srsoftware.tools.Path path, HttpExchange ex) throws IOException { public boolean doPatch(de.srsoftware.tools.Path path, HttpExchange ex) throws IOException {
addCors(ex); addCors(ex);