Sicherheit, PDF-Robustheit, Stueckzahl und Besteuerung ueberarbeitet

Sicherheit:
- debug=True aus app.run() entfernt. Der Werkzeug-Debugger erlaubt
  Codeausfuehrung ueber das Netz und lief bisher im LXC-Container mit.
  Neu ueber FLASK_DEBUG/HOST/PORT steuerbar; mit FLASK_DEBUG=1 bindet
  die App automatisch nur an 127.0.0.1.
- Wirkungsloses --no-reload aus dem systemd-ExecStart entfernt
  (app.run() wertet sys.argv nicht aus).
- /save_config uebernimmt nur noch Keys aus DEFAULT_VALUES.

PDF:
- pdf_text() transliteriert Zeichen, die die latin-1-Kernfonts nicht
  koennen (Gedankenstrich, typografische Anfuehrungszeichen, Euro-Zeichen).
  Vorher warf jeder solche Kundenname einen UnicodeEncodeError -> HTTP 500.
- Legacy-fpdf2-API abgeloest: ln=True -> new_x/new_y, output(dest='S')
  -> output(), Font "Arial" -> "Helvetica". Alle drei waren als
  "will soon be removed" markiert.

Kalkulation:
- Stueckzahl wirkt jetzt in der Hauptansicht. Vorher zeigte "GESAMT"
  den Preis pro Stueck, obwohl die Stueckzahl direkt darueber eingegeben
  wurde. Anzeige jetzt getrennt nach PRO STUECK und AUFTRAG.
- Rundung auf Cent-Basis des Stueckpreises, damit die Rechnungszeile
  nachrechenbar bleibt (Einzelpreis x Menge == Zeilensumme).
- as_stueckzahl() faengt '', '1.5' und 0 ab (vorher ValueError -> 500).
- Gewinnmarge im Ergebnis kommt vom Server; das Label zeigte auf der
  Startseite immer 20 %, weil es ein nicht vorhandenes Feld auslas.

Besteuerung (drei Regime statt eines Bool-Schalters):
- 'privat'           - kein Unternehmer nach §2 UStG: weder MwSt. noch
                       Steuernummer noch §19-Hinweis auf dem Beleg
- 'kleinunternehmer' - §19 UStG mit Pflichthinweis
- 'regel'            - MwSt. mit konfigurierbarem Satz (19 % / 7 %)
Steuernummer, USt-IdNr. und Leistungsdatum (§14 UStG) ergaenzt; sie
erscheinen nur in den beiden Unternehmer-Modi, unabhaengig davon, ob
die Felder in der Config gefuellt sind.

Config-Handling:
- save_config() schreibt atomar (tmp + os.replace + fsync), ein Lock
  serialisiert die Read-modify-write-Zyklen.
- load_config() wirft ConfigError bei beschaedigter YAML, statt still
  auf Defaults zu fallen -- sonst haette der naechste Autosave die
  Datei endgueltig ueberschrieben.
- _migriere() leitet steuerregime aus dem alten mwst_ausweisen ab und
  laeuft vor dem Auffuellen mit Defaults sowie nach jedem Speichern.

Frontend:
- Dark-Mode-Toggle schrieb "Light"/"Dark" als Textknoten in den
  Icon-Button und liess das Icon in der falschen Richtung stehen.
- recalculate() steigt auf Seiten ohne Ergebnisblock aus; auf der
  Einstellungsseite warf bisher jeder Tastendruck eine TypeError.
- Live-Berechnung entprellt (150 ms) statt ein Request pro Tastendruck.

Sonstiges:
- requirements.txt mit gepinnten Versionen; beide Proxmox-Skripte
  nutzen sie statt loser Paketlisten und ohne blindes --upgrade.
- Falsche Port-Ausgabe in proxmox-setup.sh korrigiert (5000 -> 5001).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Erik Thiele
2026-08-22 10:51:45 +02:00
parent c5b52f4c1c
commit b6e5a59daa
10 changed files with 543 additions and 134 deletions

View File

@@ -19,15 +19,18 @@
source .venv/bin/activate
```
**Dependencies**: `flask`, `fpdf2` (imported as `fpdf`), `PyYAML`. Alle in `.venv`.
**Dependencies**: gepinnt in `requirements.txt` (`pip install -r requirements.txt`).
`flask`, `fpdf2` (imported as `fpdf`), `PyYAML`. Alle in `.venv`.
## Run
```bash
python app.py
# -> http://127.0.0.1:5001
python app.py # Debug aus, lauscht auf 0.0.0.0:5001
FLASK_DEBUG=1 python app.py # Debug an, dann nur 127.0.0.1
```
`HOST` und `PORT` ueberschreiben die Defaults.
Kein Test, Lint, Typecheck oder CI vorhanden. README.md und AGENTS.md enthalten die Projektdokumentation.
## Routes
@@ -50,7 +53,26 @@ Kein Test, Lint, Typecheck oder CI vorhanden. README.md und AGENTS.md enthalten
## Wichtige Fallstricke
- `fpdf` kommt via `fpdf2`-Paket, Import ist `from fpdf import FPDF` (alte API).
- `fpdf` kommt via `fpdf2`-Paket, Import ist `from fpdf import FPDF`.
Die Legacy-API (`ln=True`, `output(dest='S')`, Font "Arial") ist abgeloest
durch `new_x/new_y`, `output()` und "Helvetica" -- nicht zurueckbauen.
- PDF-Kernfonts koennen nur latin-1. Jeder Freitext muss durch `pdf_text()`,
sonst wirft die PDF-Route bei Sonderzeichen einen UnicodeEncodeError.
- `load_config()` wirft `ConfigError`, wenn die YAML kaputt ist. Nicht
abfangen und auf Defaults zurueckfallen -- das wuerde die Datei beim
naechsten Autosave endgueltig ueberschreiben.
- `/save_config` uebernimmt nur Keys aus `DEFAULT_VALUES`.
- **Steuerlogik**: `steuerregime` ist die Quelle der Wahrheit und kennt drei
Werte (`STEUERREGIME`): 'privat' | 'kleinunternehmer' | 'regel'.
'privat' ist rechtlich NICHT dasselbe wie 'kleinunternehmer': ein
Privatverkaeufer ist kein Unternehmer nach §2 UStG, daher duerfen weder
§19-Hinweis noch Steuernummer/USt-IdNr. auf den Beleg -- auch dann nicht,
wenn die Felder in der Config gefuellt sind (`calc['ist_unternehmer']`). `mwst_ausweisen` ist davon *abgeleitet*
und wird in `_migriere()` gesetzt -- nie direkt schreiben. `_migriere()`
laeuft in `load_config()` VOR dem Auffuellen mit Defaults (sonst greift
die Ableitung aus Alt-Configs nicht) und in `/save_config` nach dem
Update (sonst steht Widerspruechliches auf der Platte).
- Der MwSt.-Satz kommt aus `mwst_satz`, nicht mehr hartcodiert.
- **Saeamtliche UI-Texte sind Deutsch** — Labels, Fehlermeldungen, Rechnungsinhalte.
- Nur `.venv/`, `druckkalkulation_config.yaml` und `static/logo.*` werden ignoriert.
- `3dDruckKalkulator.py` ist die alte Tkinter-Version und wird nicht mehr verwendet (nicht im Tracking).