Merge pull request #160 from gf3/pointer-value

fix: follow pointers when evaling identifiers

compiler.go

@@ -418,18 +418,23 @@ func (c *compiler) evalIdentifier(node *ast.Identifier) (interface{}, error) {
 		}
 
 		f := rv.FieldByName(node.Value)
+		if f.Kind() == reflect.Ptr {
+			if f.IsNil() {
+				return nil, nil
+			}
+			f = f.Elem()
+		}
 		if !f.IsValid() {
 			m := rv.MethodByName(node.Value)
 			if !m.IsValid() {
 				return nil, fmt.Errorf("'%s' does not have a field or method named '%s' (%s)", node.Callee.String(), node.Value, node)
 			}
 			return m.Interface(), nil
 		}
-
 		if !f.CanInterface() {
-
 			return nil, fmt.Errorf("'%s'cannot return value obtained from unexported field or method '%s' (%s)", node.Callee.String(), node.Value, node)
 		}
+
 		return f.Interface(), nil
 	}
 	if c.ctx.Has(node.Value) {

struct_test.go

@@ -106,7 +106,6 @@ func Test_Render_Struct_PointerMethod_IsNil(t *testing.T) {
 	}
 
 	for p := first; p != nil; p = p.Next {
-
 		ctx := NewContextWith(map[string]interface{}{"p": p})
 		res, err := Render(input, ctx)
 		r.NoError(err)
@@ -115,6 +114,51 @@ func Test_Render_Struct_PointerMethod_IsNil(t *testing.T) {
 	}
 }
 
+func Test_Render_Struct_PointerValue_Nil(t *testing.T) {
+	r := require.New(t)
+
+	type user struct {
+		Name  string
+		Image *string
+	}
+
+	u := user{
+		Name:  "Garn Clapstick",
+		Image: nil,
+	}
+	ctx := NewContextWith(map[string]interface{}{
+		"user": u,
+	})
+	input := `<%= user.Name %>: <%= user.Image %>`
+	res, err := Render(input, ctx)
+
+	r.NoError(err)
+	r.Equal(`Garn Clapstick: `, res)
+}
+
+func Test_Render_Struct_PointerValue_NonNil(t *testing.T) {
+	r := require.New(t)
+
+	type user struct {
+		Name  string
+		Image *string
+	}
+
+	image := "bicep.png"
+	u := user{
+		Name:  "Scrinch Archipeligo",
+		Image: &image,
+	}
+	ctx := NewContextWith(map[string]interface{}{
+		"user": u,
+	})
+	input := `<%= user.Name %>: <%= user.Image %>`
+	res, err := Render(input, ctx)
+
+	r.NoError(err)
+	r.Equal(`Scrinch Archipeligo: bicep.png`, res)
+}
+
 func Test_Render_Struct_Multiple_Access(t *testing.T) {
 	r := require.New(t)