[WebClient] ResponsePredicate create type no found

[pcariel]

package io.unknow.client;

import io.vertx.reactivex.core.buffer.Buffer;
import io.vertx.reactivex.ext.web.client.HttpResponse;
import io.vertx.reactivex.ext.web.client.predicate.ErrorConverter;
import io.vertx.reactivex.ext.web.client.predicate.ResponsePredicate;

public class UnknowClientPredicate {

  public final static ResponsePredicate predicate = ResponsePredicate.create(
    ResponsePredicate.SC_SUCCESS, ErrorConverter.createFullBody(result -> {
      HttpResponse<Buffer> response = result.response();
      if (response.getHeader("content-type").equals("application/json"));
      return new Exception();
    }));
}

Wrong 1st argument type. Found: 'io.vertx.reactivex.ext.web.client.predicate.ResponsePredicate', required: 'java.util.function.Function<io.vertx.reactivex.ext.web.client.HttpResponse<java.lang.Void>,io.vertx.reactivex.ext.web.client.predicate.ResponsePredicateResult>'

Hello everyone, I need your help.
Excuse me for my bad English

[tsegismont]

@pcariel good catch

This is due to a limitation in vertx-codegen. I'll check with @vietj how to solve it. In the meantime, here's a workaround:

    ResponsePredicate predicate = ResponsePredicate.create(response -> {
      return ResponsePredicateResult.newInstance(ResponsePredicate.JSON.getDelegate().apply(response.getDelegate()));
    }, ErrorConverter.createFullBody(result -> {
      HttpResponse<Buffer> response = result.response();
      if (response.getHeader("content-type").equals("application/json")) ;
      return new Exception();
    }));

[tsegismont]

@vietj similar to eclipse-vertx/vertx-codegen#252 in fact. We need to support Function in codegen, in addition to Iterable and Iterator.

[vietj]

I think that might go beyond what we want to do.

[tsegismont]

I think that might go beyond what we want to do.

Why? How is it different from checking if an API object implements Iterator or Iterable?

If we don't do this, what do you suggest to solve this issue?

[vietj]

yes I see what you mean better, so in practice io.vertx.reactivex.ext.web.client.predicate.ResponsePredicate should extend Function and that is what we are missing right ?

[tsegismont]

Exactly