search

fuel / parser

Fuel PHP Framework - v1.x template parser package adapters
http://fuelphp.com
64 stars 45 forks source link

Adding Multiton support for Mustache Parser #39

Closed tgriesser closed 12 years ago

tgriesser commented 12 years ago

Necessary modification for a package i'm working on which needs a separate instance of \View_Mustache::parser() outside of it's typical use with the \View::forge('template.mustache')

jschreuder commented 12 years ago

As an implementation this is too limited. The View class itself is incapable of using it as a multiton and it takes the View_Mustache out of sync with the other View classes without being technically necessary. Thus you essentially made View_Mustache a multiton frontend for the Mustache class, which is wrong on many levels.