People get confused adding a CAPTCHA on a contact form - see #2865967: Admin interface for CAPTCHA on contact forms is confusing.

We can fix it as follows:

  • Delete captcha.captcha_point.contact_message_feedback_form.yml. The contact module does not define a matching default contact form, so this is really confusing.
  • Enhance captcha_install to automatically create captcha points for contact forms similar to existing code for nodes.

See also #3005577: Create CAPTCHA points on demand not on install - could create on demand rather than on install.

Issue fork captcha-3005579

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

AdamPS created an issue. See original summary.

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Parent issue: » #3005577: Create CAPTCHA points on demand not on install
Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Related issues: +#2948379: Captcha point configuration should have a dependency on the configuration object it applies to
Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Version: 8.x-1.x-dev » 2.x-dev
Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Status: Active » Needs review

Delete captcha.captcha_point.contact_message_feedback_form.yml. The contact module does not define a matching default contact form, so this is really confusing.

has been done in the MR.

@AdamPS: Should captcha.captcha_point.contact_message_personal_form.yml also be removed or does that one exist?

All other things should indeed be done in #3005577: Create CAPTCHA points on demand not on install!

Anybody opened merge request !44

AdamPS’s picture

AdamPS CreditAttribution: AdamPS at AlbanyWeb commented
Status: Needs review » Reviewed & tested by the community

Great thanks.

@AdamPS: Should captcha.captcha_point.contact_message_personal_form.yml also be removed or does that one exist?

This one is fine and should stay - the "Personal contact form" always exists and cannot be deleted.

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Status: Reviewed & tested by the community » Fixed

Thank you @AdamPS! Merged :)

  • Anybody committed a7633a5b on 2.x 
    Issue #3005579 by Anybody: Better interface for CAPTCHA on contact forms
AdamPS’s picture

AdamPS CreditAttribution: AdamPS at AlbanyWeb commented
Status: Reviewed & tested by the community » Fixed

Great thanks

Status: Fixed » Closed (fixed)

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