Page MenuHomePhabricator

ConfirmAccount checks the skipcaptcha right directly
Closed, ResolvedPublic

Description

As a follow on to T398850: Class "ReCaptchaNoCaptcha" not found, ConfirmAccount probably shoudn't be checking the skipcaptcha right directly, it should use SimpleCaptcha::canSkipCaptcha

Event Timeline

Change #1181118 had a related patch set uploaded (by Reedy; author: Reedy):

[mediawiki/extensions/ConfirmAccount@master] Don't check the skipcaptcha right directly

https://gerrit.wikimedia.org/r/1181118

Change #1181118 merged by jenkins-bot:

[mediawiki/extensions/ConfirmAccount@master] Don't check the skipcaptcha right directly

https://gerrit.wikimedia.org/r/1181118

Change #1182546 had a related patch set uploaded (by Reedy; author: Reedy):

[mediawiki/extensions/ConfirmAccount@REL1_44] Don't check the skipcaptcha right directly

https://gerrit.wikimedia.org/r/1182546

Change #1182548 had a related patch set uploaded (by Reedy; author: Reedy):

[mediawiki/extensions/ConfirmAccount@REL1_43] Don't check the skipcaptcha right directly

https://gerrit.wikimedia.org/r/1182548

Change #1182546 merged by jenkins-bot:

[mediawiki/extensions/ConfirmAccount@REL1_44] Don't check the skipcaptcha right directly

https://gerrit.wikimedia.org/r/1182546

Reedy claimed this task.

Change #1182548 merged by jenkins-bot:

[mediawiki/extensions/ConfirmAccount@REL1_43] Don't check the skipcaptcha right directly

https://gerrit.wikimedia.org/r/1182548