Fix SetMarshalData for PutSingle request #65

dstepanov-yadro · 2024-02-15T12:41:04Z

dstepanov-yadro commented

2024-02-15 12:41:04 +00:00

Allow to reset marshal data also for put single request

dstepanov-yadro force-pushed fix/put_single_request_marshal_data from 4cafb5342c to 2e83fecbdc

2024-02-15 12:42:37 +00:00

Compare

dstepanov-yadro requested review from storage-core-committers 2024-02-15 12:42:49 +00:00

dstepanov-yadro requested review from storage-core-developers 2024-02-15 12:42:50 +00:00

dstepanov-yadro force-pushed fix/put_single_request_marshal_data from 2e83fecbdc to 634e24aba7

2024-02-15 12:44:16 +00:00

Compare

fyrchik reviewed 2024-02-15 12:44:36 +00:00

util/proto/marshal.go

					
				@ -287,6 +287,11 @@ func NestedStructureSetMarshalData[T any, M setMarshalData[T]](field int64, pare

						return 0

					}

					if parentData == nil {

fyrchik commented

2024-02-15 12:44:31 +00:00

How is this related?

dstepanov-yadro commented

2024-02-15 13:34:04 +00:00

Now r.marshalData can be nil

func (r *PutSingleRequestBody) SetMarshalData(data []byte) {
	if r == nil {
		return
	}

	r.marshalData = data

	proto.NestedStructureSetMarshalData(putSingleReqObjectField, r.marshalData, r.object)
}

So if parentData == nil then we should reset nested marshalData

Now `r.marshalData` can be `nil` ``` func (r *PutSingleRequestBody) SetMarshalData(data []byte) { if r == nil { return } r.marshalData = data proto.NestedStructureSetMarshalData(putSingleReqObjectField, r.marshalData, r.object) } ``` So if `parentData == nil` then we should reset nested marshalData