Review Request 126659: [kio_ftp] fix display of file/directory modification time/date

[Wolfgang Bauer]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126659/
-----------------------------------------------------------

(Updated Jan. 7, 2016, 1:23 nachm.)


Review request for KDE Frameworks, kdelibs and David Faure.


Changes
-------

update description as well


Bugs: 354597
    https://bugs.kde.org/show_bug.cgi?id=354597


Repository: kio


Description (updated)
-------

- QDate() treats the year literally (i.e. 90 is really year 90, not 1990), so subtracting 1900 is wrong.
- Use QDate::currentDate() instead of QDateTime::currentDateTime(), we only need the current date anyway
- Initialize day, month, and year to the current date instead of 0. In the case when no year is mentioned in the server's reply (the year is implicit), it wasn't set to the current year at all, so the result was either 0 or -1.


Diffs
-----

  src/ioslaves/ftp/ftp.cpp 2179179 

Diff: https://git.reviewboard.kde.org/r/126659/diff/


Testing
-------

Connected to an FTP server with dolphin (15.12.0). The modification times/dates are now shown correctly.


Thanks,

Wolfgang Bauer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20160107/57a2a86f/attachment.htm>
-------------- next part --------------
_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel at kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel