From d26bbfc37b5325bf844cc591c0a0de5d42e9e2f5 Mon Sep 17 00:00:00 2001
From: fds2610-bot <felix-bot@schepmann.de>
Date: Fri, 15 May 2026 16:51:38 +0200
Subject: [PATCH] =?UTF-8?q?Fix:=20Bank-Import-Duplikate=20durch=20Transakt?=
 =?UTF-8?q?ion,=20UNIQUE-Constraint=20und=20Dateiname-Pr=C3=BCfung=20verhi?=
 =?UTF-8?q?ndern?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Changes.md                                  | 15 ++++++++++++++
 migrations/051_bank_transactions_unique.sql | 22 +++++++++++++++++++++
 src/Controllers/BankController.php          | 18 ++++++++++++++---
 src/Models/BankTransaction.php              |  9 +++++++++
 src/version.txt                             |  2 +-
 5 files changed, 62 insertions(+), 4 deletions(-)
 create mode 100644 migrations/051_bank_transactions_unique.sql

diff --git a/Changes.md b/Changes.md
index 1f8259d..876561a 100644
--- a/Changes.md
+++ b/Changes.md
@@ -2,6 +2,21 @@
 
 ---
 
+## 2026-05-15 – Fix: Bank-Import-Duplikate verhindern
+
+- Migration 051: UNIQUE-Constraint `(buchungstag, wertstellung, wert, buchungstext(200))` auf `bank_transactions`; bestehende Duplikate wurden dabei bereinigt
+- `BankController::upload()`: Import läuft jetzt in einer DB-Transaktion – bei Fehler wird alles zurückgerollt statt einen unvollständigen Batch zu hinterlassen
+- `BankController::upload()`: Dateiname-Prüfung vor dem Import – erneuter Upload derselben Datei wird mit Fehlermeldung abgelehnt
+- `BankTransaction::filenameExists()`: neue Hilfsmethode zur Duplikat-Prüfung anhand Dateiname
+
+---
+
+## 2026-05-15 – Fix: Netto-Neuberechnung bei Kategorie-Wechsel in Bank-Buchungsmaske
+
+- IIFE-Wrapper für `updateAutoUst` in einen `DOMContentLoaded`-Listener verschoben, damit er erst nach der Funktionsdefinition (späteres `<script>`-Tag) gesetzt wird und nicht sofort von ihr überschrieben wird.
+
+---
+
 ## 2026-05-14 – Feature: Bank buchen
 
 - Navigation: neuer Eintrag „Bank buchen" unter Belege & Buchungen
diff --git a/migrations/051_bank_transactions_unique.sql b/migrations/051_bank_transactions_unique.sql
new file mode 100644
index 0000000..aed80d5
--- /dev/null
+++ b/migrations/051_bank_transactions_unique.sql
@@ -0,0 +1,22 @@
+-- Duplikate entfernen: pro Gruppe (buchungstag, wertstellung, wert, buchungstext) die niedrigste id behalten
+DELETE bt
+FROM bank_transactions bt
+INNER JOIN (
+    SELECT MIN(id) AS keep_id, buchungstag, wertstellung, wert, buchungstext
+    FROM bank_transactions
+    GROUP BY buchungstag, wertstellung, wert, buchungstext
+    HAVING COUNT(*) > 1
+) dup ON bt.buchungstag = dup.buchungstag
+      AND bt.wertstellung = dup.wertstellung
+      AND bt.wert = dup.wert
+      AND bt.buchungstext = dup.buchungstext
+      AND bt.id > dup.keep_id;
+
+-- Unvollständige Batches entfernen (row_count stimmt nicht mit tatsächlichen Zeilen überein)
+DELETE FROM bank_import_batches
+WHERE id NOT IN (SELECT DISTINCT batch_id FROM bank_transactions)
+  AND row_count > 0;
+
+-- UNIQUE-Constraint gegen Doppel-Importe
+ALTER TABLE bank_transactions
+    ADD CONSTRAINT uq_bank_tx UNIQUE (buchungstag, wertstellung, wert, buchungstext(200));
diff --git a/src/Controllers/BankController.php b/src/Controllers/BankController.php
index b587aab..2e70883 100644
--- a/src/Controllers/BankController.php
+++ b/src/Controllers/BankController.php
@@ -4,6 +4,7 @@ declare(strict_types=1);
 
 namespace Steuerkram\Controllers;
 
+use Steuerkram\Database;
 use Steuerkram\Models\BankImportFormat;
 use Steuerkram\Models\BankTransaction;
 use Steuerkram\Models\Beleg;
@@ -83,9 +84,20 @@ class BankController
             View::redirect('/bank/upload', 'Keine Buchungszeilen in der Datei gefunden.', 'error');
         }
 
-        $batchId = BankTransaction::createBatch($formatId, $origName, count($rows));
-        foreach ($rows as $row) {
-            BankTransaction::insertRow($batchId, $row);
+        if (BankTransaction::filenameExists($origName)) {
+            View::redirect('/bank/upload', 'Diese Datei wurde bereits importiert (' . $origName . ').', 'error');
+        }
+
+        Database::beginTransaction();
+        try {
+            $batchId = BankTransaction::createBatch($formatId, $origName, count($rows));
+            foreach ($rows as $row) {
+                BankTransaction::insertRow($batchId, $row);
+            }
+            Database::commit();
+        } catch (\Throwable $e) {
+            Database::rollBack();
+            View::redirect('/bank/upload', 'Import fehlgeschlagen: ' . $e->getMessage(), 'error');
         }
 
         View::redirect('/bank', count($rows) . ' Buchungen importiert.');
diff --git a/src/Models/BankTransaction.php b/src/Models/BankTransaction.php
index 912117e..a011e01 100644
--- a/src/Models/BankTransaction.php
+++ b/src/Models/BankTransaction.php
@@ -96,6 +96,15 @@ class BankTransaction
         );
     }
 
+    public static function filenameExists(string $filename): bool
+    {
+        $row = Database::fetchOne(
+            'SELECT id FROM bank_import_batches WHERE filename = ? LIMIT 1',
+            [$filename]
+        );
+        return $row !== null;
+    }
+
     public static function createBatch(int $formatId, string $filename, int $rowCount): int
     {
         Database::execute(
diff --git a/src/version.txt b/src/version.txt
index 442ba83..b220809 100644
--- a/src/version.txt
+++ b/src/version.txt
@@ -1 +1 @@
-1-260514
+1-260515