Codegen: Possibility to combine method arguments into a single object

For example, the C# code below

public class FooQuery
{
  public string Bar { get; set; }
  public string Baz { get; set; }
  public string FooBar { get; set; }
  public string FooBaz { get; set; }
  public int? FooBarBaz { get; set; }
}

public class FooController
{
  [HttpGet, Route]
  public ICollection<Foo> ReadAll([FromUri]FooQuery query = null)
  {
    // ...
  }
}

generates the following TypeScript method:

export class FooApi {
  readAll(bar: string | null, baz: string | null, fooBar: string | null, fooBaz: string | null, fooBarBaz: number | null): Observable<Foo | null> {
    // ...
  }    
}

Clearly, the generated method is cumbersome and error-prone to use and code using it will break if we add another property to FooQuery.

My proposal is to generate a FooApiReadAllQuery class when there are >1 arguments and use that class as an argument to the method:

export class FooApiReadAllArgs {
  bar: string | null;
  baz: string | null;
  fooBar: string | null;
  fooBaz: string | null;
  fooBarBaz: number | null;
}

export class FooApi {
  readAll(query: FooApiReadAllArgs): Observable<Foo | null> {
    // ...
  }
}

This will:

• Make adding new arguments to the API controller backward-compatible with existing code
• Make such methods easier and safer to use

Is such an implementation welcome as a pull request in this repo? It seems that it’s relatively easy to this to the templates and configuration.

[goldsam]

This would really be a great feature when implementing collection endpoints with lots of filter parameters.

[feafarot]

Any updates on this feature? It’s really painful to manage 20+ parameters.