kiota icon indicating copy to clipboard operation
kiota copied to clipboard

Path parameters don't support arrays and maps as input

Open darrelmiller opened this issue 2 years ago • 4 comments

This OpenAPI fragment

  /v2/properties/{properties}:
    get:
      operationId: v2_getSelectedProperties
      parameters:
        - name: properties
          in: path
          required: true
          schema:
            type: array
            items:
              type: string

generated the following indexer

public WithPropertiesItemRequestBuilder this[string position] { get {
            var urlTplParams = new Dictionary<string, object>(PathParameters);
            if (!string.IsNullOrWhiteSpace(position)) urlTplParams.Add("properties", position);
            return new WithPropertiesItemRequestBuilder(urlTplParams, RequestAdapter);
        } }

The type of the indexer parameter should be string[] and not string.

darrelmiller avatar Feb 19 '23 18:02 darrelmiller

Thanks for reporting this. Do we need it to be fixed for GA?

baywet avatar Feb 20 '23 14:02 baywet

No. It is not a common use case.

darrelmiller avatar Feb 21 '23 04:02 darrelmiller

Moved this to v2 as fixing it will be a breaking change

darrelmiller avatar Apr 21 '23 15:04 darrelmiller

note: we'd also need to address other types like int etc...

baywet avatar Apr 21 '23 16:04 baywet