-
Notifications
You must be signed in to change notification settings - Fork 68
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
Confirm registration validation is working properly #2321
Comments
I checked on this a little. I found that PHP also validates the data so there appears to be no danger of double-registering the same username (for example) or other possible data-corruption. The old @jusuff did you say you can look into re-activating ajax validation here or should we discuss another solution? |
Just add Webshim for the HTML5 validation, as a polyfill for browser support. |
I don't know anything about those technologies. A PR would be welcome! |
@shefik this is not about simple checks like mandatory fields. This issue is about extended validation, like username may not be already used, password must not be equal username, email address may not be already used, and so on. |
@Guite Webshim is still needed for browsers such as Safari on Mac, to add support for attributes such as "required". As an example, the attribute "required" works correctly on Chrome on Mac, and the form does not submit via HTML5 validation. |
@shefik did your PR only add the library? Did you implement any part of the solution? When I asked for a PR, I assumed it would be understood that you would also do that. |
@craigh Yes, it only added the Webshim library for the moment, as well as some validation for the installer. I can submit another pull request to specifically address the registration/user forms. I've been going through much of the core in progression, so when I get fully past the installer tickets, I can move on to User and Profile mods, and so on. |
Feel free to submit many smaller PRs instead of one large one at the end. I would rather have smaller commits anyway |
Make sure not to validate any hooks via Ajax, because this breaks certain modules (such as the Captcha module). |
I'm not seeing any usage of this in the installer. can you be more specific where it was added? |
ping @shefik feedback requested ^^^ |
@craigh To see it working, use Safari on Mac, you will see that HTML5 form validation is working on the "required" fields in installer. Without webshim, Safari did not fully support HTML 5 form validation. Therefore, the forms were able to bypass validation. The webshim displays the appropriate validation message and prevents the form from being submitted if invalid. As a comparison, view the same pages via Chrome on Mac. That particular browser already has built-in support for HTML 5 form validation. Therefore, the webshim is not used. |
What's about this ticket? Does anyone work on it? And I mean initial issue - validating via ajax user entries (eg - is username free to use). |
You are 😉 ASAP 😄 |
Ok :) |
Closed by #2377 |
#1805 was closed without merging
#1804 removed all the old prototype files and didn't replace any code.
2da396f removed same from admin side since all the files were gone anyway
it is unknown if any JS validation is occurring before a user is created.
has anyone replaced the old js validation?
@phaidon @shefik @jusuff @Guite
The text was updated successfully, but these errors were encountered: