Use PSR-17 ResponseFactory draft instead of ResponsePrototype

[codeliner]

Closes #3

See #4 for discussion

[coveralls]

Pull Request Test Coverage Report for Build 27

• 81 of 81 (100.0%) changed or added relevant lines in 36 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 100.0%

---

Totals
Change from base Build 22:	0.0%
Covered Lines:	472
Relevant Lines:	472

---

💛 - Coveralls

[prolic]

Looks good, but where is the JsonTransformer class?

[codeliner]

@prolic JsonTransformer is not part of this package.
It needs to be injected: https://github.com/prooph/event-store-http-middleware/pull/5/files#diff-9782dc5d06a5247229375e61a092313eR28
by implementing the Transformer interface. I'll add an example implementation (which uses Diactoros JsonResponse) to the docs, so users can copy & paste it, but it is userland implementation.

[prolic]

Why not pass the response factory to the json transformer? Then you can set appropriate headers and json encode the body. So the package has the json transformer implementation.

…

On Sun, Apr 8, 2018, 01:32 Alexander Miertsch ***@***.***> wrote: @prolic <https://github.com/prolic> JsonTransformer is not part of this package. It needs to be injected: https://github.com/prooph/event-store-http-middleware/pull/5/files#diff-9782dc5d06a5247229375e61a092313eR28 by implementing the Transformer interface. I'll add an example implementation (which uses Diactoros JsonResponse) to the docs, so users can copy & paste it, but it is userland implementation. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#5 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAYEvG1yh9zR9ALAvW51j8j8NyMSBt0Xks5tmPgCgaJpZM4TKoS6> .

[prolic]

I'm going to merge this for now, but please let's implement the json-transformer in a separate PR. We don't need a distinct JsonResponse implementation, the default provided by response factory is good enough.