Sprint 2: Beladungsvorlagen (Versionierung), Objekte anlegen/duplizieren, Override-Logik
CI / backend-tests (push) Successful in 29s
CI / backend-tests (push) Successful in 29s
- Beladungsvorlage/Vorlagenposition-Modelle + Endpunkte: erstellen, neue Version
(Prompt 08 Punkt 4 - alte Version bleibt unverändert lesbar statt überschrieben)
- Objekt-Anlage aus Vorlage: Objektpositionen werden initialisiert, Ist-Menge startet
bei 0 ("nicht kontrolliert", Prompt 09)
- Duplizieren: Struktur/Sollmengen-Override kopiert, Ist-Menge/SN/Ablauf/Charge NICHT,
neuer Code Pflicht (409 bei Kollision)
- Sollmengen-Auflösung (Prompt 10): Override schlägt Vorlagenwert, sonst dynamisch aus
aktueller Vorlagenposition des Objekts
- GET /objekte/code/{code} bewusst vor /objekte/{id} registriert (Routing-Reihenfolge)
- Bugfix: app.models muss vollständig importiert werden (main.py + conftest.py), sonst
schlägt FK-Auflösung zwischen Modellen fehl, die noch von keinem Endpoint berührt
werden (Fund im dritten echten CI-Lauf)
- Tests: U8 (Override schlägt Vorlage), U9 (Duplizieren-Kopierregeln + Code-Konflikt),
U10 (neue Version, alte bleibt lesbar), Rollenrechte
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01L85hmKbvX7Cqkq47KnQhFt
This commit is contained in:
@@ -1,6 +1,6 @@
|
||||
from fastapi import APIRouter
|
||||
|
||||
from app.api.v1.endpoints import auth, benutzer, health, stammdaten, zustaendigkeit
|
||||
from app.api.v1.endpoints import auth, benutzer, health, objekte, stammdaten, vorlagen, zustaendigkeit
|
||||
|
||||
api_router = APIRouter()
|
||||
api_router.include_router(health.router, tags=["health"])
|
||||
@@ -8,3 +8,5 @@ api_router.include_router(auth.router, tags=["auth"])
|
||||
api_router.include_router(stammdaten.router, tags=["stammdaten"])
|
||||
api_router.include_router(benutzer.router, tags=["benutzer"])
|
||||
api_router.include_router(zustaendigkeit.router, tags=["zustaendigkeit"])
|
||||
api_router.include_router(vorlagen.router, tags=["vorlagen"])
|
||||
api_router.include_router(objekte.router, tags=["objekte"])
|
||||
|
||||
@@ -0,0 +1,222 @@
|
||||
import uuid
|
||||
|
||||
from fastapi import APIRouter, Depends, HTTPException, status
|
||||
from sqlalchemy import select
|
||||
from sqlalchemy.exc import IntegrityError
|
||||
from sqlalchemy.ext.asyncio import AsyncSession
|
||||
|
||||
from app.api.deps import get_current_user, require_roles
|
||||
from app.db.session import get_db
|
||||
from app.models.auth import KnotenTyp, RolleTyp, Systemknoten
|
||||
from app.models.objekt import Objekt
|
||||
from app.models.objektposition import Objektposition
|
||||
from app.models.vorlage import Beladungsvorlage
|
||||
from app.schemas.objekt import (
|
||||
ObjektCreate,
|
||||
ObjektDuplizierenRequest,
|
||||
ObjektpositionCreate,
|
||||
ObjektpositionRead,
|
||||
ObjektpositionUpdate,
|
||||
ObjektRead,
|
||||
)
|
||||
from app.services.objekte import dupliziere_objekt, objekt_aus_vorlage, sollmenge_effektiv, sollmenge_map
|
||||
|
||||
router = APIRouter()
|
||||
|
||||
_admin_only = require_roles(RolleTyp.administration)
|
||||
_materialverantwortliche = require_roles(RolleTyp.administration, RolleTyp.materialverantwortlicher)
|
||||
|
||||
|
||||
@router.get("/objekte", response_model=list[ObjektRead])
|
||||
async def liste_objekte(
|
||||
standort_id: int | None = None,
|
||||
db: AsyncSession = Depends(get_db),
|
||||
_=Depends(get_current_user),
|
||||
) -> list[Objekt]:
|
||||
stmt = select(Objekt)
|
||||
if standort_id is not None:
|
||||
stmt = stmt.where(Objekt.standort_id == standort_id)
|
||||
result = await db.execute(stmt)
|
||||
return list(result.scalars().all())
|
||||
|
||||
|
||||
@router.get("/objekte/code/{code}", response_model=ObjektRead)
|
||||
async def hole_objekt_per_code(
|
||||
code: str, db: AsyncSession = Depends(get_db), _=Depends(get_current_user)
|
||||
) -> Objekt:
|
||||
"""Lookup per QR/Barcode-Code (Karte 10). Muss VOR /objekte/{id} registriert
|
||||
sein, sonst greift FastAPI/Starlette-Routing zuerst die generische ID-Route."""
|
||||
result = await db.execute(select(Objekt).where(Objekt.code == code))
|
||||
objekt = result.scalar_one_or_none()
|
||||
if objekt is None:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Objekt nicht gefunden")
|
||||
return objekt
|
||||
|
||||
|
||||
@router.get("/objekte/{objekt_id}", response_model=ObjektRead)
|
||||
async def hole_objekt(
|
||||
objekt_id: int, db: AsyncSession = Depends(get_db), _=Depends(get_current_user)
|
||||
) -> Objekt:
|
||||
objekt = await db.get(Objekt, objekt_id)
|
||||
if objekt is None:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Objekt nicht gefunden")
|
||||
return objekt
|
||||
|
||||
|
||||
@router.get("/objekte/{objekt_id}/positionen", response_model=list[ObjektpositionRead])
|
||||
async def liste_objektpositionen(
|
||||
objekt_id: int, db: AsyncSession = Depends(get_db), _=Depends(get_current_user)
|
||||
) -> list[ObjektpositionRead]:
|
||||
objekt = await db.get(Objekt, objekt_id)
|
||||
if objekt is None:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Objekt nicht gefunden")
|
||||
|
||||
vorlagen_sollmengen = await sollmenge_map(db, objekt.vorlage_id)
|
||||
result = await db.execute(select(Objektposition).where(Objektposition.objekt_id == objekt_id))
|
||||
positionen = result.scalars().all()
|
||||
return [
|
||||
ObjektpositionRead(
|
||||
id=p.id,
|
||||
objekt_id=p.objekt_id,
|
||||
material_id=p.material_id,
|
||||
sollmenge_override=p.sollmenge_override,
|
||||
sollmenge_effektiv=sollmenge_effektiv(p, vorlagen_sollmengen),
|
||||
ist_status=p.ist_status,
|
||||
istmenge=p.istmenge,
|
||||
seriennummer=p.seriennummer,
|
||||
ablaufdatum=p.ablaufdatum,
|
||||
chargennummer=p.chargennummer,
|
||||
)
|
||||
for p in positionen
|
||||
]
|
||||
|
||||
|
||||
@router.post("/objekte", response_model=ObjektRead, status_code=status.HTTP_201_CREATED)
|
||||
async def erstelle_objekt(
|
||||
payload: ObjektCreate, db: AsyncSession = Depends(get_db), _=Depends(_admin_only)
|
||||
) -> Objekt:
|
||||
"""Prompt 09: neues Objekt aus einer Beladungsvorlage, startet "nicht kontrolliert"
|
||||
(Ist-Menge 0 in allen übernommenen Positionen)."""
|
||||
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")
|
||||
|
||||
hauptserver = await db.execute(select(Systemknoten.id).where(Systemknoten.typ == KnotenTyp.haupt))
|
||||
zustaendiger_server_id = hauptserver.scalar_one()
|
||||
|
||||
try:
|
||||
return await objekt_aus_vorlage(
|
||||
db,
|
||||
code=payload.code,
|
||||
name=payload.name,
|
||||
standort_id=payload.standort_id,
|
||||
vorlage=vorlage,
|
||||
zustaendiger_server_id=zustaendiger_server_id,
|
||||
)
|
||||
except IntegrityError as exc:
|
||||
raise HTTPException(status_code=status.HTTP_409_CONFLICT, detail="Code bereits vergeben") from exc
|
||||
|
||||
|
||||
@router.post(
|
||||
"/objekte/{objekt_id}/duplizieren", response_model=ObjektRead, status_code=status.HTTP_201_CREATED
|
||||
)
|
||||
async def dupliziere(
|
||||
objekt_id: int,
|
||||
payload: ObjektDuplizierenRequest,
|
||||
db: AsyncSession = Depends(get_db),
|
||||
_=Depends(_admin_only),
|
||||
) -> Objekt:
|
||||
"""Prompt 09: Struktur/Sollmengen kopiert, Ist-Menge/SN/Ablauf/Charge NICHT,
|
||||
neuer eindeutiger Code Pflicht."""
|
||||
quelle = await db.get(Objekt, objekt_id)
|
||||
if quelle is None:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Quell-Objekt nicht gefunden")
|
||||
|
||||
try:
|
||||
return await dupliziere_objekt(
|
||||
db,
|
||||
quelle=quelle,
|
||||
neuer_code=payload.neuer_code,
|
||||
neuer_name=payload.neuer_name,
|
||||
neuer_standort_id=payload.neuer_standort_id,
|
||||
zustaendiger_server_id=quelle.zustaendiger_server_id,
|
||||
)
|
||||
except IntegrityError as exc:
|
||||
raise HTTPException(status_code=status.HTTP_409_CONFLICT, detail="Code bereits vergeben") from exc
|
||||
|
||||
|
||||
@router.patch("/objekte/{objekt_id}/positionen/{position_id}", response_model=ObjektpositionRead)
|
||||
async def aendere_objektposition(
|
||||
objekt_id: int,
|
||||
position_id: uuid.UUID,
|
||||
payload: ObjektpositionUpdate,
|
||||
db: AsyncSession = Depends(get_db),
|
||||
_=Depends(_materialverantwortliche),
|
||||
) -> ObjektpositionRead:
|
||||
"""Prompt 10: Sollmengen-Override setzen/aufheben, Material als entfernt markieren,
|
||||
Ist-Werte (Menge/SN/Ablauf/Charge) pflegen."""
|
||||
position = await db.get(Objektposition, position_id)
|
||||
if position is None or position.objekt_id != objekt_id:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Objektposition nicht gefunden")
|
||||
|
||||
for feld, wert in payload.model_dump(exclude_unset=True).items():
|
||||
setattr(position, feld, wert)
|
||||
await db.flush()
|
||||
|
||||
objekt = await db.get(Objekt, objekt_id)
|
||||
vorlagen_sollmengen = await sollmenge_map(db, objekt.vorlage_id if objekt else None)
|
||||
return ObjektpositionRead(
|
||||
id=position.id,
|
||||
objekt_id=position.objekt_id,
|
||||
material_id=position.material_id,
|
||||
sollmenge_override=position.sollmenge_override,
|
||||
sollmenge_effektiv=sollmenge_effektiv(position, vorlagen_sollmengen),
|
||||
ist_status=position.ist_status,
|
||||
istmenge=position.istmenge,
|
||||
seriennummer=position.seriennummer,
|
||||
ablaufdatum=position.ablaufdatum,
|
||||
chargennummer=position.chargennummer,
|
||||
)
|
||||
|
||||
|
||||
@router.post(
|
||||
"/objekte/{objekt_id}/positionen",
|
||||
response_model=ObjektpositionRead,
|
||||
status_code=status.HTTP_201_CREATED,
|
||||
)
|
||||
async def fuege_zusatzposition_hinzu(
|
||||
objekt_id: int,
|
||||
payload: ObjektpositionCreate,
|
||||
db: AsyncSession = Depends(get_db),
|
||||
_=Depends(_materialverantwortliche),
|
||||
) -> ObjektpositionRead:
|
||||
"""Prompt 10 Punkt 2: zusätzliches, nicht in der Vorlage enthaltenes Material."""
|
||||
objekt = await db.get(Objekt, objekt_id)
|
||||
if objekt is None:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Objekt nicht gefunden")
|
||||
|
||||
position = Objektposition(
|
||||
objekt_id=objekt_id,
|
||||
material_id=payload.material_id,
|
||||
sollmenge_override=payload.sollmenge_override,
|
||||
)
|
||||
db.add(position)
|
||||
try:
|
||||
await db.flush()
|
||||
except IntegrityError as exc:
|
||||
raise HTTPException(
|
||||
status_code=status.HTTP_409_CONFLICT, detail="Material existiert bereits an diesem Objekt"
|
||||
) from exc
|
||||
|
||||
return ObjektpositionRead(
|
||||
id=position.id,
|
||||
objekt_id=position.objekt_id,
|
||||
material_id=position.material_id,
|
||||
sollmenge_override=position.sollmenge_override,
|
||||
sollmenge_effektiv=payload.sollmenge_override,
|
||||
ist_status=position.ist_status,
|
||||
istmenge=position.istmenge,
|
||||
seriennummer=position.seriennummer,
|
||||
ablaufdatum=position.ablaufdatum,
|
||||
chargennummer=position.chargennummer,
|
||||
)
|
||||
@@ -0,0 +1,78 @@
|
||||
from fastapi import APIRouter, Depends, HTTPException, status
|
||||
from sqlalchemy import select
|
||||
from sqlalchemy.ext.asyncio import AsyncSession
|
||||
|
||||
from app.api.deps import get_current_user, require_roles
|
||||
from app.db.session import get_db
|
||||
from app.models.auth import RolleTyp
|
||||
from app.models.vorlage import Beladungsvorlage
|
||||
from app.schemas.vorlage import BeladungsvorlageCreate, BeladungsvorlageRead, VorlagenAenderung
|
||||
from app.services.vorlagen import erstelle_vorlage, hole_positionen, neue_version
|
||||
|
||||
router = APIRouter()
|
||||
|
||||
_ersteller = require_roles(RolleTyp.administration, RolleTyp.materialverantwortlicher)
|
||||
|
||||
|
||||
async def _mit_positionen(db: AsyncSession, vorlage: Beladungsvorlage) -> BeladungsvorlageRead:
|
||||
positionen = await hole_positionen(db, vorlage.id)
|
||||
return BeladungsvorlageRead(
|
||||
id=vorlage.id,
|
||||
objekttyp_id=vorlage.objekttyp_id,
|
||||
name=vorlage.name,
|
||||
version=vorlage.version,
|
||||
status=vorlage.status,
|
||||
positionen=positionen,
|
||||
)
|
||||
|
||||
|
||||
@router.get("/vorlagen", response_model=list[BeladungsvorlageRead])
|
||||
async def liste_vorlagen(
|
||||
objekttyp_id: int | None = None,
|
||||
db: AsyncSession = Depends(get_db),
|
||||
_=Depends(get_current_user),
|
||||
) -> list[BeladungsvorlageRead]:
|
||||
stmt = select(Beladungsvorlage)
|
||||
if objekttyp_id is not None:
|
||||
stmt = stmt.where(Beladungsvorlage.objekttyp_id == objekttyp_id)
|
||||
result = await db.execute(stmt)
|
||||
return [await _mit_positionen(db, v) for v in result.scalars().all()]
|
||||
|
||||
|
||||
@router.get("/vorlagen/{vorlage_id}", response_model=BeladungsvorlageRead)
|
||||
async def hole_vorlage(
|
||||
vorlage_id: int, db: AsyncSession = Depends(get_db), _=Depends(get_current_user)
|
||||
) -> BeladungsvorlageRead:
|
||||
vorlage = await db.get(Beladungsvorlage, vorlage_id)
|
||||
if vorlage is None:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Vorlage nicht gefunden")
|
||||
return await _mit_positionen(db, vorlage)
|
||||
|
||||
|
||||
@router.post("/vorlagen", response_model=BeladungsvorlageRead, status_code=status.HTTP_201_CREATED)
|
||||
async def erstelle_neue_vorlage(
|
||||
payload: BeladungsvorlageCreate, db: AsyncSession = Depends(get_db), _=Depends(_ersteller)
|
||||
) -> BeladungsvorlageRead:
|
||||
vorlage = await erstelle_vorlage(
|
||||
db, objekttyp_id=payload.objekttyp_id, name=payload.name, positionen=payload.positionen
|
||||
)
|
||||
return await _mit_positionen(db, vorlage)
|
||||
|
||||
|
||||
@router.post(
|
||||
"/vorlagen/{vorlage_id}/neue-version",
|
||||
response_model=BeladungsvorlageRead,
|
||||
status_code=status.HTTP_201_CREATED,
|
||||
)
|
||||
async def erzeuge_neue_version(
|
||||
vorlage_id: int,
|
||||
payload: VorlagenAenderung,
|
||||
db: AsyncSession = Depends(get_db),
|
||||
_=Depends(_ersteller),
|
||||
) -> BeladungsvorlageRead:
|
||||
"""Prompt 08 Punkt 4: erzeugt neue Version, alte bleibt unverändert/lesbar."""
|
||||
aktuelle_vorlage = await db.get(Beladungsvorlage, vorlage_id)
|
||||
if aktuelle_vorlage is None:
|
||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Vorlage nicht gefunden")
|
||||
neue = await neue_version(db, aktuelle_vorlage=aktuelle_vorlage, positionen=payload.positionen)
|
||||
return await _mit_positionen(db, neue)
|
||||
Reference in New Issue
Block a user