Merge pull request #155 from manyminds/issue-154

Fix renaming bug that ocured while unmarshaling

Author: sharpner

api_interfaces_test.go

@@ -13,8 +13,9 @@ import (
 )
 
 type SomeData struct {
-	ID   string `json:"-"`
-	Data string
+	ID         string `json:"-"`
+	Data       string
+	CustomerID string `jsonapi:"name=customerId"`
 }
 
 func (s SomeData) GetID() string {
@@ -103,6 +104,14 @@ var _ = Describe("Test interface api type casting", func() {
 		api.Handler().ServeHTTP(rec, req)
 		Expect(rec.Code).To(Equal(http.StatusOK))
 	})
+
+	It("Post works with lowercase renaming", func() {
+		reqBody := strings.NewReader(`{"data": [{"attributes":{"customerId": "2" }, "type": "someDatas"}]}`)
+		req, err := http.NewRequest("POST", "/v1/someDatas", reqBody)
+		Expect(err).To(BeNil())
+		api.Handler().ServeHTTP(rec, req)
+		Expect(rec.Code).To(Equal(http.StatusCreated))
+	})
 })
 
 var _ = Describe("Test return code behavior", func() {

jsonapi/unmarshal.go

@@ -260,7 +260,7 @@ func UnmarshalInto(input map[string]interface{}, targetStructType reflect.Type,
 						for x := 0; x < val.NumField(); x++ {
 							tfield := val.Type().Field(x)
 							name := GetTagValueByName(tfield, "name")
-							if name == strings.ToLower(fieldName) {
+							if strings.ToLower(name) == strings.ToLower(fieldName) {
 								field = val.Field(x)
 							}
 						}