CSSTUDIO-2572 Improve error handling when loading of a log entry is not successful.

Author: abrahamwolk

app/logbook/olog/ui/src/main/java/org/phoebus/logbook/olog/ui/LogEntryDisplayController.java

@@ -41,9 +41,14 @@
 import org.phoebus.olog.es.api.model.OlogLog;
 import org.phoebus.ui.javafx.ImageCache;
 
+import java.util.logging.Level;
+import java.util.logging.Logger;
+
 
 public class LogEntryDisplayController {
 
+    static Logger log = Logger.getLogger(LogEntryDisplayController.class.getName());
+
     @FXML
     @SuppressWarnings("unused")
     private SingleLogEntryDisplayController singleLogEntryDisplayController;
@@ -160,8 +165,13 @@ public void jumpToLogEntry() {
             return;
         }
 
-        if (logEntryTableViewController.goBackAndGoForwardActions.isPresent()) {
-            logEntryTableViewController.goBackAndGoForwardActions.get().loadLogEntryWithID(logEntryIDToJumpTo);
+        if (logEntryIDToJumpTo > 0) {
+            if (logEntryTableViewController.goBackAndGoForwardActions.isPresent()) {
+                boolean success = logEntryTableViewController.goBackAndGoForwardActions.get().loadLogEntryWithID(logEntryIDToJumpTo);
+                if (!success) {
+                    log.log(Level.WARNING, "Error loading entry with log entry ID: " + logEntryIDToJumpTo + ".");
+                }
+            }
         }
     }
 

app/logbook/olog/ui/src/main/java/org/phoebus/logbook/olog/ui/LogEntryTable.java

@@ -172,12 +172,18 @@ private void addGoForwardAction() {
             }
         }
 
-        protected void loadLogEntryWithID(Long id) {
+        protected boolean loadLogEntryWithID(Long id) {
             goForwardActions.clear();
             addGoBackAction();
 
-            LogEntry logEntry = controller.client.getLog(id);
-            gotoLogEntry(logEntry);
+            try {
+                LogEntry logEntry = controller.client.getLog(id);
+                gotoLogEntry(logEntry);
+                return true;
+            }
+            catch (RuntimeException runtimeException) {
+                return false;
+            }
         }
 
         protected void goBack() {