Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.While the documentation page does mention that the form time limit is in seconds, the configuration form (admin/config/content/honeypot) doesn't make any reference to the unit of time. This patch updates the help text to mention that the form field is in seconds.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | honeypot-form-time-suffix-1502850-3.patch | 393 bytes | yannickoo |
| 0001-honeypot-Clarify-the-unit-time-of-the-time-limit.patch | 1.38 KB | chaseontheweb |
Comments
Comment #1
yannickooThe patch looks good maerleythedog, thanks. In future you should name the patches like it says in the documentation for creating patches:
git diff > [project_name]-[short-description]-[issue-number]-[comment-number].patchTo know the issue number you should first create your issue and immediately after that you can add a comment with the patch attached.
Comment #2
loominade commentedwouldn't it be better to use
'#suffix' => t('seconds')instead?Comment #3
yannickooNice idea loominade but we should consider which idea is better. Even patch with seconds suffix is attached.
Comment #4
geerlingguy commentedProbably going to go the route of using #suffix; thanks for the patches and reviews! I'll try to commit this soon, when I get back to my laptop.
Comment #5
geerlingguy commentedCommitted: http://drupalcode.org/project/honeypot.git/commit/3262839 - makes things a bit nicer!