EdEichman Posted November 10, 2015 Share Posted November 10, 2015 In case anyone is interested, I wrote up my override of the PrestaShop "forgotten password" feature. My objections to the standard method... Two emails to the client are needed. Not all emails arrive instantly, so this can slow down a customer that wants to do business with you. A password is being sent as plain text to the user. The password auto-generated by PrestaShop is hard for the user to remember. The customer's secure key is sent out as plain text (it's the "token" parameter in the URL of the first email). Comments and criticisms welcomed. Thanks, Ed 2 Link to comment Share on other sites More sharing options...
Bill Dalton Posted November 11, 2015 Share Posted November 11, 2015 Have not yet tried it but I certainly agree with your method. Very nice of you to share what looks like a complete solution. I hope to try this on a test site in the next few days. 1 Link to comment Share on other sites More sharing options...
bellini13 Posted November 11, 2015 Share Posted November 11, 2015 I'm curious, why not just submit these changes to their Forge so they can be included in the next core release of Prestashop? 1 Link to comment Share on other sites More sharing options...
EdEichman Posted November 11, 2015 Author Share Posted November 11, 2015 Hi Bellini, Ummm, do you think so? Wow, I'm flattered, I guess LOL (literally) ;-> I know the concept of contributing to an open source project, but I've never done so - I was just trying to scratch my itch. I'll take a look after work to so how to go about it. Thanks for the push. Have a good one, Ed Link to comment Share on other sites More sharing options...
bellini13 Posted November 11, 2015 Share Posted November 11, 2015 I erroneously said Forge, but meant github, where they manage their code base https://github.com/PrestaShop/PrestaShop To learn how to contribute... http://doc.prestashop.com/display/PS16/Contributing+to+PrestaShop Link to comment Share on other sites More sharing options...
EdEichman Posted November 12, 2015 Author Share Posted November 12, 2015 No problem... I understood what you meant yesterday. Thanks! Link to comment Share on other sites More sharing options...
EdEichman Posted November 17, 2015 Author Share Posted November 17, 2015 I'm curious, why not just submit these changes to their Forge so they can be included in the next core release of Prestashop? Hi Bellini13, Before doing a large pull request, I started small with a one line fix of a clear bug I found in the order confirmation process to see how this all worked. From there came into contact with Maxime, a core developer, who was lightning fast at responding to my pull request, and pointed out that the development branch already had what appears to be a very similar approach to forgotten passwords. For those interested in doing pull requests, currently bug fixes should go to the 1.6.1.x branch, and new features to the development branch. Thanks! Ed 3 Link to comment Share on other sites More sharing options...
betrayed Posted April 24, 2016 Share Posted April 24, 2016 Hello, I used your code, i got the following " an error occured while preparing the email ", any help would be appreciated as i'm new to prestashop, thanx. Link to comment Share on other sites More sharing options...
Bill Dalton Posted April 24, 2016 Share Posted April 24, 2016 I never did find the time to try Ed's code. I'm sure it will do the job. Instead I gave this module a try and it does a good job, $45 bucks and I was done, plug & play. http://addons.prestashop.com/en/11169-password-recovery-high-security-password-storage.html Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now