From a959cf850b56ab7b51bb948957c30aca32c881f5 Mon Sep 17 00:00:00 2001 From: Paul Derscheid Date: Thu, 12 Sep 2024 15:19:55 +0200 Subject: [PATCH] Bug 37618: (follow-up) Inline single use function containsAny and remove definition --- .../intranet-tmpl/prog/js/modals/place_booking.js | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/js/modals/place_booking.js b/koha-tmpl/intranet-tmpl/prog/js/modals/place_booking.js index 8e348f73480..4ccff2a2b08 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/modals/place_booking.js +++ b/koha-tmpl/intranet-tmpl/prog/js/modals/place_booking.js @@ -8,15 +8,6 @@ let bookable_items, booking_patron, booking_itemtype_id; -/** - * @param {number[]} integers1 - The first array of integers to check. - * @param {number[]} integers2 - The second array of integers to be checked against. - * @returns {boolean} - Returns true if any element from integers1 is found in integers2, otherwise false. - */ -function containsAny(integers1, integers2) { - return integers1.some(integer => new Set(integers2).has(integer)); -} - $("#placeBookingModal").on("show.bs.modal", function (e) { // Get context let button = $(e.relatedTarget); @@ -189,7 +180,9 @@ $("#placeBookingModal").on("show.bs.modal", function (e) { $pickupSelect.empty(); const filtered_pickup_locations = response.filter(({ pickup_items }) => - containsAny(pickup_items, bookableItemnumbers) + pickup_items.some(pickup_item => + new Set(bookableItemnumbers).has(pickup_item) + ) ); $.each(filtered_pickup_locations, function (index, pickup_location) { let option = $( -- 2.47.0