Return findEntry method

JabRef · Oct 30, 2024 · c480bce · c480bce
1 parent 875ec14
commit c480bce
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 2 deletions.
diff --git a/src/main/java/org/jabref/gui/maintable/MainTable.java b/src/main/java/org/jabref/gui/maintable/MainTable.java
@@ -259,7 +259,7 @@ public void clearAndSelect(BibEntry bibEntry) {
         } else {
             // select new entry
             getSelectionModel().clearSelection();
-            model.getViewModelByIndex(database.getDatabase().indexOf(bibEntry)).ifPresent(entry -> {
+            findEntry(bibEntry).ifPresent(entry -> {
                 getSelectionModel().select(entry);
                 scrollTo(entry);
             });
@@ -489,6 +489,10 @@ public List<BibEntry> getSelectedEntries() {
                 .collect(Collectors.toList());
     }
 
+    private Optional<BibEntryTableViewModel> findEntry(BibEntry entry) {
+        return model.getViewModelByIndex(database.getDatabase().indexOf(entry));
+    }
+
     public void setCitationMergeMode(boolean citationMerge) {
         this.citationMergeMode = citationMerge;
     }

diff --git a/src/main/java/org/jabref/gui/maintable/MainTableDataModel.java b/src/main/java/org/jabref/gui/maintable/MainTableDataModel.java
@@ -38,10 +38,13 @@
 import com.tobiasdiez.easybind.EasyBind;
 import com.tobiasdiez.easybind.Subscription;
 import org.jspecify.annotations.Nullable;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import static org.jabref.model.search.PostgreConstants.ENTRY_ID;
 
 public class MainTableDataModel {
+    private final Logger LOGGER = LoggerFactory.getLogger(MainTableDataModel.class);
 
     private final ObservableList<BibEntryTableViewModel> entriesViewModel;
     private final FilteredList<BibEntryTableViewModel> entriesFiltered;
@@ -197,6 +200,7 @@ public SortedList<BibEntryTableViewModel> getEntriesFilteredAndSorted() {
 
     public Optional<BibEntryTableViewModel> getViewModelByIndex(int index) {
         if (index < 0 || index >= entriesViewModel.size()) {
+            LOGGER.warn("Tried to access out of bounds index {} in entriesViewModel", index);
             return Optional.empty();
         }
         return Optional.of(entriesViewModel.get(index));

diff --git a/src/main/java/org/jabref/model/database/BibDatabase.java b/src/main/java/org/jabref/model/database/BibDatabase.java
@@ -637,7 +637,11 @@ public String getNewLineSeparator() {
      */
     public int indexOf(BibEntry bibEntry) {
         int index = Collections.binarySearch(entries, bibEntry, Comparator.comparing(BibEntry::getId));
-        return index >= 0 ? index : -1;
+        if (index >= 0) {
+            return index;
+        }
+        LOGGER.warn("Could not find entry with ID {} in the database", bibEntry.getId());
+        return -1;
     }
 
     public BibEntry getEntryById(String id) {