no captcha support

If someone wants to create a patch for non-image captcha (e.g. arithmetic based), and make it optional, I will include it as a feature.

For the OP, you could revert to using the built-in Contact module which seems to have a good set of functionality to me: it has complete captcha support. Or is there some particular reason you need feedback?

Here's how you can hack it for now, I've never created a patch nor can I code the "optional" functionality. This hack applies to the captcha.module file.

Find 'node_form' => t('Create a node'), [It should be on line 34]
After that, insert 'feedback_mail_page' => t('Send Feedback'),

You'll now have the math based captcha (or image-based, if you've installed the textimage module and changed captcha's settings) on your Feedback page.

Hope this helps you for now; sorry I can't make this "proper".

Nick

If someone gets this working, please provide a patch.

We have to make this an option, not mandatory.

Thanks NickHBO, but it didn't work for me. It has worked for all other custom modules I've added to the captcha.module, but not this one.

I'm starting to get a little annoyed of all spam seeping through my feedback forms because otherwise I'm beginning to have the spam-problem under control...

I was able to get this working by adding this line in the captcha.module:

'feedback_mail_page' => t('Feedback Form'),

So, moving this issue to the captcha module.

It would be nice if the captcha module provides a hook so modules add their own entry into the table, and they show up in the captcha settings page.

To add support for captcha to the feedback 5.x module, you need to add this line:

'feedback_mail_form' => t('Feedback Form'),

http://drupal.org/node/134880

This issue solves the problem for feedback and all other modules that need captcha by introducing a new hook.

If this gets committed then there is no need to edit the captcha module at all for any module that needs a captcha.

Captcha already allows you to add captcha points without pain.

This solution requires a third module, which at upgrade time creates yet another dependency, and you have to wait for it.

The hook is simple enough to include, and less code is better always over the long term.

Sorry Khalid,
But using the third module allow us to give to the user a better interface, and I think that adding a hook just to do that isn't a pretty solution.
Anyway your patch do not apply in HEAD.