Sterc / FormIt

A dynamic form processing Snippet for MODX Revolution
https://docs.modx.com/current/en/extras/formit
33 stars 58 forks source link

Bug on hasHook() #182

Closed lokamaya closed 5 years ago

lokamaya commented 6 years ago

(boolean) function hasHook()

Using strpos() to evaluate hooks is very unreliable, as it is prone to hook collision. For example when using ReCaptchaV2 as hooks, its generates an error because FormIt automatically tries to load the ReCaptcha class.

See MODx Forum thread/103830

lokamaya commented 6 years ago

It is same here https://github.com/Sterc/FormIt/pull/181