fix(objekt): Fahrzeug-Zuordnung als 1:n statt n:m (fachliche Korrektur)
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:
@@ -195,9 +195,9 @@ async def erstelle_objekt(
|
||||
vorlage = await db.get(Beladungsvorlage, payload.vorlage_id)
|
||||
if vorlage is None:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Vorlage nicht gefunden")
|
||||
if payload.gezogen_von_objekt_id is not None:
|
||||
if await db.get(Objekt, payload.gezogen_von_objekt_id) is None:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Zugfahrzeug nicht gefunden")
|
||||
if payload.fahrzeug_id is not None:
|
||||
if await db.get(Objekt, payload.fahrzeug_id) is None:
|
||||
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))
|
||||
zustaendiger_server_id = hauptserver.scalar_one()
|
||||
@@ -210,7 +210,7 @@ async def erstelle_objekt(
|
||||
standort_id=payload.standort_id,
|
||||
vorlage=vorlage,
|
||||
zustaendiger_server_id=zustaendiger_server_id,
|
||||
gezogen_von_objekt_id=payload.gezogen_von_objekt_id,
|
||||
fahrzeug_id=payload.fahrzeug_id,
|
||||
)
|
||||
except IntegrityError as 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(
|
||||
objekt_id: int, payload: ObjektUpdate, db: AsyncSession = Depends(get_db), _=Depends(_admin_only)
|
||||
) -> Objekt:
|
||||
"""Fahrzeug/Anhänger-Kopplung nachträglich setzen/lösen (Nutzer-Vorgabe).
|
||||
postgres-expert-Review-Fund: Zielobjekt-Existenz und direkter Zyklus (A↔B)
|
||||
werden hier geprüft, mehrstufige Ketten sind fachlich nicht vorgesehen und
|
||||
bewusst nicht geprüft (Aufwand/Nutzen)."""
|
||||
"""Fahrzeug-Zuordnung nachträglich setzen/lösen: jedes Objekt gehört zu
|
||||
jedem Zeitpunkt höchstens einem Fahrzeug (1:n). Zielobjekt-Existenz und
|
||||
direkter Zyklus (A↔B) werden geprüft, mehrstufige Ketten sind fachlich
|
||||
nicht vorgesehen und bewusst nicht geprüft (Aufwand/Nutzen)."""
|
||||
objekt = await db.get(Objekt, objekt_id)
|
||||
if objekt is None:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Objekt nicht gefunden")
|
||||
|
||||
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:
|
||||
ziel = await db.get(Objekt, ziel_id)
|
||||
if ziel is None:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Zugfahrzeug nicht gefunden")
|
||||
if ziel.gezogen_von_objekt_id == objekt_id:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Fahrzeug nicht gefunden")
|
||||
if ziel.fahrzeug_id == objekt_id:
|
||||
raise HTTPException(
|
||||
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():
|
||||
|
||||
@@ -33,9 +33,10 @@ class Objekt(Base):
|
||||
zustaendiger_server_id: Mapped[int] = mapped_column(
|
||||
ForeignKey("systemknoten.id"), nullable=False
|
||||
)
|
||||
# Fahrzeug/Anhänger-Kopplung (Nutzer-Vorgabe): optional, beide Fälle möglich -
|
||||
# Anhänger als eigenständiges Objekt (NULL) oder fest einem Zugfahrzeug
|
||||
# zugeordnet. Selbstreferenzierend, kein eigener Anhänger-Objekttyp nötig.
|
||||
gezogen_von_objekt_id: Mapped[int | None] = mapped_column(
|
||||
# Zuordnung zu einem Fahrzeug (Nutzer-Vorgabe): jedes Objekt (Rucksack,
|
||||
# Gerät, Anhänger, ...) gehört zu jedem Zeitpunkt höchstens einem Fahrzeug
|
||||
# (1:n), NULL = eigenständig/keinem Fahrzeug zugeordnet. Selbstreferenzierend,
|
||||
# kein eigener Fahrzeug-Objekttyp nötig (Objekt bleibt generisch, Karte 11).
|
||||
fahrzeug_id: Mapped[int | None] = mapped_column(
|
||||
ForeignKey("objekt.id", ondelete="SET NULL")
|
||||
)
|
||||
|
||||
@@ -14,7 +14,7 @@ class ObjektCreate(BaseModel):
|
||||
objekttyp_id: int
|
||||
vorlage_id: int
|
||||
standort_id: int
|
||||
gezogen_von_objekt_id: int | None = None
|
||||
fahrzeug_id: int | None = None
|
||||
|
||||
|
||||
class ObjektRead(BaseModel):
|
||||
@@ -26,11 +26,11 @@ class ObjektRead(BaseModel):
|
||||
vorlage_id: int | None
|
||||
standort_id: int
|
||||
status: ObjektStatus
|
||||
gezogen_von_objekt_id: int | None
|
||||
fahrzeug_id: int | None
|
||||
|
||||
|
||||
class ObjektUpdate(BaseModel):
|
||||
gezogen_von_objekt_id: int | None = None
|
||||
fahrzeug_id: int | None = None
|
||||
|
||||
|
||||
class ObjektDuplizierenRequest(BaseModel):
|
||||
|
||||
@@ -56,12 +56,11 @@ async def objekt_aus_vorlage(
|
||||
standort_id: int,
|
||||
vorlage: Beladungsvorlage,
|
||||
zustaendiger_server_id: int,
|
||||
gezogen_von_objekt_id: int | None = None,
|
||||
fahrzeug_id: int | None = None,
|
||||
) -> Objekt:
|
||||
"""Prompt 09 (Neuanlage, nicht Duplizieren): Objektpositionen werden aus den
|
||||
Vorlagenpositionen initialisiert, Ist-Menge startet bei 0 ("nicht kontrolliert").
|
||||
gezogen_von_objekt_id: optionale Fahrzeug/Anhänger-Kopplung (Nutzer-Vorgabe),
|
||||
beide Fälle möglich - unabhängiges Objekt (NULL) oder fest zugeordnet."""
|
||||
fahrzeug_id: optionale Zuordnung zu einem Fahrzeug (1:n, Nutzer-Vorgabe)."""
|
||||
objekt = Objekt(
|
||||
code=code,
|
||||
name=name,
|
||||
@@ -69,7 +68,7 @@ async def objekt_aus_vorlage(
|
||||
vorlage_id=vorlage.id,
|
||||
standort_id=standort_id,
|
||||
zustaendiger_server_id=zustaendiger_server_id,
|
||||
gezogen_von_objekt_id=gezogen_von_objekt_id,
|
||||
fahrzeug_id=fahrzeug_id,
|
||||
)
|
||||
db.add(objekt)
|
||||
await db.flush()
|
||||
|
||||
Reference in New Issue
Block a user