From 9fce2434666505460030db1df3869c87c1ef83a7 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Wed, 2 May 2018 14:50:32 +0200 Subject: [PATCH] Bug 20695: Fix multiple upload result in tools/upload script MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Instead of calling UploadedFiles->find only once with $id, we should extract all comma separated id's from the parameter. Test plan: [1] Before this patch, upload two files at once in Tools/Upload. After doing so, you should see only one result reported. But both files should have been uploaded. [2] With this patch, repeat step 1 and see two upload results. Signed-off-by: Marcel de Rooy Patch applies and functions as described. Signed-off-by: Dilan Johnpullé Signed-off-by: Katrin Fischer Signed-off-by: Jonathan Druart --- tools/upload.pl | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/tools/upload.pl b/tools/upload.pl index dafb8dd39f..9b10b4226c 100755 --- a/tools/upload.pl +++ b/tools/upload.pl @@ -57,10 +57,14 @@ if ( $op eq 'new' ) { } elsif ( $op eq 'search' ) { my $uploads; - if( $id ) { - my $rec = Koha::UploadedFiles->find( $id ); - undef $rec if $rec && $plugin && !$rec->public; - push @$uploads, $rec->unblessed if $rec; + if( $id ) { # might be a comma separated list + my @id = split /,/, $id; + foreach my $recid (@id) { + my $rec = Koha::UploadedFiles->find( $recid ); + push @$uploads, $rec->unblessed + if $rec && ( $rec->public || !$plugin ); + # Do not show private uploads in the plugin mode (:editor) + } } else { $uploads = Koha::UploadedFiles->search_term({ term => $term, -- 2.39.5