bccampus / validate-by-domain Goto Github PK

Mapping of user role to capability only works if the default role is set to 'subscriber'

Early this morning there were two successful user registrations that by-passed the check for domains during a sign up event. The domains used were mail.ru and wugjeyxwgv.pl.

IP 114.37.245.67 is located in Taiwan
IP 191.101.65.167 is located in Germany

When the user doesn't have a valid domain, or is on the blacklist, an hard coded error message is displayed: "Please use an email address from an allowed agency or institution within British Columbia" ... bring this into the options page to allow the user to set both the blacklist and whitelist error messages.

Get this option working, to allow the user to turn off the functionality in the options page without having to disable the plugin.

Currently there is no way to set the domains on the Blacklist without editing the code, bring this into the plugin options just like the Whitelist.

the value to check against field_155 is not reliable/stable across sites.

https://github.com/BCcampus/validate-by-domain/blob/dev/public/class-validate-by-domain-public.php#L292

https://github.com/BCcampus/validate-by-domain/blob/dev/public/class-validate-by-domain-public.php#L393

https://github.com/BCcampus/validate-by-domain/blob/dev/public/class-validate-by-domain-public.php#L394

Make sure there's only one domain per line, and remove all unnecessary characters

The domains in the whitelist are supposed to stop the registration of organizer accounts without a valid domain. Dev branch currently allows registration with any domain, and ignores the whitelist.

Description

Is this plugin work with Pressbooks.

Expected behaviour

[What you expect/want to happen]

Actual behaviour

[What actually happened]

Steps to reproduce the problem

[First Step, Second Step, etc]

System Information

• Validate by Domain: [Enter Version here]
• WordPress: [Enter WP Version number here]

User currently has update the hardcoded value, bring it into the plugin options or figure out a way to make this work without the need to set the field value.

Let the user set which role that user gets

The plugin assumes that we're using BuddyPress and it's sign-up page. Let's Add a check to make sure BP plugin is active, or that a BP function exists.

Description

The similarity between bccampus/bc-validate and bccampus/validate-by-domain justifies the effort to investigate incorporating the use case for bc-validate. Maintaining two plugins that do similar things doubles the effort.

Expected behaviour

The scope of validate-by-domain should be expanded to incorporate more and broader use cases.

Actual behaviour

The scope is currently limited to one particular use case with very specific dependencies (ie buddypress, specific sign-up fields)

Let the user add the validate by domain functionality on an e-mail input field of their choice.