Abrechnungen links im Hauptfenster (Klick öffnet), Import fragt nach der Ziel-Abrechnung

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

.gitignore

@@ -16,3 +16,4 @@ build/version_info.txt
 selbsttest.txt
 /*.jpg
 /*.png
+.claude/

README.md

@@ -80,7 +80,7 @@ Das Hauptfenster ist unter Windows ein echtes Programmfenster mit Win95-Optik: a
 ziehen (Andocken), `Win`+Pfeiltasten, Doppelklick auf die Titelleiste und Ziehen am Rand funktionieren
 wie gewohnt.
 
-**Zeitraum abrechnen:** Im Export-Dialog „Nur von … bis …“ eintragen (oder „Vormonat“), wahlweise nach
+**Zeitraum abrechnen:** Im Export-Dialog „Nur von … bis …“ eintragen (oder „Vormonat“; „Noch nicht abgerechnet“ = alles Offene), wahlweise nach
 Aktivierungsdatum oder Abrechnungszeitraum des Anbieters. Exportierte Positionen werden als *abgerechnet*
 gekennzeichnet (grau) und nie wieder exportiert – auch nicht, wenn dieselbe Position später in einer
 anderen Datei oder Abrechnung auftaucht. Versehentlich? Rechtsklick ▸ *Kennzeichnung „abgerechnet“ aufheben*.

provtool/__init__.py

@@ -1,5 +1,5 @@
 """Provisions-Manager 95 – Provisionsabrechnung für Unterhändler der Handy-Insel Miesbach."""
 
-__version__ = "1.4.1"
+__version__ = "1.5.0"
 APP_NAME = "Provisions-Manager 95"
 APP_ORG = "Handy-Insel Miesbach"

provtool/gui/app.py

@@ -195,8 +195,23 @@ class App:
                                       bd=0, opaqueresize=False, sashcursor="sb_h_double_arrow")
         pw.pack(fill="both", expand=True, padx=6, pady=6)
 
+        links = tk.Frame(pw, bg=w.FACE)
+        # --- Abrechnungen (ein Klick öffnet)
+        self.f_a = w.Fenster(links, "Abrechnungen", self.icons("oeffnen"))
+        self.f_a.pack(fill="x", pady=(0, 6))
+        r, self.atree = w.scroll_tree(self.f_a.inhalt, columns=("status",), show="tree headings", height=5,
+                                      selectmode="browse")
+        self.atree.heading("#0", text="Abrechnung", anchor="w")
+        self.atree.heading("status", text="Status")
+        self.atree.column("#0", width=170, minwidth=80, stretch=False)
+        self.atree.column("status", width=90, anchor="w", stretch=False)
+        w.auto_breite(self.atree, {"#0": 1})
+        r.pack(fill="both", expand=True)
+        self.atree.bind("<<TreeviewSelect>>", self._abrechnung_gewaehlt)
+
         # --- Händler-Fenster
-        self.f_h = w.Fenster(pw, "Händler", self.icons("haendler"))
+        self.f_h = w.Fenster(links, "Händler", self.icons("haendler"))
+        self.f_h.pack(fill="both", expand=True)
         r, self.htree = w.scroll_tree(self.f_h.inhalt, columns=("anzahl", "summe"), show="tree headings",
                                       selectmode="browse")
         self.htree.heading("#0", text="Händler", anchor="w")
@@ -211,7 +226,7 @@ class App:
         r.pack(fill="both", expand=True)
         self.htree.bind("<<TreeviewSelect>>", self._haendler_gewaehlt)
         self.htree.bind("<Double-1>", lambda e: self._haendler_doppelklick())
-        pw.add(self.f_h, minsize=240, width=310)
+        pw.add(links, minsize=240, width=310)
 
         # --- Positionen-Fenster
         self.f_p = w.Fenster(pw, "Positionen", self.icons("neu"))
@@ -284,9 +299,26 @@ class App:
             self.positionen = []
             self.rahmen.set_titel(w.APP_NAME)
             self.status.set("info", " Keine Abrechnung geöffnet – Datei > Neue Abrechnung oder Import")
+        self._abrechnungsliste()
         self._haendlerliste(behalte_auswahl)
         self._tabelle()
 
+    def _abrechnungsliste(self):
+        t = self.atree
+        t.delete(*t.get_children())
+        for a in self.db.abrechnungen():
+            status = "erledigt" if a["erledigt"] else (f"{a['abgerechnet']} abgerechnet" if a["abgerechnet"] else "offen")
+            t.insert("", "end", iid=str(a["id"]), text=a["titel"], values=(status,))
+        if self.aid and t.exists(str(self.aid)):
+            t.selection_set(str(self.aid))
+            t.see(str(self.aid))
+
+    def _abrechnung_gewaehlt(self, _e=None):
+        sel = self.atree.selection()
+        if sel and int(sel[0]) != self.aid:
+            self.ansicht = ALLE
+            self.oeffnen(int(sel[0]))
+
     def _suche_planen(self):
         """Suche erst kurz nach dem letzten Tastendruck ausführen (flüssiges Tippen)."""
         if self._suche_geplant is not None:
@@ -466,7 +498,7 @@ class App:
         else:
             self._aktualisieren()
 
-    def importieren(self, dateien: list[str] | None = None):
+    def importieren(self, dateien: list[str] | None = None, ziel: int | None = None):
         if not dateien:
             dateien = list(filedialog.askopenfilenames(
                 parent=self.root, title="Provisionsabrechnungen importieren",
@@ -477,7 +509,22 @@ class App:
         if not dateien:
             return
         self.settings["import_ordner"] = str(Path(dateien[0]).parent)
-        if not self._brauche_abrechnung() or self._gesperrt():
+        if ziel is None:  # jedes Mal fragen, in welche Abrechnung
+            if not self.db.abrechnungen():
+                if not self.neu():
+                    return
+            else:
+                res = dlg.AbrechnungenDialog(self.root, self.db, self.aid, fuer_import=True).zeigen()
+                if not res:
+                    return
+                if res[0] == "neu":
+                    if not self.neu():
+                        return
+                else:
+                    self.aid, self.ansicht = res[1], ALLE
+        else:
+            self.aid = ziel
+        if self._gesperrt():
             return
         bericht = []
         self.root.configure(cursor="watch")
@@ -1018,7 +1065,7 @@ class App:
             self.ansicht = ALLE
             self._aktualisieren()
             if res["dateien"]:
-                self.importieren(res["dateien"])
+                self.importieren(res["dateien"], ziel=self.aid)
         if res["rundgang"]:
             self.root.after(200, self.rundgang)
 

provtool/gui/dialogs.py

@@ -558,10 +558,13 @@ class ProvisionsartenDialog(w.Dialog):
 # =========================================================================== Abrechnungen
 
 class AbrechnungenDialog(w.Dialog):
-    def __init__(self, master, db: Datenbank, aktuelle_id: int | None):
+    def __init__(self, master, db: Datenbank, aktuelle_id: int | None, fuer_import: bool = False):
-        super().__init__(master, "Abrechnung öffnen", resizable=True)
+        """fuer_import: Ziel eines Imports wählen (Ergebnis ("oeffnen", id) oder ("neu",))."""
+        super().__init__(master, "Import – in welche Abrechnung?" if fuer_import else "Abrechnung öffnen",
+                         resizable=True)
         self.db = db
-        tk.Label(self, text="Gespeicherte Abrechnungen:", anchor="w").pack(fill="x", padx=10, pady=(10, 2))
+        tk.Label(self, text="In welche Abrechnung sollen die Dateien importiert werden?" if fuer_import
+                 else "Gespeicherte Abrechnungen:", anchor="w").pack(fill="x", padx=10, pady=(10, 2))
         r, self.tree = w.scroll_tree(self, columns=("titel", "status", "geaendert", "anzahl", "dateien"),
                                      show="headings", height=12, selectmode="browse")
         for c, t, b in (("titel", "Bezeichnung", 180), ("status", "Status", 110), ("geaendert", "Zuletzt geändert", 120),
@@ -572,6 +575,10 @@ class AbrechnungenDialog(w.Dialog):
         self.tree.bind("<Double-1>", lambda e: self._oeffnen())
         k = tk.Frame(self, bg=w.FACE)
         k.pack(fill="x", padx=10, pady=10)
+        if fuer_import:
+            w.knopf(k, "Hierhin importieren", self._oeffnen, default="active").pack(side="left")
+            w.knopf(k, "Neue Abrechnung...", self._neu).pack(side="left", padx=6)
+        else:
             w.knopf(k, "Öffnen", self._oeffnen, breite=10, default="active").pack(side="left")
             w.knopf(k, "Umbenennen...", self._umbenennen, breite=12).pack(side="left", padx=6)
             w.knopf(k, "Löschen", self._loeschen, breite=10).pack(side="left")
@@ -600,6 +607,10 @@ class AbrechnungenDialog(w.Dialog):
             self.result = ("oeffnen", self._sel())
             self.destroy()
 
+    def _neu(self):
+        self.result = ("neu",)
+        self.destroy()
+
     def _umbenennen(self):
         aid = self._sel()
         if not aid:
@@ -917,10 +928,13 @@ class ExportDialog(w.Dialog):
         tk.Label(zf, text="bis:").pack(side="left")
         tk.Entry(zf, textvariable=self.v_bis, width=11).pack(side="left", padx=4)
         w.knopf(zf, "Vormonat", self._vormonat).pack(side="left", padx=(4, 0))
+        # ohne Zeitraum = alles, was noch nicht abgerechnet ist (abgerechnete Positionen exportiert er nie)
+        w.knopf(zf, "Noch nicht abgerechnet", lambda: (self.v_von.set(""), self.v_bis.set(""))).pack(
+            side="left", padx=(4, 0))
         self.v_basis = tk.StringVar(value=zr.BASEN[settings.get("export_basis", zr.AKTIVIERUNG)])
         ttk.Combobox(z, textvariable=self.v_basis, values=list(zr.BASEN.values()), state="readonly",
                      width=40).pack(anchor="w", pady=(4, 0))
-        tk.Label(z, text="(leer = alle offenen Positionen; Datum als TT.MM.JJJJ)", fg=w.SHADOW, anchor="w").pack(
+        tk.Label(z, text="(leer bzw. „Noch nicht abgerechnet“ = alle offenen Positionen; Datum als TT.MM.JJJJ)", fg=w.SHADOW, anchor="w").pack(
             fill="x")
         self.v_kennz = tk.BooleanVar(value=True)
         tk.Checkbutton(z, text="Exportierte Positionen als abgerechnet kennzeichnen (werden dann nicht noch "

tests/test_gui.py

@@ -1098,11 +1098,18 @@ def test_export_zeitraum_kennzeichnen_und_erledigt(lauf_modern, tmp_path, monkey
         d.v_auf.set(False)
         d._ok()
 
+    def ok_offen():
+        d = lauf.dialog(ExportDialog)
+        d.v_von.set("01.10.2026")
+        d.v_bis.set("02.10.2026")
+        next(k for k in _alle(d) if isinstance(k, tk.Button) and k.cget("text") == "Noch nicht abgerechnet").invoke()
+        ok(d.v_von.get(), d.v_bis.get())
+
     lauf.ablauf([
         (IMMER, app.exportieren, "Export Oktober 6."),
         (lambda: lauf.dialog(ExportDialog) is not None, lambda: ok("01.10.2026", "05.10.2026"), "nur bis 5.10."),
         (lambda: lauf.dialog(ExportDialog) is None, app.exportieren, "Export Rest"),
-        (lambda: lauf.dialog(ExportDialog) is not None, ok, "alles Offene"),
+        (lambda: lauf.dialog(ExportDialog) is not None, ok_offen, "Noch nicht abgerechnet"),
         (lambda: lauf.dialog(ExportDialog) is None, app.exportieren, "nochmal"),
     ])
     assert not lauf.uebrig, lauf.diagnose()
@@ -1116,3 +1123,41 @@ def test_export_zeitraum_kennzeichnen_und_erledigt(lauf_modern, tmp_path, monkey
     app.prozent()
     assert "erledigt" in meldungen[-1]
     lauf.pruefen()
+
+
+def _alle(widget):
+    for kind in widget.winfo_children():
+        yield kind
+        yield from _alle(kind)
+
+
+def test_abrechnungen_links_und_import_fragt_nach_ziel(lauf_modern, home, monkeypatch):
+    from provtool.gui import win95 as w
+    from provtool.gui.dialogs import AbrechnungenDialog, ImportBericht
+    monkeypatch.setattr(w, "frage", lambda *a, **k: True)  # „bereits importiert – trotzdem?“ (Testdatei ist schon drin)
+    lauf = lauf_modern
+    app = lauf.app
+    alt = app.aid
+    neu = app.db.abrechnung_neu("November 2026")
+    app._aktualisieren()
+    assert set(app.atree.get_children()) == {str(alt), str(neu)}, "Abrechnungen fehlen links"
+    app.atree.selection_set(str(neu))
+    lauf.root.update()
+    assert app.aid == neu and not app.positionen, "Klick links öffnet die Abrechnung nicht"
+    app.atree.selection_set(str(alt))
+    lauf.root.update()
+
+    def ziel_waehlen():
+        d = lauf.dialog(AbrechnungenDialog)
+        d.tree.selection_set(str(neu))
+        d._oeffnen()
+
+    lauf.ablauf([
+        (IMMER, lambda: app.importieren([str(home / "freenet.csv")]), "Import"),
+        (lambda: lauf.dialog(AbrechnungenDialog) is not None, ziel_waehlen, "Ziel: November"),
+        (lambda: lauf.dialog(ImportBericht) is not None, lambda: lauf.dialog(ImportBericht).abbrechen(), "Bericht zu"),
+    ])
+    assert not lauf.uebrig, lauf.diagnose()
+    assert app.aid == neu and len(app.positionen) == 3, "Import nicht in der gewählten Abrechnung"
+    assert app.atree.selection() == (str(neu),)
+    lauf.pruefen()