@@ -, +, @@ return with explicit undef removed Follow-up for PBP fixes follow-up fix POD syntax follow-up jquery has moved --- C4/UploadedFiles.pm | 2 +- cataloguing/value_builder/upload.pl | 2 +- .../prog/en/modules/cataloguing/value_builder/upload.tt | 2 +- opac/opac-retrieve-file.pl | 4 ++-- 4 files changed, 5 insertions(+), 5 deletions(-) --- a/C4/UploadedFiles.pm +++ a/C4/UploadedFiles.pm @@ -180,7 +180,7 @@ sub UploadFile { return $id; } - return undef; + return; } =head2 DelUploadedFile --- a/cataloguing/value_builder/upload.pl +++ a/cataloguing/value_builder/upload.pl @@ -140,7 +140,7 @@ sub finddirs { my $base = shift || $upload_path; my $found = 0; my @dirs; - my @files = <$base/*>; + my @files = glob("$base/*"); foreach (@files) { if (-d $_ and -w $_) { my $lastdirname = basename($_); --- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/value_builder/upload.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/value_builder/upload.tt @@ -4,7 +4,7 @@ Upload plugin - + --- a/opac/opac-retrieve-file.pl +++ a/opac/opac-retrieve-file.pl @@ -32,12 +32,12 @@ exit 1 if not $file; my $file_path = $file->{filepath}; if( -f $file_path ) { - open FH, '<', $file_path or die "Can't open file: $!"; + open my $fh, '<', $file_path or die "Can't open file: $!"; print $input->header( -type => "application/octet-stream", -attachment => $file->{filename} ); - while() { + while(<$fh>) { print $_; } } else { --