Protect: Change math check to init hook - #3686
Conversation
samhotchkiss
added
the
[Feature] Protect
jeherve
added
the
Bug
|
Fantastic! This fixes an issue with the bbPress login widget :) |
|
Funny thing though: it seems to work sometimes but most of the times it's not. Sometimes the math puzzle is shown below the bbPress login widget (when it's there, login is easy) but when it's not, login will always direct to a math puzzle which in turn (solved or not) will always direct to the main login.php which demands again a login + math puzzle... Which is very annoying. Any way to make the math puzzle show below the bbPress widget login form constantly? |
|
Showing the issue may be much clearer: https://status301.net/schermfilm-van-26-04-2016-213100/ (note the difference in the login widget form between the first and second time) |
jeherve
added
[Status] Needs Author Reply
|
Hi Ravan-- This, combined with another change we made, should significantly improve things-- once you're running all of 4.0.4, if you're still running into this on a regular basis, please submit a support request and I can help you out through that channel.\ Thanks! |
jeherve
removed
the
[Status] Needs Author Reply
* Protect: Change math check to init hook again See merge #3686 by @samhotchkiss and issue #5144 by @jeherve ... unless there is a particular reason to limit this to login_init? Additional change to line 61 for full bbPress Login Widget compatibility while keeping $pagenow fallback functional. * Update protect.php Fix action order (check_use_math after check_login_ability) while keeping Login Widget compatibility. * Update protect.php * Update protect.php
3 participants
Fixes an issue where the Protect math form doesn't show up on front end login forms
Changes proposed in this Pull Request: