From 671657d78bbebd879fb419ade80e7c4c3fb8e2ad Mon Sep 17 00:00:00 2001 From: plegall Date: Thu, 31 Jul 2025 23:56:40 +0200 Subject: [PATCH] fixes #2396 avoid duplicate image_id in link to manage uploaded photos --- admin/photos_add_direct.php | 2 +- admin/themes/default/js/photos_add_direct.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/admin/photos_add_direct.php b/admin/photos_add_direct.php index ea6cdd2f2..21fc3e372 100644 --- a/admin/photos_add_direct.php +++ b/admin/photos_add_direct.php @@ -26,7 +26,7 @@ DELETE FROM '.CADDIE_TABLE.' pwg_query($query); $inserts = array(); - foreach (explode(',', $_GET['batch']) as $image_id) + foreach (array_unique(explode(',', $_GET['batch'])) as $image_id) { $inserts[] = array( 'user_id' => $user['id'], diff --git a/admin/themes/default/js/photos_add_direct.js b/admin/themes/default/js/photos_add_direct.js index 4d361f5d7..f46a033e2 100644 --- a/admin/themes/default/js/photos_add_direct.js +++ b/admin/themes/default/js/photos_add_direct.js @@ -479,7 +479,7 @@ $(function () { // pwg.caddie.add(uploadedPhotos) instead of relying on huge GET parameter // (and remove useless code from admin/photos_add_direct.php) - $(".batchLink").attr("href", "admin.php?page=photos_add§ion=direct&batch=" + uploadedPhotos.join(",")); + $(".batchLink").attr("href", "admin.php?page=photos_add§ion=direct&batch=" + [...new Set(uploadedPhotos)].join(",")); $(".batchLink").html(sprintf(batch_Label, uploadedPhotos.length)); $(".afterUploadActions").show();