feat: Import-Vorlage richtet sich nach dem gewaehlten Konto
Beim Kontowechsel musste man die Vorlage jedes Mal neu waehlen – und las im schlimmsten Fall den Revolut-Auszug mit der Raiffeisen-Vorlage ein (genau das steht im Verlauf). Eigene Vorlagen standen schon als template_id im Verlauf; Bank-Vorlagen sind fest im Code und hinterliessen nichts. Der Verlauf merkt sich deshalb jetzt auch deren Namen (Migration 0014). Die Oberflaeche waehlt beim Kontowechsel, womit zuletzt in dieses Konto importiert wurde. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -530,6 +530,10 @@ class ImportBatch(Base):
|
||||
template_id: Mapped[int | None] = mapped_column(
|
||||
ForeignKey("import_templates.id", ondelete="SET NULL"), nullable=True
|
||||
)
|
||||
# Name der Bank-Vorlage (Raiffeisen, N26 …). Eigene Vorlagen stehen in
|
||||
# template_id; die fest eingebauten haben keine ID und wuerden sonst
|
||||
# vergessen, sobald der Import durch ist.
|
||||
preset: Mapped[str | None] = mapped_column(String(64), nullable=True)
|
||||
filename: Mapped[str] = mapped_column(String(255), nullable=False, default="")
|
||||
row_count: Mapped[int] = mapped_column(Integer, nullable=False, default=0)
|
||||
imported_count: Mapped[int] = mapped_column(Integer, nullable=False, default=0)
|
||||
|
||||
@@ -142,7 +142,8 @@ async def preview(
|
||||
@router.post("/commit", response_model=ImportBatchOut)
|
||||
def commit(payload: ImportCommitIn, db: Session = Depends(get_db), user: User = Depends(get_current_user)):
|
||||
account = get_owned(db, Account, payload.account_id, user)
|
||||
return csv_import.commit_import(db, user, account, payload.template_id, payload.filename, payload.rows)
|
||||
return csv_import.commit_import(db, user, account, payload.template_id, payload.filename,
|
||||
payload.rows, preset=payload.preset)
|
||||
|
||||
|
||||
@router.get("/batches", response_model=list[ImportBatchOut])
|
||||
|
||||
@@ -419,6 +419,7 @@ class ImportBatchOut(BaseModel):
|
||||
id: int
|
||||
account_id: int
|
||||
template_id: int | None
|
||||
preset: str | None
|
||||
filename: str
|
||||
row_count: int
|
||||
imported_count: int
|
||||
@@ -429,6 +430,7 @@ class ImportBatchOut(BaseModel):
|
||||
class ImportCommitIn(BaseModel):
|
||||
account_id: int
|
||||
template_id: int | None = None
|
||||
preset: str | None = None
|
||||
filename: str = ""
|
||||
rows: list[dict] = Field(default_factory=list)
|
||||
|
||||
|
||||
@@ -698,11 +698,12 @@ def _apply_counterpart_amount(db: Session, user: User, account: Account, row: di
|
||||
return True
|
||||
|
||||
|
||||
def commit_import(db: Session, user: User, account: Account, template_id: int | None, filename: str, rows: list[dict]) -> ImportBatch:
|
||||
def commit_import(db: Session, user: User, account: Account, template_id: int | None, filename: str,
|
||||
rows: list[dict], preset: str | None = None) -> ImportBatch:
|
||||
accounts = {a.id: a for a in db.query(Account).filter(Account.owner_id == user.id)}
|
||||
batch = ImportBatch(
|
||||
owner_id=user.id, account_id=account.id, template_id=template_id,
|
||||
filename=(filename or "")[:255], row_count=len(rows),
|
||||
preset=(preset or None), filename=(filename or "")[:255], row_count=len(rows),
|
||||
imported_count=0, status=ImportStatus.committed,
|
||||
)
|
||||
db.add(batch)
|
||||
|
||||
@@ -0,0 +1,29 @@
|
||||
"""Import-Verlauf merkt sich die verwendete Bank-Vorlage.
|
||||
|
||||
Eigene Vorlagen stehen als ``template_id`` im Verlauf, Bank-Vorlagen wie
|
||||
Raiffeisen oder N26 stecken dagegen fest im Code und hinterliessen bisher
|
||||
nichts. Damit die Oberflaeche beim Kontowechsel die zuletzt benutzte
|
||||
Einstellung wieder vorschlagen kann, wird der Name der Bank-Vorlage
|
||||
mitgeschrieben.
|
||||
|
||||
Revision ID: 0014
|
||||
Revises: 0013
|
||||
"""
|
||||
|
||||
import sqlalchemy as sa
|
||||
from alembic import op
|
||||
|
||||
revision = "0014"
|
||||
down_revision = "0013"
|
||||
branch_labels = None
|
||||
depends_on = None
|
||||
|
||||
|
||||
def upgrade() -> None:
|
||||
with op.batch_alter_table("import_batches") as batch:
|
||||
batch.add_column(sa.Column("preset", sa.String(length=64), nullable=True))
|
||||
|
||||
|
||||
def downgrade() -> None:
|
||||
with op.batch_alter_table("import_batches") as batch:
|
||||
batch.drop_column("preset")
|
||||
@@ -605,3 +605,20 @@ def test_rohvorschau_nennt_die_waehlbaren_filterwerte(admin):
|
||||
assert produkte[0]["count"] == 3
|
||||
# Spalten mit lauter verschiedenen Werten sind keine Gruppen.
|
||||
assert "3" not in daten["filter_values"]
|
||||
|
||||
|
||||
def test_verlauf_merkt_sich_die_bank_vorlage(admin):
|
||||
"""Bank-Vorlagen haben keine ID. Ohne diesen Vermerk wüsste die Oberfläche
|
||||
beim nächsten Import in dasselbe Konto nicht mehr, welches Format passt."""
|
||||
konto = _account(admin, name="Merkkonto", currency="CHF")
|
||||
r = _preview(admin, konto["id"])
|
||||
zeilen = [z for z in r.json()["rows"] if z["valid"]][:1]
|
||||
gebucht = admin.post("/api/imports/commit", json={
|
||||
"account_id": konto["id"], "preset": "Raiffeisen (CH)",
|
||||
"filename": "k.csv", "rows": zeilen,
|
||||
})
|
||||
assert gebucht.status_code == 200, gebucht.text
|
||||
assert gebucht.json()["preset"] == "Raiffeisen (CH)"
|
||||
verlauf = admin.get("/api/imports/batches").json()
|
||||
meins = [b for b in verlauf if b["account_id"] == konto["id"]]
|
||||
assert meins[0]["preset"] == "Raiffeisen (CH)"
|
||||
|
||||
Reference in New Issue
Block a user