Merge pull request #97 from /issues/96

Ensure empty slices have the same behavior as empty strings

helpers.go

@@ -38,6 +38,20 @@ func isSlice(obj any) bool {
 	return is(obj, reflect.Slice)
 }
 
+func isEmptySlice(obj any) bool {
+	if !isSlice(obj) {
+		return false
+	}
+
+	for _, v := range obj.([]any) {
+		if isTrue(v) {
+			return false
+		}
+	}
+
+	return true
+}
+
 func isTrue(obj any) bool {
 	if isBool(obj) {
 		return obj.(bool)

issues_test.go

@@ -306,11 +306,29 @@ func TestIssue81(t *testing.T) {
 	var result bytes.Buffer
 
 	err := jsonlogic.Apply(strings.NewReader(rule), strings.NewReader(data), &result)
-
 	if err != nil {
 		t.Fatal(err)
 	}
 
 	expected := `true`
 	assert.JSONEq(t, expected, result.String())
 }
+
+func TestIssue96(t *testing.T) {
+	rule := `{"map":[
+      {"var":"integers"},
+	  {"*":[{"var":[""]},2]}
+    ]}`
+
+	data := `{"integers": [1,2,3]}`
+
+	var result bytes.Buffer
+
+	err := jsonlogic.Apply(strings.NewReader(rule), strings.NewReader(data), &result)
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	expected := `[2, 4, 6]`
+	assert.JSONEq(t, expected, result.String())
+}

vars.go

@@ -17,6 +17,11 @@ func solveVars(values, data any) any {
 					continue
 				}
 
+				if isEmptySlice(value) {
+					logic["var"] = ""
+					continue
+				}
+
 				val := getVar(value, data)
 				if val != nil {
 					return val