794a81f58b
Beide kannte die API bisher nur als Anlegen und Loeschen. Wer sich vertippt hatte, musste den Eintrag wegwerfen und neu anlegen - und verlor dabei genau das, was daran haengt: Chargen zeigen auf die Lagerort-ID, Produkte und Gruppen auf die Einheiten-ID. Ein Tippfehler kostete also Zuordnungen. Neu ist je ein PATCH nach dem Vorbild der Gebinde, samt Pruefung auf doppelte Namen ohne Ruecksicht auf Gross- und Kleinschreibung. Anders als beim Loeschen duerfen auch eingebaute Einheiten umbenannt werden - auch das wie bei den Gebinden. Art und Faktor einer Einheit bleiben dagegen fest: Sie stecken in bereits umgerechneten Bestaenden, eine Aenderung wuerde die still verfaelschen. Die Tests halten fest, worum es eigentlich geht - dass die Verweise das Umbenennen ueberleben. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
71 lines
2.1 KiB
Python
71 lines
2.1 KiB
Python
from fastapi import APIRouter, Depends, HTTPException, status
|
|
from sqlalchemy import func
|
|
from sqlalchemy.orm import Session
|
|
|
|
from ..database import get_db
|
|
from ..deps import get_current_user, require_admin
|
|
from ..models import Location, User
|
|
from ..schemas import LocationCreate, LocationOut, LocationUpdate
|
|
|
|
router = APIRouter(prefix="/locations", tags=["locations"])
|
|
|
|
|
|
@router.get("", response_model=list[LocationOut])
|
|
def list_locations(
|
|
db: Session = Depends(get_db), _: User = Depends(get_current_user)
|
|
) -> list[Location]:
|
|
return db.query(Location).order_by(Location.name).all()
|
|
|
|
|
|
@router.post("", response_model=LocationOut, status_code=status.HTTP_201_CREATED)
|
|
def create_location(
|
|
payload: LocationCreate,
|
|
db: Session = Depends(get_db),
|
|
_: User = Depends(require_admin),
|
|
) -> Location:
|
|
loc = Location(name=payload.name, parent_id=payload.parent_id)
|
|
db.add(loc)
|
|
db.commit()
|
|
db.refresh(loc)
|
|
return loc
|
|
|
|
|
|
@router.patch("/{location_id}", response_model=LocationOut)
|
|
def update_location(
|
|
location_id: int,
|
|
payload: LocationUpdate,
|
|
db: Session = Depends(get_db),
|
|
_: User = Depends(require_admin),
|
|
) -> Location:
|
|
"""Umbenennen. Chargen haengen an der ID, behalten ihren Lagerort also."""
|
|
loc = db.get(Location, location_id)
|
|
if loc is None:
|
|
raise HTTPException(status.HTTP_404_NOT_FOUND, "Lagerort nicht gefunden")
|
|
|
|
name = payload.name.strip()
|
|
doppelt = (
|
|
db.query(Location)
|
|
.filter(func.lower(Location.name) == name.lower(), Location.id != location_id)
|
|
.first()
|
|
)
|
|
if doppelt is not None:
|
|
raise HTTPException(status.HTTP_409_CONFLICT, "Diesen Lagerort gibt es bereits")
|
|
|
|
loc.name = name
|
|
db.commit()
|
|
db.refresh(loc)
|
|
return loc
|
|
|
|
|
|
@router.delete("/{location_id}", status_code=status.HTTP_204_NO_CONTENT)
|
|
def delete_location(
|
|
location_id: int,
|
|
db: Session = Depends(get_db),
|
|
_: User = Depends(require_admin),
|
|
) -> None:
|
|
loc = db.get(Location, location_id)
|
|
if loc is None:
|
|
raise HTTPException(status.HTTP_404_NOT_FOUND, "Lagerort nicht gefunden")
|
|
db.delete(loc)
|
|
db.commit()
|