feat: implied-string-literals

Added support for implied-string-literals to the various JsonUrlWriter
classes.

Author: dmaccormack

module/jsonurl-core/src/main/java/org/jsonurl/JsonUrl.java

@@ -846,9 +846,9 @@ public static <V> V parseLiteral(
      * @param text  source
      * @param start offset in source
      * @param end   length of source
-     * @return dest
+     * @return true if dest was modified
      */
-    public static <T extends Appendable> T appendLiteral(// NOPMD - CyclomaticComplexity
+    public static <T extends Appendable> boolean appendLiteral(// NOPMD - CyclomaticComplexity
             T dest,
             CharSequence text,
             int start,
@@ -861,27 +861,20 @@ public static <T extends Appendable> T appendLiteral(// NOPMD - CyclomaticComple
             //
             // empty string
             //
-            if (isEmptyUnquotedStringOK) {
-                return dest;
-            }
-
-            if (isImpliedStringLiteral) {
-                //
-                // no way to represent it
-                //
-                throw new IOException(MSG_EXPECT_LITERAL.getMessageText());
+            if (isEmptyUnquotedStringOK || isImpliedStringLiteral) {
+                return false;
             }
 
             //
             // the empty string must be quoted
             //
             dest.append("''");
-            return dest;
+            return true;
         }
 
         if (isImpliedStringLiteral) {
             Encode.encode(dest, text, start, end, false, true);
-            return dest;
+            return true;
         }
 
         //
@@ -907,7 +900,7 @@ public static <T extends Appendable> T appendLiteral(// NOPMD - CyclomaticComple
                 //
                 dest.append('\'').append(text, start, end).append('\'');
             }
-            return dest;
+            return true;
         }
 
         //
@@ -933,6 +926,6 @@ public static <T extends Appendable> T appendLiteral(// NOPMD - CyclomaticComple
             break;
         }
 
-        return dest;
+        return true;
     }
 }

module/jsonurl-core/src/main/java/org/jsonurl/JsonUrlStringBuilder.java

@@ -35,6 +35,11 @@ public class JsonUrlStringBuilder extends
      */
     private static final int DEFAULT_SIZE = 1 << 10;
 
+    /**
+     * My StringBuilder.
+     */
+    private final StringBuilder builder; // NOPMD - AvoidStringBufferField
+
     /**
      * Create a new JsonUrlStringBuilder.
      *
@@ -60,6 +65,7 @@ public JsonUrlStringBuilder(int size) {
      */
     public JsonUrlStringBuilder(StringBuilder dest) {
         super(dest);
+        this.builder = dest;
     }
 
     @Override
@@ -71,4 +77,13 @@ public String build() {
     public String toString() {
         return out.toString();
     }
+
+    /**
+     * Clear the internal buffer.
+     * @return this
+     */
+    public JsonUrlStringBuilder clear() {
+        builder.setLength(0);
+        return this;
+    }
 }

module/jsonurl-core/src/main/java/org/jsonurl/JsonUrlTextAppender.java

@@ -116,51 +116,67 @@ public JsonUrlTextAppender<A,R> nameSeparator() throws IOException {
 
     @Override
     public JsonUrlTextAppender<A,R> addNull() throws IOException {
+        if (impliedStringLiterals) {
+            throw new IOException(
+                "can not represent null when implied-string-literals is true");
+        }
+        
         out.append("null");
         return this;
     }
 
     @Override
     public JsonUrlTextAppender<A,R> add(BigDecimal value) throws IOException {
+        if (value == null) {
+            return addNull();
+        }
+
         if (impliedStringLiterals) {
             add(String.valueOf(value), false);
 
         } else {
             out.append(String.valueOf(value));    
         }
+
         return this;
     }
 
     @Override
     public JsonUrlTextAppender<A,R> add(BigInteger value) throws IOException {
+        if (value == null) {
+            return addNull();
+        }
+
         if (impliedStringLiterals) {
             add(String.valueOf(value), false);
 
         } else {
             out.append(String.valueOf(value));    
         }
+
+        
         return this;
     }
 
     @Override
     public JsonUrlTextAppender<A,R> add(long value) throws IOException {
         if (impliedStringLiterals) {
             add(String.valueOf(value), false);
-
         } else {
             out.append(String.valueOf(value));    
         }
+
         return this;
     }
 
     @Override
     public JsonUrlTextAppender<A,R> add(double value) throws IOException {
         if (impliedStringLiterals) {
             add(String.valueOf(value), false);
-
         } else {
             out.append(String.valueOf(value));    
         }
+
         return this;
     }
 
@@ -172,7 +188,7 @@ public JsonUrlTextAppender<A,R> add(boolean value) throws IOException {
 
     @Override
     public JsonUrlTextAppender<A,R> add(char value) throws IOException {
-        out.append(value); // FIXME - need to encode
+        out.append(value);
         return this;
     }
 
@@ -182,6 +198,14 @@ public JsonUrlTextAppender<A,R> add(
             int start,
             int end,
             boolean isKey) throws IOException {
+
+        if (text == null) {
+            if (isKey) {
+                throw new IOException("object key can not be null");
+            }
+            return addNull();
+        }
+
         JsonUrl.appendLiteral(
                 out,
                 text,
@@ -190,6 +214,7 @@ public JsonUrlTextAppender<A,R> add(
                 isKey,
                 isKey ? allowEmptyUnquotedKey : allowEmptyUnquotedValue,
                 impliedStringLiterals);
+
         return this;
     }