When checking "Add CAPTCHA administration links to forms" checkbox in captcha settings form, user with permission "administer CAPTCHA settings" don't see the fieldset provided to add/show captcha to current form. However, we require also "skip CAPTCHA" permission for the fieldset to show on all forms. Can we change this description in captcha.admin.inc:130

We should update CAPTCHA's captcha_form_alter to reflect the proper permission.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JayKandari created an issue. See original summary.

arunkumark’s picture

I have updated the documentation and created patch for the CAPTCHA 7.x-1.x version.

arunkumark’s picture

Status: Active » Needs review
rajnishkumar’s picture

Status: Needs review » Reviewed & tested by the community

Verified changes on Captcha 7.x-1.4 . RTBC

wundo’s picture

Status: Reviewed & tested by the community » Needs work

That is actually a bug, not only a documentation issue.

wundo’s picture

Title: Change config description for "Add CAPTCHA administration links to forms" » "Add CAPTCHA administration links to forms" should be triggered for every user with "administer CAPTCHA settings" permission
Version: 7.x-1.4 » 7.x-1.5
Component: Code » Captcha API (captcha)
Assigned: Unassigned » wundo
Issue summary: View changes
Issue tags: -Novice
wundo’s picture

Here is a quick patch that should fix the bug.

  • wundo authored 4a70042 on 7.x-1.x
    Issue #2897761 by wundo, arunkumark, JayKandari, rajnishkumar: "Add...
elachlan’s picture

Status: Needs review » Fixed
mnarlock’s picture

Pardon my ignorance, but is this patch for Captcha 7.x-1.5 or is 1.5 the fixed module? I'm running 1.5 now and currently have this issue wherein only the superadmin sees the fieldset. Regular admins do not.

wundo’s picture

Hi @mnarlock, this has been committed earlier this week but we haven't released a new release with it included yet.

Status: Fixed » Closed (fixed)

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