https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22508
--- Comment #34 from Jonathan Druart <[email protected]> --- (In reply to Marcel de Rooy from comment #29) > +sub local_public_path { > + my ($self) = @_; > + my $upload_public_path = C4::Context->config('upload_public_path'); > + return unless $upload_public_path; > + $upload_public_path =~ s|/$||; > + my $filepath = "$upload_public_path/" . $self->hashvalue; > + return $filepath; > +} > > Compare with full_path sub: > my $path = File::Spec->catfile( > $self->permanent > ? $self->permanent_directory > : C4::Context->temporary_directory, > $self->dir, > $self->hashvalue. '_'. $self->filename, > ); > > You do not use the filename only the hashvalue? Yes, see commit message: * FIXME: in upload_public_path the files are not suffixed by the filename as it is for upload_path. We certainly want to be consistent here. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
