Finanzen: Offen zeigt auch Ueberfaellige, Rechnungsliste laedt nach Zuordnen neu

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
2026-10-01 15:12:18 +00:00
1 parent 1feeb18c08
commit 24a3cc41ef
@@ -21,11 +21,12 @@ const NOTE_MAX = 500
 * weicht ab, 502 nicht gespeichert …) stehen im Dialog, der offen bleibt. Eine bestehende Zuordnung (außer Rechnung)
 * lässt sich mit „Zuordnung lösen“ aufheben.
 *
- * invoices = offene und überfällige Rechnungen (invoicesLoading / invoicesError für den Ladezustand).
+ * invoices = offene und überfällige Rechnungen (invoicesLoading / invoicesError für den Ladezustand; invoicesLoaded =
+ * frisch und ohne Fehler geladen, dann sind Vorschläge zu nicht mehr offenen Rechnungen gesperrt).
 * onDone(buchung) nach Erfolg (Aufrufer lädt neu und schließt); onClose({ refresh }) bei Abbruch, refresh = ein
 * Fehler trat auf, der Stand kann sich geändert haben.
 */
-export default function AssignDialog({ transaction, invoices = [], invoicesLoading = false, invoicesError = '', onClose, onDone }) {
+export default function AssignDialog({ transaction, invoices = [], invoicesLoading = false, invoicesError = '', invoicesLoaded = false, onClose, onDone }) {
  const tx = transaction || {}
  const row = useMemo(() => transactionRows([tx])[0], [tx])
  const [fixed, setFixed] = useState({ list: [], loading: false, error: '' })
@@ -69,7 +70,7 @@ export default function AssignDialog({ transaction, invoices = [], invoicesLoadi
    }
  }, [outgoing, unassigned])

-  const model = useMemo(() => assignModel(tx, { invoices, fixedCosts: fixed.list }), [tx, invoices, fixed.list])
+  const model = useMemo(() => assignModel(tx, { invoices, fixedCosts: fixed.list, invoicesLoaded }), [tx, invoices, fixed.list, invoicesLoaded])
  const option = model.byKey[selected] || null

  function close() {
@@ -12,10 +12,12 @@ import {
  ageGroupCells,
  finalizedFilter,
  invoiceConfirmText,
+  invoiceFilterStatuses,
  invoiceDoneMessage,
  invoiceFailureMessage,
  invoiceRows,
  isFinalizedNotSent,
+  mergeInvoiceLists,
 } from '../../lib/financeView'

 const PAGE_SIZE = 50
@@ -31,8 +33,11 @@ const CALLS = { send: sendInvoice, resend: resendInvoice, void: voidInvoice, del
 * customerId filtert auf einen Kunden; `customer` (Appwrite-Dokument) legt zusätzlich den Kunden im Dialog fest
 * (Kundenansicht). onChanged() meldet jede Änderung (Übersicht neu laden). Schmal (Karte bis 700 px) wird jede
 * Zeile ein Raster: Nummer und Status / Kunde / Datum, Fällig / Betrag, Offen / Aktionen.
+ *
+ * „Offen“ zeigt offene und überfällige Rechnungen (zwei Abrufe, zusammengeführt, neueste zuerst). Ändert sich
+ * `refreshToken` (die Seite lädt nach Zuordnen, Bankabruf usw. alles neu), lädt auch die Liste neu.
 */
-export default function InvoiceTable({ altersgruppen = null, customerId = null, customer = null, onChanged }) {
+export default function InvoiceTable({ altersgruppen = null, customerId = null, customer = null, refreshToken = 0, onChanged }) {
  const filterCustomer = customerId || customer?.$id || null
  const [status, setStatus] = useState(DEFAULT_INVOICE_FILTER)
  const [reloadKey, setReloadKey] = useState(0)
@@ -59,17 +64,17 @@ export default function InvoiceTable({ altersgruppen = null, customerId = null,
  useEffect(() => {
    // Nur die jüngste Anfrage zählt (Filterwechsel während des Ladens)
    const mine = ++seqRef.current
-    getInvoices({ status, kunde: filterCustomer })
-      .then((data) => {
+    Promise.all(invoiceFilterStatuses(status).map((s) => getInvoices({ status: s, kunde: filterCustomer })))
+      .then((results) => {
        if (!aliveRef.current || mine !== seqRef.current) return
-        setResult({ key, list: Array.isArray(data?.rechnungen) ? data.rechnungen : [] })
+        setResult({ key, list: mergeInvoiceLists(results.map((r) => r?.rechnungen)) })
        setError(null)
      })
      .catch((err) => {
        if (!aliveRef.current || mine !== seqRef.current) return
        setError({ key, message: err?.message || 'Die Rechnungen konnten nicht geladen werden.' })
      })
-  }, [status, filterCustomer, key, reloadKey])
+  }, [status, filterCustomer, key, reloadKey, refreshToken])

  const reload = () => setReloadKey((k) => k + 1)
  const changed = () => {
@@ -1,31 +1,33 @@
 import { useEffect, useState } from 'react'
 import { getInvoices } from '../lib/financeApi'
+import { invoiceFilterStatuses, mergeInvoiceLists } from '../lib/financeView'

 const FALLBACK_ERROR = 'Die offenen Rechnungen konnten nicht geladen werden.'

 /**
 * Offene und überfällige Rechnungen für den Zuordnen-Dialog. Lädt jedes Mal frisch, wenn `active` true wird
 * (damit der Dialog nie auf einem alten Stand arbeitet), und leert die Liste, wenn es wieder false ist.
- * Scheitert Stripe, bleibt `invoices` leer und `error` nennt den Grund (Kategorien lassen sich trotzdem wählen).
+ * `loaded` ist true, sobald die Liste dieser Öffnung frisch und ohne Fehler da ist (erst dann darf der Dialog eine
+ * Rechnung als „nicht mehr offen“ erkennen). Scheitert Stripe, bleibt `invoices` leer und `error` nennt den Grund
+ * (Kategorien lassen sich trotzdem wählen).
 */
 export function useOpenInvoices(active) {
-  const [state, setState] = useState({ invoices: [], loading: false, error: '' })
+  const [state, setState] = useState({ invoices: [], loading: false, error: '', loaded: false })

  useEffect(() => {
    if (!active) {
-      setState({ invoices: [], loading: false, error: '' })
+      setState({ invoices: [], loading: false, error: '', loaded: false })
      return undefined
    }
    let cancelled = false
-    setState({ invoices: [], loading: true, error: '' })
-    Promise.all([getInvoices({ status: 'offen' }), getInvoices({ status: 'ueberfaellig' })])
-      .then(([open, overdue]) => {
+    setState({ invoices: [], loading: true, error: '', loaded: false })
+    Promise.all(invoiceFilterStatuses('offen').map((status) => getInvoices({ status })))
+      .then((results) => {
        if (cancelled) return
-        const all = [...(open?.rechnungen || []), ...(overdue?.rechnungen || [])]
-        setState({ invoices: all, loading: false, error: '' })
+        setState({ invoices: mergeInvoiceLists(results.map((r) => r?.rechnungen)), loading: false, error: '', loaded: true })
      })
      .catch((err) => {
-        if (!cancelled) setState({ invoices: [], loading: false, error: err?.message || FALLBACK_ERROR })
+        if (!cancelled) setState({ invoices: [], loading: false, error: err?.message || FALLBACK_ERROR, loaded: false })
      })
    return () => {
      cancelled = true
@@ -406,16 +406,53 @@ export function aboView(abos) {

 // --- Rechnungen (Block 6) ----------------------------------------------------------------------------------

-/** Filter über der Rechnungsliste (Schlüssel = `status` der API); Standard ist „Offen“. */
+/**
+ * Filter über der Rechnungsliste. `statuses` sind die API-Status, die der Filter abruft: „Offen“ zeigt offene UND
+ * überfällige Rechnungen (zwei Abrufe, zusammengeführt), alle anderen genau ihren Status. Standard ist „Offen“.
+ */
 export const INVOICE_FILTERS = [
-  { key: 'offen', label: 'Offen' },
-  { key: 'ueberfaellig', label: 'Überfällig' },
-  { key: 'bezahlt', label: 'Bezahlt' },
-  { key: 'entwurf', label: 'Entwürfe' },
-  { key: 'alle', label: 'Alle' },
+  { key: 'offen', label: 'Offen', statuses: ['offen', 'ueberfaellig'] },
+  { key: 'ueberfaellig', label: 'Überfällig', statuses: ['ueberfaellig'] },
+  { key: 'bezahlt', label: 'Bezahlt', statuses: ['bezahlt'] },
+  { key: 'entwurf', label: 'Entwürfe', statuses: ['entwurf'] },
+  { key: 'alle', label: 'Alle', statuses: ['alle'] },
 ]
 export const DEFAULT_INVOICE_FILTER = 'offen'

+/** API-Status, die ein Filter abruft (unbekannter Filter = alle). */
+export function invoiceFilterStatuses(filterKey) {
+  const filter = INVOICE_FILTERS.find((f) => f.key === filterKey)
+  return filter ? [...filter.statuses] : ['alle']
+}
+
+/**
+ * Mehrere Antwortlisten (je Status eine) zu einer Liste: neueste zuerst nach Rechnungsdatum, gleiche Tage behalten
+ * die Reihenfolge der Listen; eine Rechnung, die in mehreren vorkommt, nur einmal.
+ */
+export function mergeInvoiceLists(lists) {
+  const seen = new Set()
+  const all = []
+  for (const list of Array.isArray(lists) ? lists : []) {
+    for (const inv of Array.isArray(list) ? list : []) {
+      if (!inv || typeof inv !== 'object') continue
+      if (inv.id) {
+        if (seen.has(inv.id)) continue
+        seen.add(inv.id)
+      }
+      all.push(inv)
+    }
+  }
+  return all
+    .map((inv, index) => ({ inv, index }))
+    .sort((a, b) => {
+      const da = text(a.inv.createdDate)
+      const db = text(b.inv.createdDate)
+      if (da === db) return a.index - b.index
+      return da < db ? 1 : -1
+    })
+    .map((x) => x.inv)
+}
+
 const obj = (v) => (v && typeof v === 'object' ? v : {})
 const objects = (list) => (Array.isArray(list) ? list.filter((x) => x && typeof x === 'object') : [])
 const OPEN_INVOICE = new Set(['offen', 'ueberfaellig'])
@@ -790,6 +827,7 @@ export function transactionRows(buchungen) {

 // --- Zuordnen (Dialog) -------------------------------------------------------------------------------------

+const NOT_OPEN_ANYMORE = 'Diese Rechnung ist nicht mehr offen.'
 export const PENDING_ASSIGN_HINT = 'Vorgemerkte Buchungen können erst nach der Buchung einer Rechnung zugeordnet werden.'

 /** Bestätigungstext vor dem Bezahlt-Markieren (nicht umkehrbar). */
@@ -809,8 +847,10 @@ const fixedCostLabel = (f) => `${text(f.name) || 'Fixkosten'} · ${euro(f.amount
 *  - Eingang: Vorschläge des Servers, andere offene Rechnungen, Kategorien (auch „Einnahme ohne Stripe-Rechnung“)
 *  - Ausgang: Fixkosten (aktive), Privat/Umbuchung, Sonstiges
 * Rechnungen, deren offener Betrag vom Betrag der Buchung abweicht, sind nicht wählbar (Teilzahlungen in Stripe).
+ * invoicesLoaded = die offenen Rechnungen wurden frisch und ohne Fehler geladen: ein Vorschlag, dessen Rechnung nicht
+ * (mehr) darin steht, ist dann nicht wählbar („Diese Rechnung ist nicht mehr offen.“).
 */
-export function assignModel(tx, { invoices = [], fixedCosts = [] } = {}) {
+export function assignModel(tx, { invoices = [], fixedCosts = [], invoicesLoaded = false } = {}) {
  const t = obj(tx)
  const amount = t.amountCents
  const incoming = isNum(amount) && amount > 0
@@ -868,6 +908,7 @@ export function assignModel(tx, { invoices = [], fixedCosts = [] } = {}) {
      } else if (s.kind === 'invoice' && text(s.id) && !seen.has(s.id)) {
        seen.add(s.id)
        const known = open.find((inv) => inv.id === s.id)
+        const gone = invoicesLoaded && !known
        const differs = s.amountDiffers === true || (known ? known.remainingCents !== amount : false)
        const number = text(s.number) || text(known?.number)
        model.suggestions.push({
@@ -876,8 +917,12 @@ export function assignModel(tx, { invoices = [], fixedCosts = [] } = {}) {
          label: `Rechnung ${number || s.id}${known ? ` · ${text(known.customerName) || text(known.customerEmail)}` : ''}`,
          detail: [text(s.reason), known ? `${euro(known.remainingCents)} offen` : ''].filter(Boolean).join(' · '),
          confidence: s.confidence,
-          disabled: differs,
-          disabledReason: differs ? 'Betrag weicht ab, Teilzahlungen bitte direkt in Stripe erfassen' : '',
+          disabled: gone || differs,
+          disabledReason: gone
+            ? NOT_OPEN_ANYMORE
+            : differs
+              ? 'Betrag weicht ab, Teilzahlungen bitte direkt in Stripe erfassen'
+              : '',
          number,
          body: { invoiceId: s.id },
        })
@@ -26,6 +26,8 @@ export default function FinancePage() {
  // Buchung, die gerade zugeordnet werden soll (Knopf in Block 7 oder Aufgabe „Zahlungseingang zuordnen“)
  const [assignTxId, setAssignTxId] = useState(null)
  const [assignNotice, setAssignNotice] = useState('')
+  // Zähler, der die Rechnungsliste (Block 6) zum Neuladen bringt: sie hält ihre Liste selbst
+  const [invoicesRefresh, setInvoicesRefresh] = useState(0)
  const refreshLock = useRef(false)
  const retriedRef = useRef(null)

@@ -34,8 +36,12 @@ export default function FinancePage() {
  const needsInvoices = Boolean(assignTx && !assignTx.category && assignTx.status !== 'PDNG' && assignTx.amountCents > 0)
  const openInvoices = useOpenInvoices(needsInvoices)

-  // Übersicht und Buchungen zusammen neu laden (nach Zuordnen, Rechnungsänderungen und Bankabruf)
-  const reloadAll = useCallback(() => Promise.all([reload(), reloadTransactions()]), [reload, reloadTransactions])
+  // Übersicht, Buchungen und Rechnungsliste zusammen neu laden (nach Zuordnen, Rechnungsänderungen und Bankabruf):
+  // eine eben bezahlte Rechnung soll sofort aus „Offen“ verschwinden
+  const reloadAll = useCallback(() => {
+    setInvoicesRefresh((n) => n + 1)
+    return Promise.all([reload(), reloadTransactions()])
+  }, [reload, reloadTransactions])

  // Eine Aufgabe kann eine Buchung nennen, die in der geladenen Liste fehlt (neu abgerufen): einmal nachladen,
  // dann melden statt still nichts zu tun
@@ -169,7 +175,7 @@ export default function FinancePage() {
        <div className="fin-cols fin-cols-half">
          <AboCard abos={data.abos} />
        </div>
-        <InvoiceTable altersgruppen={data.rechnungen?.altersgruppen} onChanged={reloadAll} />
+        <InvoiceTable altersgruppen={data.rechnungen?.altersgruppen} refreshToken={invoicesRefresh} onChanged={reloadAll} />
        <TransactionTable
          transactions={txs.transactions}
          stripeFehler={txs.stripeFehler}
@@ -200,6 +206,7 @@ export default function FinancePage() {
          invoices={openInvoices.invoices}
          invoicesLoading={openInvoices.loading}
          invoicesError={openInvoices.error}
+          invoicesLoaded={openInvoices.loaded}
          onClose={handleAssignClose}
          onDone={handleAssignDone}
        />