Комментарии 7
Что-то я не понял, а почему тут пусто?
https://github.com/evgentus/antispam/blob/master/classes/kohana/antispam.php
Спс. Исправляю.
Ещё одно замечание. Раз уж делаете в стиле Koahana, то
self:: везде замените на имя конечного класса (Antispam), иначе весь transparent extension коту под хвостСпасибо за замечание. Поправил.
Жень, тогда уж и isHuman() ==> is_human()
kohanaframework.org/3.0/guide/kohana/conventions
kohanaframework.org/3.0/guide/kohana/conventions
1. Методы isHuman() и getForm() статические, поэтому и вызывать их надо без использования фабрики.
2. Какой смысл в конструкторе несколько раз грузить конфиг? Достаточно один раз — когда $botobor_class пустой. Аналогично с правилами, секретом и т.д. — их можно добавить в Botobor через статические методы ОДИН раз.
В целом, складывается ощущение жуткой поделки на коленке, с использованием неизученной библиотеки. Абы как.
PS. В голову пришла мысль — такие вещи было бы прикольно прикручивать к штатному валидатору, как правило (callback). Очень полезная штучка получилась бы.
2. Какой смысл в конструкторе несколько раз грузить конфиг? Достаточно один раз — когда $botobor_class пустой. Аналогично с правилами, секретом и т.д. — их можно добавить в Botobor через статические методы ОДИН раз.
В целом, складывается ощущение жуткой поделки на коленке, с использованием неизученной библиотеки. Абы как.
PS. В голову пришла мысль — такие вещи было бы прикольно прикручивать к штатному валидатору, как правило (callback). Очень полезная штучка получилась бы.
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Анти-спам модуль для Kohana