Problem/Motivation

From discussion in comment #9 in issue #2808087: Improve captcha_install(), correctly create dynamic captcha points, move static captcha points to default config
The second drupal_set_message is broken (page cache is a module now, not a setting).
Other modules don't do this either, the modules page has links to settings and help built-in.

Proposed resolution

Possibly add a link to the settings instead

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#8 2811777-8-8.x-1.x-dev.patch2.2 KBamit.drupal
#6 interdiff-6.txt1.29 KBprashant.c
#6 2811777-6-8.x-1.x-dev.patch1.61 KBprashant.c
#2 changes_2811777.png31.05 KBlomasr
#2 2811777_2.patch1.61 KBlomasr

Comments

Ginovski created an issue. See original summary.

lomasr’s picture

lomasr commented
StatusFileSize
new 2811777_2.patch1.61 KB
new changes_2811777.png31.05 KB

I agree with the proposed solution . Added a patch.

lomasr’s picture

lomasr commented
Status: Active » Needs review
ddrozdik’s picture

ddrozdik commented

I guess this is an important message, I suggest to make it bold, to take user attention on it.

ddrozdik’s picture

ddrozdik commented
Status: Needs review » Needs work
prashant.c’s picture

prashant.c
he/him
Primary language English
Location Dharamshala
commented
Status: Needs work » Needs review
StatusFileSize
new 2811777-6-8.x-1.x-dev.patch1.61 KB
new interdiff-6.txt1.29 KB

Re-submitting the patch by adding <b> tag to the returned output.

ddrozdik’s picture

ddrozdik commented
Status: Needs review » Reviewed & tested by the community

looks good.

amit.drupal’s picture

amit.drupal commented
StatusFileSize
new 2811777-8-8.x-1.x-dev.patch2.2 KB

Re-submitting Patch #6 .

+++ b/captcha.module

+ return $output;
return render array rest looks good.

return ['#markup' => $output];

wundo’s picture

wundo commented
wundo’s picture

wundo commented
Status: Reviewed & tested by the community » Fixed

  • wundo committed 6ded687 on 8.x-1.x authored by amit.drupal 
    Issue #2811777 by lomasr, Prashant.c, amit.drupal, ddrozdik, Ginovski,...

Status: Fixed » Closed (fixed)

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

jomik’s picture

jomik commented

Hi all,
when I trey to install CAPTCHA by drupal 8, I get this message:
Unable to install CAPTCHA, captcha.captcha_point.contact_message_feedback_form, captcha.captcha_point.contact_message_personal_form, captcha.captcha_point.user_login_form, captcha.captcha_point.user_pass, captcha.captcha_point.user_register_form, captcha.settings already exist in active configuration.
Can anybody help me?
Thanks in advance,
Joseph