Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make sure cap user is active user. #319

Merged
merged 3 commits into from
Oct 23, 2024
Merged

Conversation

StevenDufresne
Copy link
Contributor

@StevenDufresne StevenDufresne commented Oct 22, 2024

I think this solves an issue with displaying the get_enable_2fa_notice in the support forums. It is being displayed because it's running for every user included in the forum table. Run it only for current user.

wporg-two-factor.php Outdated Show resolved Hide resolved
@StevenDufresne StevenDufresne marked this pull request as ready for review October 23, 2024 01:16
@StevenDufresne StevenDufresne merged commit fe0a981 into trunk Oct 23, 2024
2 checks passed
@StevenDufresne StevenDufresne deleted the fix/bbpress-2fa-alert branch October 23, 2024 01:16
@StevenDufresne StevenDufresne changed the title Don't rely on user passed into user_has_cap. Make sure cap user is active user. Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants