fix order[items][][properties][] unmarshaling for empty values

marshaling.go

@@ -92,6 +92,40 @@ func (p *OrderPayments) UnmarshalJSON(data []byte) error {
 	return nil
 }
 
+func (p *Properties) UnmarshalJSON(data []byte) error {
+	var i interface{}
+	var m Properties
+	if err := json.Unmarshal(data, &i); err != nil {
+		return err
+	}
+
+	switch e := i.(type) {
+	case map[string]interface{}:
+		m = make(Properties, len(e))
+		for idx, val := range e {
+			var res Property
+			err := unmarshalMap(val.(map[string]interface{}), &res)
+			if err != nil {
+				return err
+			}
+			m[idx] = res
+		}
+	case []interface{}:
+		m = make(Properties, len(e))
+		for idx, val := range e {
+			var res Property
+			err := unmarshalMap(val.(map[string]interface{}), &res)
+			if err != nil {
+				return err
+			}
+			m[strconv.Itoa(idx)] = res
+		}
+	}
+
+	*p = m
+	return nil
+}
+
 func unmarshalMap(m map[string]interface{}, v interface{}) (err error) {
 	var data []byte
 	data, err = json.Marshal(m)

marshaling_test.go

@@ -38,6 +38,9 @@ func TestAPIErrorsList_UnmarshalJSON(t *testing.T) {
 	assert.Len(t, list, 2)
 	assert.Equal(t, list["a"], "first")
 	assert.Equal(t, list["b"], "second")
+
+	require.NoError(t, json.Unmarshal([]byte(`[]`), &list))
+	assert.Len(t, list, 0)
 }
 
 func TestCustomFieldsList_UnmarshalJSON(t *testing.T) {
@@ -52,6 +55,9 @@ func TestCustomFieldsList_UnmarshalJSON(t *testing.T) {
 	assert.Len(t, list, 2)
 	assert.Equal(t, list["a"], "first")
 	assert.Equal(t, list["b"], "second")
+
+	require.NoError(t, json.Unmarshal([]byte(`[]`), &list))
+	assert.Len(t, list, 0)
 }
 
 func TestOrderPayments_UnmarshalJSON(t *testing.T) {
@@ -66,4 +72,24 @@ func TestOrderPayments_UnmarshalJSON(t *testing.T) {
 	assert.Len(t, list, 2)
 	assert.Equal(t, list["a"], OrderPayment{ID: 1})
 	assert.Equal(t, list["b"], OrderPayment{ID: 2})
+
+	require.NoError(t, json.Unmarshal([]byte(`[]`), &list))
+	assert.Len(t, list, 0)
+}
+
+func TestProperties_UnmarshalJSON(t *testing.T) {
+	var list Properties
+
+	require.NoError(t, json.Unmarshal([]byte(`[{"code": "first"}, {"code": "second"}]`), &list))
+	assert.Len(t, list, 2)
+	assert.Equal(t, list["0"], Property{Code: "first"})
+	assert.Equal(t, list["1"], Property{Code: "second"})
+
+	require.NoError(t, json.Unmarshal([]byte(`{"a": {"code": "first"}, "b": {"code": "second"}}`), &list))
+	assert.Len(t, list, 2)
+	assert.Equal(t, list["a"], Property{Code: "first"})
+	assert.Equal(t, list["b"], Property{Code: "second"})
+
+	require.NoError(t, json.Unmarshal([]byte(`[]`), &list))
+	assert.Len(t, list, 0)
 }

types.go

@@ -275,6 +275,7 @@ Order related types
 
 type OrderPayments map[string]OrderPayment
 type StringMap map[string]string
+type Properties map[string]Property
 
 // Order type.
 type Order struct {
@@ -456,7 +457,7 @@ type OrderItem struct {
 	Comment               string     `json:"comment,omitempty"`
 	IsCanceled            bool       `json:"isCanceled,omitempty"`
 	Offer                 Offer      `json:"offer,omitempty"`
-	Properties            map[string]Property `json:"properties,omitempty"`
+	Properties            Properties `json:"properties,omitempty"`
 	PriceType             *PriceType `json:"priceType,omitempty"`
 }