Make logger in `ErrorHandler` optional

[vjik]

[xepozz]

Why?

[vjik]

ErrorHandler can work without logger.

[xepozz]

There're no reasons to make it optional

[vjik]

There're no reasons to make it optional

Optional always is better than a mandatory one. Creating of object is simplier.

[xepozz]

NullLogger is there for this purpose

[vjik]

NullLogger is there for this purpose

Yes, optional: private readonly LoggerInterface $logger = new NullLogger()

[olegbaturin]

NullLogger is there for this purpose

There are no reasons to force user to provide logger in case logging in not required, even a NullLogger. Nullable param is more expected, in general.