https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42992
--- Comment #2 from David Nind <[email protected]> --- Created attachment 201501 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201501&action=edit Bug 42992: FTP file transport: fix list_files() corrupting filenames Koha::File::Transport::FTP::_list_files parsed the LIST / "ls -l" output with a non-greedy date capture that only consumed the first of the three date tokens (the month), leaving the day and time glued to the front of the filename. Every filename returned for an FTP transport was therefore prefixed with the listing date, e.g. -rw-r--r-- 1 user group 1234 Jan 01 12:00 QUOTES_413514.CEQ => filename "01 12:00 QUOTES_413514.CEQ" Callers (Koha::Edifact::Transport, marc_ordering_process.pl) then asked the server for a file that does not exist, so no files were downloaded, renamed or deleted over FTP. SFTP and Local transports were unaffected. This regressed when the EDI transport was moved onto the file_transports system (bug 38489). The previous Koha::Edifact::Transport::ftp_download listed files with Net::FTP->ls (NLST, bare names); the refactor switched to Net::FTP->dir (LIST) so it could return the richer hashref shape used by the SFTP and Local transports, but the parse was faulty. The date field in "ls -l" output is a fixed run of three whitespace separated tokens (Mon DD HH:MM or Mon DD YYYY). Consume it explicitly so the filename is captured cleanly, preserving spaces and still skipping directories. The LIST-based approach and the filename/longname/size/perms return shape are kept so the structure stays consistent across transport classes (relied on by consumers, including out-of-tree ones). Test plan: 1) prove t/db_dependent/Koha/File/Transport/FTP.t 2) git checkout Koha/File/Transport/FTP.pm and re-run: the list_files() subtest fails on the filename/size/perms assertions 3) prove t/db_dependent/Koha/Edifact/Transport.t Signed-off-by: David Nind <[email protected]> -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://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/
