call smb_cutime on the correct url to actually set mtime properly
ClosedPublic

Authored by sitter on Wed, Feb 12, 3:38 PM.

Details

Summary

since the introduction of partial resume dstUrl would can be the .part url
whereas dstOrigUrl is the final destination url.

i.e.
we called smbc_utime on smb://foo/bar.part rather than smb://foo/bar

BUG: 356651
FIXED-IN: 19.12.3

Test Plan

moving file to smb retains mtime

Diff Detail

Repository
R320 KIO Extras
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sitter created this revision.Wed, Feb 12, 3:38 PM
Restricted Application added projects: Dolphin, Frameworks. · View Herald TranscriptWed, Feb 12, 3:38 PM
Restricted Application added subscribers: kfm-devel, kde-frameworks-devel. · View Herald Transcript
sitter requested review of this revision.Wed, Feb 12, 3:38 PM
ngraham accepted this revision.Wed, Feb 12, 4:42 PM

It works! Fantastic!

This revision is now accepted and ready to land.Wed, Feb 12, 4:42 PM
This revision was automatically updated to reflect the committed changes.