Wrong generated code with generic type as argument

[woprandi]

Wrong generated code with generic type as argument

[TekExplorer]

Note: retrofit has no way of knowing what to call feomJson on, so it would need to implement something like genericArgumentFactories where we pass in the fromJson to the method

[trevorwang]

There's no reflection for lutter. So, you must tell retrofit the exact type before it generates code for you. Refer ApiResult<t> for an example. @woprandi @ekuleshov

[ekuleshov]

@trevorwang appreciate response. I'm aware of the issue involved with both generics and absence of reflection.

However it is a valid case to have rest api to receive or return a generic argument, when the type is not known at the compilation time. For such types the JsonSerializable can generate special to/fromJson() methods that are taking additional parameters (i.e. genericArgumentFactories option).

Currently retrofit generator is generating an invalid code like T.fromJson(_result.data!) for such classes and it doesn't seem like it allows to pass generic arg factories as code generated by the JsonSerializable is expecting.

It would be really helpful and would allow to deal with the issue without writing a bunch of boilerplate code, if we could specify additional parameters for the retrofit endpoint method, so the consuming code could provide appropriate generic factories and these factories would be wired with the to/fromJson calls generated by retrofit generator.

For example, a rest endpoint declaration could look like this:

  @POST('/api/config/save')
  @Headers({'Content-Type': 'application/json'})
  Future<Config<T>> saveConfig<T>(
          Config<T> config,
          Object? Function(T value) toJson,
          T Function(Object? json) fromJson);

Note that Config<T> class is declared like that and defined the same generic arg factories params in to/fromJson methods.

@JsonSerializable(genericArgumentFactories: true)
class Config<T> {
  ...
  T details;
  
  factory Config.fromJson(Map<String, dynamic> json, T Function(Object? json) fromJsonT)
       => _$ReportConfigFromJson(json, fromJsonT);

  Map<String, dynamic> toJson(Object? Function(T value) toJsonT) => _$ReportConfigToJson(this, toJsonT);
}

[Raviteja11122]

Is there any work around for this issue?