Bug 204605 - konqueror leaves files downloaded via ftp:// with .part extension
Summary: konqueror leaves files downloaded via ftp:// with .part extension
Status: RESOLVED FIXED
Alias: None
Product: kio
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: David Faure
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-21 03:54 UTC by Maciej Mrozowski
Modified: 2009-11-23 15:13 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
possible fix (apply to kdelibs/kioslave/ftp/ftp.cpp.diff) (2.27 KB, patch)
2009-11-20 02:11 UTC, David Faure
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Maciej Mrozowski 2009-08-21 03:54:29 UTC
Version:           KDE 4.3 branch, r1012788 (kdelibs r1013848) (using Devel)
Compiler:          gcc (Gentoo 4.3.2-r3 p1.6, pie-10.1.5) 4.3.2 
OS:                Linux
Installed from:    Compiled sources

This is 4.3 branch

Steps to reproduce:
1. Open konqueror and enter URL (for example ftp://ftp.kde.org/pub/kde/unstable/kdevelop/3.9.95/src
2. Select and copy files to clipboard and paste in some local directory.
3. Wait for them to be downloaded (there will be knotify message)
4. Go to destination dir and see that files are not completely downloaded (at least marked with .part suffix)

I it some ftp kioslave regression or plain konqueror bug?
It would be nice to have it fixed before 4.3.1 is tagged.

Thanks in advance
Comment 1 FiNeX 2009-08-21 10:11:39 UTC
Did you have stopped the download before its end?
Comment 2 Maciej Mrozowski 2009-08-21 14:39:34 UTC
Well, please read step 3.
Comment 3 FiNeX 2009-08-21 15:19:01 UTC
When the file download is completed the .part file should be renamed. Is the checksum of the .part file correct matching the correct one?
Comment 4 Maciej Mrozowski 2009-08-21 15:53:03 UTC
Right, I could have added it to initial bug report.
Yes, file is fully downloaded, just .part suffix not removed.
Please try to reproduce for yourself though (in 4.3 branch)
Comment 5 FiNeX 2009-08-21 17:05:42 UTC
(Un)fortunatly I cannot reproduce the bug using current trunk. I don't have KDE 4.3 installed now.
Comment 6 FiNeX 2009-08-22 01:17:33 UTC
Can you reproduce in dolphin too? Probably this is a dup of bug #203819 .
Comment 7 Maciej Mrozowski 2009-08-22 01:52:35 UTC
Reproducible with dolphin as well.
Bug #203819 refers to smb:// protocol anyway (ftp:// here), but it's indeed possible that fixing one will fix the other. I'd keep them both anyway for now until fixed.
Comment 8 Hikaru 2009-08-22 16:41:25 UTC
I cannot reproduce on KDE 4.3 (ArchLinux), both using konqueror and dolphin. ".part" suffix is removed correctly after download.

I cannot repdoce bug #203819 too.
Comment 9 Maciej Mrozowski 2009-08-23 01:27:48 UTC
Well, it may have been patched in ArchLinux or regression introduced after 4.3.0 has been released (I reported bug against 4.3 svn branch).
Someone with recent snapshot of 4.3 branch (so 4.3.1 wannabe) would need to test it.
Comment 10 Spiros Georgaras 2009-09-18 15:50:27 UTC
This bug is still here ( kde 4.3.1 )
Comment 11 Maciej Mrozowski 2009-11-03 12:46:57 UTC
I just checked it, and this is the status:

Still reproducible with 4.3.3, but...
bug should be called: "konqueror ignores 'Mark partially uploaded files' option.
When I select this option:
- konqueror marks partially downloaded file appending .part to filename AND renames file back to original one when download completed

When I deselect this option:
- konqueror STILL marks partially downloaded file with .part suffix (bug) and LEAVES it not renamed (correct behaviour) when completed

Haven't checked whether trunk is affected yet, but definitely 4.3 branch, reproduced on clean account.
Comment 12 FiNeX 2009-11-08 15:46:21 UTC
Is this a dup of bug #209070 ?
Comment 13 David Faure 2009-11-20 02:11:18 UTC
Created attachment 38452 [details]
possible fix (apply to kdelibs/kioslave/ftp/ftp.cpp.diff)

Here's a possible fix for this problem, kio_ftp was indeed not handling correctly the case where .part files are disabled.

I coded this on a plane so I couldn't test it, please give it a try. AFAICS it should work :-)

Reply to comment #12: No, I don't think so.
Comment 14 Maciej Mrozowski 2009-11-20 13:35:49 UTC
Tested with 4.3 branch - confirmed to fix this issue (it should fix for trunk as well).
David, do the honors! :)

Thanks
Comment 15 David Faure 2009-11-23 15:11:49 UTC
SVN commit 1053170 by dfaure:

Fix kio_ftp downloading files as "foo.part" instead of "foo" when disabling(!) the "Mark partially uploaded files" feature.
CCBUG: 204605


 M  +12 -12    ftp.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1053170
Comment 16 David Faure 2009-11-23 15:13:47 UTC
SVN commit 1053171 by dfaure:

Fix kio_ftp downloading files as "foo.part" instead of "foo" when disabling(!) the "Mark partially uploaded files" feature.
Fixed for: 4.3.4
BUG: 204605


 M  +12 -12    ftp.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1053171