length property not working in handlebars-jackson2 fix #598

Author: jknack

handlebars-jackson2/src/main/java/com/github/jknack/handlebars/JsonNodeValueResolver.java

@@ -58,6 +58,9 @@ public Object resolve(final Object context, final String name) {
     Object value = null;
     if (context instanceof ArrayNode) {
       try {
+        if (name.equals("length")) {
+          return ((ArrayNode) context).size();
+        }
         value = resolve(((ArrayNode) context).get(Integer.parseInt(name)));
       } catch (NumberFormatException ex) {
         // ignore undefined key and move on, see https://github.com/jknack/handlebars.java/pull/280

handlebars-jackson2/src/test/java/com/github/jknack/handlebars/Issue598.java

@@ -0,0 +1,24 @@
+package com.github.jknack.handlebars;
+
+import com.fasterxml.jackson.databind.JsonNode;
+import com.fasterxml.jackson.databind.ObjectMapper;
+import com.github.jknack.handlebars.context.MapValueResolver;
+import org.junit.Test;
+
+import java.io.IOException;
+
+public class Issue598 extends AbstractTest {
+
+  @Override
+  protected Object configureContext(final Object model) {
+    return Context.newBuilder(model)
+        .resolver(MapValueResolver.INSTANCE, JsonNodeValueResolver.INSTANCE).build();
+  }
+
+  @Test
+  public void shouldExtendJsonArrayWithLenghtProperty() throws IOException {
+    JsonNode tree = new ObjectMapper().readTree("[1, 2, 3]");
+    shouldCompileTo("{{this.length}}", tree, "3");
+  }
+
+}