Merge pull request #101 from jamslinger/fix-size

Count number of characters in size filter instead of number of bytes

filters/standard_filters_test.go

@@ -83,6 +83,8 @@ var filterTests = []struct {
 	// sequence (array or string) filters
 	{`"Ground control to Major Tom." | size`, 28},
 	{`"apples, oranges, peaches, plums" | split: ", " | size`, 4},
+	// count chars, not bytes
+	{`"Straße" | size`, 6},
 
 	// string filters
 	{`"Take my protein pills and put my helmet on" | replace: "my", "your"`, "Take your protein pills and put your helmet on"},

values/arrays.go

@@ -2,6 +2,7 @@ package values
 
 import (
 	"reflect"
+	"unicode/utf8"
 )
 
 // TODO Length is now only used by the "size" filter.
@@ -13,8 +14,10 @@ func Length(value any) int {
 	value = ToLiquid(value)
 	ref := reflect.ValueOf(value)
 	switch ref.Kind() {
-	case reflect.Array, reflect.Slice, reflect.String:
+	case reflect.Array, reflect.Slice:
 		return ref.Len()
+	case reflect.String:
+		return utf8.RuneCountInString(ref.String())
 	default:
 		return 0
 	}