https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41020
Martin Renvoize (ashimema) <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #188147|0 |1 is obsolete| | --- Comment #33 from Martin Renvoize (ashimema) <[email protected]> --- Created attachment 188154 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188154&action=edit Bug 41020: Standardize _abort_operation() signature in FTP transport This patch updates Koha::File::Transport::FTP::_abort_operation() to accept an optional $path parameter, matching the signature used in Koha::File::Transport::SFTP. This improves error reporting by including the path/file that caused the operation to fail in the error message payload. Previously, the FTP implementation only included the error detail and message, making it harder to debug which file or directory caused the problem. Changes: - Updated _abort_operation($operation) to _abort_operation($operation, $path) - Updated change_directory() call to pass $remote_directory to _abort_operation - Added path to error payload, using either the passed path or pwd() as fallback This standardization ensures consistent error reporting across all transport implementations and makes debugging transport issues easier. Test plan: 1. Verify FTP transport operations still work correctly 2. Check that error messages now include the path that failed 3. Confirm no regressions in existing FTP functionality Signed-off-by: Martin Renvoize <[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/
