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

[Martin Kostolný]

> On Úno. 27, 2016, 11:11 dop., David Faure wrote:
> > 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? :-)
> 
> Martin Kostolný wrote:
>     Sorry for my late response. I agree with every word you said, David. :-) Thanks for taking your time to discuss this.
>     
>     My use-case is indeed the (one specific) remote SMB server has problems with renaming and it breaks the upload. So I'd like to offer an option to disable this otherwise very useful ".part" functionality.
>     
>     I agree that global-mark-partial-option is not a good idea but I think we should come up with something. So here is my first proposal: I'm up for the overblown configuration dialog which would be hidden behind "Avanced upload options..." button placed in Connection Preferences. Here is one wireframe (warning - very unfinished! I had little time...): https://sojcaci.cz/temp/disable-mark-partial-mockup.png. Both FTP options would be moved to mentioned configuration dialog as well (to their dedicated tab).
>     
>     As I'm looking to the configuration dialog it is really overblown. Second proposal: maybe we could use an entirelly different approach like a runtime option to temporarily but globally disable MarkPartial. Something similar to "Enable Power Management" button available in expanded battery plasmoid.
> 
> Martin Kostolný wrote:
>     One more proposal: Add "DisableMarkPartial" to JobFlag enum as a hint to be used (in KIO::file_copy(..) etc.) in various file managers. That way we can delegate this settings to be picked up by file managers like krusader, dolphin or other tools using KIO.
> 
> David Faure wrote:
>     We can't delegate the setting to the app. All KIO-based apps use this, not just file managers. If you save a document in calligra onto an FTP server, kio_ftp is used, and needs to know whether to use MarkPartial.
>     
>     Not sure if I prefer the global option switched on/off temporarily, or the precise per-host per-protocol setting. Maybe the VDG (visual design group) has an opinion and/or other ideas...

Sorry about this rookie question - what is the right procedure to present this problem to VDG? Is there such a group to add as reviewers or should I ask the question on a forum?


- Martin


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


On Úno. 24, 2016, 11:52 odp., Martin Kostolný wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127154/
> -----------------------------------------------------------
> 
> (Updated Úno. 24, 2016, 11:52 odp.)
> 
> 
> 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/20160311/d762d4b4/attachment.html>