search

laminas / laminas-cli

Console command runner, exposing commands written in Laminas MVC and Mezzio components and applications
https://docs.laminas.dev/laminas-cli
BSD 3-Clause "New" or "Revised" License
55 stars 22 forks source link

Fixed code sample showing listener being added, use subscribers instead #95

Closed dkmuir closed 2 years ago

dkmuir commented 2 years ago

ErrorListener needs to be attached using addSubscriber() instead of addListener()

Signed-off-by: dkmuir david@davidkmuir.com

Q A
Documentation yes
Bugfix no
BC Break no
New Feature no
RFC no
QA no

Description

Sample code for ConsoleEventDispatcherFactory is wrong.

Ocramius commented 2 years ago

CI failures are unrelated:

Error: docs/book/index.md:6:2 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
Error: docs/book/index.md:8:2 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
Error: docs/book/index.md:10:2 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
Error: docs/book/index.md:12:2 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
Error: docs/book/index.md:14:2 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
Error: docs/book/index.md:16:2 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
Error: docs/book/index.md:18:2 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]