Hacker-Software/opencrm
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Kunde-E-Mail: Pflicht nur beim Anlegen, nicht beim Bearbeiten

Browse Source 
Bestandskunden ohne E-Mail bleiben editierbar. Beim Anlegen ist E-Mail
weiterhin Pflicht (Frontend required + Backend createCustomer). Die
Domain-Pruefung (keine verwaltete Provider-Domain) greift unveraendert bei
create UND update, falls eine E-Mail gesetzt wird.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
duffyduck
2026-08-18 14:26:03 +02:00
co-authored by Claude Opus 4.8
parent 15142676ef
commit 8964820542
3 changed files with 8 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
backend/src/controllers/customer.controller.ts
+4 -7
View File
@@ -181,11 +181,9 @@ export async function updateCustomer(req: Request, res: Response): Promise<void>
return;
}
const data: any = pickCustomerUpdate(req.body);
// E-Mail ist Pflichtfeld – darf per Update nicht geleert werden.
if ('email' in data && (!data.email || !String(data.email).trim())) {
res.status(400).json({ success: false, error: 'E-Mail-Adresse ist ein Pflichtfeld.' } as ApiResponse);
return;
}
// E-Mail ist nur beim ANLEGEN Pflicht – beim Bearbeiten darf ein
// Bestandskunde ohne E-Mail bleiben (die Domain-Prüfung oben greift aber
// weiterhin, falls doch eine gesetzt wird).
// Pentest 60.3 (MEDIUM, 2026-06-01): pickCustomerUpdate macht nur
// stripHtml; CRLF und andere Control-Chars überlebten. Phone/Mobile
@@ -214,8 +212,7 @@ export async function updateCustomer(req: Request, res: Response): Promise<void>
data.birthDate = new Date(data.birthDate);
}
// Leere Strings in optionalen Feldern zu null konvertieren
// 'email' NICHT nullable – ist Pflichtfeld (leer wird oben schon abgelehnt).
const nullableFields = ['salutation', 'birthPlace', 'phone', 'mobile', 'companyName', 'taxNumber', 'businessRegistration', 'commercialRegister', 'commercialRegisterNumber', 'notes'];
const nullableFields = ['salutation', 'birthPlace', 'phone', 'mobile', 'email', 'companyName', 'taxNumber', 'businessRegistration', 'commercialRegister', 'commercialRegisterNumber', 'notes'];
for (const field of nullableFields) {
if (data[field] === '') data[field] = null;
}