Bug 204322

Summary: Copy with rename file name is broken if url has protocol
Product: [Unmaintained] kio Reporter: Giuseppe Della Bianca <giusdbg>
Component: generalAssignee: David Faure <faure>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Giuseppe Della Bianca 2009-08-18 20:49:21 UTC
Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

Hi.

In

file
jobuidelegate.cpp

function
KIO::RenameDialog_Result KIO::JobUiDelegate::askFileRename(KJob * job,
                                                           const QString & caption,
                                                           const QString& src,
                                                           const QString & dest,
                                                           KIO::RenameDialog_Mode mode,
                                                           QString& newDest,
                                                           KIO::filesize_t sizeSrc,
                                                           KIO::filesize_t sizeDest,
                                                           time_t ctimeSrc,
                                                           time_t ctimeDest,
                                                           time_t mtimeSrc,
                                                           time_t mtimeDest)

section
    KIO::RenameDialog_Result res = static_cast<RenameDialog_Result>(dlg.exec());
    newDest = dlg.newDestUrl().toLocalFile();

The newDest = dlg.newDestUrl().toLocalFile(); not work with url with protocol, I think is good dlg.newDestUrl().path();
Comment 1 David Faure 2009-08-19 12:45:28 UTC
SVN commit 1013226 by dfaure:

Giuseppe Della Bianca is completely right, this should be path(), not toLocalFile(). Wrong Windows porting...
Fix will be in 4.3.1.
BUG: 204322


 M  +1 -1      jobuidelegate.cpp  


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