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

Password Validation js link 500 error #2679

Closed
jHards opened this issue Dec 31, 2022 · 3 comments · Fixed by #2698
Closed

Password Validation js link 500 error #2679

jHards opened this issue Dec 31, 2022 · 3 comments · Fixed by #2698
Labels
Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on.

Comments

@jHards
Copy link
Contributor

jHards commented Dec 31, 2022

On version 6.4.3, using tag {exp:member:validation_url fields="password"} returns a 500 error. The docs suggest that this should be allowed.

The issue seems to be in mod.member.php line 2719 where function passwordValidate is called dynamically but doesn't exist.

I'm happy to propose a fix but I'm not sure what the original intention of that code block is - any direction appreciated.

@intoeetive intoeetive added the Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. label Jan 4, 2023
@intoeetive
Copy link
Contributor

@jHards looks like those functions have been refactored, but the Member module wasn't updated properly. Thanks!

@jHards
Copy link
Contributor Author

jHards commented Jan 4, 2023

@jHards looks like those functions have been refactored, but the Member module wasn't updated properly. Thanks!

Happy to support if you point me in the right direction?

@intoeetive
Copy link
Contributor

Do not worry, I'll get the fix ready soon!

bryannielsen added a commit that referenced this issue Jan 9, 2023
…ation-url

Resolved #2679 where password validation URL was giving PHP error
bryannielsen added a commit that referenced this issue Jan 9, 2023
…ation-url

Resolved #2679 where password validation URL was giving PHP error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants