In function honeypot_get_protected_forms() there is a following SQL request:
SELECT name FROM {variable} WHERE name LIKE '%honeypot_form_%'

As I understand, it should be:
SELECT name FROM {variable} WHERE name LIKE 'honeypot_form_%'

It will be faster and there will be no chances to get some wrong variables (like "something_honeypot_form_something").

CommentFileSizeAuthor
#1 Fix_SQL_pattern-1450438-1.patch1.19 KBdmitriy.trt

Comments

dmitriy.trt’s picture

StatusFileSize
new1.19 KB

Patch attached

geerlingguy’s picture

Version: 6.x-1.9 » 6.x-1.x-dev
Status: Needs review » Reviewed & tested by the community

Right you are - I'll fix this up in 6.x and 7.x. Shouldn't make too big a difference performance-wise, but is definitely the right way to go :)

geerlingguy’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.