Use proper function signature for recaptcha_captcha().

CommentFileSizeAuthor
#4 recaptcha-2232701-4.patch617 bytescs_shadow
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

  • Commit 6d10b06 on 7.x-1.x by Liam Morland: 
    Issue #2232701: Use proper function signature for recaptcha_captcha().
Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented
Status: Active » Fixed
Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented
Version: 7.x-1.x-dev » 8.x-1.x-dev
Status: Fixed » Patch (to be ported)
cs_shadow’s picture

cs_shadow CreditAttribution: cs_shadow commented
Status: Patch (to be ported) » Needs review
FileSize
recaptcha-2232701-4.patch617 bytes

Attaching a patch backported for 8.x-1.x

  • Liam Morland committed 6d10b06 on 7.x-1.x, 7.x-2.x 
    Issue #2232701: Use proper function signature for recaptcha_captcha().

  • Liam Morland committed 9d52bf2 on 7.x-1.x 
    Issue #2232701: Pass arguments to captcha_captcha() in recaptcha_captcha...

  • Liam Morland committed 61627cc on 7.x-1.x 
    Issue #2232701: Proper function signature for captcha_captcha().
Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented
Status: Needs review » Needs work

Please check my latest commits; the 8.x-1.x should be similar and can have the change done all as one commit!

  • Liam Morland committed 923762d on 8.x-1.x 
    Issue #2232701: Use proper function signature for recaptcha_captcha().
Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented
Status: Needs work » Fixed

Status: Fixed » Needs work

The last submitted patch, 4: recaptcha-2232701-4.patch, failed testing.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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