The user object field value needs to be updated, commenting out 190 breaks this and results in out of date broken passwords on resync (when needed); I do not recommend this change. See the if check comparing the field value to the password.
Line 225 accounts for spacing and is multi-platform (windows and linux), using str_replace you'll end up with whitespace and user won't be able to login. Your change will also cause partial matches. For example bob:123 will match for abob:12345 breaking "abob" login.
It's already designed to update the password on profile update. Don't see how these changes would fix this if it was broken. Please clarify your proposal and as to what the exact issue is so can be investigated and fixed if issue is found. Please provide your server OS as well as will help determining what usage is failing.
Kyle (Krileon) Community Builder Team Member Before posting on forums:
Read FAQ thoroughly
+
Read our Documentation
+
Search the forums CB links:
Documentation
-
Localization
-
CB Quickstart
-
CB Paid Subscriptions
-
Add-Ons
-
Forge
-- If you are a Professional, Developer, or CB Paid Subscriptions subscriber and have a support issue please always post in your respective support forums for best results!
-- If I've missed your support post with a delay of 3 days or greater and are a Professional, Developer, or CBSubs subscriber please
send me a private message
with your thread and will reply when possible!
-- Please note I am available Monday - Friday from 8:00 AM CST to 4:00 PM CST. I am away on weekends (Saturday and Sunday) and if I've missed your post on or before a weekend after business hours please wait for the next following business day (Monday) and will get to your issue as soon as possible, thank you.
-- My role here is to provide guidance and assistance. I cannot provide custom code for each custom requirement. Please do not inquire me about custom development.
krileon
