sync change in prettyprinting (#86)

Signed-off-by: Lukas Jungmann <[email protected]>

Author: lukasj

impl/src/main/java/org/glassfish/json/JsonGeneratorImpl.java

@@ -505,12 +505,16 @@ public JsonGenerator writeEnd() {
     }
 
     protected void writeComma() {
-        if (!currentContext.first && currentContext.scope != Scope.IN_FIELD) {
+        if (isCommaAllowed()) {
             writeChar(',');
         }
         currentContext.first = false;
     }
 
+    boolean isCommaAllowed() {
+        return !currentContext.first && currentContext.scope != Scope.IN_FIELD;
+    }
+
     protected void writeColon() {
         writeChar(':');
     }

impl/src/main/java/org/glassfish/json/JsonPrettyGeneratorImpl.java

@@ -1,7 +1,7 @@
 /*
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
  *
- * Copyright (c) 2012-2017 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2012-2018 Oracle and/or its affiliates. All rights reserved.
  *
  * The contents of this file are subject to the terms of either the GNU
  * General Public License Version 2 only ("GPL") or the Common Development
@@ -112,8 +112,10 @@ private void writeIndent() {
     @Override
     protected void writeComma() {
         super.writeComma();
-        writeChar('\n');
-        writeIndent();
+        if (isCommaAllowed()) {
+            writeChar('\n');
+            writeIndent();
+        }
     }
 
     @Override

tests/src/test/java/org/glassfish/json/tests/JsonGeneratorTest.java

@@ -1,7 +1,7 @@
 /*
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
  *
- * Copyright (c) 2012-2017 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2012-2018 Oracle and/or its affiliates. All rights reserved.
  *
  * The contents of this file are subject to the terms of either the GNU
  * General Public License Version 2 only ("GPL") or the Common Development
@@ -235,6 +235,31 @@ public void testPrettyObjectWriter() throws Exception {
         JsonObjectTest.testPerson(person);
     }
 
+    public void testPrettyPrinting() throws Exception {
+        String[][] lines = {{"firstName", "John"}, {"lastName", "Smith"}};
+        StringWriter writer = new StringWriter();
+        Map<String, Object> config = new HashMap<>();
+        config.put(JsonGenerator.PRETTY_PRINTING, true);
+        JsonGenerator generator = Json.createGeneratorFactory(config)
+                .createGenerator(writer);
+        generator.writeStartObject()
+                .write("firstName", "John")
+                .write("lastName", "Smith")
+                .writeEnd();
+        generator.close();
+        writer.close();
+         BufferedReader reader = new BufferedReader(new StringReader(writer.toString().trim()));
+        int numberOfLines = 0;
+        String line;
+        while ((line = reader.readLine()) != null) {
+            numberOfLines++;
+            if (numberOfLines > 1 && numberOfLines < 4) {
+                assertTrue(line.contains("\"" + lines[numberOfLines - 2][0] + "\": \"" + lines[numberOfLines - 2][1] + "\""));
+            }
+        }
+        assertEquals(4, numberOfLines);
+    }
+
     public void testPrettyObjectStream() throws Exception {
         ByteArrayOutputStream out = new ByteArrayOutputStream();
         Map<String, Object> config = new HashMap<>();