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

Wolfgang Bauer <wba...@tmo.at> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
      Latest Commit|                            |https://commits.kde.org/kio
                   |                            |/5e791ef216c230efc15b853c08
                   |                            |b21d41eca65e0c
         Resolution|---                         |FIXED
   Version Fixed In|                            |5.60.0

--- Comment #8 from Wolfgang Bauer <wba...@tmo.at> ---
Git commit 5e791ef216c230efc15b853c08b21d41eca65e0c by Wolfgang Bauer.
Committed on 21/06/2019 at 13:30.
Pushed by wbauer into branch 'master'.

[copyjob] Only set modification time if the kio-slave provided it

If the slave didn't pass a modification time (e.g. the http slave
doesn't), it was set to -1, resulting in setting a wrong modification
time for the destination file in copyNextFile() later on because that
case wasn't checked.

So only set info.mtime when the slave actually provided a value.

There's no need for further checks later in copyNextFile() (where
FileCopyJob::setModificationTime() is called) because FileCopyJob checks
for validity anyway.
FIXED-IN: 5.60.0
Differential Revision: https://phabricator.kde.org/D21955

M  +4    -1    src/core/copyjob.cpp

https://commits.kde.org/kio/5e791ef216c230efc15b853c08b21d41eca65e0c

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to