fix(objekt): Fahrzeug-Zuordnung als 1:n statt n:m (fachliche Korrektur)
CI / backend-tests (push) Failing after 1m23s
CI / frontend-build (push) Successful in 29s

Nutzer-Korrektur: "Ein Objekt gehört immer nur zu genau einem Fahrzeug"
- Spalte gezogen_von_objekt_id -> fahrzeug_id (Rename-Migration 0010)
- Zuvor begonnener n:m-Ansatz (objekt_kopplung Tabelle/Endpoints/Panel)
  wieder entfernt, da fachlich falsch
- PATCH /objekte/{id} prüft Existenz + direkten Zyklus (A<->B)
- Frontend: einfache Single-Select "Fahrzeug zuordnen" statt Kopplungs-Panel

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CVgbozhYmuEhiEJHffRXCV
This commit is contained in:
2026-09-05 01:11:25 +02:00
co-authored by Claude Sonnet 5
parent ebd6a9589e
commit 38b8ce1795
8 changed files with 144 additions and 109 deletions
@@ -0,0 +1,36 @@
"""objekt.gezogen_von_objekt_id -> fahrzeug_id (Umbenennung, gleiche Semantik)
Revision ID: 0010_objekt_kopplung
Revises: 0009_gezogen_von
Create Date: 2026-09-05
Nutzer-Klarstellung zu Migration 0009: es geht nicht spezifisch um Anhänger-
Zugfahrzeug-Kopplung, sondern allgemein um die Zuordnung eines Objekts
(Rucksack, Gerät, Anhänger, ...) zu genau einem Fahrzeug (1:n, ein Objekt
gehört jederzeit höchstens einem Fahrzeug). Strukturell identisch mit 0009,
nur die Spalte/Constraint/Index sauberer benannt statt "gezogen_von".
"""
from typing import Sequence, Union
from alembic import op
revision: str = "0010_objekt_kopplung"
down_revision: Union[str, None] = "0009_gezogen_von"
branch_labels: Union[str, Sequence[str], None] = None
depends_on: Union[str, Sequence[str], None] = None
def upgrade() -> None:
op.execute("ALTER TABLE objekt RENAME COLUMN gezogen_von_objekt_id TO fahrzeug_id")
op.execute("ALTER INDEX idx_objekt_gezogen_von RENAME TO idx_objekt_fahrzeug")
op.execute(
"ALTER TABLE objekt RENAME CONSTRAINT objekt_kein_selbstbezug TO objekt_fahrzeug_kein_selbstbezug"
)
def downgrade() -> None:
op.execute(
"ALTER TABLE objekt RENAME CONSTRAINT objekt_fahrzeug_kein_selbstbezug TO objekt_kein_selbstbezug"
)
op.execute("ALTER INDEX idx_objekt_fahrzeug RENAME TO idx_objekt_gezogen_von")
op.execute("ALTER TABLE objekt RENAME COLUMN fahrzeug_id TO gezogen_von_objekt_id")
+12 -12
View File
@@ -195,9 +195,9 @@ async def erstelle_objekt(
vorlage = await db.get(Beladungsvorlage, payload.vorlage_id) vorlage = await db.get(Beladungsvorlage, payload.vorlage_id)
if vorlage is None: if vorlage is None:
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Vorlage nicht gefunden") raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Vorlage nicht gefunden")
if payload.gezogen_von_objekt_id is not None: if payload.fahrzeug_id is not None:
if await db.get(Objekt, payload.gezogen_von_objekt_id) is None: if await db.get(Objekt, payload.fahrzeug_id) is None:
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Zugfahrzeug nicht gefunden") raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Fahrzeug nicht gefunden")
hauptserver = await db.execute(select(Systemknoten.id).where(Systemknoten.typ == KnotenTyp.haupt)) hauptserver = await db.execute(select(Systemknoten.id).where(Systemknoten.typ == KnotenTyp.haupt))
zustaendiger_server_id = hauptserver.scalar_one() zustaendiger_server_id = hauptserver.scalar_one()
@@ -210,7 +210,7 @@ async def erstelle_objekt(
standort_id=payload.standort_id, standort_id=payload.standort_id,
vorlage=vorlage, vorlage=vorlage,
zustaendiger_server_id=zustaendiger_server_id, zustaendiger_server_id=zustaendiger_server_id,
gezogen_von_objekt_id=payload.gezogen_von_objekt_id, fahrzeug_id=payload.fahrzeug_id,
) )
except IntegrityError as exc: except IntegrityError as exc:
raise HTTPException(status_code=status.HTTP_409_CONFLICT, detail="Code bereits vergeben") from exc raise HTTPException(status_code=status.HTTP_409_CONFLICT, detail="Code bereits vergeben") from exc
@@ -220,24 +220,24 @@ async def erstelle_objekt(
async def aendere_objekt( async def aendere_objekt(
objekt_id: int, payload: ObjektUpdate, db: AsyncSession = Depends(get_db), _=Depends(_admin_only) objekt_id: int, payload: ObjektUpdate, db: AsyncSession = Depends(get_db), _=Depends(_admin_only)
) -> Objekt: ) -> Objekt:
"""Fahrzeug/Anhänger-Kopplung nachträglich setzen/lösen (Nutzer-Vorgabe). """Fahrzeug-Zuordnung nachträglich setzen/lösen: jedes Objekt gehört zu
postgres-expert-Review-Fund: Zielobjekt-Existenz und direkter Zyklus (A↔B) jedem Zeitpunkt höchstens einem Fahrzeug (1:n). Zielobjekt-Existenz und
werden hier geprüft, mehrstufige Ketten sind fachlich nicht vorgesehen und direkter Zyklus (A↔B) werden geprüft, mehrstufige Ketten sind fachlich
bewusst nicht geprüft (Aufwand/Nutzen).""" nicht vorgesehen und bewusst nicht geprüft (Aufwand/Nutzen)."""
objekt = await db.get(Objekt, objekt_id) objekt = await db.get(Objekt, objekt_id)
if objekt is None: if objekt is None:
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Objekt nicht gefunden") raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Objekt nicht gefunden")
daten = payload.model_dump(exclude_unset=True) daten = payload.model_dump(exclude_unset=True)
ziel_id = daten.get("gezogen_von_objekt_id") ziel_id = daten.get("fahrzeug_id")
if ziel_id is not None: if ziel_id is not None:
ziel = await db.get(Objekt, ziel_id) ziel = await db.get(Objekt, ziel_id)
if ziel is None: if ziel is None:
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Zugfahrzeug nicht gefunden") raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Fahrzeug nicht gefunden")
if ziel.gezogen_von_objekt_id == objekt_id: if ziel.fahrzeug_id == objekt_id:
raise HTTPException( raise HTTPException(
status_code=status.HTTP_409_CONFLICT, status_code=status.HTTP_409_CONFLICT,
detail="Zyklus: Zielobjekt wird bereits von diesem Objekt gezogen", detail="Zyklus: Zielobjekt ist bereits diesem Objekt zugeordnet",
) )
for feld, wert in daten.items(): for feld, wert in daten.items():
+5 -4
View File
@@ -33,9 +33,10 @@ class Objekt(Base):
zustaendiger_server_id: Mapped[int] = mapped_column( zustaendiger_server_id: Mapped[int] = mapped_column(
ForeignKey("systemknoten.id"), nullable=False ForeignKey("systemknoten.id"), nullable=False
) )
# Fahrzeug/Anhänger-Kopplung (Nutzer-Vorgabe): optional, beide Fälle möglich - # Zuordnung zu einem Fahrzeug (Nutzer-Vorgabe): jedes Objekt (Rucksack,
# Anhänger als eigenständiges Objekt (NULL) oder fest einem Zugfahrzeug # Gerät, Anhänger, ...) gehört zu jedem Zeitpunkt höchstens einem Fahrzeug
# zugeordnet. Selbstreferenzierend, kein eigener Anhänger-Objekttyp nötig. # (1:n), NULL = eigenständig/keinem Fahrzeug zugeordnet. Selbstreferenzierend,
gezogen_von_objekt_id: Mapped[int | None] = mapped_column( # kein eigener Fahrzeug-Objekttyp nötig (Objekt bleibt generisch, Karte 11).
fahrzeug_id: Mapped[int | None] = mapped_column(
ForeignKey("objekt.id", ondelete="SET NULL") ForeignKey("objekt.id", ondelete="SET NULL")
) )
+3 -3
View File
@@ -14,7 +14,7 @@ class ObjektCreate(BaseModel):
objekttyp_id: int objekttyp_id: int
vorlage_id: int vorlage_id: int
standort_id: int standort_id: int
gezogen_von_objekt_id: int | None = None fahrzeug_id: int | None = None
class ObjektRead(BaseModel): class ObjektRead(BaseModel):
@@ -26,11 +26,11 @@ class ObjektRead(BaseModel):
vorlage_id: int | None vorlage_id: int | None
standort_id: int standort_id: int
status: ObjektStatus status: ObjektStatus
gezogen_von_objekt_id: int | None fahrzeug_id: int | None
class ObjektUpdate(BaseModel): class ObjektUpdate(BaseModel):
gezogen_von_objekt_id: int | None = None fahrzeug_id: int | None = None
class ObjektDuplizierenRequest(BaseModel): class ObjektDuplizierenRequest(BaseModel):
+3 -4
View File
@@ -56,12 +56,11 @@ async def objekt_aus_vorlage(
standort_id: int, standort_id: int,
vorlage: Beladungsvorlage, vorlage: Beladungsvorlage,
zustaendiger_server_id: int, zustaendiger_server_id: int,
gezogen_von_objekt_id: int | None = None, fahrzeug_id: int | None = None,
) -> Objekt: ) -> Objekt:
"""Prompt 09 (Neuanlage, nicht Duplizieren): Objektpositionen werden aus den """Prompt 09 (Neuanlage, nicht Duplizieren): Objektpositionen werden aus den
Vorlagenpositionen initialisiert, Ist-Menge startet bei 0 ("nicht kontrolliert"). Vorlagenpositionen initialisiert, Ist-Menge startet bei 0 ("nicht kontrolliert").
gezogen_von_objekt_id: optionale Fahrzeug/Anhänger-Kopplung (Nutzer-Vorgabe), fahrzeug_id: optionale Zuordnung zu einem Fahrzeug (1:n, Nutzer-Vorgabe)."""
beide Fälle möglich - unabhängiges Objekt (NULL) oder fest zugeordnet."""
objekt = Objekt( objekt = Objekt(
code=code, code=code,
name=name, name=name,
@@ -69,7 +68,7 @@ async def objekt_aus_vorlage(
vorlage_id=vorlage.id, vorlage_id=vorlage.id,
standort_id=standort_id, standort_id=standort_id,
zustaendiger_server_id=zustaendiger_server_id, zustaendiger_server_id=zustaendiger_server_id,
gezogen_von_objekt_id=gezogen_von_objekt_id, fahrzeug_id=fahrzeug_id,
) )
db.add(objekt) db.add(objekt)
await db.flush() await db.flush()
+35 -40
View File
@@ -300,23 +300,25 @@ async def test_objektposition_label_pdf_ohne_code_ist_404(
assert mit_code.content.startswith(b"%PDF") assert mit_code.content.startswith(b"%PDF")
@pytest.mark.asyncio @pytest.mark.asyncio
async def test_anhaenger_kopplung_bei_anlage_und_nachtraeglich( async def test_fahrzeug_zuordnung_bei_anlage_und_nachtraeglich(
client, db_session, objekttyp_factory, standort_factory, admin_user client, db_session, objekttyp_factory, standort_factory, admin_user
): ):
"""Nutzer-Vorgabe: Fahrzeug/Anhänger beide als eigenständige Objekte möglich, """Nutzer-Vorgabe: jedes Objekt (Rucksack/Gerät/Anhänger) gehört zu jedem
optional per gezogen_von_objekt_id gekoppelt.""" Zeitpunkt höchstens einem Fahrzeug (1:n) - Fahrzeug selbst ist einfach ein
objekttyp = await objekttyp_factory("Fahrzeug/Anhänger") Objekt ohne eigene fahrzeug_id."""
standort = await standort_factory("Wache-Kopplung") objekttyp = await objekttyp_factory("Fahrzeug/Zuordnung")
standort = await standort_factory("Wache-Zuordnung")
vorlage = await erstelle_vorlage( vorlage = await erstelle_vorlage(
db_session, objekttyp_id=objekttyp.id, name="Kopplung Standard", positionen=[] db_session, objekttyp_id=objekttyp.id, name="Zuordnung Standard", positionen=[]
) )
token = await login(client, "admin1") token = await login(client, "admin1")
fahrzeug = await client.post( fahrzeug = await client.post(
"/api/v1/objekte", "/api/v1/objekte",
json={ json={
"code": "FZ-K1", "code": "ZO-FZ",
"name": "Zugfahrzeug", "name": "Zugfahrzeug",
"objekttyp_id": objekttyp.id, "objekttyp_id": objekttyp.id,
"vorlage_id": vorlage.id, "vorlage_id": vorlage.id,
@@ -325,29 +327,29 @@ async def test_anhaenger_kopplung_bei_anlage_und_nachtraeglich(
headers=auth_header(token), headers=auth_header(token),
) )
fahrzeug_id = fahrzeug.json()["id"] fahrzeug_id = fahrzeug.json()["id"]
assert fahrzeug.json()["gezogen_von_objekt_id"] is None assert fahrzeug.json()["fahrzeug_id"] is None
anhaenger = await client.post( zugeordnet = await client.post(
"/api/v1/objekte", "/api/v1/objekte",
json={ json={
"code": "AH-K1", "code": "ZO-RK",
"name": "Anhänger", "name": "Rucksack",
"objekttyp_id": objekttyp.id, "objekttyp_id": objekttyp.id,
"vorlage_id": vorlage.id, "vorlage_id": vorlage.id,
"standort_id": standort.id, "standort_id": standort.id,
"gezogen_von_objekt_id": fahrzeug_id, "fahrzeug_id": fahrzeug_id,
}, },
headers=auth_header(token), headers=auth_header(token),
) )
assert anhaenger.status_code == 201 assert zugeordnet.status_code == 201
assert anhaenger.json()["gezogen_von_objekt_id"] == fahrzeug_id assert zugeordnet.json()["fahrzeug_id"] == fahrzeug_id
# unabhängiger Anhänger, erst nachträglich gekoppelt # unabhängiges Objekt, erst nachträglich zugeordnet
unabhaengig = await client.post( unabhaengig = await client.post(
"/api/v1/objekte", "/api/v1/objekte",
json={ json={
"code": "AH-K2", "code": "ZO-GR",
"name": "Anhänger 2", "name": "Gerät",
"objekttyp_id": objekttyp.id, "objekttyp_id": objekttyp.id,
"vorlage_id": vorlage.id, "vorlage_id": vorlage.id,
"standort_id": standort.id, "standort_id": standort.id,
@@ -355,39 +357,35 @@ async def test_anhaenger_kopplung_bei_anlage_und_nachtraeglich(
headers=auth_header(token), headers=auth_header(token),
) )
unabhaengig_id = unabhaengig.json()["id"] unabhaengig_id = unabhaengig.json()["id"]
assert unabhaengig.json()["gezogen_von_objekt_id"] is None assert unabhaengig.json()["fahrzeug_id"] is None
geaendert = await client.patch( geaendert = await client.patch(
f"/api/v1/objekte/{unabhaengig_id}", f"/api/v1/objekte/{unabhaengig_id}", json={"fahrzeug_id": fahrzeug_id}, headers=auth_header(token)
json={"gezogen_von_objekt_id": fahrzeug_id},
headers=auth_header(token),
) )
assert geaendert.status_code == 200 assert geaendert.status_code == 200
assert geaendert.json()["gezogen_von_objekt_id"] == fahrzeug_id assert geaendert.json()["fahrzeug_id"] == fahrzeug_id
geloest = await client.patch( geloest = await client.patch(
f"/api/v1/objekte/{unabhaengig_id}", f"/api/v1/objekte/{unabhaengig_id}", json={"fahrzeug_id": None}, headers=auth_header(token)
json={"gezogen_von_objekt_id": None},
headers=auth_header(token),
) )
assert geloest.json()["gezogen_von_objekt_id"] is None assert geloest.json()["fahrzeug_id"] is None
@pytest.mark.asyncio @pytest.mark.asyncio
async def test_anhaenger_kopplung_lehnt_selbstbezug_und_zyklus_ab( async def test_fahrzeug_zuordnung_lehnt_selbstbezug_und_zyklus_ab(
client, db_session, objekttyp_factory, standort_factory, admin_user client, db_session, objekttyp_factory, standort_factory, admin_user
): ):
objekttyp = await objekttyp_factory("Fahrzeug/Anhänger Zyklus") objekttyp = await objekttyp_factory("Fahrzeug/Zuordnung Zyklus")
standort = await standort_factory("Wache-Zyklus") standort = await standort_factory("Wache-Zyklus-Zuordnung")
vorlage = await erstelle_vorlage( vorlage = await erstelle_vorlage(
db_session, objekttyp_id=objekttyp.id, name="Zyklus Standard", positionen=[] db_session, objekttyp_id=objekttyp.id, name="Zyklus-Zuordnung Standard", positionen=[]
) )
token = await login(client, "admin1") token = await login(client, "admin1")
a = await client.post( a = await client.post(
"/api/v1/objekte", "/api/v1/objekte",
json={ json={
"code": "ZY-A", "code": "ZYO-A",
"name": "Objekt A", "name": "Objekt A",
"objekttyp_id": objekttyp.id, "objekttyp_id": objekttyp.id,
"vorlage_id": vorlage.id, "vorlage_id": vorlage.id,
@@ -397,35 +395,32 @@ async def test_anhaenger_kopplung_lehnt_selbstbezug_und_zyklus_ab(
) )
a_id = a.json()["id"] a_id = a.json()["id"]
# Selbstbezug: A zieht sich selbst
selbstbezug = await client.patch( selbstbezug = await client.patch(
f"/api/v1/objekte/{a_id}", json={"gezogen_von_objekt_id": a_id}, headers=auth_header(token) f"/api/v1/objekte/{a_id}", json={"fahrzeug_id": a_id}, headers=auth_header(token)
) )
assert selbstbezug.status_code == 409 assert selbstbezug.status_code == 409
# nicht existierendes Zielobjekt
ungueltig = await client.patch( ungueltig = await client.patch(
f"/api/v1/objekte/{a_id}", json={"gezogen_von_objekt_id": 999999}, headers=auth_header(token) f"/api/v1/objekte/{a_id}", json={"fahrzeug_id": 999999}, headers=auth_header(token)
) )
assert ungueltig.status_code == 404 assert ungueltig.status_code == 404
b = await client.post( b = await client.post(
"/api/v1/objekte", "/api/v1/objekte",
json={ json={
"code": "ZY-B", "code": "ZYO-B",
"name": "Objekt B", "name": "Objekt B",
"objekttyp_id": objekttyp.id, "objekttyp_id": objekttyp.id,
"vorlage_id": vorlage.id, "vorlage_id": vorlage.id,
"standort_id": standort.id, "standort_id": standort.id,
"gezogen_von_objekt_id": a_id, "fahrzeug_id": a_id,
}, },
headers=auth_header(token), headers=auth_header(token),
) )
b_id = b.json()["id"] b_id = b.json()["id"]
assert b.json()["gezogen_von_objekt_id"] == a_id assert b.json()["fahrzeug_id"] == a_id
# Zyklus: A soll jetzt von B gezogen werden (B wird bereits von A gezogen)
zyklus = await client.patch( zyklus = await client.patch(
f"/api/v1/objekte/{a_id}", json={"gezogen_von_objekt_id": b_id}, headers=auth_header(token) f"/api/v1/objekte/{a_id}", json={"fahrzeug_id": b_id}, headers=auth_header(token)
) )
assert zyklus.status_code == 409 assert zyklus.status_code == 409
+1 -1
View File
@@ -6,7 +6,7 @@ export interface Objekt {
vorlage_id: number | null; vorlage_id: number | null;
standort_id: number; standort_id: number;
status: string; status: string;
gezogen_von_objekt_id: number | null; fahrzeug_id: number | null;
} }
export interface Objektposition { export interface Objektposition {
+49 -45
View File
@@ -33,6 +33,7 @@ export function ObjektSection({
const [name, setName] = useState(""); const [name, setName] = useState("");
const [vorlageId, setVorlageId] = useState<number | "">(""); const [vorlageId, setVorlageId] = useState<number | "">("");
const [standortId, setStandortId] = useState<number | "">(""); const [standortId, setStandortId] = useState<number | "">("");
const [fahrzeugId, setFahrzeugId] = useState<number | "">("");
const [wirdAngelegt, setWirdAngelegt] = useState(false); const [wirdAngelegt, setWirdAngelegt] = useState(false);
const [ausgeklapptId, setAusgeklapptId] = useState<number | null>(null); const [ausgeklapptId, setAusgeklapptId] = useState<number | null>(null);
const [duplizierId, setDuplizierId] = useState<number | null>(null); const [duplizierId, setDuplizierId] = useState<number | null>(null);
@@ -40,11 +41,38 @@ export function ObjektSection({
const [neuerCode, setNeuerCode] = useState(""); const [neuerCode, setNeuerCode] = useState("");
const [neuerName, setNeuerName] = useState(""); const [neuerName, setNeuerName] = useState("");
const [neuerStandortId, setNeuerStandortId] = useState<number | "">(""); const [neuerStandortId, setNeuerStandortId] = useState<number | "">("");
const [gezogenVonId, setGezogenVonId] = useState<number | "">(""); const [fahrzeugBearbeitenId, setFahrzeugBearbeitenId] = useState<number | null>(null);
const [kopplungBearbeitenId, setKopplungBearbeitenId] = useState<number | null>(null); const [fahrzeugAuswahl, setFahrzeugAuswahl] = useState<number | "">("");
const [kopplungAuswahl, setKopplungAuswahl] = useState<number | "">("");
const [suche, setSuche] = useState(""); const [suche, setSuche] = useState("");
function objektName(id: number | null): string {
if (id === null) return "";
const gefunden = objekte.find((x) => x.id === id);
return gefunden ? `${gefunden.name} (${gefunden.code})` : `#${id}`;
}
function fahrzeugBearbeiten(o: Objekt) {
if (fahrzeugBearbeitenId === o.id) {
setFahrzeugBearbeitenId(null);
return;
}
setFahrzeugBearbeitenId(o.id);
setFahrzeugAuswahl(o.fahrzeug_id ?? "");
}
async function fahrzeugSpeichern(objektId: number) {
try {
await apiRequest(`/objekte/${objektId}`, {
method: "PATCH",
body: { fahrzeug_id: fahrzeugAuswahl === "" ? null : fahrzeugAuswahl },
});
setFahrzeugBearbeitenId(null);
await onGeaendert();
} catch {
onFehler("Fahrzeug-Zuordnung konnte nicht gespeichert werden.");
}
}
async function codeVorschlagen() { async function codeVorschlagen() {
if (!praefix) { if (!praefix) {
onFehler("Präfix eingeben (z. B. RK, FZ)."); onFehler("Präfix eingeben (z. B. RK, FZ).");
@@ -91,13 +119,13 @@ export function ObjektSection({
objekttyp_id: vorlage?.objekttyp_id, objekttyp_id: vorlage?.objekttyp_id,
vorlage_id: vorlageId, vorlage_id: vorlageId,
standort_id: standortId, standort_id: standortId,
gezogen_von_objekt_id: gezogenVonId || null, fahrzeug_id: fahrzeugId === "" ? null : fahrzeugId,
}, },
}); });
setPraefix(""); setPraefix("");
setCode(""); setCode("");
setName(""); setName("");
setGezogenVonId(""); setFahrzeugId("");
await onGeaendert(); await onGeaendert();
} catch { } catch {
onFehler("Objekt konnte nicht angelegt werden (Code evtl. schon vergeben)."); onFehler("Objekt konnte nicht angelegt werden (Code evtl. schon vergeben).");
@@ -106,30 +134,6 @@ export function ObjektSection({
} }
} }
function objektName(id: number | null): string {
if (id === null) return "";
const o = objekte.find((x) => x.id === id);
return o ? `${o.name} (${o.code})` : `Objekt ${id}`;
}
function kopplungBearbeiten(o: Objekt) {
setKopplungBearbeitenId(kopplungBearbeitenId === o.id ? null : o.id);
setKopplungAuswahl(o.gezogen_von_objekt_id ?? "");
}
async function kopplungSpeichern(objektId: number) {
try {
await apiRequest(`/objekte/${objektId}`, {
method: "PATCH",
body: { gezogen_von_objekt_id: kopplungAuswahl || null },
});
setKopplungBearbeitenId(null);
await onGeaendert();
} catch {
onFehler("Kopplung konnte nicht geändert werden.");
}
}
function duplizierenStarten(o: Objekt) { function duplizierenStarten(o: Objekt) {
setDuplizierId(duplizierId === o.id ? null : o.id); setDuplizierId(duplizierId === o.id ? null : o.id);
setNeuerPraefix(""); setNeuerPraefix("");
@@ -222,16 +226,16 @@ export function ObjektSection({
<input className="input" value={name} onChange={(e) => setName(e.target.value)} /> <input className="input" value={name} onChange={(e) => setName(e.target.value)} />
</div> </div>
<div className="field"> <div className="field">
<label>Gezogen von (optional, Anhänger-Kopplung)</label> <label>Fahrzeug (optional)</label>
<select <select
className="input" className="input"
value={gezogenVonId} value={fahrzeugId}
onChange={(e) => setGezogenVonId(Number(e.target.value) || "")} onChange={(e) => setFahrzeugId(Number(e.target.value) || "")}
> >
<option value="">Kein Zugfahrzeug</option> <option value="">Kein Fahrzeug</option>
{objekte.map((o) => ( {objekte.map((x) => (
<option key={o.id} value={o.id}> <option key={x.id} value={x.id}>
{o.name} ({o.code}) {x.name} ({x.code})
</option> </option>
))} ))}
</select> </select>
@@ -260,8 +264,8 @@ export function ObjektSection({
<div className="row-between" style={{ padding: "0.4rem 0" }}> <div className="row-between" style={{ padding: "0.4rem 0" }}>
<span> <span>
{o.name} <span className="text-muted">({o.code})</span> {o.name} <span className="text-muted">({o.code})</span>
{o.gezogen_von_objekt_id !== null && ( {o.fahrzeug_id !== null && (
<span className="text-muted"> gezogen von {objektName(o.gezogen_von_objekt_id)}</span> <span className="text-muted"> Fahrzeug: {objektName(o.fahrzeug_id)}</span>
)} )}
</span> </span>
<div className="row"> <div className="row">
@@ -272,8 +276,8 @@ export function ObjektSection({
{ausgeklapptId === o.id ? "Positionen ausblenden" : "Ablauf/Charge/SN pflegen"} {ausgeklapptId === o.id ? "Positionen ausblenden" : "Ablauf/Charge/SN pflegen"}
</button> </button>
{!nurPflegen && ( {!nurPflegen && (
<button className="btn btn-secondary" onClick={() => kopplungBearbeiten(o)}> <button className="btn btn-secondary" onClick={() => fahrzeugBearbeiten(o)}>
{kopplungBearbeitenId === o.id ? "Schließen" : "Kopplung ändern"} {fahrzeugBearbeitenId === o.id ? "Schließen" : "Fahrzeug zuordnen"}
</button> </button>
)} )}
{!nurPflegen && ( {!nurPflegen && (
@@ -283,14 +287,14 @@ export function ObjektSection({
)} )}
</div> </div>
</div> </div>
{kopplungBearbeitenId === o.id && ( {fahrzeugBearbeitenId === o.id && (
<div className="row" style={{ marginTop: "0.5rem", flexWrap: "wrap" }}> <div className="row" style={{ marginTop: "0.5rem", flexWrap: "wrap" }}>
<select <select
className="input" className="input"
value={kopplungAuswahl} value={fahrzeugAuswahl}
onChange={(e) => setKopplungAuswahl(Number(e.target.value) || "")} onChange={(e) => setFahrzeugAuswahl(Number(e.target.value) || "")}
> >
<option value="">Kein Zugfahrzeug</option> <option value="">Kein Fahrzeug</option>
{objekte {objekte
.filter((x) => x.id !== o.id) .filter((x) => x.id !== o.id)
.map((x) => ( .map((x) => (
@@ -299,7 +303,7 @@ export function ObjektSection({
</option> </option>
))} ))}
</select> </select>
<button className="btn btn-primary" onClick={() => kopplungSpeichern(o.id)}> <button className="btn btn-primary" onClick={() => fahrzeugSpeichern(o.id)}>
Speichern Speichern
</button> </button>
</div> </div>