struct: ignore unexported fields when reflect (#239)

go-ini · May 9, 2020 · ad8a106 · ad8a106
1 parent 0b997d7
commit ad8a106
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/struct.go b/struct.go
@@ -564,6 +564,10 @@ func (s *Section) reflectFrom(val reflect.Value) error {
 	typ := val.Type()
 
 	for i := 0; i < typ.NumField(); i++ {
+		if !val.Field(i).CanInterface() {
+			continue
+		}
+
 		field := val.Field(i)
 		tpField := typ.Field(i)
 

diff --git a/struct_test.go b/struct_test.go
@@ -260,7 +260,6 @@ func Test_MapToStruct(t *testing.T) {
 			So(ts.TimePtrNil, ShouldEqual, nil)
 			So(*ts.DurationPtr, ShouldEqual, 0)
 			So(ts.DurationPtrNil, ShouldEqual, nil)
-
 		})
 
 		Convey("Map section to struct", func() {
@@ -440,12 +439,13 @@ func Test_ReflectFromStruct(t *testing.T) {
 			GPA       float64
 			Date      time.Time
 			NeverMind string `ini:"-"`
+			ignored   string
 			*Embeded  `ini:"infos" comment:"Embeded section"`
 		}
 
 		t, err := time.Parse(time.RFC3339, "1993-10-07T20:17:05Z")
 		So(err, ShouldBeNil)
-		a := &Author{"Unknwon", true, nil, 21, 100, 2.8, t, "",
+		a := &Author{"Unknwon", true, nil, 21, 100, 2.8, t, "", "ignored",
 			&Embeded{
 				[]time.Time{t, t},
 				[]string{"HangZhou", "Boston"},