Fix unmarshalAttribute to support unmarshaling into a json.RawMessage

Author: michele

request.go

@@ -393,6 +393,11 @@ func unmarshalAttribute(
 		return
 	}
 
+	// Handle field of type json.RawMessage
+	if fieldValue.Type() == reflect.TypeOf(json.RawMessage{}) {
+		value, err = handleJSONRawMessage(attribute)
+	}
+
 	// Handle field of type time.Time
 	if fieldValue.Type() == reflect.TypeOf(time.Time{}) ||
 		fieldValue.Type() == reflect.TypeOf(new(time.Time)) {
@@ -444,6 +449,14 @@ func handleStringSlice(attribute interface{}) (reflect.Value, error) {
 	return reflect.ValueOf(values), nil
 }
 
+func handleJSONRawMessage(attribute interface{}) (reflect.Value, error) {
+	tmp, err := json.Marshal(attribute)
+	if err != nil {
+		return reflect.Value{}, err
+	}
+	return reflect.ValueOf(json.RawMessage(tmp)), nil
+}
+
 func handleTime(attribute interface{}, args []string, fieldValue reflect.Value) (reflect.Value, error) {
 	var isIso8601 bool
 	v := reflect.ValueOf(attribute)

request_test.go

@@ -1320,7 +1320,8 @@ func (mca *MyCustomAttribute) UnmarshalJSON(bts []byte) error {
 
 type StructForTest struct {
 	ID     string            `jsonapi:"primary,tests"`
-	Custom MyCustomAttribute `jsonapi:"attr,custom"`
+	Custom MyCustomAttribute `jsonapi:"attr,custom,omitempty"`
+	Raw    json.RawMessage   `jsonapi:"attr,raw,omitempty"`
 }
 
 func TestUnmarshalWithCustomType(t *testing.T) {
@@ -1347,3 +1348,33 @@ func TestUnmarshalWithCustomType(t *testing.T) {
 			sft.Custom.Field, newSft.Custom.Field)
 	}
 }
+
+func TestUnmarshalWithJSONRawMessage(t *testing.T) {
+	tests := [][]byte{
+		[]byte(`{"really":{"deep":true},"test":"toast"}`),
+		[]byte(`"just a string"`),
+		[]byte(`123`),
+	}
+	for _, v := range tests {
+		sft := &StructForTest{
+			ID:  "my-id",
+			Raw: v,
+		}
+		buf := new(bytes.Buffer)
+		err := MarshalPayload(buf, sft)
+		if err != nil {
+			t.Fatal(err)
+		}
+		newSft := &StructForTest{}
+		err = UnmarshalPayload(buf, newSft)
+
+		if err != nil {
+			t.Fatal(err)
+		}
+
+		if bytes.Compare(sft.Raw, newSft.Raw) != 0 {
+			t.Fatalf("json.RawMessage wasn't properly unmarshalled: Expected to have `%s` but got `%s`",
+				string(sft.Raw), string(newSft.Raw))
+		}
+	}
+}