Review Request 127154: GUI option to configure global MarkPartial for KIO slaves

[David Faure]

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



It seems weird to have a setting for FTP on one side, and a setting for "SMB and SFTP and any slave where this is implemented in the future".
I understand that it was the path of least code changes, but I fear this creates an inconsistency which will lead to more bug reports down the line.
Either we should just have the global option, or it should be per protocol.

Can you tell me more about your use case for this option, so we can understand better the reason for all this?

In general I find that .part is good, it 1) tells me that the file isn't ready to be used yet, 2) enables the resuming feature in case the connection drops.

I can very well imagine that some servers disable renaming and then the feature breaks the upload in the first place. Is this is the issue here?
But then it's not even per-protocol, it's per-protocol-and-host.... which might lead to an overblown configuration dialog. But the alternative is for the user to turn off some global setting (like the one you just added) and then (forget to) turn it back on. Right? :-)

- David Faure


On Feb. 24, 2016, 11:52 p.m., Martin Kostolný wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127154/
> -----------------------------------------------------------
> 
> (Updated Feb. 24, 2016, 11:52 p.m.)
> 
> 
> Review request for KDE Frameworks, David Edmundson, David Faure, and Marco Martin.
> 
> 
> Repository: kio
> 
> 
> Description
> -------
> 
> We have an option called "Mark partially uploaded files" in System Settings -> Network: Settings -> Connection Preferences -> FTP Options. This one toggles adding ".part" extension to partially transferred files through FTP protocol.
> 
> Other KIO slaves (such as SMB, SFTP) has already working code that supports MarkPartial property read from kioslaverc. This patch is just exposing the property from GUI, specifically I've created "Global Options" section (with one checkbox) above existing "FTP Options" section.
> 
> Originally I've created a bug report for this: https://bugs.kde.org/show_bug.cgi?id=359636
> 
> 
> Diffs
> -----
> 
>   src/kcms/kio/kioslave.kcfg d34b691 
>   src/kcms/kio/netpref.h f7a0abc 
>   src/kcms/kio/netpref.cpp 24273a3 
> 
> Diff: https://git.reviewboard.kde.org/r/127154/diff/
> 
> 
> Testing
> -------
> 
> Tested toggling MarkPartial on SMB, SFTP slaves.
> 
> FISH slave was and still is ignoring it (no .part extension).
> 
> I didn't test NFS and others.
> 
> 
> Thanks,
> 
> Martin Kostolný
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160227/3fe3c626/attachment.html>