Vasyka/autocrm
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(migration): case-insensitive dedupe on units backfill + use insertOrIgnore

Browse Source 
MariaDB default collation is case-insensitive on VARCHAR unique indexes,
so 'L' and 'l' clash on units_company_id_code_unique. When a tenant had
'L' in parts.unit (uppercase) and the migration seeds 'l' (lowercase),
the extras loop tried to re-insert 'L' and errored out — blocking all
subsequent migrations (including the new injector_protocols tables).

Fix: lowercase the comparison + switch the extras insert to
insertOrIgnore so a residual race is a no-op.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Vasyka
2026-07-26 11:39:34 +00:00
parent 47eb4f62c1
commit f484167b51
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
database/migrations/2026_07_16_000001_create_units_and_link_unit_id.php
+6 -3
View File
@@ -81,11 +81,14 @@ return new class extends Migration
} }
$extraCodes = $extraCodes->unique(); $extraCodes = $extraCodes->unique();
$existingCodes = DB::table('units')->where('company_id', $companyId)->pluck('code')->all(); // Case-insensitive dedupe: MariaDB default collation treats 'L' == 'l'
// as the same value on the unique index, so we compare lowercased.
$existingCodes = DB::table('units')->where('company_id', $companyId)
->pluck('code')->map(fn ($c) => mb_strtolower((string) $c))->all();
foreach ($extraCodes as $code) { foreach ($extraCodes as $code) {
if (in_array($code, $existingCodes, true)) continue; if (in_array(mb_strtolower($code), $existingCodes, true)) continue;
if (mb_strlen($code) > 16) continue; if (mb_strlen($code) > 16) continue;
DB::table('units')->insert([ DB::table('units')->insertOrIgnore([
'company_id' => $companyId, 'company_id' => $companyId,
'code' => $code, 'code' => $code,
'name_ro' => $code, 'name_ro' => $code,