Bug 371825

Summary: Places->Network doesn't lead to anywhere
Product: [Plasma] plasmashell Reporter: Boris <kde>
Component: generalAssignee: David Edmundson <kde>
Severity: normal CC: arojas, bhush94, elvis.angelaccio, notmart, plasma-bugs
Priority: NOR    
Version: master   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 5.10

Description Boris 2016-10-29 14:11:44 UTC
User-Agent:       Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:49.0) Gecko/20100101 Firefox/49.0
Build Identifier: 16.04.3

So, I'm on xubuntu 16.10 and I installed dolphin from repository. Clicking Network in Places side panel results in no action, location doesn't change from current dirrectory. If I right click it and select open in new panel, it brings up blank remote:/ location. I still can see network shares in other browsers (thunar and nemo) normally.

Reproducible: Always

Steps to Reproduce:
1. On xubuntu 16.10
2. open Dolphin
3.Click Network in left panel
Alternatively to 3.: right click Network in Places menu and choose open in new tab
Actual Results:  
No change, or blank remote:/ location shows up in new panel

Expected Results:  
There should be local network samba shares visile like it is in other browsers on same machine.
Comment 1 Boris 2016-10-29 14:48:58 UTC
Update: This problem doesn't exist in Dolphin4 on same system
Comment 2 Boris 2016-10-29 15:13:40 UTC
Update: workaround: This problem can be fixed by installing plasma-workspace
Comment 3 Elvis Angelaccio 2016-11-01 11:15:52 UTC
Yep, remote:// is provided by plasma-workspace.
Comment 4 Christoph Feck 2016-11-02 16:41:27 UTC
It's actually in "kio-extras", you do not need to install the Plasma workspace.
Comment 5 Christoph Feck 2016-11-02 16:45:29 UTC
Pardon, I confused "remote:" and "network:". Not sure if the remote-kioslave can be moved to kio-extras; it should not depend on the Plasma workspace.

Reassigning to Plasma developers to check if it can be splitted from there.
Comment 6 David Edmundson 2016-11-02 22:46:29 UTC
it is in plasma-workspace. applications:// and desktop:// are also in there.

Arguably kio-extras is also a bad place. desktop:// and remote:// are fairly core - and the whole point of extras is that they are...extra.

and because kio-extras is in applications moving is a complex nightmare as we no longer have a simple release schedule where we could do this. 

Pragmatically this ends up breaking things for more users than we end up fixing it for...but it is technically currently wrong.
Comment 7 Antonio Rojas 2016-11-09 11:52:19 UTC
Wherever this ends up, knetattach (from plasma-desktop) should be moved together with it IMO, as it is tightly integrated in kio-remote and unrelated to Plasma.
Comment 8 Marco Martin 2016-11-17 12:46:14 UTC
only way would be i guess renaming the files, like kio_desktop2.so etc
Comment 9 Elvis Angelaccio 2017-03-29 21:14:57 UTC
Git commit f81c73b8e25b06a58d89ef42a8ccfd338c6e556f by Elvis Angelaccio.
Committed on 29/03/2017 at 21:14.
Pushed by elvisangelaccio into branch 'master'.

Drop remote ioslave, which is now in KIO 5.32

As discussed on plasma-devel [1], the remote ioslave has been moved to kio [2].

This bumps the minimum KF5 version to 5.32

[1]: https://mail.kde.org/pipermail/plasma-devel/2016-November/061750.html
[2]: https://phabricator.kde.org/D4690
FIXED-IN: 5.10

Differential Revision: https://phabricator.kde.org/D3531

M  +1    -1    CMakeLists.txt
M  +0    -1    kioslave/CMakeLists.txt
D  +0    -19   kioslave/remote/CMakeLists.txt
D  +0    -2    kioslave/remote/Messages.sh
D  +0    -6    kioslave/remote/kdedmodule/CMakeLists.txt
D  +0    -42   kioslave/remote/kdedmodule/remotedirnotify.cpp
D  +0    -105  kioslave/remote/kdedmodule/remotedirnotify.desktop
D  +0    -40   kioslave/remote/kdedmodule/remotedirnotify.h
D  +0    -33   kioslave/remote/kdedmodule/remotedirnotifymodule.cpp
D  +0    -35   kioslave/remote/kdedmodule/remotedirnotifymodule.h
D  +0    -216  kioslave/remote/kio_remote.cpp
D  +0    -44   kioslave/remote/kio_remote.h
D  +0    -17   kioslave/remote/remote.protocol
D  +0    -287  kioslave/remote/remoteimpl.cpp
D  +0    -50   kioslave/remote/remoteimpl.h