Additional information in save-and-restore log entry

Author: georgweiss

app/save-and-restore/logging/src/main/java/org/phoebus/applications/saveandrestore/logging/Messages.java

@@ -23,6 +23,8 @@
 public class Messages {
 
     public static String LogbookCredentialsMissing;
+    public static String SnapshotCreated;
+    public static String SnapshotRestored;
 
     static
     {

app/save-and-restore/logging/src/main/java/org/phoebus/applications/saveandrestore/logging/SaveAndRestoreEventLogger.java

@@ -32,6 +32,7 @@
 import org.phoebus.security.tokens.ScopedAuthenticationToken;
 import org.phoebus.security.tokens.SimpleAuthenticationToken;
 
+import java.text.MessageFormat;
 import java.time.Instant;
 import java.util.ArrayList;
 import java.util.Collection;
@@ -116,6 +117,7 @@ protected String getSnapshotInfoTable(Node node) {
         // This is needed!
         stringBuilder.append("|-|-|\n");
         stringBuilder.append("| Snapshot name | ").append(node.getName()).append(" |\n");
+        stringBuilder.append("| Comment | ").append(node.getProperty("comment")).append(" |\n");
         stringBuilder.append("| Created | ").append(node.getCreated()).append(" |\n");
         String isGolden = node.getProperty("golden");
         stringBuilder.append("| Golden | ").append("true".equals(isGolden) ? "yes" : "no").append(" |\n");
@@ -155,7 +157,7 @@ public String getOwner() {
 
         @Override
         public String getTitle() {
-            return "New snapshot created";
+            return MessageFormat.format(Messages.SnapshotCreated, node.getName());
         }
 
         @Override
@@ -267,7 +269,7 @@ public LogEntryRestoreImpl(Node node, List<String> failedPVs) {
 
         @Override
         public String getTitle() {
-            return "Snapshot restored";
+            return MessageFormat.format(Messages.SnapshotRestored, node.getName());
         }
 
         @Override

app/save-and-restore/logging/src/main/resources/org/phoebus/applications/saveandrestore/logging/messages.properties

@@ -1 +1,3 @@
-LogbookCredentialsMissing=No logbook credentials found
+LogbookCredentialsMissing=No logbook credentials found
+SnapshotCreated=Snapshot {0} created
+SnapshotRestored=Snapshot {0} restored