Update ActionHelper.java

JabRef · Mar 14, 2020 · c3df843 · c3df843
1 parent 1bd2770
commit c3df843
Showing 1 changed file with 3 additions and 12 deletions.
diff --git a/src/main/java/org/jabref/gui/actions/ActionHelper.java b/src/main/java/org/jabref/gui/actions/ActionHelper.java
@@ -33,22 +33,13 @@ public static BooleanExpression isFieldSetForSelectedEntry(Field field, StateMan
     public static BooleanExpression isAnyFieldSetForSelectedEntry(List<Field> fields, StateManager stateManager) {
         ObservableList<BibEntry> selectedEntries = stateManager.getSelectedEntries();
 
-        // binding should be recreated on every right click
-        // not sure why selectedEntries might be empty, see https://github.com/JabRef/jabref/issues/6085
-        if (selectedEntries.isEmpty()) {
-            return Bindings.createBooleanBinding(() -> false, selectedEntries);
-        }
-
         ObjectBinding<BibEntry> entry = Bindings.valueAt(selectedEntries, 0);
         return Bindings.createBooleanBinding(() -> {
-            if (entry.get() == null)
+            if (entry.get() == null) {
                 return false;
-            else
+            } else {
                return entry.get().getFields().stream().anyMatch(fields::contains);
+            }
         }, entry, entry.getFieldsObserable());
-        return Bindings.createBooleanBinding(
-                () -> entry.getFields().stream().anyMatch(fields::contains),
-                entry.getFieldsObservable(),
-                selectedEntries);
     }
 }