There's an extra token in the query of the module. This patch fixes it.

CommentFileSizeAuthor
#2 fix-captcha-on-node-form-columns-1393090-2.patch809 bytesptrl
nodeformcolum_captcha_error.patch611 bytesAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aitala’s picture

HI,

I'm getting a notice now...

Notice: Undefined index: buttons in nodeformcols_nodeformcols_base_form_alter() (line 24 of ..../sites/all/modules/contrib/nodeformcols/nfccaptcha.module).

Eric

ptrl’s picture

Issue summary: View changes
Status: Active » Needs review
FileSize
809 bytes

This module doesn't seem to be so active anymore (and certainly not the CAPTCHA compatability submodule) but we use it on a old site so there is a updated patch that makes the CAPTCHA compatability module work again.

The two things this patches does is:

  1. Count on column captcha_type instead of module
  2. And check weight on action instead of buttons
Hugo Wetterberg’s picture

Hi ptrl, you're right about the module not being active, modules tend to slip when you no longer depend on them yourself. Would you like to become a maintainer?

ptrl’s picture

I understand completely Hugo, hope i didn't come of as hostile, wasn't my intend.

Thanks for the question but since I also no longer depend on it i wouldn't help much, think only I have active project that still uses the module. But I'll continue submit patches if I'll run into any problems on that site with nodeformcols.

Hugo Wetterberg’s picture

Ah, no worries, my reply was short because I was replying from the phone and was short on time, not because of grumpiness.

I guess that I'll just have to look through the issue queue and find some candidates and get the rtbc patches in.

Hugo Wetterberg’s picture

Status: Needs review » Closed (fixed)