feat: Ein Uebertrag faengt seine Gegenseite ab
Ein Uebertrag legte auf dem Zielkonto nichts an. Der Duplikatschutz konnte die Gegenseite deshalb nicht sehen – er vergleicht import_hash, den eine Uebertrags-Buchung nicht hat, und Eingangs-Eintraege, die es dort nicht gab. Die schon vorliegende Gegenseite liess sich ein zweites Mal buchen, die spaeter von der Bank gelieferte kam ungeprueft an: das Geld zweimal auf dem Zielkonto. Jetzt entsteht auf dem Zielkonto ein Eingangs-Eintrag, der auf DIESELBE Buchung zeigt. Liegt dort schon ein passender offener Eintrag, wird dieser verbraucht statt einen zweiten anzulegen – ueber denselben Abgleich wie beim Import, nicht ueber eine zweite Vergleichslogik. Damit greift auch der vorhandene Schutz gegen die spaeter eintreffende Gegenseite, ohne dass etwas Neues dafuer gebaut werden musste. Die Uebertrags-Logik liegt dafuer in einem eigenen Dienst. Zwei Wege fuehren hinein – ein offener Vorschlag und eine schon gebuchte Buchung – und beide brauchen dieselben vier Regeln. Der PATCH-Weg des Buchungs-Routers taete es nicht: Er dreht die Richtung bei einer Einnahme nicht um und legt keine Gegenseite an. Die Sperre richtet sich nicht mehr nach der Waehrung, sondern danach, ob der Betrag der Gegenseite BEKANNT oder geschaetzt ist. Bei gleicher Waehrung ist er exakt, die Gegenseite wird sicher wiedererkannt – dann geht ein Uebertrag auf ein angebundenes Konto ohne Nachfrage durch. Ueber Waehrungen hinweg bleibt die Warnung, weil der EZB-Kurs nicht der Kurs der Bank ist. Und Verwerfen raeumt beide Seiten: Bei einem Uebertrag zeigen zwei Eintraege auf dieselbe Buchung, und der Fremdschluessel haette die andere als 'gebucht' ohne Buchung zurueckgelassen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Guido
Claude Opus 5
@@ -0,0 +1,236 @@
|
||||
"""Etwas zum Kontouebertrag machen – aus dem Eingang oder nachtraeglich.
|
||||
|
||||
Zwei Wege fuehren hierher: ein offener Vorschlag im Eingang und eine schon
|
||||
gebuchte Ausgabe oder Einnahme. Beide muessen dieselben vier Dinge einhalten,
|
||||
deshalb liegen sie hier und nicht in den Routern:
|
||||
|
||||
1. Die Richtung kommt aus dem Vorzeichen. Ein Abgang geht von hier weg, ein
|
||||
Zugang kommt vom Gegenkonto her.
|
||||
2. Der Betrag der Gegenseite ist bekannt oder geschaetzt – das entscheidet
|
||||
ueber Punkt 3.
|
||||
3. Ist er geschaetzt und haengt das Zielkonto selbst an einer Bank, wird
|
||||
nachgefragt: Die Bank meldet den Zugang von allein, und ein geschaetzter
|
||||
Betrag laesst sich beim Abgleich nicht wiedererkennen.
|
||||
4. Auf dem Zielkonto entsteht ein Eingangs-Eintrag, der auf DIESELBE Buchung
|
||||
zeigt. Er faengt die Gegenseite ab – die schon vorliegende, indem er sie
|
||||
verbraucht, und die spaeter eintreffende, weil ``detect_rows`` gegen
|
||||
gebuchte Eingangs-Eintraege prueft.
|
||||
|
||||
Ohne Punkt 4 stuende das Geld zweimal auf dem Zielkonto: Der Duplikatschutz
|
||||
vergleicht ``import_hash`` – den hat eine Uebertrags-Buchung nicht – und
|
||||
Eingangs-Eintraege, und ohne diesen legte der Uebertrag auf dem Zielkonto
|
||||
keinen an.
|
||||
|
||||
Nicht betroffen ist das Buchungsformular: Wer dort von Hand einen Uebertrag
|
||||
anlegt, waehlt beide Konten bewusst und hat keine Gegenseite, die noch kommt.
|
||||
"""
|
||||
|
||||
from __future__ import annotations
|
||||
|
||||
from sqlalchemy.orm import Session
|
||||
|
||||
from ..models import (
|
||||
Account,
|
||||
BankAccountLink,
|
||||
InboxItem,
|
||||
InboxSource,
|
||||
InboxStatus,
|
||||
Transaction,
|
||||
TxnType,
|
||||
User,
|
||||
)
|
||||
from . import buchung, csv_import, currency
|
||||
|
||||
|
||||
class UebertragFehler(RuntimeError):
|
||||
"""Fehler mit einem Text, den man dem Nutzer zeigen kann."""
|
||||
|
||||
def __init__(self, text: str, nachfragen: bool = False):
|
||||
super().__init__(text)
|
||||
# True heisst: Es ist kein Fehler, sondern eine Warnung, die sich mit
|
||||
# ``trotzdem`` uebergehen laesst.
|
||||
self.nachfragen = nachfragen
|
||||
|
||||
|
||||
def ist_angebunden(db: Session, user: User, konto: Account) -> bool:
|
||||
"""Holt Certain die Umsaetze dieses Kontos selbst von der Bank?
|
||||
|
||||
Entscheidend fuer einen Uebertrag: Bei einem angebundenen Gegenkonto
|
||||
liefert die Bank die Gegenbuchung selbst.
|
||||
"""
|
||||
return db.query(BankAccountLink.id).filter(
|
||||
BankAccountLink.owner_id == user.id,
|
||||
BankAccountLink.account_id == konto.id,
|
||||
).first() is not None
|
||||
|
||||
|
||||
def _gegenbetrag(db: Session, konto: Account, ziel: Account, betrag: int,
|
||||
datum) -> tuple[int | None, bool]:
|
||||
"""Betrag auf der Zielseite und ob er BEKANNT oder geschaetzt ist.
|
||||
|
||||
Gleiche Waehrung: derselbe Betrag, exakt. Sonst der EZB-Kurs – eine
|
||||
Naeherung, denn die Bank rechnet mit ihrem eigenen Kurs.
|
||||
"""
|
||||
if ziel.currency == konto.currency:
|
||||
return betrag, True
|
||||
umgerechnet = currency.convert(db, betrag, konto.currency, ziel.currency, datum)
|
||||
return umgerechnet, False
|
||||
|
||||
|
||||
def _pruefe(db: Session, user: User, konto: Account, ziel: Account,
|
||||
exakt: bool, trotzdem: bool) -> None:
|
||||
if ziel.id == konto.id:
|
||||
raise UebertragFehler("Ein Übertrag braucht ein anderes Zielkonto")
|
||||
if exakt or trotzdem:
|
||||
return
|
||||
if not ist_angebunden(db, user, ziel):
|
||||
return
|
||||
raise UebertragFehler(
|
||||
f"Der Betrag auf '{ziel.name}' ist eine Umrechnung über den EZB-Kurs, "
|
||||
f"nicht der Betrag, den die Bank meldet. Da das Konto selbst angebunden "
|
||||
f"ist, kommt der Zugang von dort noch einmal – und liesse sich mit einem "
|
||||
f"geschätzten Betrag nicht als derselbe erkennen. Sicherer: hier als "
|
||||
f"Ausgabe buchen und beide Seiten danach verbinden lassen.",
|
||||
nachfragen=True,
|
||||
)
|
||||
|
||||
|
||||
def _lege_an(db: Session, user: User, konto: Account, ziel: Account,
|
||||
betrag_signiert: int, ziel_betrag: int | None, datum,
|
||||
payee: str | None, beschreibung: str | None) -> Transaction:
|
||||
"""Die Uebertrags-Buchung, richtig herum.
|
||||
|
||||
Bei einem Zugang kennen wir den Betrag auf DIESEM Konto exakt – der gehoert
|
||||
dann auf die Zielseite der Buchung, und der Abgang wird umgerechnet.
|
||||
"""
|
||||
betrag = abs(betrag_signiert)
|
||||
if betrag_signiert < 0:
|
||||
von, nach = konto, ziel
|
||||
von_betrag, nach_betrag = betrag, ziel_betrag
|
||||
else:
|
||||
von, nach = ziel, konto
|
||||
von_betrag, nach_betrag = (ziel_betrag if ziel_betrag is not None else betrag), betrag
|
||||
|
||||
txn = Transaction(
|
||||
owner_id=user.id,
|
||||
type=TxnType.transfer,
|
||||
date=datum,
|
||||
account_id=von.id,
|
||||
to_account_id=nach.id,
|
||||
amount_cents=von_betrag,
|
||||
to_amount_cents=nach_betrag,
|
||||
payee=payee,
|
||||
description=beschreibung,
|
||||
)
|
||||
buchung.setze_betraege(db, txn, von, nach)
|
||||
db.add(txn)
|
||||
db.flush()
|
||||
return txn
|
||||
|
||||
|
||||
def _gegenseite(db: Session, user: User, txn: Transaction, ziel: Account,
|
||||
betrag_signiert: int, ziel_betrag: int, datum) -> InboxItem:
|
||||
"""Den Eingangs-Eintrag auf dem Zielkonto – verbrauchen oder anlegen.
|
||||
|
||||
Verbraucht wird ein noch OFFENER Eintrag: Er beschreibt dieselbe Bewegung,
|
||||
nur von der anderen Seite gesehen, und wuerde sonst ein zweites Mal gebucht.
|
||||
"""
|
||||
# Auf dem Zielkonto ist die Bewegung anders herum.
|
||||
gesucht = -1 if betrag_signiert > 0 else 1
|
||||
gesucht *= ziel_betrag
|
||||
|
||||
offen = [
|
||||
(eid, edatum, ebetrag)
|
||||
for eid, edatum, ebetrag in db.query(
|
||||
InboxItem.id, InboxItem.date, InboxItem.amount_cents
|
||||
).filter(
|
||||
InboxItem.owner_id == user.id,
|
||||
InboxItem.account_id == ziel.id,
|
||||
InboxItem.status == InboxStatus.pending,
|
||||
)
|
||||
]
|
||||
# Derselbe Abgleich wie beim Import – nicht eine zweite Vergleichslogik.
|
||||
treffer = csv_import.finde_eingangs_treffer(offen, set(), datum, gesucht)
|
||||
|
||||
eintrag = db.get(InboxItem, treffer) if treffer is not None else None
|
||||
if eintrag is None:
|
||||
eintrag = InboxItem(
|
||||
owner_id=user.id,
|
||||
source=InboxSource.transfer,
|
||||
account_id=ziel.id,
|
||||
date=datum,
|
||||
amount_cents=gesucht,
|
||||
currency=ziel.currency,
|
||||
payee=txn.payee,
|
||||
note=txn.description,
|
||||
)
|
||||
db.add(eintrag)
|
||||
eintrag.status = InboxStatus.booked
|
||||
eintrag.transaction_id = txn.id
|
||||
# Ein Uebertrag traegt keine Kategorie – auch nicht auf der Gegenseite.
|
||||
eintrag.category_id = None
|
||||
db.flush()
|
||||
return eintrag
|
||||
|
||||
|
||||
def aus_eingang(db: Session, user: User, eintrag: InboxItem, ziel: Account,
|
||||
trotzdem: bool = False) -> Transaction:
|
||||
"""Einen offenen Vorschlag als Kontouebertrag buchen."""
|
||||
if eintrag.status == InboxStatus.booked or eintrag.transaction_id is not None:
|
||||
raise UebertragFehler("Dieser Eintrag ist schon gebucht")
|
||||
if eintrag.account_id is None:
|
||||
raise UebertragFehler("Kein Konto zugeordnet")
|
||||
konto = db.get(Account, eintrag.account_id)
|
||||
if konto is None or konto.owner_id != user.id:
|
||||
raise UebertragFehler("Kein Konto zugeordnet")
|
||||
|
||||
betrag = abs(eintrag.amount_cents)
|
||||
ziel_betrag, exakt = _gegenbetrag(db, konto, ziel, betrag, eintrag.date)
|
||||
_pruefe(db, user, konto, ziel, exakt, trotzdem)
|
||||
|
||||
txn = _lege_an(db, user, konto, ziel, eintrag.amount_cents, ziel_betrag,
|
||||
eintrag.date, eintrag.payee, eintrag.note)
|
||||
eintrag.transaction_id = txn.id
|
||||
eintrag.status = InboxStatus.booked
|
||||
eintrag.category_id = None
|
||||
_gegenseite(db, user, txn, ziel, eintrag.amount_cents,
|
||||
ziel_betrag if ziel_betrag is not None else betrag, eintrag.date)
|
||||
return txn
|
||||
|
||||
|
||||
def aus_buchung(db: Session, user: User, txn: Transaction, ziel: Account,
|
||||
trotzdem: bool = False) -> Transaction:
|
||||
"""Eine schon gebuchte Ausgabe oder Einnahme zum Uebertrag machen.
|
||||
|
||||
Nicht ueber den PATCH-Weg des Buchungs-Routers: Der dreht die Richtung bei
|
||||
einer Einnahme nicht um und legt keine Gegenseite an.
|
||||
"""
|
||||
if txn.type == TxnType.transfer:
|
||||
raise UebertragFehler("Diese Buchung ist bereits ein Übertrag")
|
||||
konto = db.get(Account, txn.account_id)
|
||||
if konto is None or konto.owner_id != user.id:
|
||||
raise UebertragFehler("Buchung ohne Konto")
|
||||
|
||||
signiert = txn.amount_cents if txn.type == TxnType.income else -txn.amount_cents
|
||||
ziel_betrag, exakt = _gegenbetrag(db, konto, ziel, abs(signiert), txn.date)
|
||||
_pruefe(db, user, konto, ziel, exakt, trotzdem)
|
||||
|
||||
# Die vorhandene Buchung umschreiben statt eine zweite anzulegen: Alles, was
|
||||
# auf sie zeigt – Eingangs-Eintrag, Verknuepfungen – bleibt damit gueltig.
|
||||
if signiert < 0:
|
||||
von, nach = konto, ziel
|
||||
txn.amount_cents, txn.to_amount_cents = abs(signiert), ziel_betrag
|
||||
else:
|
||||
von, nach = ziel, konto
|
||||
txn.amount_cents = ziel_betrag if ziel_betrag is not None else abs(signiert)
|
||||
txn.to_amount_cents = abs(signiert)
|
||||
txn.type = TxnType.transfer
|
||||
txn.account_id = von.id
|
||||
txn.to_account_id = nach.id
|
||||
txn.category_id = None
|
||||
buchung.setze_betraege(db, txn, von, nach)
|
||||
db.flush()
|
||||
|
||||
_gegenseite(db, user, txn, ziel, signiert,
|
||||
ziel_betrag if ziel_betrag is not None else abs(signiert), txn.date)
|
||||
return txn
|
||||
Reference in New Issue
Block a user