fix: Konto umbenennen scheitert nicht mehr an Identifikator-UNIQUE
_set_identifiers glich die Liste per clear()+neu-Anlegen ab; SQLAlchemy fügte dabei die (unveränderten) Identifikatoren vor dem Löschen ein und verletzte so UNIQUE(account_id, kind, value_norm) -> Internal Server Error beim Speichern. Jetzt als Diff: unveränderte behalten, nur fehlende hinzufügen / überzählige entfernen. Betrifft alle Identifikator-Arten (IBAN/Nummer/Text). Regressionstest: Umbenennen mit unverändertem Identifikator + Entfernen (76 grün). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
@@ -50,18 +50,31 @@ def to_out(db: Session, acc: Account, base: str, on: date) -> AccountOut:
|
||||
|
||||
|
||||
def _set_identifiers(acc: Account, items) -> None:
|
||||
"""Ersetzt die Identifikator-Liste (dedupliziert nach Art + Normalform)."""
|
||||
acc.identifiers.clear()
|
||||
seen: set[tuple[str, str]] = set()
|
||||
"""Gleicht die Identifikator-Liste auf den Ziel-Zustand ab (dedupliziert nach
|
||||
Art + Normalform). Bewusst als Diff (unveränderte behalten, nur fehlende
|
||||
hinzufügen / überzählige entfernen): ein einfaches clear()+neu-Anlegen würde
|
||||
beim Speichern zuerst einfügen und dann löschen und so die UNIQUE-Bedingung
|
||||
(account_id, kind, value_norm) verletzen (Fehler beim Umbenennen)."""
|
||||
desired: dict[tuple[str, str], tuple] = {}
|
||||
for it in items:
|
||||
norm = normalize_identifier(it.value, it.kind)
|
||||
key = (it.kind.value, norm)
|
||||
if not norm or key in seen:
|
||||
if not norm:
|
||||
continue
|
||||
seen.add(key)
|
||||
acc.identifiers.append(
|
||||
AccountIdentifier(kind=it.kind, value=it.value.strip(), value_norm=norm)
|
||||
)
|
||||
key = (it.kind.value, norm)
|
||||
desired.setdefault(key, (it.kind, it.value.strip(), norm))
|
||||
|
||||
keep: set[tuple[str, str]] = set()
|
||||
for existing in list(acc.identifiers):
|
||||
key = (existing.kind.value, existing.value_norm)
|
||||
if key in desired:
|
||||
keep.add(key)
|
||||
existing.value = desired[key][1] # Original-Schreibweise ggf. angleichen
|
||||
else:
|
||||
acc.identifiers.remove(existing)
|
||||
|
||||
for key, (kind, value, norm) in desired.items():
|
||||
if key not in keep:
|
||||
acc.identifiers.append(AccountIdentifier(kind=kind, value=value, value_norm=norm))
|
||||
|
||||
|
||||
@router.get("", response_model=list[AccountOut])
|
||||
|
||||
Reference in New Issue
Block a user