Merge pull request #71 from wonderpush/fix-map-struct

Fix map filter with slice of structs

filters/standard_filters.go

@@ -50,9 +50,11 @@ func AddStandardFilters(fd FilterDictionary) { // nolint: gocyclo
 		return append(append(result, a...), b...)
 	})
 	fd.AddFilter("join", joinFilter)
-	fd.AddFilter("map", func(a []map[string]interface{}, key string) (result []interface{}) {
+	fd.AddFilter("map", func(a []interface{}, key string) (result []interface{}) {
+		keyValue := values.ValueOf(key)
 		for _, obj := range a {
-			result = append(result, obj[key])
+			value := values.ValueOf(obj)
+			result = append(result, value.PropertyValue(keyValue).Interface())
 		}
 		return result
 	})

filters/standard_filters_test.go

@@ -63,6 +63,8 @@ var filterTests = []struct {
 	{`map_slice_dup | join`, `a a b`},
 	{`map_slice_dup | uniq | join`, `a b`},
 
+	{`struct_slice | map: "str" | join`, `a b c`},
+
 	// date filters
 	{`article.published_at | date`, "Fri, Jul 17, 15"},
 	{`article.published_at | date: "%a, %b %d, %y"`, "Fri, Jul 17, 15"},
@@ -244,6 +246,13 @@ var filterTestBindings = map[string]interface{}{
 		{"name": "page 6"},
 		{"name": "page 7", "category": "technology"},
 	},
+	"struct_slice": []struct {
+		Str string `liquid:"str"`
+	}{
+		{Str: "a"},
+		{Str: "b"},
+		{Str: "c"},
+	},
 }
 
 func TestFilters(t *testing.T) {