Extract response generator from ErrorCatcher middleware

[olegbaturin]

Q	A
Is bugfix?	❌
New feature?	✔️
Breaks BC?	❌
Fixed issues	#129

"psr/event-dispatcher" must be in the composer.json "require" section?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.31%. Comparing base (033468a) to head (e8f5683).
Report is 42 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master     #133      +/-   ##
============================================
- Coverage     78.86%   77.31%   -1.56%     
- Complexity      170      180      +10     
============================================
  Files            13       14       +1     
  Lines           511      551      +40     
============================================
+ Hits            403      426      +23     
- Misses          108      125      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[samdark]

Good one overall but needs minor adjustments before being merged.

[olegbaturin]

What about "psr/event-dispatcher" dependency? It's absent in the master branch. Should it be added to the composer.json in this pr?

[vjik]

What about "psr/event-dispatcher" dependency? It's absent in the master branch. Should it be added to the composer.json in this pr?

Not need. psr/event-dispatcher is optional dependency.

[olegbaturin]

Not need. psr/event-dispatcher is optional dependency.

I've added it to the "require-dev", because it required for tests.

[vjik]

Good refactoring 👍