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

User can bypass mandatory custom registration field #99

Closed
Lazylord opened this issue Apr 15, 2014 · 4 comments
Closed

User can bypass mandatory custom registration field #99

Lazylord opened this issue Apr 15, 2014 · 4 comments

Comments

@Lazylord
Copy link

Hello,

I have setup a mandatory custom field for user registration. However while using Firefox, if the user does not input anything, after one dialog box asking to enter the value it shows an option to "disable page from creating additional dialogs". Once this option is checked the user can bypass the custom registration field and will be registered successfully. This report is also present in my log file if you need it.

Pls help so users can't bypass the required field.

@jaswrks
Copy link
Contributor

jaswrks commented Apr 18, 2014

Thank you for the report.

This is a known limitation in the current release. Custom fields are validated via client-side JS only; not server-side too. I will leave this issue open while we work to improve on this.

@jaswrks
Copy link
Contributor

jaswrks commented May 18, 2014

  • Build validation routine
  • Integrate with Pro Forms
  • Integrate with Profile Editing Form
  • Integrate with BuddyPress Profile Editing
  • Integrate with WordPress default Registration Form

@jaswrks
Copy link
Contributor

jaswrks commented May 19, 2014

This has been completed.
This improvement will get pushed out with the next maintenance release of s2Member.

@jaswrks jaswrks closed this as completed May 19, 2014
jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue May 19, 2014
jaswrks pushed a commit that referenced this issue May 19, 2014
jaswrks pushed a commit that referenced this issue May 19, 2014
jaswrks pushed a commit to wpsharks/s2member-pro that referenced this issue May 19, 2014
@jaswrks
Copy link
Contributor

jaswrks commented May 19, 2014

Testing complete. Ready-to-go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants