Hi, thanks for your Pull Request.
Although this functionality is fine, I am not convinced that it is the right way to mix in commands to the gui into the ParserResult, which is as mere model/logic only class. So I think this has to be implemented differently. Maybe there needs to be a whole different apprach to send ui commands from the command line to the gui.

What I immediately imagine would be a List of sth like JumpToEntry implements UiCommand, OpenDatabase implements UiCommand or InsertBibEntries implements UiCommands. This could be handled inside JabRefGUI. I sense this could be some training on cutting edge java pattern matching stuff. Actually implementing this could be fun! 😄

@calixtus Thank you for the feedback! I assume UiCommand is some interface that I would create, however I am not sure exactly what would be included in this interface. Unless you are referring to https://docs.oracle.com/javaee/7/api/javax/faces/component/UICommand.html. I also not familiar with Java pattern matching so I am not sure where to apply pattern matching to this problem.

Considering my current approach does not easily allow me to check for the argument citation key in recently opened databases, I was thinking of adding a String field to JabRefFrame storing the key which gets passed from the Launcher class. Then during the openDatabase() method I could check all databases for the key and open the respective library and entry. Would this approach be too simplistic or even applicable given it woud require changing multiple Constructors and class fields to pass this string to the JabRef.
I might continue with this approach and refactor where necessary :)

I made UiCommand up when i was commenting, I did not know there is already a idk class the same name.

Give me some days, I will try to support you with this, although I'm a bit of a very tight schedule the next days.

@calixtus Thank ou again. If my changes are useful I could submit a PR and someone else could pick this up where I left off. Otherwise I can also continue working on this in the coming weeks, however I also am very busy the next few weeks.

• Remote messages should be logged (visible through the menu)
• Opening in running JabRef (whith library opened) does not work
• Opening a file in running JabRef does not work (JabRef does not have library opened)
• Opening a file in new JabRef instance does not work.

Test setting:

@Article{test,
}

@Article{lala,
}

@Article{coverpage,
  title = {YEAH},
}

@Comment{jabref-meta: databaseType:bibtex;}

gradle run configuration

run --args="-j coverpage C:\temp\b.bib"

The functionality is WRONG.

I have OPENED LIBRARIES. The jump-to-entry should search in the OPENED LIBRARIES!

It is OK if additionally one can jump to an entry of a library which is given by the command line. However, the idea is just to call jabref.bat -j entryId23 and then JabRef searches entryId23 first in the current library. If not found, it searches the other libraries.

jumpToKey should only use the String citation key. No parser result needed

Played a little bit around with bindings:
What about this one instead of the change listener (not yet tested)?

        BooleanBinding loadingBinding = Bindings.createBooleanBinding(
                () -> loadings.stream().anyMatch(ObservableBooleanValue::get),
                loadings.toArray(Observable[]::new)
        );

        EasyBind.subscribe(loadingBinding, loading -> {
            if (!loading) {
                future.complete(null);
            }
        });

I have following output

TRACE: Waiting for all tabs beging loaded

Even if all tabs are loaded...

When a file is loaded from outside, onDatabaseLoadingSucceed is not called.

As far as i can approve koppors changes, @Siedlerchr should also approve.