mekras / botobor Goto Github PK

Есть ли лишние методы? Все ли методы находятся в своём классе? Есть ли большие методы, которые следует разбить на более мелкие?

Нужна возможность отключать определённые проверки. Например, в некоторых, невыясненных, случаях Opera 11.5x не передаёт Referer и проверка на этот заголовок проваливается.

Можно сохранять в сессии мета-данные формы или их контрольную сумму. Т. к. в них хранится время создания формы, они будут уникальными.

Пример API:

if (Botobor_Keeper::isResubmit())
{
  …
}

Если отключена проверка honeypots (через Botobor или Botobor_Form), поля-приманки не должны создаваться.

Добавить возможность хранить мета-данные в сессии, а в форме передавать только их идентификатор. Возможно сделать это основным способом передачи мета-данных.

Иногда может быть полезным узнать, какая именно проверка не прошла. Это может быть нужно в целях отладки или для показа пояснительного сообщения пользователю.

В большинстве случаев надо проверять именно на то, что форма заполнена роботом, а не на то, что она заполнена человеком. Т. е. чаще всего пишется такой код:

if (!Botobor_Keeper::isHuman())

так что представляется логичным инвертировать смысл проверки.

Можно сделать в качестве дополнительной, отдельно подключаемой возможности, разбор кода формы при помощи парсера HTML при модификации этой формы.

Добавить проверку с внедрением кода JavaScript и описать в README использование её совместно с CAPTCHA при отключённом JavaScript.