"open directory" for attached file does not work #8679

wujastyk · 2022-04-15T18:04:16Z

JabRef version

Latest development branch build (please note build date below)

Operating system

GNU / Linux

Details on version and operating system

JabRef 5.6--2022-04-15--f13ada0 Linux 5.13.0-39-generic amd64 Java 17.0.2 JavaFX 18+12

Checked with the latest development build

I made a backup of my libraries before testing the latest development version.
I have tested the latest development version and the problem persists

Steps to reproduce the behaviour

Select an entry
Go to the "General" tab
Select an attached PDF file
Right click to open context menu
select "open folder" (or ctrl-shift-O)
nothing happens.

More or less the same as #8093, but in a different context.

Appendix

No response

Siedlerchr · 2022-04-15T19:53:30Z

Hi just asking some additional information, snap or deb?
Out of curiosity, I recently learned that java.awt.desktop apparently only works with libgnome-2.0. I am not 100% sure here in this case, but do you have that installed or not?

@LyzardKing You worked on this, if I remember correctly, any further ideas?

wujastyk · 2022-04-15T20:39:41Z

deb
Not sure about libgnome-2.0. Here's what synaptic says:
77

PS https://wiki.gnome.org/Attic/LibgnomeMustDie

Siedlerchr · 2022-04-17T19:44:56Z

Ah, I now remember that we changed this, not sure how this would need to be adapted

jabref/src/main/java/org/jabref/gui/desktop/os/Linux.java

Lines 89 to 102 in f13ada0

    
           public void openFolderAndSelectFile(Path filePath) throws IOException { 
        
               String desktopSession = System.getenv("DESKTOP_SESSION"); 
        
               String cmd = "xdg-open " + filePath.toAbsolutePath().getParent().toString(); // default command 
        
               if (desktopSession != null) { 
        
                   desktopSession = desktopSession.toLowerCase(Locale.ROOT); 
        
                   if (desktopSession.contains("gnome")) { 
        
                       cmd = "nautilus" + filePath.toString().replace(" ", "\\ "); 
        
                   } else if (desktopSession.contains("kde")) { 
        
                       cmd = "dolphin --select " + filePath.toString().replace(" ", "\\ "); 
        
                   } 
        
               } 
        
               Runtime.getRuntime().exec(cmd);

LyzardKing · 2022-04-20T07:58:46Z

The last time I looked at it there was no way to port java.awt.desktop to using more modern systems to launch programs/file managers/etc, not easily at least.
I'll give it another go in the next few days and let you know what I come up with

LyzardKing · 2022-04-20T08:03:46Z

@wujastyk out of curiosity, what is the output of running env | grep DESKTOP_SESSION in a terminal?

Even in edge cases the default xdg-open command should run fine anyway

wujastyk · 2022-04-20T16:39:04Z

I get

$ env | grep DESKTOP_SESSION
GNOME_DESKTOP_SESSION_ID=this-is-deprecated
DESKTOP_SESSION=cinnamon

LyzardKing · 2022-04-20T16:43:10Z

If this command works (the path must be absolute) we could replace all the custom parts with this.
It should work on all distros/desktop environments

dbus-send --session --print-reply --dest=org.freedesktop.FileManager1 --type=method_call /org/freedesktop/FileManager1 org.freedesktop.FileManager1.ShowItems array:string:"file://<insert full file path>" string:""

calixtus · 2022-04-20T20:17:08Z

Reveal folder works for me, open terminal here not, because i dont have xterm installed, but konsole.

env | grep DESKTOP
DESKTOP_SESSION=plasmawayland
XDG_CURRENT_DESKTOP=KDE
XDG_SESSION_DESKTOP=KDE

LyzardKing · 2022-04-21T08:57:06Z

@wujastyk what file manager are you using?
@calixtus there is a condition specific for konsole. What package are you running?

wujastyk · 2022-04-21T17:32:20Z

@LyzardKing I'm using Nemo, the file manager of Cinnamon.

LyzardKing · 2022-04-21T19:06:11Z

Strange. It should work with the default (running xdg-open <path to folder>)
We can add another option specifically for nemo, but the default command should work anyway. Could you try it in a terminal?

wujastyk · 2022-04-21T19:35:28Z

try what in a terminal? Nemo? xdg-open . Both work fine.

mlep · 2022-04-22T07:16:45Z

FYI: I am using Nemo and cinnamon too, and I do not have this issue: "open folder" opens nemo with the proper directory.

JabRef 5.6--2022-04-20--f220575
Linux 4.19.0-20-amd64 amd64
Java 17.0.2
JavaFX 18+12

ThiloteE · 2022-05-18T10:44:49Z

Using Linux Mint default file explorer

JabRef 5.7--2022-05-16--6a2332f
Linux 5.4.0-110-generic amd64
Java 18.0.1
JavaFX unknown

Having this issue as well.

ThiloteE · 2022-05-18T12:44:59Z

#8679 (comment)

If this command works (the path must be absolute) we could replace all the custom parts with this. It should work on all distros/desktop environments

dbus-send --session --print-reply --dest=org.freedesktop.FileManager1 --type=method_call /org/freedesktop/FileManager1 org.freedesktop.FileManager1.ShowItems array:string:"file://<insert full file path>" string:""

I will mark this issue as a good first issue. Maybe somebody can try what you propose here and open a pull request. If it does not work, we can close the pull request.

LyzardKing · 2022-05-19T06:20:20Z

actually on further look it's not a good solution.
The dbus service it is using is not available in confined packages like flatpak or snap (as far as I can tell, but I have to look at it more).

LyzardKing · 2022-05-19T06:30:27Z

I opened the PR to add Nemo and Caja in the list of file managers. It still doesn't solve the snap and flatpak.

ThiloteE · 2022-05-19T11:57:55Z

Relevant documentation for possibly implementing/fixing this for Flatpak:

not sure but could be that flatpak has not implemented it yet. See this issue

ThiloteE · 2022-05-19T12:41:35Z

Unfortunately #8831 did not fix the issue for me. :/ I just tried the newest development version as Linux portable

I am on Linux Mint 20.3 (Cinnamon 64-bit) with Nemo

JabRef 5.7--2022-05-19--4baf2a1
Linux 5.4.0-110-generic amd64
Java 18.0.1
JavaFX unknown

ThiloteE · 2022-05-19T14:02:15Z

Being as curious as I am, I stumpled upon this guide: https://linuxhint.com/installing_nemo_file_manager_linux/

When I checked on my system which mimetypes it is being used for, it showed some other file managers than Nemo attached to the same mimetype, if i understand it correctly. So maybe for this type of action Nemo is not prioritized? I am afraid of completely removing the other ones from this mimetype and making Nemo the defacto default, because it might break other things.

Not sure if I understand what is going on, but maybe it helps somebody solving this riddle.

ThiloteE · 2022-05-25T10:34:56Z

Same problem for

1.  click on entry in entry table.
2.  right click on entry and select open folder. or press ctrl+shift+O.

ThiloteE · 2022-05-25T14:27:18Z

Could the d-bus solution be used as fallback? Or alternatively xdg-open as fall-back instead? Or is this an EITHER / OR type of problem?

Related discussion: https://unix.stackexchange.com/questions/364997/open-a-directory-in-the-default-file-manager-and-select-a-file

ThiloteE · 2022-05-25T14:59:12Z

Btw.:

JabRef 5.7--2022-05-23--117bb1c
Linux 5.4.0-113-generic amd64
Java 18.0.1
JavaFX unknown

on Linux Mint 20.3 with Nemo.

thilo@thilo-Aspire-ES1-311 ~ $ env | grep DESKTOP_SESSION
GNOME_DESKTOP_SESSION_ID=this-is-deprecated
DESKTOP_SESSION=cinnamon

and

thilo@thilo-Aspire-ES1-311 ~ $ env | grep DESKTOP
GNOME_DESKTOP_SESSION_ID=this-is-deprecated
DESKTOP_SESSION=cinnamon
XDG_SESSION_DESKTOP=cinnamon
XDG_CURRENT_DESKTOP=X-Cinnamon

wujastyk · 2022-05-25T15:44:22Z

JabRef 5.7--2022-05-23--117bb1c
Linux 5.13.0-44-generic amd64
Java 18.0.1
JavaFX unknown

"Open file F4 "now works. "Open folder ctrl+sht+O" does not.

LyzardKing · 2022-05-26T06:20:54Z

@ThiloteE xdg-open is set to be used as a fallback.
Could you try the latest build from https://builds.jabref.org/main/?
In your case the command to run should be nemo --select <filename>

ThiloteE · 2022-05-26T07:20:42Z

Ok, I think I found the problem!

thilo@thilo-Aspire-ES1-311 ~ $ nemo --select test.txt
Could not parse arguments: Unbekannte Option --select

Please remove the --select for nemo
Apparently they removed this option. Nemo is a fork of Nautilus, so yes there are similarities, but at one point they must have removed it.

https://linuxcommandlibrary.com/man/nemo.html

Here up to date commands for nemo:

thilo@thilo-Aspire-ES1-311 ~ $ nemo --help
Aufruf:
  nemo [OPTION …] [Adresse …] 

Browse the file system with the file manager

Hilfeoptionen:
  -h, --help                   Hilfeoptionen anzeigen
  --help-all                   Alle Hilfeoptionen anzeigen
  --help-gtk                   GTK+-Optionen anzeigen

Anwendungsoptionen:
  --version                    Die Programmversion anzeigen
  -g, --geometry=GEOMETRIE     Das Anfangsfenster mit der angegebenen Geometrie erstellen. Beispiele: nemo --geometry=+100+100, nemo --geometry=600x400, nemo --geometry=600x400+100+100+100.
  -n, --no-default-window      Nur für ausdrücklich angegebene Adressen die Fenster erstellen.
  --no-desktop                 Ignorierter Ausdruck – wird nur zur Kompatibilität zugelassen.
  -t, --tabs                   Adressen in Reitern öffnen.
  --existing-window            Adressen in einem bestehenden Fenster öffnen.
  --fix-cache                  Bitte den Benutzervorschaubildpuffer reparieren - das kann nützlich sein, wenn Sie Probleme mit den Vorschaubildern haben. Muss als Systemverwalter ausgeführt werden.
  --debug                      Enable debugging code.  Example usage: 'NEMO_DEBUG=Actions,Window nemo --debug'.  Use NEMO_DEBUG=help for more topics.
  -q, --quit                   Nemo beenden.
  --display=ANZEIGE            X-Anzeige, die verwendet werden soll

I did some tests and using the command nemo Downloads/test.txt via terminal to open my downloads folder and to select the file test.txt worked just fine!

ThiloteE · 2022-05-26T07:29:54Z

Maybe it was only working in an older version of Linux Mint:

FYI: I am using Nemo and cinnamon too, and I do not have this issue: "open folder" opens nemo with the proper directory.

JabRef 5.6--2022-04-20--f220575
Linux 4.19.0-20-amd64 amd64
Java 17.0.2
JavaFX 18+12

ThiloteE · 2022-05-26T14:36:02Z

Flatpak compatible solution:

https://github.com/flatpak/xdg-desktop-portal
https://flatpak.github.io/xdg-desktop-portal/

Stumpled upon it while reading https://gitlab.freedesktop.org/xdg/xdg-specs/-/issues/64

ThiloteE · 2022-05-27T17:02:03Z

Hello everybody. Christoph and me tried another method that should fix this issue. The new method also works for Linux Mint Cinnamon (Nemo file manager), but we have not tested the other distributions. Would you mind testing the PR 8853? It can be downloaded here: https://builds.jabref.org/pull/8853/merge/

wujastyk · 2022-05-27T18:00:02Z

Yes! It worked for me.

JabRef 5.7-PullRequest8853.81--2022-05-26--7636804
Linux 5.13.0-44-generic amd64
Java 18.0.1
JavaFX unknown

Kernel: 5.13.0-44-generic x86_64 bits: 64 compiler: N/A Desktop: Cinnamon 5.2.7
wm: muffin dm: LightDM Distro: Linux Mint 20.3 Una base: Ubuntu 20.04 focal

Thank you all!

Siedlerchr added the os: linux label Apr 15, 2022

ThiloteE mentioned this issue May 18, 2022

General --> File --> right mouse button --> Open Folder opens pdf viewer #6744

Closed

ThiloteE added the bug Confirmed bugs or reports that are very likely to be bugs label May 18, 2022

ThiloteE added this to Normal priority in Bugs via automation May 18, 2022

ThiloteE added the good first issue An issue intended for project-newcomers. Varies in difficulty. label May 18, 2022

LyzardKing mentioned this issue May 19, 2022

Add Nemo file manager #8831

Merged

6 tasks

calixtus closed this as completed in #8831 May 19, 2022

Bugs automation moved this from Normal priority to Done May 19, 2022

ThiloteE reopened this May 19, 2022

ThiloteE removed the good first issue An issue intended for project-newcomers. Varies in difficulty. label May 19, 2022

koppor moved this from Done to Normal priority in Bugs May 23, 2022

ThiloteE mentioned this issue May 25, 2022

cannot "open folder" #8093

Closed

2 tasks

ThiloteE mentioned this issue May 25, 2022

"open folder" does not select the attached file(s) of the entry #8849

Closed

ThiloteE mentioned this issue May 27, 2022

Fix linux file opening by using Process Builder #8853

Merged

6 tasks

Siedlerchr closed this as completed in #8853 Jun 5, 2022

Bugs automation moved this from Normal priority to Done Jun 5, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

"open directory" for attached file does not work #8679

"open directory" for attached file does not work #8679

wujastyk commented Apr 15, 2022 •

edited

Loading

Siedlerchr commented Apr 15, 2022

wujastyk commented Apr 15, 2022 •

edited

Loading

Siedlerchr commented Apr 17, 2022

LyzardKing commented Apr 20, 2022

LyzardKing commented Apr 20, 2022

wujastyk commented Apr 20, 2022

LyzardKing commented Apr 20, 2022

calixtus commented Apr 20, 2022

LyzardKing commented Apr 21, 2022

wujastyk commented Apr 21, 2022

LyzardKing commented Apr 21, 2022

wujastyk commented Apr 21, 2022

mlep commented Apr 22, 2022

ThiloteE commented May 18, 2022

ThiloteE commented May 18, 2022 •

edited

Loading

LyzardKing commented May 19, 2022

LyzardKing commented May 19, 2022

ThiloteE commented May 19, 2022 •

edited

Loading

ThiloteE commented May 19, 2022 •

edited

Loading

ThiloteE commented May 19, 2022

ThiloteE commented May 25, 2022

ThiloteE commented May 25, 2022

ThiloteE commented May 25, 2022 •

edited

Loading

wujastyk commented May 25, 2022

LyzardKing commented May 26, 2022 •

edited

Loading

ThiloteE commented May 26, 2022 •

edited

Loading

ThiloteE commented May 26, 2022

ThiloteE commented May 26, 2022

ThiloteE commented May 27, 2022

wujastyk commented May 27, 2022

"open directory" for attached file does not work #8679

"open directory" for attached file does not work #8679

Comments

wujastyk commented Apr 15, 2022 • edited Loading

JabRef version

Operating system

Details on version and operating system

Checked with the latest development build

Steps to reproduce the behaviour

Appendix

Siedlerchr commented Apr 15, 2022

wujastyk commented Apr 15, 2022 • edited Loading

Siedlerchr commented Apr 17, 2022

LyzardKing commented Apr 20, 2022

LyzardKing commented Apr 20, 2022

wujastyk commented Apr 20, 2022

LyzardKing commented Apr 20, 2022

calixtus commented Apr 20, 2022

LyzardKing commented Apr 21, 2022

wujastyk commented Apr 21, 2022

LyzardKing commented Apr 21, 2022

wujastyk commented Apr 21, 2022

mlep commented Apr 22, 2022

ThiloteE commented May 18, 2022

ThiloteE commented May 18, 2022 • edited Loading

LyzardKing commented May 19, 2022

LyzardKing commented May 19, 2022

ThiloteE commented May 19, 2022 • edited Loading

ThiloteE commented May 19, 2022 • edited Loading

ThiloteE commented May 19, 2022

ThiloteE commented May 25, 2022

ThiloteE commented May 25, 2022

ThiloteE commented May 25, 2022 • edited Loading

wujastyk commented May 25, 2022

LyzardKing commented May 26, 2022 • edited Loading

ThiloteE commented May 26, 2022 • edited Loading

ThiloteE commented May 26, 2022

ThiloteE commented May 26, 2022

ThiloteE commented May 27, 2022

wujastyk commented May 27, 2022

wujastyk commented Apr 15, 2022 •

edited

Loading

wujastyk commented Apr 15, 2022 •

edited

Loading

ThiloteE commented May 18, 2022 •

edited

Loading

ThiloteE commented May 19, 2022 •

edited

Loading

ThiloteE commented May 19, 2022 •

edited

Loading

ThiloteE commented May 25, 2022 •

edited

Loading

LyzardKing commented May 26, 2022 •

edited

Loading

ThiloteE commented May 26, 2022 •

edited

Loading