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 user password mismatch #2

Closed
mindblender opened this issue Feb 28, 2015 · 0 comments
Closed

New user password mismatch #2

mindblender opened this issue Feb 28, 2015 · 0 comments

Comments

@mindblender
Copy link
Member

@mindblender mindblender commented Feb 28, 2015

From SF #296 (https://sourceforge.net/p/phpcollab/bugs/296/)
I have just installed phpCollab and tried to create the first "proper" user. Because I discovered that the system does not cope with special symbols (ticket #245), I have specified password generator parameters so as to disallow special symbols and make length 16 chars to keep reasonable password length.
I pasted the generated password to the new user "password" and "retype password" fields, only to discover that the system complains about the "mismatched passwords". HTML inpector in Chrome showed that the "password" field max length is 15, while the "retype" field max length is 16. So my 16-long password was trimmed during text paste in the first box, but not the second, hence mismatch.

mindblender added a commit that referenced this issue Mar 7, 2015
Reorganized the language array to make it more readable.

Refactored the language detection code to set the default language if one is not already set in the settings.php file.  Also added logic so that the app can detect the user's browser preferences and display the site accordingly.

Created a method that returns a language drop down list.  This is better than having multiple static lists in various places.

Made the includes for the various language files more solid.  It checks to see if the file exists before trying to include it.  This should intercept any errors with the include.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant