search

fenom-template / fenom

Template Engine for PHP. Maintainers wanted!
Other
446 stars 108 forks source link

Вопросы по коду + баги #59

Closed WinterSilence closed 10 years ago

WinterSilence commented 10 years ago

для чего используется опция disable_statics ? она встречается во всем коде только 1 раз https://github.com/bzick/fenom/blob/master/src/Fenom.php#L69

Почему метод clean объявлен статическим и открытым? нашел его использование только в еще одном статическом методе rm, который кроме как в тестах нигде не используется. https://github.com/bzick/fenom/blob/master/src/Fenom/Provider.php#L27

WinterSilence commented 10 years ago

да и еще в Provider'e вызывается метод clearstatcache, у вас первым параметром null, а в документации написано что должно быть булево значение clearstatcache ([ bool $clear_realpath_cache = false [, string $filename ]] ), кроме того если использовать null, то смысла задавать второй параметр нет т.к. используется только если параметр clear_realpath_cache установлен в TRUE.

bzick commented 10 years ago

О, спасибо за code review :) disable_statics - это резерев на следующую версию, отключает использование статических методов в шаблоне {MyClass::method}, но не успел до 1.4 и отложил недочистив. метод clean() этого некоторого рода допущение, нужен для тестов и очистки компиленых шаблонов. просто я так и не решил куда его вынести. c clearstatcache, действительно, какая-то бага

WinterSilence commented 10 years ago

а сейчас разве конструкции вида {MyClass::method} работают? у меня с ними как раз самая главная проблема возникла - пишу модуль интеграции фреймворка kohana и fenom'a, нужно подключить классы-хэлперы, но никак не удается из-за их статичности (

bzick commented 10 years ago

Пока нет, но с 1.5 будет поддержка

WinterSilence commented 10 years ago

спасибо за работу)