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

New Option to Allow Override "Anyone Can Register" Wordpress Setting #55

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vinull
Copy link

@vinull vinull commented Sep 7, 2016

This adds a new option that let's a user override the Wordpress option "users_can_register" to let WP-OAuth register new users while having the site closed to new Wordpress accounts.

The use case for this is wanting to have some users (admins) log in with Wordpress accounts, thus keeping the wordpress login form visible, while user accounts require OAuth logins. Enabling "Anyone Can Register" leaves the site owner open to spam registrations. I know, I had this problem which is why I made this pull request =)

Please note, the file register.php had multiple line ending types detected in the same file and even my beloved vim could not cope. The only change is line 12, the rest are just fixing the line endings.

Also, there is a small change on the style sheet to fix the options page. On my firefox the "toggle" options were pushing down col 2 and causing col 1 to stack beneath it.

Fixes style issue that caused options page columns to stack instead of
display side-by-side
This adds an option to override the "Anyone Can Register" option in
Wordpress, allowing the site to be closed to new normal accounts yet
open to new accounts with WP-OAuth
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.

1 participant