Bug 55155 - copy to (kuickcopy) does not allow copying to remote (sftp/fish/ftp) URLS
Summary: copy to (kuickcopy) does not allow copying to remote (sftp/fish/ftp) URLS
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR major
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-02-25 13:46 UTC by Moritz Moeller-Herrmann
Modified: 2004-04-15 14:23 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Very quick hack, comments? (682 bytes, patch)
2004-04-12 03:13 UTC, Teemu Rytilahti
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Moritz Moeller-Herrmann 2003-02-25 13:46:58 UTC
Version:            (using KDE KDE 3.1)
Installed from:    SuSE RPMs
OS:          Linux

It is not possible to copy a file to a remote location using copy/move to action in konqueror. This breaks the kioslave principle.

You get a confused message after choosing a remote directory: You can only choose local FILEs.
Comment 1 Moritz Moeller-Herrmann 2003-11-14 13:52:20 UTC
The bug has not been fixed, but has become WORSE. Now if I copy to an sftp/fish-ioslave dir (I had to add a link to the choose folder dialog), the file gets copied to $HOME for some reason.

This might cause unintended data loss therefore it should be fixed for KDE-3.2. I have set severity to major.
Comment 2 Teemu Rytilahti 2004-04-12 03:13:47 UTC
Created attachment 5598 [details]
Very quick hack, comments?

Seems to work here with this patch, but please test and comment.
Comment 3 Waldo Bastian 2004-04-15 14:23:19 UTC
CVS commit by waba: 

Add support for copying/moving to remote URLs (BR55155)
Based on a patch by Teemu Rytilahti
CCMAIL: 55155-done@bugs.kde.org


  M +2 -2      kmetamenu.cpp   1.21.2.1
  M +2 -2      kuick_plugin.cpp   1.28.2.1


--- kdeaddons/konq-plugins/kuick/kmetamenu.cpp  #1.21:1.21.2.1
@@ -135,6 +135,6 @@ void KMetaMenu::writeConfig( const QStri
 }
 void KMetaMenu::slotBrowse() {
-     QString dest = KFileDialog::getExistingDirectory();
+     KURL dest = KFileDialog::getExistingURL();
      if( dest.isEmpty() ) return;
-     slotFileChosen( dest );
+     slotFileChosen( dest.isLocalFile() ? dest.path() : dest.url() );
 }

--- kdeaddons/konq-plugins/kuick/kuick_plugin.cpp  #1.28:1.28.2.1
@@ -58,5 +58,5 @@ void KTestMenu::slotPopupMaeh( ){
 }
 void KTestMenu::slotStartCopyJob( const QString &path ) {
-     KURL url( path );
+     KURL url = KURL::fromPathOrURL( path );
      KIO::CopyJob *copy;
      copy = KIO::copy( popup->popupURLList(), url);
@@ -64,5 +64,5 @@ void KTestMenu::slotStartCopyJob( const 
 }
 void KTestMenu::slotStartMoveJob( const QString &path) {
-     KURL url( path );
+     KURL url = KURL::fromPathOrURL( path );
      KIO::CopyJob *move;
      move = KIO::move( popup->popupURLList(), url );