Warenwirtschaft: Doku — AP-52 in Plan V7 und README; Gegenprüfung Abschnitt 7 (unabhängige Bestätigung R-01/R-02) eingecheckt
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This commit is contained in:
parent
3abb9e644e
commit
db2f7fe1a3
3 changed files with 47 additions and 2 deletions
|
|
@ -31,6 +31,7 @@ Die mit **`#`** markierten Punkte hat der Kunde als **buggy/dringend** priorisie
|
|||
| **AP-43** Zwischenspeichern (Entwurf) + „abschließen" | ✅ erledigt (abd202a) — Migration `status` auf Dev-/Live-DB |
|
||||
| **AP-44** Rezeptur-Darstellung (Optik/iPad) | ✅ erledigt (2a6daf2) |
|
||||
| **AP-38** WooCommerce-Ampel (Cron/Auth) | ✅ erledigt (0c8d321 ERP+Connector, b6ca332 Cron-Selbstheilung) — Live-Ursache behoben |
|
||||
| **AP-52** Verpackung: Bestand/Differenz/Lagerort in der Produktion (Nachtrag 09.09.) | ✅ erledigt (3abb9e6) — Spalte einheitlich „Lagerort" |
|
||||
| übrige APs | — |
|
||||
|
||||
Suite (26.08.2026): **338 grün**; nach Nacharbeit, Nachprüfung und Gegenprüfung (08./09.09.2026): **370 grün**. Erledigt: **AP-34…AP-51 vollständig** (AP-34/35/36/37/38/39/40/41/42/43/44/45/46/47/48/49/50/51). **Keine offenen APs mehr.** Verbleibend nur der **Connector-Rollout v1.6.0** in den Shop (AP-34 + AP-38).
|
||||
|
|
@ -180,6 +181,11 @@ Aktuell nach `pos` sortiert (`Product::manufacturer_ingredients() ->orderByPivot
|
|||
**Lösung:** Gleiche Lagerplatz-Zuordnung für Produkte und Verpackungsartikel + Anzeige an den passenden Stellen (Produktbestand, Produktverpackung). Nutzt die bestehende Storage-Hierarchie (`StorageRoom/Shelf/Slot`).
|
||||
**Aufwand:** 1,5–2,5 PT.
|
||||
|
||||
### AP-52 — Produktion: Bestand/Differenz/Lagerort auch für die Verpackung (Nachtrag 09.09.2026)
|
||||
**Kundenwunsch (Nachtrag zu „Produktion"):** Wie bei den Rohstoffen oben (AP-40) sollen in der Verpackungs-Vorschau darunter der Bestand des Verpackungsartikels und die Differenz (Plus/Minus) stehen, damit schon bei der Planung sichtbar ist, ob alles da ist. Dazu eine Spalte mit dem Lagerplatz — einheitlich mit der INCI-Tabelle, Bezeichnung **„Lagerort"**.
|
||||
**Lösung:** Verpackungs-Vorschau mit Spalten Artikel · Stück gesamt · Lagerort · Bestand · Differenz (rot = Fehlmenge, grün = ausreichend, gleiche Darstellung wie AP-40). Bestand aus `InventoryService::remainingByPackagingItem()`, Lagerort aus den AP-51-Feldern. Die INCI-Spalte „Ablage" heißt jetzt ebenfalls „Lagerort". Bei bereits gebuchten Produktionen (Snapshot, F-05/N-04) wird keine Differenz angezeigt, da der Bestand den Abzug bereits enthält.
|
||||
**Umsetzung:** ✅ 09.09.2026, Commit 3abb9e6.
|
||||
|
||||
---
|
||||
|
||||
## D. Klärungspunkte — beantwortet (Kundenentscheide 26.08.2026)
|
||||
|
|
|
|||
|
|
@ -1,5 +1,7 @@
|
|||
# Warenwirtschaft: Gegenprüfung der zweiten Nacharbeit vom 09.09.2026
|
||||
|
||||
**Aktueller Stand nach erneuter Verifikation:** R-01 und R-02 sind im geprüften Umfang behoben. Die Bestätigung mit Code-Prüfstand und Testergebnissen steht in **Abschnitt 7**. Die folgenden Abschnitte 1–5 dokumentieren den früheren Stand vor dieser letzten Korrektur; Abschnitt 6 dokumentiert die anschließende Umsetzung.
|
||||
|
||||
**Ergebnis: Vier der sechs Restbefunde sind im geprüften Umfang behoben. Bei N-02 und N-04 bleiben jeweils konkrete Randfälle offen.** Die ursprünglichen Reproduktionsfälle funktionieren jetzt; die Aussage „alle sechs Restbefunde vollständig umgesetzt“ ist dennoch zu weitgehend.
|
||||
|
||||
Die gemeldete Testsuite wurde bestätigt: **368 Tests bestanden, 1.482 Assertions**. Darüber hinaus wurden die vorherigen Diagnosen erneut ausgeführt und um die unten beschriebenen Fälle erweitert. Die beiden verbleibenden Fehler wurden anhand von Testrequests, Datenbankzustand und tatsächlich ausgeführter Formularlogik überprüft.
|
||||
|
|
@ -108,3 +110,40 @@ Die bereits bekannten offenen Betriebsnachweise bleiben unverändert: MySQL-Para
|
|||
| R-02 / N-04 | ✅ `bdaac3b` | Das Bearbeitungsformular übermittelt getrennt, ob ein historischer Snapshot gilt (`data-packaging-snapshot-applies`, = gebucht). Bei unveränderten Produktsummen zeigt die Vorschau auch einen leeren Snapshot („Kein Verpackungsverbrauch gebucht“); Entwürfe rechnen weiter mit der Stückliste. Regression: ohne Verpackung buchen → Stückliste ergänzen → öffnen (leer) → Notiz speichern (0) → Menge ändern (Neuberechnung). |
|
||||
|
||||
Suite danach: **370 Tests grün, 1.506 Assertions**. Offen bleiben unverändert die Betriebsnachweise (MySQL-Parallelität, iPad, Migrationen auf Dev-/Live-DB, Connector 1.6.0) und die nachgelagerten Pakete.
|
||||
|
||||
## 7. Erneute unabhängige Verifikation von R-01/R-02 (09.09.2026)
|
||||
|
||||
**Ergebnis: Beide Korrekturen sind im geprüften Umfang bestätigt.** Die früheren Reproduktionsfälle schlagen nicht mehr fehl. Zusammen mit der vorherigen Bestätigung von N-01, N-03, N-05 und N-06 können damit die sechs Restbefunde für die geprüften Abläufe als behoben geführt werden. In dieser erneuten Prüfung wurde kein weiterer konkreter Fehler dieser Abläufe festgestellt.
|
||||
|
||||
**Prüfstand:** `22e218d5708a96c16e115212db44bffe86527ffe` einschließlich der bereits zuvor vorhandenen lokalen Änderungen. Geprüft wurden insbesondere `fda37d5` und `bdaac3b`, die ID-Felder im Artikelformular, die Zuordnung im Repository sowie Statusübergabe und Darstellung der Verpackungsvorschau. Anwendungscode, bestehende Tests, Migrationen und Konfigurationen wurden nicht verändert. Im Repository wurde ausschließlich dieser Bericht ergänzt.
|
||||
|
||||
### 7.1 Bestätigung der beiden Korrekturen
|
||||
|
||||
| Befund | Verifiziertes Verhalten | Ergebnis |
|
||||
|---|---|---|
|
||||
| R-01 / N-02 – Phasenzuordnung | Neue Textphase voranstellen und bestehende Phase umbenennen: Die bisherigen Phasen behalten ihre IDs; der Produktionsentwurf zeigt weiterhin 6 g und 4 g. | **Behoben** |
|
||||
| R-01 – Vorrang der expliziten IDs | Zusätzlicher HTTP-Test mit Einfügen, Umordnen und vertauschten Phasennamen: Die übermittelten IDs haben Vorrang vor Namen und Reihenfolge. Im ausgeführten Formular erscheinen die Mengen entsprechend der neuen Reihenfolge korrekt als 4 g und 6 g. | **Bestätigt** |
|
||||
| R-02 / N-04 – Leerer historischer Snapshot | Produktion ohne Verpackungsverbrauch, danach Stückliste ergänzen: Das ausgeführte Formular zeigt „Kein Verpackungsverbrauch gebucht.“; eine reine Notizkorrektur behält den Verbrauch null. | **Behoben** |
|
||||
| R-02 – Änderung und Rückkehr | Ergebnisstückzahl von 1 auf 3 ändern: Vorschau berechnet bei zwei Tiegeln je Stück korrekt sechs Tiegel und erklärt die Neuberechnung. Zurück auf 1: wieder leerer historischer Verbrauch. | **Bestätigt** |
|
||||
| R-02 – Entwurf | Ein Entwurf ohne Snapshot verwendet die heutige Stückliste und zeigt bei einem Ergebnisstück korrekt zwei Tiegel. | **Bestätigt** |
|
||||
|
||||
Die vorhandene Suite bestätigt außerdem die Neuberechnung beim tatsächlichen Speichern einer geänderten Ergebnismenge. Die zusätzlichen früheren Prüfungen zu Entwurfsmenge ohne Charge, unverändertem Artikelspeichern, Ablehnung doppelter Rohstoffzeilen innerhalb einer Phase, Ergebnis-Chargensplit, Alt-Ausschuss und ml-Kopie bestehen weiterhin.
|
||||
|
||||
Die Zuordnung ohne übermittelte IDs verwendet weiterhin Namen und Rohstoffüberschneidungen. Der frühere Reproduktionsfall mit eindeutig zuordenbaren Rohstoffen besteht auch auf diesem Weg. Daraus folgt keine allgemeine Eindeutigkeitsgarantie für beliebige alte Aufrufe ohne IDs; der reguläre Artikelformularweg führt nun die stabilen IDs mit.
|
||||
|
||||
### 7.2 Ausgeführte Prüfungen
|
||||
|
||||
| Prüfung | Ergebnis |
|
||||
|---|---|
|
||||
| `php artisan test` | **370 Tests bestanden, 1.506 Assertions**, 5,88 s |
|
||||
| Unveränderte PHP-Diagnosen aus der letzten Gegenprüfung: `/tmp/WawiThirdProbeTest.php` | **9 Tests bestanden, 56 Assertions**, 0,422 s |
|
||||
| Erweiterte PHP-Diagnosen: `/tmp/WawiFourthProbeTest.php` | **10 Tests bestanden, 67 Assertions**, 0,559 s; enthalten die neun vorherigen Fälle, zusätzliche Formularausgabe für den Entwurf und einen weiteren ID-Zuordnungstest |
|
||||
| Ausgeführte Formularlogik: `node /tmp/wawi-fourth-dom.cjs` | **21 Einzelprüfungen bestanden, 0 fehlgeschlagen** |
|
||||
|
||||
Die zusätzlichen PHP-Diagnosen liefen mit `vendor/bin/phpunit -c phpunit.xml --do-not-cache-result` und ausschließlich SQLite `:memory:`. Die DOM-Prüfungen verwenden aktuell über Laravel-Testrequests erzeugtes HTML und Rezeptur-JSON, vorhandenes jsdom/jQuery sowie Select2 für den Einkaufsablauf. Die Erwartung für den leeren Snapshot wurde auf die jetzt ausdrücklich ausgegebene Meldung angepasst; die Fälle für Mengenänderung, Rückkehr und Entwurf wurden zusätzlich ausgeführt. Temporäre Prüfprogramme und Formularartefakte liegen außerhalb des Repositories unter `/tmp`.
|
||||
|
||||
PHPUnit meldet weiterhin lediglich die bekannte Deprecation zum XML-Konfigurationsschema; die zusätzlichen Tests haben keine fachlichen Fehlschläge. Die beiden PHP-Diagnoseläufe überlappen und sind deshalb nicht als 19 unterschiedliche Testfälle zu zählen.
|
||||
|
||||
### 7.3 Verbleibende Abnahmegrenzen
|
||||
|
||||
Die Bestätigung gilt für die beschriebenen Code-, HTTP-, Datenbank- und Formularabläufe. Sie bestätigt keinen bereits erfolgten Live-Rollout. Unverändert offen bleiben MySQL-Parallelitätsnachweis, reale Browser-/iPad-Abnahme, Anwendung der drei Migrationen auf den vorgesehenen Entwicklungs-/Live-Datenbanken, Connector-Rollout 1.6.0 und die gesondert nachgelagerten Pakete. Diese Punkte sind von den nun bestätigten Fehlerkorrekturen getrennt zu führen.
|
||||
|
|
|
|||
|
|
@ -2,7 +2,7 @@
|
|||
|
||||
> **Stand:** 09.09.2026
|
||||
> **Status:** Modul ist **live** und in der **Kundentestphase** (Dev-Container mit eingespielter Live-Datenbank inkl. Testdaten).
|
||||
> **Testsuite:** 370 Tests grün (1.506 Assertions, 0 Failures).
|
||||
> **Testsuite:** 371 Tests grün (1.512 Assertions, 0 Failures).
|
||||
|
||||
Dieses Dokument ist der Einstieg in den Ordner. Es skizziert den Gesamtstand,
|
||||
listet die erledigten und die noch offenen Arbeitspakete und verweist auf die
|
||||
|
|
@ -101,7 +101,7 @@ WooCommerce-Ampel ist voraussichtlich **kein** Partnercenter-Fehler, sondern
|
|||
shopseitig (Status-Cron/Auth). Rohfassung des Feedbacks:
|
||||
[entwicklungsplan-aktualisiert-2026-08-26.md](01-entwicklungsplaene/entwicklungsplan-aktualisiert-2026-08-26.md).
|
||||
Offene Klärungspunkte (D-1…D-6) im Plan blockieren den Start einzelner Pakete.
|
||||
**Stand 08.09.2026:** AP-34…AP-51 sind umgesetzt (siehe Umsetzungsstand in V7.0).
|
||||
**Stand 09.09.2026:** AP-34…AP-51 sind umgesetzt (siehe Umsetzungsstand in V7.0); Kundennachtrag **AP-52** (Verpackungs-Vorschau in der Produktion mit Bestand, Differenz und „Lagerort", einheitlich mit der INCI-Tabelle) ebenfalls umgesetzt.
|
||||
|
||||
## Umsetzungsprüfung 08.09.2026 → Nacharbeit
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue