From cf23cc661384c8db3a6ce9aedf6ecc80f2ec1a5d Mon Sep 17 00:00:00 2001
From: Roger Rehnelt <noreply@maennchen1.de>
Date: Mon, 28 Sep 2026 10:17:22 +0200
Subject: Altbestand: Zustand nach Dateien statt Übernahme-Vermerk; sauber
 migrierte Shops können direkt löschen
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 views/mods/mod_rechnungen/archive.phtml | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/views/mods/mod_rechnungen/archive.phtml b/views/mods/mod_rechnungen/archive.phtml
index ba88672..cbc57c6 100644
--- a/views/mods/mod_rechnungen/archive.phtml
+++ b/views/mods/mod_rechnungen/archive.phtml
@@ -15,18 +15,16 @@
 	};
 
 	$legacy_path = 'uploads/wpsg_rechnungen/';
-	$nTodo = sizeof($arLegacy['scan']['todo'] ?? []);
-	$nUnknown = ($arLegacy['result'] === null)?(sizeof($arLegacy['scan']['unknown'] ?? []) + sizeof($arLegacy['scan']['conflict'] ?? [])):($arLegacy['result']['unknown'] + $arLegacy['result']['conflict']);
+	// Zustand nach den Dateien, nicht nach dem Übernahme-Vermerk: siehe doc/code-kommentar.md#alte-rechnungsablage
+	$nTodo = sizeof($arLegacy['scan']['todo']);
+	$nDone = $arLegacy['scan']['done'];
+	$nUnknown = sizeof($arLegacy['scan']['unknown']) + sizeof($arLegacy['scan']['conflict']);
 
 ?>
 <div class="wpsg_archive_legacy">
 	<h3><?php echo __('Altbestand', 'wpsg'); ?></h3>
-	<?php if ($arLegacy['result'] === null) { ?>
-	<?php if ($nTodo === 0) { ?>
-	<p><?php echo wpsg_translate(__('Die alte Ablage #1# aus wpShopGermany 3 wurde noch nicht übernommen.', 'wpsg'), '<code>'.$legacy_path.'</code>'); ?></p>
-	<?php } else { ?>
+	<?php if ($nTodo > 0) { ?>
 	<p><?php echo __('Die Übernahme der Rechnungen aus wpShopGermany 3 ist nicht sauber durchgelaufen.', 'wpsg').' '.wpsg_translate(_n('Unter #1# liegt #2# Rechnung, die in der aktuellen Ablage fehlt und sich im Shop nicht herunterladen lässt.', 'Unter #1# liegen #2# Rechnungen, die in der aktuellen Ablage fehlen und sich im Shop nicht herunterladen lassen.', $nTodo, 'wpsg'), '<code>'.$legacy_path.'</code>', $nTodo); ?></p>
-	<?php } ?>
 	<?php if ($nUnknown > 0) { ?>
 	<p><?php echo wpsg_translate(_n('#1# Datei lässt sich keiner Rechnung zuordnen und bleibt dort liegen.', '#1# Dateien lassen sich keiner Rechnung zuordnen und bleiben dort liegen.', $nUnknown, 'wpsg'), $nUnknown); ?></p>
 	<?php } ?>
@@ -35,11 +33,15 @@
 	<?php if ($arLegacy['result'] !== null) { ?>
 	<p><?php echo wpsg_translate(__('Alte Ablage übernommen am #1#: #2#, #3#.', 'wpsg'), wpsg_formatTimestamp(strtotime($arLegacy['result']['date']), true),
 		wpsg_translate(_n('#1# Datei kopiert', '#1# Dateien kopiert', $arLegacy['result']['copied'], 'wpsg'), $arLegacy['result']['copied']),
-		wpsg_translate(_n('#1# war schon vorhanden', '#1# waren schon vorhanden', $arLegacy['result']['done'], 'wpsg'), $arLegacy['result']['done'])); ?>
-	<?php if ($nUnknown > 0) echo wpsg_translate(_n('#1# Datei ließ sich keiner Rechnung zuordnen und liegt nur in der alten Ablage.', '#1# Dateien ließen sich keiner Rechnung zuordnen und liegen nur in der alten Ablage.', $nUnknown, 'wpsg'), $nUnknown); ?></p>
+		wpsg_translate(_n('#1# war schon vorhanden', '#1# waren schon vorhanden', $arLegacy['result']['done'], 'wpsg'), $arLegacy['result']['done'])); ?></p>
+	<?php } else { ?>
+	<p><?php echo wpsg_translate(_n('Die Rechnungen aus wpShopGermany 3 wurden vollständig übernommen: #1# Datei liegt auch in der aktuellen Ablage.', 'Die Rechnungen aus wpShopGermany 3 wurden vollständig übernommen: #1# Dateien liegen auch in der aktuellen Ablage.', $nDone, 'wpsg'), $nDone); ?></p>
+	<?php } ?>
+	<?php if ($nUnknown > 0) { ?>
+	<p><?php echo wpsg_translate(_n('#1# Datei lässt sich keiner Rechnung zuordnen und liegt nur in der alten Ablage.', '#1# Dateien lassen sich keiner Rechnung zuordnen und liegen nur in der alten Ablage.', $nUnknown, 'wpsg'), $nUnknown); ?></p>
 	<?php } ?>
 	<p><?php echo wpsg_translate(__('Datenaltbestand aus wpShopGermany 3: Bitte den Ordner #1# sichern und anschließend löschen.', 'wpsg'), '<code>'.$legacy_path.'</code>'); ?></p>
-	<p><a href="<?php echo esc_url($legacy_url('legacyDelete')); ?>" class="button" onclick="return confirm('<?php echo esc_js((($nUnknown > 0)?wpsg_translate(_n('Achtung: #1# Datei ließ sich keiner Rechnung zuordnen und geht mit dem Löschen verloren.', 'Achtung: #1# Dateien ließen sich keiner Rechnung zuordnen und gehen mit dem Löschen verloren.', $nUnknown, 'wpsg'), $nUnknown)."\n\n":'').wpsg_translate(_n('Den Ordner #1# mit #2# Datei endgültig löschen? Vorher sichern!', 'Den Ordner #1# mit #2# Dateien endgültig löschen? Vorher sichern!', $arLegacy['files'], 'wpsg'), $legacy_path, $arLegacy['files'])); ?>');"><?php echo __('Alte Ablage löschen', 'wpsg'); ?></a></p>
+	<p><a href="<?php echo esc_url($legacy_url('legacyDelete')); ?>" class="button" onclick="return confirm('<?php echo esc_js((($nUnknown > 0)?wpsg_translate(_n('Achtung: #1# Datei lässt sich keiner Rechnung zuordnen und geht mit dem Löschen verloren.', 'Achtung: #1# Dateien lassen sich keiner Rechnung zuordnen und gehen mit dem Löschen verloren.', $nUnknown, 'wpsg'), $nUnknown)."\n\n":'').wpsg_translate(_n('Den Ordner #1# mit #2# Datei endgültig löschen? Vorher sichern!', 'Den Ordner #1# mit #2# Dateien endgültig löschen? Vorher sichern!', $arLegacy['files'], 'wpsg'), $legacy_path, $arLegacy['files'])); ?>');"><?php echo __('Alte Ablage löschen', 'wpsg'); ?></a></p>
 	<?php } ?>
 </div>
 <?php } ?>
-- 
cgit v1.2.3