Syco/YuGiOh-Database-API
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix duplicate image_id crash: deduplicate within batch, restore ON DUPLICATE KEY UPDATE
ci/woodpecker/push/woodpecker Pipeline was successful
Details

Browse Source
This commit is contained in:
Syco21
2026-05-15 22:05:59 +02:00
parent 382d0ffd43
commit 8fe83f41e3
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/controllers/importController.js
+4 -1
View File
@@ -92,6 +92,7 @@ async function importCardsInternal() {
const cardValues = [], cardParams = []; const cardValues = [], cardParams = [];
const imageValues = [], imageParams = []; const imageValues = [], imageParams = [];
const csrValues = [], csrParams = []; const csrValues = [], csrParams = [];
const batchImageIds = new Set();
for (const card of batch) { for (const card of batch) {
const fp = cardFingerprint(card); const fp = cardFingerprint(card);
@@ -107,7 +108,8 @@ async function importCardsInternal() {
if (Array.isArray(card.card_images)) { if (Array.isArray(card.card_images)) {
for (const img of card.card_images) { for (const img of card.card_images) {
if (!existingImageIds.has(img.id)) { if (!existingImageIds.has(img.id) && !batchImageIds.has(img.id)) {
batchImageIds.add(img.id);
imageValues.push('(?, ?, ?)'); imageValues.push('(?, ?, ?)');
imageParams.push(img.id, card.id, img.image_url); imageParams.push(img.id, card.id, img.image_url);
addedImages++; addedImages++;
@@ -151,6 +153,7 @@ async function importCardsInternal() {
if (imageValues.length) { if (imageValues.length) {
await conn.execute(` await conn.execute(`
INSERT INTO card_images (image_id, card_id, image_url) VALUES ${imageValues.join(', ')} INSERT INTO card_images (image_id, card_id, image_url) VALUES ${imageValues.join(', ')}
ON DUPLICATE KEY UPDATE image_url = VALUES(image_url)
`, imageParams); `, imageParams);
} }