Skip to content

Commit c5e10ff

Browse files
committed
Ensure all listener aggregates are attached correctly.
This patch ensures that all listener aggregates use the form: ```php $aggregate->attach($events); ``` vs ```php $events->attach($aggregate); ``` as the latter is unsupported in v3 versions of zend-eventmanager.
1 parent fc1c765 commit c5e10ff

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

Module.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ public function onBootstrap(MvcEvent $e)
3838
$serviceManager = $app->getServiceManager();
3939
$eventManager = $app->getEventManager();
4040

41-
$eventManager->attach($serviceManager->get('ZF\ApiProblem\ApiProblemListener'));
41+
$serviceManager->get('ZF\ApiProblem\ApiProblemListener')->attach($eventmanager);
4242
$eventManager->attach(MvcEvent::EVENT_RENDER, [$this, 'onRender'], 100);
4343

4444
$sendResponseListener = $serviceManager->get('SendResponseListener');
@@ -67,7 +67,7 @@ public function onRender(MvcEvent $e)
6767

6868
// register at high priority, to "beat" normal json strategy registered
6969
// via view manager, as well as HAL strategy.
70-
$events->attach($services->get('ZF\ApiProblem\ApiProblemStrategy'), 400);
70+
$services->get('ZF\ApiProblem\ApiProblemStrategy')->attach($events, 400);
7171
}
7272
}
7373
}

src/Listener/ApiProblemListener.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ public function onDispatch(MvcEvent $e)
137137

138138
// Attach the ApiProblem render.error listener
139139
$events = $app->getEventManager();
140-
$events->attach($services->get('ZF\ApiProblem\RenderErrorListener'));
140+
$services->get('ZF\ApiProblem\RenderErrorListener')->attach($events);
141141
}
142142

143143
/**

0 commit comments

Comments
 (0)