search

zendframework / zend-expressive-template

Template subcomponent for Expressive
BSD 3-Clause "New" or "Revised" License
20 stars 5 forks source link

Remove stdlib #4

Closed bakura10 closed 8 years ago

bakura10 commented 8 years ago

ping @weierophinney

This follows the same approach as in service manager :).

Also, if this is merged, it replaces #2 .

bakura10 commented 8 years ago

As a side note, isn't there a way to deprecate at some point those strange "MergeInterface"? Is there really someone using those? Do we need to have those in the inlined polyfill (especially here?)

weierophinney commented 8 years ago

@bakura10 They exist because developers needed the functionality. If you cannot completely control the merge order (e.g., your configuration is distributed as part of a 3rd party library), these interfaces allow you to shape how your configuration is merged. As such: we need to keep them.

weierophinney commented 8 years ago

Actually, in this case, we do not need those features; we can assume that those were taken care of during configuration merging.

I'll update the patch during merge.