search

zfcampus / zf-versioning

BSD 3-Clause "New" or "Revised" License
11 stars 7 forks source link

Make segment route type the default #17

Closed fabiang closed 6 years ago

fabiang commented 8 years ago

segment should be set as type when merging routes. Could effect PR #13

weierophinney commented 7 years ago

What does this solve, exactly?

fabiang commented 7 years ago

This patch prevents errors, when user add zf-versioning to their existing routes. This happend to me and was not easy to debug.

weierophinney commented 6 years ago

@fabiang It also breaks functionality if the route is another type (e.g., regex). Forcing the route to be a segment route in this way introduces new problems.