-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/106636/
-----------------------------------------------------------
(Updated Oct. 2, 2012, 4:08 a.m.)
Review request for kdelibs and David Faure.
Changes
-------
Improved version of the patch based on David's feedback. Mainly avoid sending
requests back to the server while directory listing is still going on.
Description
-------
The attached patch fixes a regression caused by a commit to fix bug# 88575.
Namely, it fixed a problem where filenames with whitespaces in them were not
handled correctly by kio_ftp. That is because the filenames were automatically
trimmed when read from the directory. However, the fix then re-introduced the
original bug and the reason why names were automatically trimmed in the first
place. Some ftp servers add bogus whitespace between the date and filename in
their listings. Hence, we need need to fix both of these opposing issues
without breaking the other. This patch tries to do just that by actually
validating each name entry that starts with a whitespace. That way the correct
name is sent to the client.
This addresses bug 300988.
http://bugs.kde.org/show_bug.cgi?id=300988
Diffs (updated)
-----
kioslave/ftp/ftp.h 2465a4b
kioslave/ftp/ftp.cpp 26be283
Diff: http://git.reviewboard.kde.org/r/106636/diff/
Testing
-------
Thanks,
Dawit Alemayehu