From c48d201dc22c4f2bb8d0b3a9ab81706c49a213b7 Mon Sep 17 00:00:00 2001 From: plegall Date: Thu, 31 Jul 2025 23:57:21 +0200 Subject: [PATCH] (cp 671657d78) 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 e66fbbbc7..c19797990 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 975052ff8..5991166c4 100644 --- a/admin/themes/default/js/photos_add_direct.js +++ b/admin/themes/default/js/photos_add_direct.js @@ -345,7 +345,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();