From 8fe43d68020138630b007beceef6cc43eb004545 Mon Sep 17 00:00:00 2001
From: Moirtz Wagner <m0w1337@gmail.com>
Date: Thu, 17 Sep 2026 11:27:27 +0200
Subject: [PATCH] Rechnungen: Anrede beim Bearbeiten, lokale Testumgebung, BOMs

Beim Test der Oberflaeche im Browser gefunden:

- addrechnung.tpl fragte die gespeicherte Anrede als $rechnung.anr1 ab
  (die Daten heissen anrede1) und gab sie ohne Klammern aus. Beim
  Bearbeiten von Angeboten/Briefen erschien deshalb immer die
  Standard-Anrede, eine geaenderte ging beim erneuten Speichern verloren.
- |escape innerhalb von {if}-Bedingungen wieder entfernt (dort ohne
  Wirkung, gehoert nicht dorthin).
- UTF-8-BOM aus sechs Dateien entfernt; makepdf.php gab es auf jeder
  Rechnungsseite vor dem HTML aus.
- makeImageFromPDF() loeschte bei der Aufraeumrunde auch .gitkeep;
  Pfade fuer convert per escapeshellarg().
- constants.php: Instanzname auch unter Windows (getcwd() mit
  Backslashes) korrekt; KV_CONFIG_PATH als Config-Pfad fuer lokale Tests.

Im Browser getestet (lokaler PHP-Webserver, migrierte Testdatenbank):
Login mit Sonderzeichen-Passwort, Entwurf automatisch speichern und
laden, Rechnung mit Umlauten/Anfuehrungszeichen/HTML im Artikelnamen und
Komma-Betraegen speichern, Anzeige als PDF (Original und Kopie),
Kundensuche nach Dokumenten mit Umlaut, Artikelnamen escaped, Storno,
Kopie auf andere Adresse, Umschlag, Angebot mit eigener Anrede anlegen und
wieder bearbeiten.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 CLAUDE.md                  |  2 ++
 ajax/searchterm.php        |  2 +-
 incs/constants.php         | 12 ++++++++----
 incs/makepdf.php           |  8 ++++----
 incs/users.php             |  2 +-
 sites/stexp.php            |  2 +-
 templates/addrechnung.tpl  | 38 +++++++++++++++++++-------------------
 templates/mailrechnung.tpl | 12 ++++++------
 templates/termlist.tpl     |  2 +-
 9 files changed, 43 insertions(+), 37 deletions(-)

diff --git a/CLAUDE.md b/CLAUDE.md
index 969f70a..888c69e 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -16,6 +16,8 @@ for f in $(git ls-files "*.php" | grep -vE "^(incs/(tcpdf|pChart|PHPMailer|WebAu
 
 Zum Ausführen braucht es einen Webserver mit PHP, eine MariaDB/MySQL-Instanz und die externe Konfigurationsdatei (siehe unten). Ohne die Konfiguration springt `index.php` in den Setup-Modus (`update/firststart.php`). **Nur** dann: Das Setup läuft ohne Login, schreibt die Config und kann die Datenbank leeren. Deshalb wird `action=setup` bei gültiger Config verworfen, und `firststart.php` und `rebuild-settings.php` brechen selbst ab. Werte, die in die Config geschrieben werden, immer über `var_export()` maskieren – die Config ist ausführbarer PHP-Code.
 
+**Lokal testen** (auch unter Windows): Die Umgebungsvariable `KV_CONFIG_PATH` ersetzt `CONFIG_PATH`. Die Config-Datei heißt nach dem Verzeichnisnamen des Checkouts, also z.B. `Kundenverwaltung_settings.conf`; `$lokale_netze` muss `127.0.0.1`/`::1` enthalten, sonst gibt es kein Passwortfeld. Start im Repo-Verzeichnis mit `php -S 127.0.0.1:8765`. Die PDF-Vorschau als Bild (`makeImageFromPDF()`) braucht ImageMagick unter `/opt/bin/convert` und funktioniert lokal nicht.
+
 ## Die zwei zentralen Mechanismen
 
 ### 1. Das Verzeichnis bestimmt die Instanz
diff --git a/ajax/searchterm.php b/ajax/searchterm.php
index 0169b09..5f3ebe2 100644
--- a/ajax/searchterm.php
+++ b/ajax/searchterm.php
@@ -1,4 +1,4 @@
-<?php 
+<?php 
 require_once("../incs/connectmysql.php");
 require_once("../incs/funcs.php");
 require_once("../incs/listfuncs.php");
diff --git a/incs/constants.php b/incs/constants.php
index 41f92ca..f6d62ab 100644
--- a/incs/constants.php
+++ b/incs/constants.php
@@ -1,15 +1,19 @@
 <?php 
-$dirs = explode("/",getcwd());
+// Unter Windows liefert getcwd() Backslashes
+$dirs = preg_split('#[/\\\\]#', getcwd());
 
 $dir = $dirs[count($dirs)-1];
 if($dir == "ajax")
 	$dir = $dirs[count($dirs)-2];
 else if($dir == "auto")
 	$dir = $dirs[count($dirs)-3];
-	
-if(DIRECTORY_SEPARATOR == "/")
+
+// KV_CONFIG_PATH erlaubt einen anderen Ort, etwa für eine lokale Testumgebung
+if(getenv("KV_CONFIG_PATH"))
+	define("CONFIG_PATH", rtrim(getenv("KV_CONFIG_PATH"), "/\\")."/");
+elseif(DIRECTORY_SEPARATOR == "/")
 	define("CONFIG_PATH", "/volume1/homes/wagner/kundenconfig/");
-else 
+else
 	define("CONFIG_PATH", "c:/kundenverwaltung/config/");
 define("CONFIG_FILE", $dir."_settings.conf");
 
diff --git a/incs/makepdf.php b/incs/makepdf.php
index 47793e0..015ad3d 100644
--- a/incs/makepdf.php
+++ b/incs/makepdf.php
@@ -1,4 +1,4 @@
-<?php
+<?php
 
 require_once('tcpdf/tcpdf.php');
 function makerawPDF($ajax)
@@ -1144,8 +1144,8 @@ function makeImageFromPDF($PDFpath)
 	$handle=opendir($PDFpathpart);
 	while ($file = readdir($handle))
 	{
-		if (preg_match("=^\.{1,2}$=", $file))
-			continue;   # . und .. Dateien ignorieren
+		if (preg_match("=^\.=", $file))
+			continue;   # ., .. und versteckte Dateien (.gitkeep) ignorieren
     
 	    if (is_dir($PDFpathpart.$file))
 	    {
@@ -1164,7 +1164,7 @@ function makeImageFromPDF($PDFpath)
 	}	
 	mkdir(dirname($PDFpath)."/".basename($PDFpath,".pdf")."/");
 	//exec('');
-	exec("export PATH=\"/opt/bin\";/opt/bin/convert $PDFpath ".dirname($PDFpath)."/".basename($PDFpath,".pdf")."/".basename($PDFpath,".pdf").".png");
+	exec("export PATH=\"/opt/bin\";/opt/bin/convert ".escapeshellarg($PDFpath)." ".escapeshellarg(dirname($PDFpath)."/".basename($PDFpath,".pdf")."/".basename($PDFpath,".pdf").".png"));
 	$i="0";
 	$ret = "";
 	if(is_file(dirname($PDFpath)."/".basename($PDFpath,".pdf")."/".basename($PDFpath,".pdf").".png"))
diff --git a/incs/users.php b/incs/users.php
index f6044ac..56a6cc5 100644
--- a/incs/users.php
+++ b/incs/users.php
@@ -1,4 +1,4 @@
-<?php 
+<?php 
 
 function generateSalt() {
 
diff --git a/sites/stexp.php b/sites/stexp.php
index f2b231a..8093c38 100644
--- a/sites/stexp.php
+++ b/sites/stexp.php
@@ -1,4 +1,4 @@
-<?php
+<?php
 require_once ("./incs/rechnung.php");
 $smarty->assign("heading","Steuerexport"); 
 $smarty->display("heading.tpl");
diff --git a/templates/addrechnung.tpl b/templates/addrechnung.tpl
index 11a24b0..8e01b84 100644
--- a/templates/addrechnung.tpl
+++ b/templates/addrechnung.tpl
@@ -1,4 +1,4 @@
-<div class="centersidebyside">
+<div class="centersidebyside">
 <div class="sidebyside">
 {if $rechnung.typ_id && !$newtype}
 	{$type = $rechnung.typ_id}
@@ -114,11 +114,11 @@
 	<p>
 	<br />
 	<label>Anrede:</label>
-	{if $anrs.{$adresse.anrede_id|escape} == "Herrn" || $anrs.{$adresse.anrede_id|escape} == "Herr"}
+	{if $anrs.{$adresse.anrede_id} == "Herrn" || $anrs.{$adresse.anrede_id|escape} == "Herr"}
 		{strip}
 		<input value='
-			{if $rechnung.anr1}
-				$rechnung.anr1
+			{if $rechnung.anrede1}
+				{$rechnung.anrede1|escape}
 			{else}
 				Sehr geehrter Herr{" "} 
 				{if $adresse.titel_id > 1}
@@ -131,11 +131,11 @@
 			{/if}
 			' id='anr1' maxlength='150' name='anr1' /><br />
 		{/strip}
-	{elseif $anrs.{$adresse.anrede2_id|escape} == "Herrn" || $anrs.{$adresse.anrede_id|escape} == "Herr"}
+	{elseif $anrs.{$adresse.anrede2_id} == "Herrn" || $anrs.{$adresse.anrede_id|escape} == "Herr"}
 		{strip}
 		<input value='
-			{if $rechnung.anr1}
-				$rechnung.anr1
+			{if $rechnung.anrede1}
+				{$rechnung.anrede1|escape}
 			{else}
 				Sehr geehrter Herr{" "} 
 				{if $adresse.titel2_id > 1}
@@ -148,11 +148,11 @@
 			{/if}
 			' id='anr1' maxlength='150' name='anr1' /><br />
 		{/strip}
-	{elseif $anrs.{$adresse.anrede_id|escape} == "Frau"}
+	{elseif $anrs.{$adresse.anrede_id} == "Frau"}
 		{strip}
 		<input value='
-			{if $rechnung.anr1}
-				$rechnung.anr1
+			{if $rechnung.anrede1}
+				{$rechnung.anrede1|escape}
 			{else}
 				Sehr geehrte Frau{" "} 
 				{if $adresse.titel_id > 1}
@@ -165,11 +165,11 @@
 			{/if}
 				' id='anr1' maxlength='150' name='anr1' /><br />
 		{/strip}
-	{elseif $anrs.{$adresse.anrede2_id|escape} == "Frau"}
+	{elseif $anrs.{$adresse.anrede2_id} == "Frau"}
 		{strip}
 		<input value='
-			{if $rechnung.anr1}
-				$rechnung.anr1
+			{if $rechnung.anrede1}
+				{$rechnung.anrede1|escape}
 			{else}
 				Sehr geehrte Frau{" "} 
 				{if $adresse.titel2_id > 1}
@@ -187,11 +187,11 @@
 	{/if}
 	
 	<label>Zweite Anrede:</label>
-	{if $anrs.{$adresse.anrede_id|escape} == "Frau" && $anrs.{$adresse.anrede2_id|escape} == "Herrn"}
+	{if $anrs.{$adresse.anrede_id} == "Frau" && $anrs.{$adresse.anrede2_id|escape} == "Herrn"}
 		{strip}
 		<input value='
-			{if $rechnung.anr2}
-				$rechnung.anr2
+			{if $rechnung.anrede2}
+				{$rechnung.anrede2|escape}
 			{else}
 				Sehr geehrte Frau{" "} 
 				{if $adresse.titel_id > 1}
@@ -204,11 +204,11 @@
 			{/if}
 			' id='anr2' maxlength='150' name='anr2' /><br />
 		{/strip}
-	{elseif $anrs.{$adresse.anrede2_id|escape} == "Frau"}
+	{elseif $anrs.{$adresse.anrede2_id} == "Frau"}
 		{strip}
 		<input value='
-			{if $rechnung.anr2}
-				$rechnung.anr2
+			{if $rechnung.anrede2}
+				{$rechnung.anrede2|escape}
 			{else}
 				Sehr geehrte Frau{" "} 
 				{if $adresse.titel2_id > 1}
diff --git a/templates/mailrechnung.tpl b/templates/mailrechnung.tpl
index 5dba58f..c03b7c6 100644
--- a/templates/mailrechnung.tpl
+++ b/templates/mailrechnung.tpl
@@ -30,7 +30,7 @@
 <label>Betreff:</label>
 <input value='{$smarty.session.firma|escape}: anbei das von Ihnen gewünschte Dokument' id='sub' maxlength='150' name='sub' /><br />
 <label>Anrede:</label>
-{if $anrs.{$adresse.anrede_id|escape} == "Herrn" || $anrs.{$adresse.anrede_id|escape} == "Herr"}
+{if $anrs.{$adresse.anrede_id} == "Herrn" || $anrs.{$adresse.anrede_id|escape} == "Herr"}
 	{strip}
 	<input value='
 		{if $rechnung.anr1}
@@ -47,7 +47,7 @@
 		{/if}
 		' id='anr1' maxlength='150' name='anr1' /><br />
 	{/strip}
-{elseif $anrs.{$adresse.anrede2_id|escape} == "Herrn" || $anrs.{$adresse.anrede_id|escape} == "Herr"}
+{elseif $anrs.{$adresse.anrede2_id} == "Herrn" || $anrs.{$adresse.anrede_id|escape} == "Herr"}
 	{strip}
 	<input value='
 		{if $rechnung.anr1}
@@ -64,7 +64,7 @@
 		{/if}
 		' id='anr1' maxlength='150' name='anr1' /><br />
 	{/strip}
-{elseif $anrs.{$adresse.anrede_id|escape} == "Frau"}
+{elseif $anrs.{$adresse.anrede_id} == "Frau"}
 	{strip}
 	<input value='
 		{if $rechnung.anr1}
@@ -81,7 +81,7 @@
 		{/if}
 			' id='anr1' maxlength='150' name='anr1' /><br />
 	{/strip}
-{elseif $anrs.{$adresse.anrede2_id|escape} == "Frau"}
+{elseif $anrs.{$adresse.anrede2_id} == "Frau"}
 	{strip}
 	<input value='
 		{if $rechnung.anr1}
@@ -103,7 +103,7 @@
 {/if}
 
 <label>Zweite Anrede:</label>
-{if $anrs.{$adresse.anrede_id|escape} == "Frau" &&  $anrs.{$adresse.anrede2_id|escape} == "Herr" && $anrs.{$adresse.anrede2_id|escape} == "Herrn"}
+{if $anrs.{$adresse.anrede_id} == "Frau" &&  $anrs.{$adresse.anrede2_id|escape} == "Herr" && $anrs.{$adresse.anrede2_id|escape} == "Herrn"}
 	{strip}
 	<input value='
 		{if $rechnung.anr2}
@@ -120,7 +120,7 @@
 		{/if}
 		' id='anr2' maxlength='150' name='anr2' /><br />
 	{/strip}
-{elseif $anrs.{$adresse.anrede2_id|escape} == "Frau"}
+{elseif $anrs.{$adresse.anrede2_id} == "Frau"}
 	{strip}
 	<input value='
 		{if $rechnung.anr2}
diff --git a/templates/termlist.tpl b/templates/termlist.tpl
index 5f0baa1..c7b2f1e 100644
--- a/templates/termlist.tpl
+++ b/templates/termlist.tpl
@@ -1,4 +1,4 @@
-<a href='?action=termin'><span class='cal' title="Terminkalender anzeigen" style='margin-left: 50%;'></span></a><br />
+<a href='?action=termin'><span class='cal' title="Terminkalender anzeigen" style='margin-left: 50%;'></span></a><br />
 <div id="tabs">
 {* ANZEIGE PANE *}
 <div class='widthlimiter'>