Skip to content


Subversion checkout URL

You can clone with
Download ZIP


Unnecessary path assembly on return in Zend\Mvc\Router\Http\TreeRouteStack->assemble() line 236 #2857

Blinden opened this Issue · 3 comments

3 participants


The return statement of Router\Http\TreeRouteStack::assemble line 236 could return $path that is already build and assigned on line 219.


$options is not exactly the same in both cases, in the first (l:219) may have the option 'uri' set

I think that is possible add a else at the line 233 with return $path; inside.

Can you send us a patch with the improvement?


I issued PR #3013 with the patch.

Instead of adding another else I refactored the if so that on $options['only_return_path'] it returns the assembled path immediatly. The return at the end can then just return $path.

@Blinden Blinden closed this

Please do not close an issue before the patch has been merged.

@weierophinney weierophinney reopened this
@ghost Unknown referenced this issue from a commit
@weierophinney weierophinney Merge branch 'hotfix/3013'
Close #3013
Fix #2857
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.