shop-old/modules/ADMIN2000_NOTES.md

Admin2000 - Development Notes

Aktuelles Problem: Save funktioniert nicht (2025-10-25)

Problem-Analyse

• Customer Editor lädt korrekt, alle Daten werden angezeigt
• Beim Speichern: Error 500
• Debug-Log zeigt: Crash in Legacy Customer->update() (customer.class.php:1474)
• Keine weiteren Logs nach "Calling customer->save()"

Root Cause

Legacy Customer->update() Methode:

• Baut dynamisch SQL UPDATE mit allen übergebenen Feldern
• Crasht wahrscheinlich bei nicht-existierenden DB-Spalten
• Problem-Felder: pass1, pass2, admin_role_id (0)
• pass1/pass2 sind UI-only Felder (sollten zu pass gemapped werden)
• admin_role_id = 0 sollte gefiltert werden oder leer bleiben
• Kein Error-Handling, kein Logging im Legacy-Code

Debug-Dateien

• /var/www/vhosts/intelectra.de/httpdocs/admin2000_debug.log - Live Debug Output
• Letzter Eintrag: "Calling customer->save()" - danach Crash

---

Lösung: CustomerManager-Klasse (Grüne Wiese)

Konzept

Skalpell-Modus für Legacy:

• Legacy Customer-Klasse bleibt READ-ONLY
• Nur get_data(), get_by_id() etc. nutzen
• KEINE Änderungen am Legacy-Code

Neue Admin2000 CRUD-Logik:

• Eigene CustomerManager-Klasse für Admin2000
• Sauberes Error-Handling + Logging
• UPDATE und CREATE sauber getrennt
• Nur existierende DB-Spalten updaten

Geplante Struktur

/modules/admin2000/
├── admin2000.php (Controller)
└── classes/
    └── CustomerManager.php (neue CRUD-Klasse)
        ├── get($id) - liest aus Legacy
        ├── update($id, $data) - eigene UPDATE-Logik
        ├── create($data) - eigene CREATE-Logik
        ├── validate($data) - Eingabe-Validierung
        └── mapFormToDb($form_data) - Feld-Mapping

CustomerManager Features

1. Feld-Mapping:

   • pass1 + pass2 → pass (md5 hash)
   • admin_role_id nur wenn > 0
   • Leere Strings bei Datums-Feldern filtern ("0000-00-00")

2. Whitelist für DB-Felder:

   • Nur Spalten updaten die wirklich existieren
   • Schützt vor SQL Errors

3. Error-Handling:

   • Try-Catch um SQL-Queries
   • Logging in eigene Datei
   • Sinnvolle Fehlermeldungen

4. Validierung:

   • E-Mail Format
   • Required Fields
   • Duplikat-Check

---

Nächste Schritte

Phase 1: CustomerManager erstellen

• modules/admin2000/classes/CustomerManager.php erstellen
• Database-Spalten-Whitelist definieren
• update() Methode mit Error-Handling
• Feld-Mapping implementieren (pass1/pass2 → pass)

Phase 2: Integration

• In admin2000.php CustomerManager statt Legacy Customer nutzen
• save_customer() umbauen auf CustomerManager->update()
• Debug-Logging entfernen wenn fertig

Phase 3: Testing

• Save testen mit verschiedenen Feldkombinationen
• Passwort-Änderung testen
• Leere Felder testen
• Admin-Role-Änderung testen

---

DB-Spalten (customers Tabelle)

Existierende Spalten (aus get_data() bekannt):

• id, number, locked, group_id
• honorific, firstname, surname
• phone, mobile, fax, email, website
• show_tax, burth_date
• company, tax_number, tax_id, eu_business
• account_holder, account_number, bank, bank_number, iban, swift_bic
• direct_debit_active, rechnung_active, skonto
• freetextfield1-6, has_item_export, newsletter_sub, info
• pass (md5 hash)
• admin_role_id (vermutlich)

UI-only Felder (nicht in DB):

• pass1, pass2 (werden zu pass gemapped)
• customer_password_action (Dropdown für Passwort-Aktionen)

---

Lessons Learned

1. Legacy = READ ONLY:

   • Nie wieder Legacy CRUD-Methoden nutzen ohne Analyse
   • Nur GET-Methoden sind safe
   • UPDATE/CREATE = grüne Wiese

2. Debugging:

   • file_put_contents() zu Debug-Datei funktioniert
   • error_log() erscheint nicht in Plesk PHP-FPM Logs
   • Browser DevTools zeigt nur "500" - kein Stack Trace

3. Skalpell-Modus:

   • "Ich habe gedacht" = VERBOTEN
   • Nur Fakten aus Legacy-Code nutzen
   • Keine Annahmen, keine "Optimierungen"

---

Kontakt / Fragen

Bei Fragen zu Admin2000:

• Debug-Log: admin2000_debug.log im httpdocs root
• Diese Datei: /modules/ADMIN2000_NOTES.md