Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show in File Manager Notification does not open Dolphin #282

Closed
crouthamela opened this issue Feb 18, 2018 · 11 comments
Closed

Show in File Manager Notification does not open Dolphin #282

crouthamela opened this issue Feb 18, 2018 · 11 comments

Comments

@crouthamela
Copy link

After saving a video, a notification appears with a button, stating the file has been saved, and it may be viewed in File Manager. Clicking the button does not open Dolphin to view the file.

@gort818
Copy link
Contributor

gort818 commented Feb 18, 2018

Can you post the output from the terminal?

@crouthamela
Copy link
Author

Sure, here is the output of me running Peek as a Flatpak (my normal method) and recording/saving a WebM, then clicking the pop-up. The last error message occurs when clicking the pop-up.

andrew@andrew-XPS-13-9360:~$ flatpak run com.uploadedlobster.peek

** (peek:3): WARNING **: Couldn't connect to accessibility bus: Failed to connect to socket /tmp/dbus-vsgYli2CJG: Connection refused

** (peek:3): WARNING **: Binding '<Ctrl><Alt>R' failed!
Using screen recorder backend ffmpeg

(peek:3): GLib-GIO-CRITICAL **: g_dbus_proxy_new: assertion 'G_IS_DBUS_CONNECTION (connection)' failed

(peek:3): GLib-GIO-CRITICAL **: g_dbus_proxy_new: assertion 'G_IS_DBUS_CONNECTION (connection)' failed

(peek:3): GLib-GIO-CRITICAL **: g_dbus_proxy_new: assertion 'G_IS_DBUS_CONNECTION (connection)' failed

(peek:3): GLib-GIO-CRITICAL **: g_dbus_proxy_new: assertion 'G_IS_DBUS_CONNECTION (connection)' failed

(peek:3): GLib-GIO-CRITICAL **: g_dbus_proxy_new: assertion 'G_IS_DBUS_CONNECTION (connection)' failed

(peek:3): GLib-GIO-CRITICAL **: g_simple_action_activate: assertion 'simple->parameter_type == NULL ? parameter == NULL : (parameter != NULL && g_variant_is_of_type (parameter, simple->parameter_type))' failed
andrew@andrew-XPS-13-9360:~$

I am running:

Peek 1.2.2
KDE Plasma Version: 5.12.1
KDE Frameworks Version: 5.43.0
Qt Version: 5.9.1
Kernel Version 4.13.0-32-generic
OS Type: 64-bit

@phw phw added the bug label Feb 19, 2018
@phw
Copy link
Owner

phw commented Feb 19, 2018

Some more questions:

  1. Does opening Dolhpin work if you enable the "Show file manager after saving" option in Peek's preferences (this will circumvent the notification and show the file manager directly)?

  2. Please give me the output of:

     G_MESSAGES_DEBUG=all flatpak run com.uploadedlobster.peek
    
  3. What distro is this?

  4. If possible it would be helpful to see whether a natively installed Peek (non-flatpak) works as expected

I tried reproducing this on my KDE Neon VM with Plasma 5.12, but unfortunately the Flatpak version doesn't show me notificatioins at all (even though Peek is sending one). I am probably missing something. With a native installed Peek everything works as it should.

@crouthamela
Copy link
Author

crouthamela commented Feb 19, 2018

  1. It does not. The terminal has no output either regarding a Dolphin opening attempt either.
  2. Peek Debug.txt
  3. Kubuntu 17.10 with Backports PPA
  4. I installed native Peek 1.2.2 from the Peek Stable PPA and it works as expected.

@phw
Copy link
Owner

phw commented Feb 19, 2018

Not sure, but something must be wrong with the way Flatpak is setup here. Just to make sure your DBus is working as it should, does the following command open your file manager with the home folder open?

dbus-send --session --print-reply --dest=org.freedesktop.FileManager1 /org/freedesktop/FileManager1 org.freedesktop.FileManager1.ShowFolders array:string:"file://$HOME" string:""

Also do you have xdg-desktop-portal-kde installed? Not sure that is needed in this case, but it at least it is required for a fully functional Flatpak setup.

@crouthamela
Copy link
Author

The dbus command works in a terminal.

Interestingly I did not have xdg-desktop-portal-kde installed, I suppose I'll let the KDE Discover team know that would be a good dependency for plasma-discover-flatpak-backend. After installing, the terminal message g_simple_action_activate: has gone away when I click the notification, but Dolphin still fails to open.

@phw
Copy link
Owner

phw commented Feb 19, 2018

Ok, we are now a step further :-) Thanks for the help so far. Now that this error is gone the output of G_MESSAGES_DEBUG=all flatpak run com.uploadedlobster.peek would be helpful again. It should tell us how Peek is trying to open the file manager (in case of Flatpak it should use the DBus method, but Peek uses some fallbacks if this doesn't work).

@crouthamela
Copy link
Author

crouthamela commented Feb 19, 2018

No problem, I've been writing KDE bugs for days too :)

Debug: Peek Debug 2.txt

The notification appears (and I click on it) after Temp file deleted: true is shown in the debug, but I don't see anything appear regarding DBus.

@crouthamela
Copy link
Author

FYI - Apparently everyone is on their computer right now, and the KDE Neon and Kubuntu teams have already pushed a patch to fix dependencies so the xdg-desktop-portal-kde package will be included in the future. :)

@rikmills
Copy link

FYI - Apparently everyone is on their computer right now, and the KDE Neon and Kubuntu teams have already pushed a patch to fix dependencies so the xdg-desktop-portal-kde package will be included in the future. :)

https://launchpad.net/ubuntu/+source/plasma-discover/5.12.1-0ubuntu2

@phw phw added the upstream label Mar 20, 2018
@phw
Copy link
Owner

phw commented Mar 20, 2018

Sorry, took me some time to get back to this. For me this looks like a general Flatpak issue. From the log I would say Peek is not even receiving the signal that opening the file was requested. Also the many other DBus related issues in the log indicate that there is something wrong in your setup regarding Flatpak and DBus. I cannot reproduce any of this on my KDE Neon VM.

I would suggest you bring this up with the Flatpak team, or maybe see if this will work better in Ubuntu 18.04. But it doesn't look like I can do anything in Peek to handle this.

@phw phw closed this as completed Mar 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants