From 2012f228c4779cbdd72fad0b35caf4bf9efde6d8 Mon Sep 17 00:00:00 2001
From: fds2610-bot <claude@sonnengetrock.net>
Date: Mon, 6 Jul 2026 23:57:37 +0200
Subject: [PATCH] =?UTF-8?q?Fix=20#114:=20Beleg-L=C3=B6schung=20nicht=20meh?=
 =?UTF-8?q?r=20durch=20Datei-Permission-Fehler=20unterbrochen?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Changes.md                             |  6 ++++
 src/Models/Beleg.php                   |  6 ++--
 tests/49_BelegDeleteUnlinkFailTest.php | 42 ++++++++++++++++++++++++++
 3 files changed, 52 insertions(+), 2 deletions(-)
 create mode 100644 tests/49_BelegDeleteUnlinkFailTest.php

diff --git a/Changes.md b/Changes.md
index bab3732..28e6c30 100644
--- a/Changes.md
+++ b/Changes.md
@@ -2,6 +2,12 @@
 
 ---
 
+## 2026-07-06 – Fix: Fehlermeldung beim Beleg-Löschen, falls Datei nicht löschbar ist (Issue #114)
+
+- **`src/Models/Beleg.php`:** `delete()` nutzte ein ungeschütztes `unlink()`; schlug dieses auf dem Server mit „Permission denied" fehl, wurde eine PHP-Warning ausgegeben, die den nachfolgenden Redirect-Header zerstörte („headers already sent"). Der Datensatz wurde dabei trotzdem gelöscht, was zu Verwirrung führte. Fehler beim Datei-Löschen wird jetzt abgefangen und nur noch geloggt (`error_log`), die Löschung des Datensatzes läuft unverändert durch.
+
+---
+
 ## 2026-07-02 – Feature: Zusätzliche CC-E-Mail-Adressen pro Kunde (Issue #111)
 
 - **`migrations/084_customer_cc_emails.sql`:** Neue Spalten `email_cc1`, `email_cc2` in `customers`
diff --git a/src/Models/Beleg.php b/src/Models/Beleg.php
index 56781c7..edaa37c 100644
--- a/src/Models/Beleg.php
+++ b/src/Models/Beleg.php
@@ -249,9 +249,11 @@ class Beleg
         $cnt = Database::fetchOne('SELECT COUNT(*) as cnt FROM bookings WHERE beleg_id=?', [$id]);
         if ($cnt && $cnt['cnt'] > 0) return false;
 
-        // Delete file
+        // Delete file (Datei-Löschung darf DB-Löschung nicht durch Warnings/Header-Fehler stören)
         $fullPath = dirname(__DIR__, 2) . '/public/' . $beleg['storage_path'];
-        if (file_exists($fullPath)) unlink($fullPath);
+        if (file_exists($fullPath) && !@unlink($fullPath)) {
+            error_log('Beleg::delete: Datei konnte nicht gelöscht werden: ' . $fullPath);
+        }
 
         Database::execute('DELETE FROM belege WHERE id=?', [$id]);
         return true;
diff --git a/tests/49_BelegDeleteUnlinkFailTest.php b/tests/49_BelegDeleteUnlinkFailTest.php
new file mode 100644
index 0000000..aff945a
--- /dev/null
+++ b/tests/49_BelegDeleteUnlinkFailTest.php
@@ -0,0 +1,42 @@
+<?php
+
+declare(strict_types=1);
+
+use Steuerkram\Models\Beleg;
+
+class BelegDeleteUnlinkFailTest extends TestCase
+{
+    protected string $testName = 'Test 49: Beleg-Löschung trotz Datei-Permission-Fehler';
+
+    public function run(): void
+    {
+        $dir  = dirname(__DIR__) . '/public/static/uploads/test_49_unlink_fail';
+        $file = $dir . '/original.pdf';
+
+        mkdir($dir, 0755, true);
+        file_put_contents($file, 'dummy');
+        // Schreibschutz auf das Verzeichnis: unlink() der Datei schlägt fehl (Permission denied),
+        // genau wie im gemeldeten Fehler auf dem Produktiv-Server.
+        chmod($dir, 0500);
+
+        try {
+            $id = Beleg::create([
+                'beleg_nr'     => 'BLG-TEST-49',
+                'filename'     => 'original.pdf',
+                'storage_path' => 'static/uploads/test_49_unlink_fail/original.pdf',
+                'mime_type'    => 'application/pdf',
+                'file_hash'    => 'testhash49',
+                'beleg_datum'  => '2026-01-01',
+            ]);
+            $this->assertGreaterThan(0, $id, 'Beleg angelegt');
+
+            $result = Beleg::delete($id);
+            $this->assert($result === true, 'delete() liefert true, obwohl unlink() fehlschlägt');
+            $this->assert(Beleg::find($id) === false, 'Beleg-Datensatz wurde trotz Datei-Fehler aus der DB gelöscht');
+        } finally {
+            chmod($dir, 0755);
+            if (file_exists($file)) unlink($file);
+            if (is_dir($dir)) rmdir($dir);
+        }
+    }
+}