Fixes #4936

[kusmierz]

It will fix #4936 issue.

[Ocramius]

Requires test case

[kusmierz]

@Ocramius I've added. Dunno how it write it better.

[Ocramius]

@kusmierz the test should be related to Chain, not to the stdlib component: you changed code only in the Mvc component after all ;-)

[kusmierz]

Indeed, you've right. I hope there is no more usages of "key()" function on PriorityList in code (I've checked it quickly).

Is it correct now?

[Ocramius]

This test should be on ChainTest, as that's where the code changes are.

[kusmierz]

I've finally found time to write it. @Ocramius could you check it if it's ok now? Thanks in advance.

Without my fix, the new test will fail on:

Failed asserting that two strings are equal.
Expected :/foo
Actual   :/foo/bar
 <Click to see difference>

zf2/tests/ZendTest/Mvc/Router/Http/ChainTest.php:180
 phar:///usr/share/php/phpunit.phar/phpunit/TextUI/Command.php:151

[kusmierz]

@Ocramius, have you added this to v2.3.5?

[Ocramius]

@kusmierz no, as you can see it's scheduled for 2.4.0