search

zfcampus / zf-api-problem

BSD 3-Clause "New" or "Revised" License
27 stars 26 forks source link

Fix event priority. #13

Closed gsc-frank closed 10 years ago

gsc-frank commented 10 years ago

Hal's renderer listener is attached with priority 100 too, so the 'ZF\ApiProblem\ApiProblemStrategy' listener is executed before the rendering just because right now the module ApiProblem loade first than the module Hal, and that is not cool. I pick 400 as priority value becuase is important that this listener be executed after the ApiProblemListener::onRender listener, and the priority of that listener is 1000.

See https://github.com/zfcampus/zf-apigility/pull/17 for related issue.

gsc-frank commented 10 years ago

Sorry, my bad, I confused ZF\ApiProblem\ApiProblemStrategy with ZF\ApiProblem\RenderErrorListener, so this change is wrong, I will close the PR.