Added Template::____call() for BC #38
Conversation
| */ | ||
| public function __call($name, $args) | ||
| { | ||
| trigger_error('Invoking filters on Template object outside of template file is deprecated, use your filter directly.', E_USER_DEPRECATED); |
There was a problem hiding this comment.
Something like …use getLatte()->invokeFilter() is better, isn't?
There was a problem hiding this comment.
Filters are usualy shortcuts for existing functions/methods. If this is deprecated, spaghetti like getLatte()->invokeFilter() is definitely a no-go and it's better to directly use that function/method, that the filter is an alias for.
There was a problem hiding this comment.
But it is lie, it is not deprecated, i.e. invokeFilter is not deprecated.
There was a problem hiding this comment.
I believe using the filter directly is better than spaghetti. If you don't like the formulation, write here how you want it and I will change it.
There was a problem hiding this comment.
You said „I know it's not a best practise, but often I would use the template object helpers in presenter or control.“ :-)
I belive that E_DEPRECATED message should explaind, how to quickly fix it.
fprochazka
force-pushed
the
bc/tpl-call-2
branch
4 times, most recently
from
20625fb to
a5e76f7
Compare
fprochazka
force-pushed
the
bc/tpl-call-2
branch
from
a5e76f7 to
91f5644
Compare
|
merged f6e5a09 |
Related #37