merged branch fabpot/tokenparserbroker (PR #903)
authorFabien Potencier <fabien.potencier@gmail.com>
Fri, 16 Nov 2012 17:44:55 +0000 (18:44 +0100)
committerFabien Potencier <fabien.potencier@gmail.com>
Fri, 16 Nov 2012 17:44:55 +0000 (18:44 +0100)
commit7072d8f54f79b878cd17c4192d8a4a1f45bef173
tree46557d9fa13c04171f719b32f59566590ee2643b
parent158ed4e88cc336724c9605f5f15a6ef859466703
parent8563e040c9c45b3c07d73cd8397be1788a6ff49f
merged branch fabpot/tokenparserbroker (PR #903)

This PR was merged into the master branch.

Commits
-------

8563e04 deprecated the token parser broker sub-system

Discussion
----------

deprecated the token parser broker sub-system

This is the first of a series of PRs where I want to deprecate some features that made sense at some point but do not anymore or features that are barely used. Deprecated features will still work in all versions of Twig 1.x but they will be removed in Twig 2.0.

This first one should not be controversial. The token parser broker was added at a time where functions in Twig were not available. Nowadays, creating a new tag is so rare that I cannot see any usage for the broker system.

As far as I know, nobody uses the broker system anyway (this feature is not even documented), except [Zwig](https://github.com/arnaud-lb/Zwig). And Zwig should probably create functions instead of tags for ZF helpers anyways.

ping @arnaud-lb

---------------------------------------------------------------------------

by arnaud-lb at 2012-11-16T17:27:02Z

IIRC the main reason for using this in Zwig was that ZF makes it hard to list helpers, and easy to get an helper by name. So the broken allows to use ZF helpers without listing them. The reason for using tags instead of functions is that many helpers in ZF are not used for their return value and/or printing. (Zwig also exposes helpers as functions, through registerUndefinedFunctionCallback, for those used for their return value or printing.)

No objection for removing in 2.x though.