Bug 161384 - Network/Add Network Folder doesn't add folders in Dolphin
Summary: Network/Add Network Folder doesn't add folders in Dolphin
Status: RESOLVED FIXED
Alias: None
Product: knetattach
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Peter Penz
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-28 20:43 UTC by Davide Ferrari
Modified: 2012-01-01 05:17 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Use KRun instead of KToolInvocation::invokeBrowser (775 bytes, patch)
2008-06-03 19:58 UTC, Elias Probst
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Davide Ferrari 2008-04-28 20:43:00 UTC
Version:            (using Devel)
Installed from:    Compiled sources
OS:                Linux

Go to "Network" in Dolphin Places panel, click on Network. Click on "Add network folder". Select some remote connection type (for example, samba or sftp), fill with your data, click save&connect and... Konqueror (not Dolphin) opens showing the smb:// or fish:// location. I've started a process in Dolphin and ended with a Konqueror window opened and Dolphin with no clues about this new remote location.

Expected behavior: Dolphin should add by default a new location to the Places panel, using the name the user labelled the connection in the wizard, and content should be displayed (after authorization, of course) in Dolphin.
Comment 1 Davide Ferrari 2008-04-28 20:44:57 UTC
I know that this bug is probably not directly related to Dolphin (remote:// is not Dolphin) but I cannot find the proper place where to fill the bug. Please reassign if needed, and please consider that this could be a junior job and that I'm willing in helping fix the problem. Thanks.
Comment 2 Elias Probst 2008-06-03 19:58:16 UTC
Created attachment 25091 [details]
Use KRun instead of KToolInvocation::invokeBrowser

This patch _should_ fix the behavior, that the new connection is opened in
Konqueror instead of the responsible app for the according URL.
But it currently doesn't work as expected, after finishing the wizard nothing
happens - maybe someone with some more KDE/Qt knowledge can take a look at
this.
Comment 3 Elias Probst 2008-06-06 02:02:34 UTC
Several bugs are 'waiting' for KToolInvocation::invokeFileManager(QString), which will be provided in KDE 4.2 as stated by 'aseigo' on #plasma:

01:42:37 < eliasp> aseigo: do you know, whether there are any plans to provide
                   KToolInvocation::invokeFileManager() as you wrote here:
                   http://bugs.kde.org/show_bug.cgi?id=151474#c2
01:43:09 < eliasp> could also be handy for bug 161384
01:43:36 < eliasp> and just discovered another one, where it might be handy...                 
                   entering ~/ or /etc or another local URL into krunner starts
                   konqueror...
01:44:25 <@aseigo> eliasp: yeah, i plan on that in 4.2. i'm going to also add
                   an entry for that to the Default Applications control panel,
                   and then hook those two things together
01:45:03 <@aseigo> eliasp: that's the main reason i haven't done the
                   KToolInvocation thing yet
01:45:39 < eliasp> aseigo: nice to hear... so this bugs shouldn't be solved yet
                   using a temporary workaround.. just wait, until kdelibs 
                   provides this in 4.2 ..

###

So forget the patch I provided previously... just wait until the necessary KToolInvocation features are provided by the upcoming 4.2/SVN and make KToolInvocation run the filemanager instead of the browser.
Comment 4 Jekyll Wu 2012-01-01 05:17:18 UTC
I think this problem has really been fixed.