spouliot/PowderCoatingLogix
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix customer import: normalize blank email to null, not empty string

Browse Source 
The UNIQUE index on (CompanyId, Email) uses HasFilter([Email] IS NOT NULL),
so NULL allows multiple rows but empty string '' does not — every blank-email
customer after the first was hitting a duplicate-key violation at save time.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Scott Pouliot
2026-05-27 13:21:56 -04:00
parent be89327c01
commit 3cee1307fc
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/PowderCoating.Infrastructure/Services/CsvImportService.cs
+6 -1
View File
@@ -480,7 +480,12 @@ public class CsvImportService : ICsvImportService
{ {
// Strip any literal quote characters that QB/Excel may wrap around field values // Strip any literal quote characters that QB/Excel may wrap around field values
var cleanCompanyName = StripQuotes(record.CompanyName); var cleanCompanyName = StripQuotes(record.CompanyName);
var cleanEmail = StripQuotes(record.Email); // Normalise to null (not empty string) — the UNIQUE index on (CompanyId, Email)
// uses HasFilter("[Email] IS NOT NULL"), so NULL is allowed for multiple rows
// but "" (empty string) is not NULL and would cause a unique-constraint violation
// on the second blank-email customer saved.
var rawEmail = StripQuotes(record.Email);
var cleanEmail = string.IsNullOrWhiteSpace(rawEmail) ? null : rawEmail;
var firstName = StripQuotes(record.ContactFirstName)?.Trim(); var firstName = StripQuotes(record.ContactFirstName)?.Trim();
var lastName = StripQuotes(record.ContactLastName)?.Trim(); var lastName = StripQuotes(record.ContactLastName)?.Trim();