Add handler for map slice type (#2)

* Add map string slice type

Author: omarismail

request.go

@@ -396,6 +396,12 @@ func unmarshalAttribute(
 		return
 	}
 
+	// Handle field of type map[string][]string
+	if fieldValue.Type() == reflect.TypeOf(map[string][]string{}) {
+		value, err = handleMapStringSlice(attribute, fieldValue)
+		return
+	}
+
 	// Handle field of type time.Time
 	if fieldValue.Type() == reflect.TypeOf(time.Time{}) ||
 		fieldValue.Type() == reflect.TypeOf(new(time.Time)) {
@@ -447,6 +453,19 @@ func handleStringSlice(attribute interface{}) (reflect.Value, error) {
 	return reflect.ValueOf(values), nil
 }
 
+func handleMapStringSlice(attribute interface{}, fieldValue reflect.Value) (reflect.Value, error) {
+	values := map[string][]string{}
+	for key, valSlice := range attribute.(map[string]interface{}) {
+		sliceValues := make([]string, 0)
+		for _, v := range valSlice.([]interface{}) {
+			sliceValues = append(sliceValues, v.(string))
+		}
+		values[key] = sliceValues
+	}
+
+	return reflect.ValueOf(values), nil
+}
+
 func handleTime(attribute interface{}, args []string, fieldValue reflect.Value) (reflect.Value, error) {
 	var isIso8601 bool
 	var isRFC3339 bool

request_test.go

@@ -1387,3 +1387,54 @@ func TestUnmarshalNestedStructSlice(t *testing.T) {
 			out.Teams[0].Members[0].Firstname)
 	}
 }
+
+type HeaderData struct {
+	ID      string              `jsonapi:"primary,header-data"`
+	Name    string              `jsonapi:"attr,name"`
+	Headers map[string][]string `jsonapi:"attr,headers"`
+}
+
+func TestUnmarshalMapStringSlice(t *testing.T) {
+	headers := map[string][]string{
+		"cache-control":  {"private"},
+		"content-length": {"129"},
+	}
+	sample := map[string]interface{}{
+		"data": map[string]interface{}{
+			"type": "header-data",
+			"id":   "123",
+			"attributes": map[string]interface{}{
+				"name":    "Planet Express",
+				"headers": headers,
+			},
+		},
+	}
+
+	data, err := json.Marshal(sample)
+	if err != nil {
+		t.Fatal(err)
+	}
+	in := bytes.NewReader(data)
+	out := new(HeaderData)
+
+	if err := UnmarshalPayload(in, out); err != nil {
+		t.Fatal(err)
+	}
+
+	if len(out.Headers["cache-control"]) == 0 {
+		t.Fatalf("Expected Headers to have Cache Control values, but got:`%s`", out.Headers["cache-control"])
+	}
+
+	if len(out.Headers["content-length"]) == 0 {
+		t.Fatalf("Expected Headers to have Content Length values, but got:`%s`", out.Headers["content-length"])
+	}
+
+	if out.Headers["cache-control"][0] != "private" {
+		t.Fatalf("Expected Cache Control Header to have 'private' value, but got:`%s`", out.Headers["cache-control"])
+	}
+
+	if out.Headers["content-length"][0] != "129" {
+		t.Fatalf("Expected Content Length Header to have '129' value, but got:`%s`", out.Headers["content-length"])
+	}
+
+}