Refactors server log handling for improved efficiency

Changes the ArrayAppender to construct a ServerLogItem directly
instead of passing individual log properties.

The ServerLogProvider now accepts a ServerLogItem, extracts the log
properties, adds the server ID and log ID, and then passes the log
item to the log controller.

This change simplifies the code and makes it more efficient by
reducing the number of parameters passed and centralizing the
creation of the ServerLogItem.

Author: NicoPiel

server/src/com/mirth/connect/plugins/serverlog/ArrayAppender.java

@@ -11,6 +11,8 @@
 
 import java.io.Serializable;
 import java.util.Date;
+import java.util.HashMap;
+import java.util.Map;
 
 import org.apache.logging.log4j.core.Layout;
 import org.apache.logging.log4j.core.LogEvent;
@@ -69,6 +71,17 @@ public void append(LogEvent logEvent) {
             throwableInformation = logText.toString();
         }
 
-        serverLogProvider.newServerLogReceived(channelId, channelName, level, date, threadName, category, lineNumber, message, throwableInformation);
+        Map<String, Object> properties = new HashMap<>();
+        properties.put("channelId", channelId);
+        properties.put("channelName", channelName);
+        properties.put("level", level);
+        properties.put("date", date);
+        properties.put("threadName", threadName);
+        properties.put("category", category);
+        properties.put("lineNumber", lineNumber);
+        properties.put("message", message);
+        properties.put("throwableInformation", throwableInformation);
+
+        serverLogProvider.newServerLogReceived(new ServerLogItem(properties));
     }
 }

server/src/com/mirth/connect/plugins/serverlog/ServerLogProvider.java

@@ -57,22 +57,12 @@ public void init(Properties properties) {
         initialize();
     }
 
-    public synchronized void newServerLogReceived(String channelId, String channelName, String level, Date date, String threadName, String category, String lineNumber, String message, String throwableInformation) {
+    public synchronized void newServerLogReceived(ServerLogItem svi) {
         if (logController != null) {
-            Map<String, Object> properties = new HashMap<>();
-            properties.put("serverId", serverId);
-            properties.put("id", logId);
-            properties.put("channelId", channelId);
-            properties.put("channelName", channelName);
-            properties.put("level", level);
-            properties.put("date", date);
-            properties.put("threadName", threadName);
-            properties.put("category", category);
-            properties.put("lineNumber", lineNumber);
-            properties.put("message", message);
-            properties.put("throwableInformation", throwableInformation);
-
-            logController.addLogItem(new ServerLogItem(properties));
+            svi.getContext().put("serverId", serverId);
+            svi.getContext().put("id", logId);
+
+            logController.addLogItem(svi);
             logId++;
         }
     }