From 489827b2d9e01ae9d29b71108550807089987f71 Mon Sep 17 00:00:00 2001
From: Timm Fitschen <t.fitschen@indiscale.com>
Date: Tue, 24 Nov 2020 20:29:32 +0100
Subject: [PATCH] WIP: cleanup

---
 .../database/proto/VersionHistoryItem.java    | 11 ++++++++--
 .../java/org/caosdb/server/utils/Info.java    | 20 +++++++++----------
 2 files changed, 18 insertions(+), 13 deletions(-)

diff --git a/src/main/java/org/caosdb/server/database/proto/VersionHistoryItem.java b/src/main/java/org/caosdb/server/database/proto/VersionHistoryItem.java
index f76b3bc6..4c0a60ce 100644
--- a/src/main/java/org/caosdb/server/database/proto/VersionHistoryItem.java
+++ b/src/main/java/org/caosdb/server/database/proto/VersionHistoryItem.java
@@ -1,13 +1,20 @@
 package org.caosdb.server.database.proto;
 
 import java.io.Serializable;
-import java.util.LinkedList;
+import java.util.List;
 
+/**
+ * This class is a flat, data-only representation of a single item of version information. This
+ * class is an intermediate representation which abstracts away the data base results and comes in a
+ * form which is easily cacheable.
+ *
+ * @author Timm Fitschen (t.fitschen@indiscale.com)
+ */
 public class VersionHistoryItem implements Serializable {
 
   private static final long serialVersionUID = 428030617967255942L;
   public String id = null;
-  public LinkedList<String> parents = null;
+  public List<String> parents = null;
   public Long seconds = null;
   public Integer nanos = null;
   public String username = null;
diff --git a/src/main/java/org/caosdb/server/utils/Info.java b/src/main/java/org/caosdb/server/utils/Info.java
index 3a488a1c..19f792cf 100644
--- a/src/main/java/org/caosdb/server/utils/Info.java
+++ b/src/main/java/org/caosdb/server/utils/Info.java
@@ -202,10 +202,6 @@ public class Info extends AbstractObservable implements Observer, TransactionInt
     }
     dropOffBox = new File(FileSystem.getDropOffBox());
     final Element info = new Element("Stats");
-    if (CaosDBServer.isDebugMode()) {
-      info.setAttribute("debug", "true");
-    }
-
     final Element counts = new Element("counts");
     counts.setAttribute("records", Integer.toString(recordsCount));
     counts.setAttribute("properties", Integer.toString(propertiesCount));
@@ -213,16 +209,18 @@ public class Info extends AbstractObservable implements Observer, TransactionInt
     counts.setAttribute("files", Integer.toString(filesCount));
     counts.setAttribute("fssize", fssize);
     counts.setAttribute("tmpfiles", Integer.toString(tmpFilesCount));
-
-    final Element dropOffBoxElem = new Element("dropOffBox");
+    if (CaosDBServer.isDebugMode()) {
+      counts.setAttribute("debug", "true");
+    }
+    final Element e = new Element("dropOffBox");
     if (dropOffBox.isDirectory()) {
       if (dropOffBox.canRead()) {
         if (tree) {
-          dropOffBoxElem.setAttribute("path", dropOffBox.getAbsolutePath());
-          dropOffBoxElem.addContent(getTree(dropOffBox.listFiles()));
+          e.setAttribute("path", dropOffBox.getAbsolutePath());
+          e.addContent(getTree(dropOffBox.listFiles()));
         } else {
-          dropOffBoxElem.setAttribute("path", dropOffBox.getAbsolutePath());
-          dropOffBoxElem.addContent(getFlatList(dropOffBox.listFiles()));
+          e.setAttribute("path", dropOffBox.getAbsolutePath());
+          e.addContent(getFlatList(dropOffBox.listFiles()));
         }
       } else {
         // TODO: return a message that the DropOffBox is not readable.
@@ -232,7 +230,7 @@ public class Info extends AbstractObservable implements Observer, TransactionInt
       // present.
     }
     info.addContent(counts);
-    info.addContent(dropOffBoxElem);
+    info.addContent(e);
     return info;
   }
 
-- 
GitLab