Serialize the JSON node only if there is content

opentripplanner · Sep 5, 2016 · 00d9b98 · 00d9b98
1 parent 2a314df
commit 00d9b98
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/src/main/java/org/opentripplanner/graph_builder/module/EmbedConfig.java b/src/main/java/org/opentripplanner/graph_builder/module/EmbedConfig.java
@@ -1,7 +1,9 @@
 package org.opentripplanner.graph_builder.module;
 
+import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
+import com.fasterxml.jackson.databind.ObjectWriter;
 import org.opentripplanner.graph_builder.services.GraphBuilderModule;
 import org.opentripplanner.routing.graph.Graph;
 
@@ -26,10 +28,9 @@ public EmbedConfig(JsonNode builderConfig, JsonNode routerConfig) {
 
     @Override
     public void buildGraph(Graph graph, HashMap<Class<?>, Object> extra) {
-        ObjectMapper mapper = new ObjectMapper();
         try {
-            graph.builderConfig = mapper.writerWithDefaultPrettyPrinter().writeValueAsString(builderConfig);
-            graph.routerConfig = mapper.writerWithDefaultPrettyPrinter().writeValueAsString(routerConfig);
+            graph.builderConfig = serializedConfiguration(builderConfig);
+            graph.routerConfig = serializedConfiguration(routerConfig);
         } catch (Exception ex) {
             throw new RuntimeException(ex);
         }
@@ -40,4 +41,10 @@ public void checkInputs() {
 
     }
 
+    private String serializedConfiguration(JsonNode config) throws JsonProcessingException{
+        ObjectMapper mapper = new ObjectMapper();
+        ObjectWriter writer = mapper.writerWithDefaultPrettyPrinter();
+        return config.isMissingNode() ? null : writer.writeValueAsString(config);
+    }
+
 }