Skip to content
This repository has been archived by the owner on Oct 26, 2019. It is now read-only.

Declaration of transformResponseBody() must be compatible #356

Open
4n70w4 opened this issue Jul 9, 2019 · 6 comments
Open

Declaration of transformResponseBody() must be compatible #356

4n70w4 opened this issue Jul 9, 2019 · 6 comments

Comments

@4n70w4
Copy link

4n70w4 commented Jul 9, 2019

Declaration of Jane\\OpenApiRuntime\\Client\\AmpArtaxEndpointTrait::transformResponseBody(string $body, int $status, Symfony\\Component\\Serializer\\SerializerInterface $serializer) must be compatible with Jane\\OpenApiRuntime\\Client\\Psr7HttplugEndpointTrait::transformResponseBody(string $body, int $status, Symfony\\Component\\Serializer\\SerializerInterface $serializer, ?string $contentType = NULL)
jane-php/json-schema-runtime          v4.4.0                                Jane runtime Library
jane-php/open-api-runtime             v4.4.0                                Jane OpenAPI Runtime Library, dependenc...
@4n70w4
Copy link
Author

4n70w4 commented Jul 9, 2019

Workaround. Add require to composer.json:

        "jane-php/open-api-runtime": "4.2",

@tlovett1
Copy link

This is broken currently. Why is there no composer.lock in docker-php-api?

@xtlsoft
Copy link

xtlsoft commented Jul 24, 2019

I've tested on PHP7.1.9 and PHP7.2.4, the issue still remains.

@alexjose
Copy link

alexjose commented Aug 1, 2019

seems like nobody is working on the repo these days.

@dangminhtruong
Copy link

dangminhtruong commented Aug 16, 2019

Go to this file vendor/jane-php/open-api-runtime/Client/Psr7HttplugEndpointTrait.php
Chage abstract function Declaration:

// FROM 

 transformResponseBody(string $body, int $status, SerializerInterface $serializer, string $contentType = null);

// TO

 transformResponseBody(string $body, int $status, SerializerInterface $serializer);

dangminhtruong added a commit to dangminhtruong/open-api-runtime that referenced this issue Sep 7, 2019
@diluy
Copy link

diluy commented Oct 12, 2019

Workaround. Add require to composer.json:

        "jane-php/open-api-runtime": "4.2",

It works for me, thanks @4n70w4

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants