Open boesing opened 5 years ago
/cc @weierophinney @webimpress
@boesing I think there might be some references to this class in the documentation so we need update it. Other than that LGTM 👍
This repository has been closed and moved to laminas/laminas-stratigility; a new issue has been opened at https://github.com/laminas/laminas-stratigility/issues/2.
This repository has been moved to laminas/laminas-stratigility. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:
git rebase -i origin/{branch}
)
As already mentioned in #190, there is a naming conflict in the
ErrorHandler
.I had some feedback of trainees which asked me, why the
ErrorHandler
ain't aRequestHandler
, as its name says. That made me think about the naming and I realized, its just because of the missingMiddleware
postfix.However, I'd like to rename the
ErrorHandler
toErrorHandlerMiddleware
to avoid future confusions about where the difference betweenRequestHandlerInterface
andMiddlewareInterface
is.