Fix .toJSON() with embedded lists in SQL

Resolves: #8236
orientechnologies · Apr 23, 2018 · f0a1a2e · f0a1a2e
1 parent 794069f
commit f0a1a2e
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/.../src/main/java/com/orientechnologies/orient/core/sql/functions/text/OSQLMethodToJSON.java b/.../src/main/java/com/orientechnologies/orient/core/sql/functions/text/OSQLMethodToJSON.java
@@ -27,7 +27,7 @@
 
 /**
  * Converts a document in JSON string.
- * 
+ *
  * @author Johann Sorel (Geomatys)
  * @author Luca Garulli (l.garulli--(at)--orientdb.com)
  */
@@ -66,8 +66,13 @@ public Object execute(Object iThis, OIdentifiable iCurrentRecord, OCommandContex
 
       StringBuilder builder = new StringBuilder();
       builder.append("[");
+      boolean first = true;
       for (Object o : OMultiValue.getMultiValueIterable(iThis, false)) {
+        if (!first) {
+          builder.append(",");
+        }
         builder.append(execute(o, iCurrentRecord, iContext, ioResult, iParams));
+        first = false;
       }
 
       builder.append("]");