Merge pull request #2 from pylon/deep-nesting

Support deeply nested paths in templates
schultyy · Oct 5, 2017 · b3ab5d4 · b3ab5d4
2 parents fb95395 + a51050f
commit b3ab5d4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/lib/mustache.ex b/lib/mustache.ex
@@ -75,11 +75,11 @@ defmodule Mustache do
 
   defp scan_for_dot(template, data) do
     regex = regex("{{", "}}", "\\w+(\\.\\w+)+")
-    scans = Regex.scan(regex, template) |> List.flatten
-    case scans do
-      [] -> template
+    matches = Regex.run(regex, template)
+    case matches do
+      nil -> template
       _  ->
-        path = List.first(scans) |> clean(["{{", "}}"])
+        path = List.first(matches) |> clean(["{{", "}}"])
         scan_for_dot(interpolate(template, data, path), data)
     end
   end
@@ -151,7 +151,7 @@ defmodule Mustache do
         fn(template, data) -> process_section(template, data) end },
     { fn(template) -> Regex.match?(triple_regex(), template) end,
         fn(template, data) -> triple_mustaches(template, data) end},
-    { fn(template) -> Regex.match?(regex("{{", "}}", "\\w+\\.\\w+"), template) end,
+    { fn(template) -> Regex.match?(regex("{{", "}}", "\\w+(\\.\\w+)+"), template) end,
         fn(template, data) -> scan_for_dot(template, data) end },
     { fn(template) -> Regex.match?(double_regex(), template) end,
         fn(template, data) -> double_mustaches(template, data) end}]

diff --git a/test/mustache_feature_test.exs b/test/mustache_feature_test.exs
@@ -61,7 +61,9 @@ defmodule MustacheFeatureTest do
 
   test "Dotted Names" do
     assert Mustache.render("\"{{person.name}}\" == \"Joe\"",
-              %{person: %{name: "Joe"}}) == "\"Joe\" == \"Joe\""
+             %{person: %{name: "Joe"}}) == "\"Joe\" == \"Joe\""
+    assert Mustache.render("\"{{person.name.first}}\" == \"Joe\"",
+             %{person: %{name: %{first: "Joe"}}}) == "\"Joe\" == \"Joe\""
   end
 
   #Sections