feat(personal): PERS-001/002 Person fachlich von Benutzer getrennt
Neue Tabelle person (Migration 0034, Backfill bestehender Benutzer 1:1 per Zeilennummer, keine Datenverluste). Benutzer.person_id optional - Login kann bestehende Person verknüpfen oder legt automatisch eine neue an (Rückwärtskompatibilität zum bisherigen Anlegen-Flow). /personen-CRUD (admin-only), neue PersonSection.tsx im Personal-Tab, BenutzerSection um Personen-Auswahl erweitert. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CVgbozhYmuEhiEJHffRXCV
This commit is contained in:
@@ -11347,3 +11347,99 @@ Keine Commits in dieser Session.
|
|||||||
- frontend/src/pages/admin/ObjekttypSection.tsx | 170 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------
|
- frontend/src/pages/admin/ObjekttypSection.tsx | 170 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------
|
||||||
|
|
||||||
---
|
---
|
||||||
|
## 2026-09-08 22:05 – 22:06 (0m)
|
||||||
|
**Beschreibung:** Claude Code Session
|
||||||
|
**Projekt:** frontend
|
||||||
|
|
||||||
|
### Commits
|
||||||
|
- 847c303 feat(wartung): MAINT-005 Ersatzteile/Kosten je Wartungsauftrag
|
||||||
|
|
||||||
|
### Geänderte Dateien
|
||||||
|
- .claude/agents/DEVLOG.md | 11 +
|
||||||
|
- DEVLOG.md | 4565 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
|
||||||
|
- arbeitskacheln/00_index.md | 11 +-
|
||||||
|
- arbeitskacheln/22_ui_redesign_v2.md | 107 +++
|
||||||
|
- backend/DEVLOG.md | 50 +
|
||||||
|
- backend/alembic/versions/0033_wartungsauftrag_teil.py | 33 +
|
||||||
|
- backend/app/api/v1/endpoints/wartung.py | 49 +-
|
||||||
|
- backend/app/models/wartung.py | 10 +
|
||||||
|
- backend/app/schemas/wartung.py | 15 +
|
||||||
|
- backend/app/services/wartung.py | 22 +
|
||||||
|
- backend/tests/test_wartung.py | 70 ++
|
||||||
|
- frontend/DEVLOG.md | 176 ++++
|
||||||
|
- frontend/src/api/types.ts | 8 +
|
||||||
|
- frontend/src/components/WartungSection.tsx | 111 ++-
|
||||||
|
|
||||||
|
---
|
||||||
|
## 2026-09-08 22:06 – 22:06 (0m)
|
||||||
|
**Beschreibung:** Claude Code Session
|
||||||
|
**Projekt:** asb-material
|
||||||
|
|
||||||
|
### Commits
|
||||||
|
Keine Commits in dieser Session.
|
||||||
|
|
||||||
|
### Geänderte Dateien
|
||||||
|
- .claude/agents/DEVLOG.md | 11 +
|
||||||
|
- DEVLOG.md | 4565 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
|
||||||
|
- arbeitskacheln/00_index.md | 11 +-
|
||||||
|
- arbeitskacheln/22_ui_redesign_v2.md | 107 +++
|
||||||
|
- backend/DEVLOG.md | 50 +
|
||||||
|
- backend/alembic/versions/0033_wartungsauftrag_teil.py | 33 +
|
||||||
|
- backend/app/api/v1/endpoints/wartung.py | 49 +-
|
||||||
|
- backend/app/models/wartung.py | 10 +
|
||||||
|
- backend/app/schemas/wartung.py | 15 +
|
||||||
|
- backend/app/services/wartung.py | 22 +
|
||||||
|
- backend/tests/test_wartung.py | 70 ++
|
||||||
|
- frontend/DEVLOG.md | 176 ++++
|
||||||
|
- frontend/src/api/types.ts | 8 +
|
||||||
|
- frontend/src/components/WartungSection.tsx | 111 ++-
|
||||||
|
|
||||||
|
---
|
||||||
|
## 2026-09-08 22:13 – 22:13 (0m)
|
||||||
|
**Beschreibung:** Claude Code Session
|
||||||
|
**Projekt:** asb-material
|
||||||
|
|
||||||
|
### Commits
|
||||||
|
Keine Commits in dieser Session.
|
||||||
|
|
||||||
|
### Geänderte Dateien
|
||||||
|
- .claude/agents/DEVLOG.md | 11 +
|
||||||
|
- DEVLOG.md | 4565 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
|
||||||
|
- arbeitskacheln/00_index.md | 11 +-
|
||||||
|
- arbeitskacheln/22_ui_redesign_v2.md | 107 +++
|
||||||
|
- backend/DEVLOG.md | 50 +
|
||||||
|
- backend/alembic/versions/0033_wartungsauftrag_teil.py | 33 +
|
||||||
|
- backend/app/api/v1/endpoints/wartung.py | 49 +-
|
||||||
|
- backend/app/models/wartung.py | 10 +
|
||||||
|
- backend/app/schemas/wartung.py | 15 +
|
||||||
|
- backend/app/services/wartung.py | 22 +
|
||||||
|
- backend/tests/test_wartung.py | 70 ++
|
||||||
|
- frontend/DEVLOG.md | 176 ++++
|
||||||
|
- frontend/src/api/types.ts | 8 +
|
||||||
|
- frontend/src/components/WartungSection.tsx | 111 ++-
|
||||||
|
|
||||||
|
---
|
||||||
|
## 2026-09-08 22:13 – 22:24 (10m)
|
||||||
|
**Beschreibung:** Claude Code Session
|
||||||
|
**Projekt:** asb-material
|
||||||
|
|
||||||
|
### Commits
|
||||||
|
Keine Commits in dieser Session.
|
||||||
|
|
||||||
|
### Geänderte Dateien
|
||||||
|
- .claude/agents/DEVLOG.md | 11 +
|
||||||
|
- DEVLOG.md | 4565 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
|
||||||
|
- arbeitskacheln/00_index.md | 11 +-
|
||||||
|
- arbeitskacheln/22_ui_redesign_v2.md | 107 +++
|
||||||
|
- backend/DEVLOG.md | 50 +
|
||||||
|
- backend/alembic/versions/0033_wartungsauftrag_teil.py | 33 +
|
||||||
|
- backend/app/api/v1/endpoints/wartung.py | 49 +-
|
||||||
|
- backend/app/models/wartung.py | 10 +
|
||||||
|
- backend/app/schemas/wartung.py | 15 +
|
||||||
|
- backend/app/services/wartung.py | 22 +
|
||||||
|
- backend/tests/test_wartung.py | 70 ++
|
||||||
|
- frontend/DEVLOG.md | 176 ++++
|
||||||
|
- frontend/src/api/types.ts | 8 +
|
||||||
|
- frontend/src/components/WartungSection.tsx | 111 ++-
|
||||||
|
|
||||||
|
---
|
||||||
|
|||||||
@@ -202,8 +202,8 @@ vergeben statt geraten.
|
|||||||
| DEFECT-003 | Defects | Verantwortlicher & Reparaturzuordnung | P1 | S | LOW | DEFECT-002, PERS-002 | ✅ (Zustaendigkeit/Kontrollverantwortung koppelt an Mangel-Objekt) |
|
| DEFECT-003 | Defects | Verantwortlicher & Reparaturzuordnung | P1 | S | LOW | DEFECT-002, PERS-002 | ✅ (Zustaendigkeit/Kontrollverantwortung koppelt an Mangel-Objekt) |
|
||||||
| DEFECT-004 | Defects | Abschluss & Dokumentation | P1 | S | LOW | DEFECT-003 | ✅ (`PATCH /maengel/{id}` Statuswechsel inkl. Abschluss) |
|
| DEFECT-004 | Defects | Abschluss & Dokumentation | P1 | S | LOW | DEFECT-003 | ✅ (`PATCH /maengel/{id}` Statuswechsel inkl. Abschluss) |
|
||||||
| DEFECT-005 | Defects | Readiness-Einfluss (kritisch) | P1 | S | MEDIUM | DEFECT-002, READY-001 | ✅ (`services/mangel.py`: Priorität "hoch" macht Objekt "nicht einsatzbereit") |
|
| DEFECT-005 | Defects | Readiness-Einfluss (kritisch) | P1 | S | MEDIUM | DEFECT-002, READY-001 | ✅ (`services/mangel.py`: Priorität "hoch" macht Objekt "nicht einsatzbereit") |
|
||||||
| PERS-001 | Personnel | Person (fachlich) | P1 | M | HIGH | FOUND-002 | ⬜ (laut DECISION-Abschnitt der Datei selbst nicht getrennt, nur `Benutzer`) |
|
| PERS-001 | Personnel | Person (fachlich) | P1 | M | HIGH | FOUND-002 | ✅ (`Person`-Modell, Migration 0034 mit Backfill bestehender Benutzer, `/personen`-CRUD, Admin-UI `PersonSection.tsx`) |
|
||||||
| PERS-002 | Personnel | Benutzer (technisch, verknüpft) | P0 | M | HIGH | PERS-001, FOUND-003 | ✅ (`Benutzer`-Modell, `benutzer.py`-Endpunkt) |
|
| PERS-002 | Personnel | Benutzer (technisch, verknüpft) | P0 | M | HIGH | PERS-001, FOUND-003 | ✅ (`Benutzer.person_id` FK, `/benutzer` verknüpft bestehende oder legt automatisch neue Person an) |
|
||||||
| PERS-003 | Personnel | Einheiten/Organisationsstruktur | P1 | S | LOW | PERS-001 | ✅ (`Einheit`-Modell, hierarchisch, `einheiten`-Endpunkt) |
|
| PERS-003 | Personnel | Einheiten/Organisationsstruktur | P1 | S | LOW | PERS-001 | ✅ (`Einheit`-Modell, hierarchisch, `einheiten`-Endpunkt) |
|
||||||
| PERS-004 | Personnel | Funktionen/Rollen im Einsatzkontext | P2 | S | LOW | PERS-003 | ✅ (`Einsatzfunktion`-Stammdaten, Migration 0031, `benutzer.einsatzfunktion_id`, Admin-UI in BenutzerSection) |
|
| PERS-004 | Personnel | Funktionen/Rollen im Einsatzkontext | P2 | S | LOW | PERS-003 | ✅ (`Einsatzfunktion`-Stammdaten, Migration 0031, `benutzer.einsatzfunktion_id`, Admin-UI in BenutzerSection) |
|
||||||
| PERS-005 | Personnel | Qualifikationen/Lehrgänge | P1 | S | LOW | PERS-001 | ✅ (`Qualifikationstyp`, `BenutzerQualifikation`) |
|
| PERS-005 | Personnel | Qualifikationen/Lehrgänge | P1 | S | LOW | PERS-001 | ✅ (`Qualifikationstyp`, `BenutzerQualifikation`) |
|
||||||
|
|||||||
@@ -0,0 +1,58 @@
|
|||||||
|
"""PERS-001/002: Person fachlich getrennt von Benutzer (Login), Backfill bestehender Benutzer.
|
||||||
|
|
||||||
|
Revision ID: 0034_person
|
||||||
|
Revises: 0033_wartungsauftrag_teil
|
||||||
|
Create Date: 2026-09-08
|
||||||
|
"""
|
||||||
|
from typing import Sequence, Union
|
||||||
|
|
||||||
|
from alembic import op
|
||||||
|
|
||||||
|
revision: str = "0034_person"
|
||||||
|
down_revision: Union[str, None] = "0033_wartungsauftrag_teil"
|
||||||
|
branch_labels: Union[str, Sequence[str], None] = None
|
||||||
|
depends_on: Union[str, Sequence[str], None] = None
|
||||||
|
|
||||||
|
|
||||||
|
def upgrade() -> None:
|
||||||
|
op.execute(
|
||||||
|
"""
|
||||||
|
CREATE TABLE person (
|
||||||
|
id SERIAL PRIMARY KEY,
|
||||||
|
name TEXT NOT NULL,
|
||||||
|
geburtsdatum DATE,
|
||||||
|
kontakt TEXT
|
||||||
|
)
|
||||||
|
"""
|
||||||
|
)
|
||||||
|
op.execute("ALTER TABLE benutzer ADD COLUMN person_id INTEGER REFERENCES person(id)")
|
||||||
|
op.execute("CREATE INDEX ix_benutzer_person_id ON benutzer (person_id)")
|
||||||
|
|
||||||
|
# Backfill: jeder bestehende Benutzer bekommt eine EIGENE Person mit gleichem
|
||||||
|
# Namen (1:1 per Zeilennummer statt Namensvergleich, sonst würden Benutzer
|
||||||
|
# mit identischem Namen fälschlich derselben Person zugeordnet).
|
||||||
|
op.execute(
|
||||||
|
"""
|
||||||
|
INSERT INTO person (name)
|
||||||
|
SELECT name FROM benutzer ORDER BY id
|
||||||
|
"""
|
||||||
|
)
|
||||||
|
op.execute(
|
||||||
|
"""
|
||||||
|
WITH benutzer_rang AS (
|
||||||
|
SELECT id, ROW_NUMBER() OVER (ORDER BY id) AS rang FROM benutzer
|
||||||
|
), person_rang AS (
|
||||||
|
SELECT id, ROW_NUMBER() OVER (ORDER BY id) AS rang FROM person
|
||||||
|
)
|
||||||
|
UPDATE benutzer
|
||||||
|
SET person_id = person_rang.id
|
||||||
|
FROM benutzer_rang
|
||||||
|
JOIN person_rang ON person_rang.rang = benutzer_rang.rang
|
||||||
|
WHERE benutzer.id = benutzer_rang.id
|
||||||
|
"""
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
def downgrade() -> None:
|
||||||
|
op.execute("ALTER TABLE benutzer DROP COLUMN person_id")
|
||||||
|
op.execute("DROP TABLE person")
|
||||||
@@ -6,6 +6,7 @@ from app.api.deps import require_roles
|
|||||||
from app.core.security import hash_password
|
from app.core.security import hash_password
|
||||||
from app.db.session import get_db
|
from app.db.session import get_db
|
||||||
from app.models.auth import Benutzer, BenutzerRolle, RolleTyp
|
from app.models.auth import Benutzer, BenutzerRolle, RolleTyp
|
||||||
|
from app.models.personal import Person
|
||||||
from app.schemas.benutzer import BenutzerCreate, BenutzerRead, BenutzerUpdate
|
from app.schemas.benutzer import BenutzerCreate, BenutzerRead, BenutzerUpdate
|
||||||
|
|
||||||
router = APIRouter()
|
router = APIRouter()
|
||||||
@@ -31,6 +32,19 @@ async def erstelle_benutzer(
|
|||||||
if result.scalar_one_or_none() is not None:
|
if result.scalar_one_or_none() is not None:
|
||||||
raise HTTPException(status_code=status.HTTP_409_CONFLICT, detail="Login bereits vergeben")
|
raise HTTPException(status_code=status.HTTP_409_CONFLICT, detail="Login bereits vergeben")
|
||||||
|
|
||||||
|
person_id = payload.person_id
|
||||||
|
if person_id is not None:
|
||||||
|
if await db.get(Person, person_id) is None:
|
||||||
|
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Person nicht gefunden")
|
||||||
|
else:
|
||||||
|
# PERS-002: kein bestehender Person-Datensatz gewählt -> automatisch
|
||||||
|
# anlegen, damit jeder Benutzer weiterhin ohne Zusatzschritt eine
|
||||||
|
# verknüpfte Person hat (Rückwärtskompatibilität zum bisherigen Flow).
|
||||||
|
person = Person(name=payload.name)
|
||||||
|
db.add(person)
|
||||||
|
await db.flush()
|
||||||
|
person_id = person.id
|
||||||
|
|
||||||
benutzer = Benutzer(
|
benutzer = Benutzer(
|
||||||
name=payload.name,
|
name=payload.name,
|
||||||
login=payload.login,
|
login=payload.login,
|
||||||
@@ -39,6 +53,7 @@ async def erstelle_benutzer(
|
|||||||
aktiv=True,
|
aktiv=True,
|
||||||
einheit_id=payload.einheit_id,
|
einheit_id=payload.einheit_id,
|
||||||
einsatzfunktion_id=payload.einsatzfunktion_id,
|
einsatzfunktion_id=payload.einsatzfunktion_id,
|
||||||
|
person_id=person_id,
|
||||||
)
|
)
|
||||||
db.add(benutzer)
|
db.add(benutzer)
|
||||||
await db.flush()
|
await db.flush()
|
||||||
|
|||||||
@@ -11,6 +11,7 @@ from app.models.personal import (
|
|||||||
Einheit,
|
Einheit,
|
||||||
Einsatzfunktion,
|
Einsatzfunktion,
|
||||||
ObjekttypQualifikationsanforderung,
|
ObjekttypQualifikationsanforderung,
|
||||||
|
Person,
|
||||||
Qualifikationstyp,
|
Qualifikationstyp,
|
||||||
)
|
)
|
||||||
from app.schemas.personal import (
|
from app.schemas.personal import (
|
||||||
@@ -24,12 +25,15 @@ from app.schemas.personal import (
|
|||||||
EinsatzfunktionRead,
|
EinsatzfunktionRead,
|
||||||
ObjekttypQualifikationsanforderungCreate,
|
ObjekttypQualifikationsanforderungCreate,
|
||||||
ObjekttypQualifikationsanforderungRead,
|
ObjekttypQualifikationsanforderungRead,
|
||||||
|
PersonCreate,
|
||||||
|
PersonRead,
|
||||||
|
PersonUpdate,
|
||||||
QualifikationstypCreate,
|
QualifikationstypCreate,
|
||||||
QualifikationstypRead,
|
QualifikationstypRead,
|
||||||
QualifikationstypUpdate,
|
QualifikationstypUpdate,
|
||||||
)
|
)
|
||||||
from app.services import historie as historie_service
|
from app.services import historie as historie_service
|
||||||
from app.services.personal import pruefe_berechtigung
|
from app.services.personal import erstelle_person, liste_personen, pruefe_berechtigung
|
||||||
|
|
||||||
router = APIRouter()
|
router = APIRouter()
|
||||||
|
|
||||||
@@ -41,6 +45,39 @@ async def _hauptserver_id(db: AsyncSession) -> int:
|
|||||||
_admin_only = require_roles(RolleTyp.administration)
|
_admin_only = require_roles(RolleTyp.administration)
|
||||||
|
|
||||||
|
|
||||||
|
# -- Person (PERS-001) ---------------------------------------------------------
|
||||||
|
|
||||||
|
@router.get("/personen", response_model=list[PersonRead])
|
||||||
|
async def liste_personen_endpunkt(db: AsyncSession = Depends(get_db), _=Depends(_admin_only)) -> list[Person]:
|
||||||
|
return await liste_personen(db)
|
||||||
|
|
||||||
|
|
||||||
|
@router.post("/personen", response_model=PersonRead, status_code=status.HTTP_201_CREATED)
|
||||||
|
async def erstelle_person_endpunkt(
|
||||||
|
payload: PersonCreate, db: AsyncSession = Depends(get_db), _=Depends(_admin_only)
|
||||||
|
) -> Person:
|
||||||
|
return await erstelle_person(
|
||||||
|
db, name=payload.name, geburtsdatum=payload.geburtsdatum, kontakt=payload.kontakt
|
||||||
|
)
|
||||||
|
|
||||||
|
|
||||||
|
@router.patch("/personen/{person_id}", response_model=PersonRead)
|
||||||
|
async def aendere_person(
|
||||||
|
person_id: int, payload: PersonUpdate, db: AsyncSession = Depends(get_db), _=Depends(_admin_only)
|
||||||
|
) -> Person:
|
||||||
|
person = await db.get(Person, person_id)
|
||||||
|
if person is None:
|
||||||
|
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Person nicht gefunden")
|
||||||
|
if payload.name is not None:
|
||||||
|
person.name = payload.name
|
||||||
|
if "geburtsdatum" in payload.model_fields_set:
|
||||||
|
person.geburtsdatum = payload.geburtsdatum
|
||||||
|
if "kontakt" in payload.model_fields_set:
|
||||||
|
person.kontakt = payload.kontakt
|
||||||
|
await db.flush()
|
||||||
|
return person
|
||||||
|
|
||||||
|
|
||||||
# -- Einheit ------------------------------------------------------------------
|
# -- Einheit ------------------------------------------------------------------
|
||||||
|
|
||||||
@router.get("/einheiten", response_model=list[EinheitRead])
|
@router.get("/einheiten", response_model=list[EinheitRead])
|
||||||
|
|||||||
@@ -47,6 +47,11 @@ class Benutzer(Base):
|
|||||||
einheit_id: Mapped[int | None] = mapped_column(ForeignKey("einheit.id")) # Personal-Modul (Migration 0012)
|
einheit_id: Mapped[int | None] = mapped_column(ForeignKey("einheit.id")) # Personal-Modul (Migration 0012)
|
||||||
# PERS-004: primäre Funktion im Einsatzkontext, optional.
|
# PERS-004: primäre Funktion im Einsatzkontext, optional.
|
||||||
einsatzfunktion_id: Mapped[int | None] = mapped_column(ForeignKey("einsatzfunktion.id"))
|
einsatzfunktion_id: Mapped[int | None] = mapped_column(ForeignKey("einsatzfunktion.id"))
|
||||||
|
# PERS-002: Verknüpfung zum fachlichen Person-Datensatz. Bestehende Benutzer
|
||||||
|
# wurden per Migration 0034 rückwirkend mit einer Person verknüpft, bleibt
|
||||||
|
# aber nullable - ein Login ohne fachliche Person(Übergang) wäre sonst nicht
|
||||||
|
# anlegbar, wenn die Migration künftig übersprungen wird.
|
||||||
|
person_id: Mapped[int | None] = mapped_column(ForeignKey("person.id"))
|
||||||
|
|
||||||
rollen: Mapped[list["BenutzerRolle"]] = relationship(back_populates="benutzer", lazy="selectin")
|
rollen: Mapped[list["BenutzerRolle"]] = relationship(back_populates="benutzer", lazy="selectin")
|
||||||
|
|
||||||
|
|||||||
@@ -20,6 +20,19 @@ qualifikationskategorie_pg = PgEnum(
|
|||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
class Person(Base):
|
||||||
|
"""PERS-001: fachliches Konzept einer Person, getrennt vom technischen Login
|
||||||
|
(Benutzer). Kann ohne Benutzerkonto existieren (z.B. Jugendgruppe, externe
|
||||||
|
Helfer ohne Systemzugang) - Benutzer.person_id verweist optional hierher."""
|
||||||
|
|
||||||
|
__tablename__ = "person"
|
||||||
|
|
||||||
|
id: Mapped[int] = mapped_column(primary_key=True)
|
||||||
|
name: Mapped[str] = mapped_column(String, nullable=False)
|
||||||
|
geburtsdatum: Mapped[date | None] = mapped_column(Date)
|
||||||
|
kontakt: Mapped[str | None] = mapped_column(String)
|
||||||
|
|
||||||
|
|
||||||
class Einheit(Base):
|
class Einheit(Base):
|
||||||
"""Roadmap-Erweiterung (Personal-Modul): Organisationseinheit (Zug/Gruppe),
|
"""Roadmap-Erweiterung (Personal-Modul): Organisationseinheit (Zug/Gruppe),
|
||||||
self-referenzierend für Hierarchie (analog Kategorie.ueberkategorie_id).
|
self-referenzierend für Hierarchie (analog Kategorie.ueberkategorie_id).
|
||||||
|
|||||||
@@ -11,6 +11,10 @@ class BenutzerCreate(BaseModel):
|
|||||||
rollen: list[RolleTyp] = []
|
rollen: list[RolleTyp] = []
|
||||||
einheit_id: int | None = None
|
einheit_id: int | None = None
|
||||||
einsatzfunktion_id: int | None = None
|
einsatzfunktion_id: int | None = None
|
||||||
|
# PERS-002: bestehende Person verknüpfen statt automatisch eine neue anzulegen
|
||||||
|
# (z.B. wenn eine bereits erfasste Person - Jugendgruppe o.ä. - nachträglich
|
||||||
|
# einen Login bekommt).
|
||||||
|
person_id: int | None = None
|
||||||
|
|
||||||
|
|
||||||
class BenutzerRead(BaseModel):
|
class BenutzerRead(BaseModel):
|
||||||
@@ -23,6 +27,7 @@ class BenutzerRead(BaseModel):
|
|||||||
rollen: list[str]
|
rollen: list[str]
|
||||||
einheit_id: int | None
|
einheit_id: int | None
|
||||||
einsatzfunktion_id: int | None
|
einsatzfunktion_id: int | None
|
||||||
|
person_id: int | None
|
||||||
|
|
||||||
@classmethod
|
@classmethod
|
||||||
def from_orm_benutzer(cls, benutzer) -> "BenutzerRead":
|
def from_orm_benutzer(cls, benutzer) -> "BenutzerRead":
|
||||||
@@ -35,6 +40,7 @@ class BenutzerRead(BaseModel):
|
|||||||
rollen=benutzer.rollen_namen,
|
rollen=benutzer.rollen_namen,
|
||||||
einheit_id=benutzer.einheit_id,
|
einheit_id=benutzer.einheit_id,
|
||||||
einsatzfunktion_id=benutzer.einsatzfunktion_id,
|
einsatzfunktion_id=benutzer.einsatzfunktion_id,
|
||||||
|
person_id=benutzer.person_id,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -6,6 +6,26 @@ from pydantic import BaseModel, ConfigDict
|
|||||||
from app.models.personal import Qualifikationskategorie
|
from app.models.personal import Qualifikationskategorie
|
||||||
|
|
||||||
|
|
||||||
|
class PersonCreate(BaseModel):
|
||||||
|
name: str
|
||||||
|
geburtsdatum: date | None = None
|
||||||
|
kontakt: str | None = None
|
||||||
|
|
||||||
|
|
||||||
|
class PersonRead(BaseModel):
|
||||||
|
model_config = ConfigDict(from_attributes=True)
|
||||||
|
id: int
|
||||||
|
name: str
|
||||||
|
geburtsdatum: date | None
|
||||||
|
kontakt: str | None
|
||||||
|
|
||||||
|
|
||||||
|
class PersonUpdate(BaseModel):
|
||||||
|
name: str | None = None
|
||||||
|
geburtsdatum: date | None = None
|
||||||
|
kontakt: str | None = None
|
||||||
|
|
||||||
|
|
||||||
class EinheitCreate(BaseModel):
|
class EinheitCreate(BaseModel):
|
||||||
name: str
|
name: str
|
||||||
uebergeordnete_einheit_id: int | None = None
|
uebergeordnete_einheit_id: int | None = None
|
||||||
|
|||||||
@@ -7,10 +7,23 @@ from app.models.objekt import Objekt
|
|||||||
from app.models.personal import (
|
from app.models.personal import (
|
||||||
BenutzerQualifikation,
|
BenutzerQualifikation,
|
||||||
ObjekttypQualifikationsanforderung,
|
ObjekttypQualifikationsanforderung,
|
||||||
|
Person,
|
||||||
Qualifikationstyp,
|
Qualifikationstyp,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
async def erstelle_person(db: AsyncSession, *, name: str, geburtsdatum, kontakt: str | None) -> Person:
|
||||||
|
person = Person(name=name, geburtsdatum=geburtsdatum, kontakt=kontakt)
|
||||||
|
db.add(person)
|
||||||
|
await db.flush()
|
||||||
|
return person
|
||||||
|
|
||||||
|
|
||||||
|
async def liste_personen(db: AsyncSession) -> list[Person]:
|
||||||
|
result = await db.execute(select(Person))
|
||||||
|
return list(result.scalars().all())
|
||||||
|
|
||||||
|
|
||||||
async def pruefe_berechtigung(db: AsyncSession, *, benutzer_id: int, objekt_id: int) -> list[Qualifikationstyp]:
|
async def pruefe_berechtigung(db: AsyncSession, *, benutzer_id: int, objekt_id: int) -> list[Qualifikationstyp]:
|
||||||
"""Nutzer-Beispiel Personal-Modul: "wer darf dieses Fahrzeug fahren?".
|
"""Nutzer-Beispiel Personal-Modul: "wer darf dieses Fahrzeug fahren?".
|
||||||
Liefert die Qualifikationstypen, die der Objekttyp verlangt, aber der
|
Liefert die Qualifikationstypen, die der Objekttyp verlangt, aber der
|
||||||
|
|||||||
@@ -243,6 +243,95 @@ async def test_unbefristete_qualifikation_erscheint_nicht_im_dashboard(client, a
|
|||||||
assert response.json() == []
|
assert response.json() == []
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.mark.asyncio
|
||||||
|
async def test_person_ohne_benutzer_anlegbar(client, admin_user):
|
||||||
|
"""PERS-001: Person kann ohne jeden Systemzugang existieren (z.B. Jugendgruppe)."""
|
||||||
|
token = await login(client, "admin1")
|
||||||
|
person = await client.post(
|
||||||
|
"/api/v1/personen",
|
||||||
|
json={"name": "Max Mustermann (Jugendgruppe)", "geburtsdatum": "2012-04-01"},
|
||||||
|
headers=auth_header(token),
|
||||||
|
)
|
||||||
|
assert person.status_code == 201
|
||||||
|
body = person.json()
|
||||||
|
assert body["name"] == "Max Mustermann (Jugendgruppe)"
|
||||||
|
assert body["geburtsdatum"] == "2012-04-01"
|
||||||
|
|
||||||
|
liste = await client.get("/api/v1/personen", headers=auth_header(token))
|
||||||
|
assert any(p["id"] == body["id"] for p in liste.json())
|
||||||
|
|
||||||
|
geaendert = await client.patch(
|
||||||
|
f"/api/v1/personen/{body['id']}", json={"kontakt": "Erziehungsberechtigter: 0170..."}, headers=auth_header(token)
|
||||||
|
)
|
||||||
|
assert geaendert.json()["kontakt"] == "Erziehungsberechtigter: 0170..."
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.mark.asyncio
|
||||||
|
async def test_mitarbeiter_darf_person_nicht_anlegen(client, mitarbeiter_user):
|
||||||
|
token = await login(client, "mitarbeiter1")
|
||||||
|
response = await client.post("/api/v1/personen", json={"name": "Jemand"}, headers=auth_header(token))
|
||||||
|
assert response.status_code == 403
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.mark.asyncio
|
||||||
|
async def test_benutzer_bekommt_automatisch_verknuepfte_person(client, admin_user):
|
||||||
|
"""PERS-002: bestehender Flow (Benutzer ohne explizite person_id anlegen)
|
||||||
|
bleibt unverändert - Person wird automatisch mitangelegt."""
|
||||||
|
token = await login(client, "admin1")
|
||||||
|
benutzer = await client.post(
|
||||||
|
"/api/v1/benutzer",
|
||||||
|
json={"name": "Auto-Person Helfer", "login": "autoperson1", "passwort": "test-passwort-123"},
|
||||||
|
headers=auth_header(token),
|
||||||
|
)
|
||||||
|
assert benutzer.status_code == 201
|
||||||
|
person_id = benutzer.json()["person_id"]
|
||||||
|
assert person_id is not None
|
||||||
|
|
||||||
|
personen = await client.get("/api/v1/personen", headers=auth_header(token))
|
||||||
|
assert any(p["id"] == person_id and p["name"] == "Auto-Person Helfer" for p in personen.json())
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.mark.asyncio
|
||||||
|
async def test_benutzer_mit_bestehender_person_verknuepfen(client, admin_user):
|
||||||
|
"""PERS-002: bereits erfasste Person (z.B. aus der Jugendgruppe) bekommt
|
||||||
|
nachträglich einen Login, ohne dass eine zweite Person entsteht."""
|
||||||
|
token = await login(client, "admin1")
|
||||||
|
person = await client.post("/api/v1/personen", json={"name": "Bestehende Person"}, headers=auth_header(token))
|
||||||
|
person_id = person.json()["id"]
|
||||||
|
|
||||||
|
benutzer = await client.post(
|
||||||
|
"/api/v1/benutzer",
|
||||||
|
json={
|
||||||
|
"name": "Bestehende Person",
|
||||||
|
"login": "bestehendeperson1",
|
||||||
|
"passwort": "test-passwort-123",
|
||||||
|
"person_id": person_id,
|
||||||
|
},
|
||||||
|
headers=auth_header(token),
|
||||||
|
)
|
||||||
|
assert benutzer.status_code == 201
|
||||||
|
assert benutzer.json()["person_id"] == person_id
|
||||||
|
|
||||||
|
personen = await client.get("/api/v1/personen", headers=auth_header(token))
|
||||||
|
assert len([p for p in personen.json() if p["id"] == person_id]) == 1
|
||||||
|
|
||||||
|
|
||||||
|
@pytest.mark.asyncio
|
||||||
|
async def test_benutzer_mit_unbekannter_person_id_abgelehnt(client, admin_user):
|
||||||
|
token = await login(client, "admin1")
|
||||||
|
response = await client.post(
|
||||||
|
"/api/v1/benutzer",
|
||||||
|
json={
|
||||||
|
"name": "X",
|
||||||
|
"login": "unbekanntperson1",
|
||||||
|
"passwort": "test-passwort-123",
|
||||||
|
"person_id": 999999,
|
||||||
|
},
|
||||||
|
headers=auth_header(token),
|
||||||
|
)
|
||||||
|
assert response.status_code == 404
|
||||||
|
|
||||||
|
|
||||||
@pytest.mark.asyncio
|
@pytest.mark.asyncio
|
||||||
async def test_qualifikation_erfassen_steht_in_historie(client, admin_user, mitarbeiter_user):
|
async def test_qualifikation_erfassen_steht_in_historie(client, admin_user, mitarbeiter_user):
|
||||||
from datetime import date
|
from datetime import date
|
||||||
|
|||||||
@@ -803,3 +803,27 @@ Keine Commits in dieser Session.
|
|||||||
- frontend/src/pages/admin/ObjekttypSection.tsx | 170 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------
|
- frontend/src/pages/admin/ObjekttypSection.tsx | 170 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------
|
||||||
|
|
||||||
---
|
---
|
||||||
|
## 2026-09-08 22:25 – 22:30 (5m)
|
||||||
|
**Beschreibung:** Claude Code Session
|
||||||
|
**Projekt:** asb-material
|
||||||
|
|
||||||
|
### Commits
|
||||||
|
Keine Commits in dieser Session.
|
||||||
|
|
||||||
|
### Geänderte Dateien
|
||||||
|
- .claude/agents/DEVLOG.md | 11 +
|
||||||
|
- DEVLOG.md | 4565 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
|
||||||
|
- arbeitskacheln/00_index.md | 11 +-
|
||||||
|
- arbeitskacheln/22_ui_redesign_v2.md | 107 +++
|
||||||
|
- backend/DEVLOG.md | 50 +
|
||||||
|
- backend/alembic/versions/0033_wartungsauftrag_teil.py | 33 +
|
||||||
|
- backend/app/api/v1/endpoints/wartung.py | 49 +-
|
||||||
|
- backend/app/models/wartung.py | 10 +
|
||||||
|
- backend/app/schemas/wartung.py | 15 +
|
||||||
|
- backend/app/services/wartung.py | 22 +
|
||||||
|
- backend/tests/test_wartung.py | 70 ++
|
||||||
|
- frontend/DEVLOG.md | 176 ++++
|
||||||
|
- frontend/src/api/types.ts | 8 +
|
||||||
|
- frontend/src/components/WartungSection.tsx | 111 ++-
|
||||||
|
|
||||||
|
---
|
||||||
|
|||||||
@@ -408,6 +408,14 @@ export interface Benutzer {
|
|||||||
rollen: string[];
|
rollen: string[];
|
||||||
einheit_id: number | null;
|
einheit_id: number | null;
|
||||||
einsatzfunktion_id: number | null;
|
einsatzfunktion_id: number | null;
|
||||||
|
person_id: number | null;
|
||||||
|
}
|
||||||
|
|
||||||
|
export interface Person {
|
||||||
|
id: number;
|
||||||
|
name: string;
|
||||||
|
geburtsdatum: string | null;
|
||||||
|
kontakt: string | null;
|
||||||
}
|
}
|
||||||
|
|
||||||
export interface Einsatzfunktion {
|
export interface Einsatzfunktion {
|
||||||
|
|||||||
@@ -4,7 +4,7 @@ import { useSearchParams } from "react-router-dom";
|
|||||||
import { apiRequest } from "../api/client";
|
import { apiRequest } from "../api/client";
|
||||||
import { AusgabeSection } from "./admin/AusgabeSection";
|
import { AusgabeSection } from "./admin/AusgabeSection";
|
||||||
import { useAuth } from "../auth/AuthContext";
|
import { useAuth } from "../auth/AuthContext";
|
||||||
import type { Benutzer, Bereich, Beladungsvorlage, Einheit, Einsatzfunktion, Fach, Kategorie, Material, Nummernschema, Objekt, Objekttyp, Qualifikationstyp, Standort } from "../api/types";
|
import type { Benutzer, Bereich, Beladungsvorlage, Einheit, Einsatzfunktion, Fach, Kategorie, Material, Nummernschema, Objekt, Objekttyp, Person, Qualifikationstyp, Standort } from "../api/types";
|
||||||
import { BenutzerSection } from "./admin/BenutzerSection";
|
import { BenutzerSection } from "./admin/BenutzerSection";
|
||||||
import { BereichSection } from "./admin/BereichSection";
|
import { BereichSection } from "./admin/BereichSection";
|
||||||
import { EinheitSection } from "./admin/EinheitSection";
|
import { EinheitSection } from "./admin/EinheitSection";
|
||||||
@@ -19,6 +19,7 @@ import { KontrollverantwortungSection } from "./admin/KontrollverantwortungSecti
|
|||||||
import { MaterialSection } from "./admin/MaterialSection";
|
import { MaterialSection } from "./admin/MaterialSection";
|
||||||
import { ObjektSection } from "./admin/ObjektSection";
|
import { ObjektSection } from "./admin/ObjektSection";
|
||||||
import { ObjekttypSection } from "./admin/ObjekttypSection";
|
import { ObjekttypSection } from "./admin/ObjekttypSection";
|
||||||
|
import { PersonSection } from "./admin/PersonSection";
|
||||||
import { QualifikationSection } from "./admin/QualifikationSection";
|
import { QualifikationSection } from "./admin/QualifikationSection";
|
||||||
import { StandortSection } from "./admin/StandortSection";
|
import { StandortSection } from "./admin/StandortSection";
|
||||||
import { VorlageSection } from "./admin/VorlageSection";
|
import { VorlageSection } from "./admin/VorlageSection";
|
||||||
@@ -110,6 +111,7 @@ export function AdminPage() {
|
|||||||
const [einheiten, setEinheiten] = useState<Einheit[]>([]);
|
const [einheiten, setEinheiten] = useState<Einheit[]>([]);
|
||||||
const [einsatzfunktionen, setEinsatzfunktionen] = useState<Einsatzfunktion[]>([]);
|
const [einsatzfunktionen, setEinsatzfunktionen] = useState<Einsatzfunktion[]>([]);
|
||||||
const [nummernschemata, setNummernschemata] = useState<Nummernschema[]>([]);
|
const [nummernschemata, setNummernschemata] = useState<Nummernschema[]>([]);
|
||||||
|
const [personen, setPersonen] = useState<Person[]>([]);
|
||||||
const [qualifikationstypen, setQualifikationstypen] = useState<Qualifikationstyp[]>([]);
|
const [qualifikationstypen, setQualifikationstypen] = useState<Qualifikationstyp[]>([]);
|
||||||
const [fehler, setFehler] = useState<string | null>(null);
|
const [fehler, setFehler] = useState<string | null>(null);
|
||||||
const [laedt, setLaedt] = useState(true);
|
const [laedt, setLaedt] = useState(true);
|
||||||
@@ -129,7 +131,7 @@ export function AdminPage() {
|
|||||||
// Seite als "nicht geladen" zeigen, obwohl deren Tabs (z.B. Kontroll-
|
// Seite als "nicht geladen" zeigen, obwohl deren Tabs (z.B. Kontroll-
|
||||||
// verantwortung) trotzdem funktionieren sollen. allSettled + leerer
|
// verantwortung) trotzdem funktionieren sollen. allSettled + leerer
|
||||||
// Fallback statt hartem Fehler.
|
// Fallback statt hartem Fehler.
|
||||||
const [be, ka, ot, s, v, o, m, b, f, ei, qt, ef, ns] = await Promise.allSettled([
|
const [be, ka, ot, s, v, o, m, b, f, ei, qt, ef, ns, pe] = await Promise.allSettled([
|
||||||
apiRequest<Bereich[]>("/bereiche"),
|
apiRequest<Bereich[]>("/bereiche"),
|
||||||
apiRequest<Kategorie[]>("/kategorien"),
|
apiRequest<Kategorie[]>("/kategorien"),
|
||||||
apiRequest<Objekttyp[]>("/objekttypen"),
|
apiRequest<Objekttyp[]>("/objekttypen"),
|
||||||
@@ -143,6 +145,7 @@ export function AdminPage() {
|
|||||||
apiRequest<Qualifikationstyp[]>("/qualifikationstypen"),
|
apiRequest<Qualifikationstyp[]>("/qualifikationstypen"),
|
||||||
apiRequest<Einsatzfunktion[]>("/einsatzfunktionen"),
|
apiRequest<Einsatzfunktion[]>("/einsatzfunktionen"),
|
||||||
apiRequest<Nummernschema[]>("/nummernschemata"),
|
apiRequest<Nummernschema[]>("/nummernschemata"),
|
||||||
|
apiRequest<Person[]>("/personen"),
|
||||||
]);
|
]);
|
||||||
const wert = <T,>(r: PromiseSettledResult<T[]>): T[] => (r.status === "fulfilled" ? r.value : []);
|
const wert = <T,>(r: PromiseSettledResult<T[]>): T[] => (r.status === "fulfilled" ? r.value : []);
|
||||||
setBereiche(wert(be));
|
setBereiche(wert(be));
|
||||||
@@ -158,6 +161,7 @@ export function AdminPage() {
|
|||||||
setQualifikationstypen(wert(qt));
|
setQualifikationstypen(wert(qt));
|
||||||
setEinsatzfunktionen(wert(ef));
|
setEinsatzfunktionen(wert(ef));
|
||||||
setNummernschemata(wert(ns));
|
setNummernschemata(wert(ns));
|
||||||
|
setPersonen(wert(pe));
|
||||||
}
|
}
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
@@ -249,6 +253,7 @@ export function AdminPage() {
|
|||||||
benutzer={benutzer}
|
benutzer={benutzer}
|
||||||
einheiten={einheiten}
|
einheiten={einheiten}
|
||||||
einsatzfunktionen={einsatzfunktionen}
|
einsatzfunktionen={einsatzfunktionen}
|
||||||
|
personen={personen}
|
||||||
onGeaendert={ladeAlles}
|
onGeaendert={ladeAlles}
|
||||||
onFehler={setFehler}
|
onFehler={setFehler}
|
||||||
/>
|
/>
|
||||||
@@ -274,6 +279,7 @@ export function AdminPage() {
|
|||||||
{aktiverTab === "eskalation" && <EskalationSection onFehler={setFehler} />}
|
{aktiverTab === "eskalation" && <EskalationSection onFehler={setFehler} />}
|
||||||
{aktiverTab === "personal" && (
|
{aktiverTab === "personal" && (
|
||||||
<>
|
<>
|
||||||
|
<PersonSection personen={personen} onGeaendert={ladeAlles} onFehler={setFehler} />
|
||||||
<EinheitSection einheiten={einheiten} standorte={standorte} onGeaendert={ladeAlles} onFehler={setFehler} />
|
<EinheitSection einheiten={einheiten} standorte={standorte} onGeaendert={ladeAlles} onFehler={setFehler} />
|
||||||
<QualifikationSection
|
<QualifikationSection
|
||||||
qualifikationstypen={qualifikationstypen}
|
qualifikationstypen={qualifikationstypen}
|
||||||
|
|||||||
@@ -1,12 +1,13 @@
|
|||||||
import { useState } from "react";
|
import { useState } from "react";
|
||||||
|
|
||||||
import { apiRequest } from "../../api/client";
|
import { apiRequest } from "../../api/client";
|
||||||
import type { Benutzer, Einheit, Einsatzfunktion, Rolle } from "../../api/types";
|
import type { Benutzer, Einheit, Einsatzfunktion, Person, Rolle } from "../../api/types";
|
||||||
|
|
||||||
interface Props {
|
interface Props {
|
||||||
benutzer: Benutzer[];
|
benutzer: Benutzer[];
|
||||||
einheiten: Einheit[];
|
einheiten: Einheit[];
|
||||||
einsatzfunktionen: Einsatzfunktion[];
|
einsatzfunktionen: Einsatzfunktion[];
|
||||||
|
personen: Person[];
|
||||||
onGeaendert: () => Promise<void>;
|
onGeaendert: () => Promise<void>;
|
||||||
onFehler: (text: string) => void;
|
onFehler: (text: string) => void;
|
||||||
}
|
}
|
||||||
@@ -18,13 +19,14 @@ const ALLE_ROLLEN: Rolle[] = [
|
|||||||
"administration",
|
"administration",
|
||||||
];
|
];
|
||||||
|
|
||||||
export function BenutzerSection({ benutzer, einheiten, einsatzfunktionen, onGeaendert, onFehler }: Props) {
|
export function BenutzerSection({ benutzer, einheiten, einsatzfunktionen, personen, onGeaendert, onFehler }: Props) {
|
||||||
const [name, setName] = useState("");
|
const [name, setName] = useState("");
|
||||||
const [login, setLogin] = useState("");
|
const [login, setLogin] = useState("");
|
||||||
const [passwort, setPasswort] = useState("");
|
const [passwort, setPasswort] = useState("");
|
||||||
const [email, setEmail] = useState("");
|
const [email, setEmail] = useState("");
|
||||||
const [einheitId, setEinheitId] = useState<number | "">("");
|
const [einheitId, setEinheitId] = useState<number | "">("");
|
||||||
const [einsatzfunktionId, setEinsatzfunktionId] = useState<number | "">("");
|
const [einsatzfunktionId, setEinsatzfunktionId] = useState<number | "">("");
|
||||||
|
const [personId, setPersonId] = useState<number | "">("");
|
||||||
const [neueEinsatzfunktion, setNeueEinsatzfunktion] = useState("");
|
const [neueEinsatzfunktion, setNeueEinsatzfunktion] = useState("");
|
||||||
const [rollen, setRollen] = useState<Rolle[]>(["mitarbeiter"]);
|
const [rollen, setRollen] = useState<Rolle[]>(["mitarbeiter"]);
|
||||||
const [wirdAngelegt, setWirdAngelegt] = useState(false);
|
const [wirdAngelegt, setWirdAngelegt] = useState(false);
|
||||||
@@ -65,6 +67,7 @@ export function BenutzerSection({ benutzer, einheiten, einsatzfunktionen, onGeae
|
|||||||
email: email || null,
|
email: email || null,
|
||||||
einheit_id: einheitId === "" ? null : einheitId,
|
einheit_id: einheitId === "" ? null : einheitId,
|
||||||
einsatzfunktion_id: einsatzfunktionId === "" ? null : einsatzfunktionId,
|
einsatzfunktion_id: einsatzfunktionId === "" ? null : einsatzfunktionId,
|
||||||
|
person_id: personId === "" ? null : personId,
|
||||||
rollen,
|
rollen,
|
||||||
},
|
},
|
||||||
});
|
});
|
||||||
@@ -74,6 +77,7 @@ export function BenutzerSection({ benutzer, einheiten, einsatzfunktionen, onGeae
|
|||||||
setEmail("");
|
setEmail("");
|
||||||
setEinheitId("");
|
setEinheitId("");
|
||||||
setEinsatzfunktionId("");
|
setEinsatzfunktionId("");
|
||||||
|
setPersonId("");
|
||||||
setRollen(["mitarbeiter"]);
|
setRollen(["mitarbeiter"]);
|
||||||
await onGeaendert();
|
await onGeaendert();
|
||||||
} catch {
|
} catch {
|
||||||
@@ -177,6 +181,14 @@ export function BenutzerSection({ benutzer, einheiten, einsatzfunktionen, onGeae
|
|||||||
</option>
|
</option>
|
||||||
))}
|
))}
|
||||||
</select>
|
</select>
|
||||||
|
<select className="input" value={personId} onChange={(e) => setPersonId(Number(e.target.value) || "")}>
|
||||||
|
<option value="">Neue Person anlegen (Standard)</option>
|
||||||
|
{personen.map((p) => (
|
||||||
|
<option key={p.id} value={p.id}>
|
||||||
|
Bestehende Person: {p.name}
|
||||||
|
</option>
|
||||||
|
))}
|
||||||
|
</select>
|
||||||
<input
|
<input
|
||||||
className="input"
|
className="input"
|
||||||
value={neueEinsatzfunktion}
|
value={neueEinsatzfunktion}
|
||||||
|
|||||||
@@ -0,0 +1,143 @@
|
|||||||
|
import { useState } from "react";
|
||||||
|
|
||||||
|
import { apiRequest } from "../../api/client";
|
||||||
|
import type { Person } from "../../api/types";
|
||||||
|
|
||||||
|
interface Props {
|
||||||
|
personen: Person[];
|
||||||
|
onGeaendert: () => Promise<void>;
|
||||||
|
onFehler: (text: string) => void;
|
||||||
|
}
|
||||||
|
|
||||||
|
export function PersonSection({ personen, onGeaendert, onFehler }: Props) {
|
||||||
|
const [name, setName] = useState("");
|
||||||
|
const [geburtsdatum, setGeburtsdatum] = useState("");
|
||||||
|
const [kontakt, setKontakt] = useState("");
|
||||||
|
const [wirdAngelegt, setWirdAngelegt] = useState(false);
|
||||||
|
const [bearbeiteteId, setBearbeiteteId] = useState<number | null>(null);
|
||||||
|
const [bearbeitungsName, setBearbeitungsName] = useState("");
|
||||||
|
const [bearbeitungsGeburtsdatum, setBearbeitungsGeburtsdatum] = useState("");
|
||||||
|
const [bearbeitungsKontakt, setBearbeitungsKontakt] = useState("");
|
||||||
|
|
||||||
|
async function anlegen() {
|
||||||
|
setWirdAngelegt(true);
|
||||||
|
try {
|
||||||
|
await apiRequest("/personen", {
|
||||||
|
method: "POST",
|
||||||
|
body: { name, geburtsdatum: geburtsdatum || null, kontakt: kontakt || null },
|
||||||
|
});
|
||||||
|
setName("");
|
||||||
|
setGeburtsdatum("");
|
||||||
|
setKontakt("");
|
||||||
|
await onGeaendert();
|
||||||
|
} catch {
|
||||||
|
onFehler("Person konnte nicht angelegt werden.");
|
||||||
|
} finally {
|
||||||
|
setWirdAngelegt(false);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
function bearbeiten(p: Person) {
|
||||||
|
setBearbeiteteId(p.id);
|
||||||
|
setBearbeitungsName(p.name);
|
||||||
|
setBearbeitungsGeburtsdatum(p.geburtsdatum ?? "");
|
||||||
|
setBearbeitungsKontakt(p.kontakt ?? "");
|
||||||
|
}
|
||||||
|
|
||||||
|
async function speichern(id: number) {
|
||||||
|
try {
|
||||||
|
await apiRequest(`/personen/${id}`, {
|
||||||
|
method: "PATCH",
|
||||||
|
body: {
|
||||||
|
name: bearbeitungsName,
|
||||||
|
geburtsdatum: bearbeitungsGeburtsdatum || null,
|
||||||
|
kontakt: bearbeitungsKontakt || null,
|
||||||
|
},
|
||||||
|
});
|
||||||
|
setBearbeiteteId(null);
|
||||||
|
await onGeaendert();
|
||||||
|
} catch {
|
||||||
|
onFehler("Person konnte nicht geändert werden.");
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return (
|
||||||
|
<section className="section">
|
||||||
|
<h2>Personen</h2>
|
||||||
|
<p className="text-muted" style={{ marginTop: 0 }}>
|
||||||
|
Fachliche Personen unabhängig vom Systemzugang – z. B. Jugendgruppe oder externe
|
||||||
|
Helfer ohne Login. Ein Login (Tab „Benutzer") kann optional mit einer bestehenden
|
||||||
|
Person verknüpft werden.
|
||||||
|
</p>
|
||||||
|
<div className="card">
|
||||||
|
<div className="row" style={{ flexWrap: "wrap", gap: "0.5rem", marginBottom: "0.75rem" }}>
|
||||||
|
<input className="input" value={name} onChange={(e) => setName(e.target.value)} placeholder="Name" />
|
||||||
|
<input
|
||||||
|
className="input"
|
||||||
|
type="date"
|
||||||
|
value={geburtsdatum}
|
||||||
|
onChange={(e) => setGeburtsdatum(e.target.value)}
|
||||||
|
/>
|
||||||
|
<input
|
||||||
|
className="input"
|
||||||
|
value={kontakt}
|
||||||
|
onChange={(e) => setKontakt(e.target.value)}
|
||||||
|
placeholder="Kontakt (optional)"
|
||||||
|
/>
|
||||||
|
<button className="btn btn-primary" onClick={anlegen} disabled={!name || wirdAngelegt}>
|
||||||
|
Anlegen
|
||||||
|
</button>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{personen.length === 0 ? (
|
||||||
|
<p className="text-muted">Keine Personen erfasst.</p>
|
||||||
|
) : (
|
||||||
|
<ul className="card-list">
|
||||||
|
{personen.map((p) => (
|
||||||
|
<li key={p.id} style={{ padding: "0.4rem 0" }}>
|
||||||
|
{bearbeiteteId === p.id ? (
|
||||||
|
<div className="row" style={{ flexWrap: "wrap", gap: "0.5rem" }}>
|
||||||
|
<input
|
||||||
|
className="input"
|
||||||
|
value={bearbeitungsName}
|
||||||
|
onChange={(e) => setBearbeitungsName(e.target.value)}
|
||||||
|
/>
|
||||||
|
<input
|
||||||
|
className="input"
|
||||||
|
type="date"
|
||||||
|
value={bearbeitungsGeburtsdatum}
|
||||||
|
onChange={(e) => setBearbeitungsGeburtsdatum(e.target.value)}
|
||||||
|
/>
|
||||||
|
<input
|
||||||
|
className="input"
|
||||||
|
value={bearbeitungsKontakt}
|
||||||
|
onChange={(e) => setBearbeitungsKontakt(e.target.value)}
|
||||||
|
placeholder="Kontakt"
|
||||||
|
/>
|
||||||
|
<button className="btn btn-primary" onClick={() => speichern(p.id)}>
|
||||||
|
Speichern
|
||||||
|
</button>
|
||||||
|
<button className="btn btn-secondary" onClick={() => setBearbeiteteId(null)}>
|
||||||
|
Abbrechen
|
||||||
|
</button>
|
||||||
|
</div>
|
||||||
|
) : (
|
||||||
|
<div className="row-between">
|
||||||
|
<span>
|
||||||
|
{p.name}
|
||||||
|
{p.geburtsdatum && <span className="text-muted"> · geb. {p.geburtsdatum}</span>}
|
||||||
|
{p.kontakt && <span className="text-muted"> · {p.kontakt}</span>}
|
||||||
|
</span>
|
||||||
|
<button className="btn btn-secondary" onClick={() => bearbeiten(p)}>
|
||||||
|
Bearbeiten
|
||||||
|
</button>
|
||||||
|
</div>
|
||||||
|
)}
|
||||||
|
</li>
|
||||||
|
))}
|
||||||
|
</ul>
|
||||||
|
)}
|
||||||
|
</div>
|
||||||
|
</section>
|
||||||
|
);
|
||||||
|
}
|
||||||
Reference in New Issue
Block a user