Pierre-Luc Posted July 27, 2015 Share Posted July 27, 2015 I just published a pre-release of my new password reset module. It will enable identification from any field of your choice, and is fully translatable. Integration is just 2 lines of code, the first is calling the controller, the second is loading a script (no dependencies). You can download the code on GitHub at https://github.com/plauclair/PasswordReset. Also, have a look at the reset process in this video https://vid.me/eEVY. This exemple is not styled, but there should be all you need in there to style it. Comments and feature requests very welcome! 12 Link to comment Share on other sites More sharing options...
Macrura Posted July 28, 2015 Share Posted July 28, 2015 this is great - last project we needed this on we did something like this, which worked ok: <?php if($notices->count) { echo '<ul id="notices">'; foreach($notices as $notice) { echo "<li>$notice->text</li>"; } echo '</ul>'; } else { echo "<div class='form-container'>"; $controller = new ProcessController(); $controller->setProcessName('ProcessForgotPassword'); echo $controller->execute(); echo "</div>"; } one problem i noticed was the link in plaintext was breaking, so wasn't clickable, i was thinking that it would be good if the email could be sent in html where the link could be in an anchor tag Link to comment Share on other sites More sharing options...
Pierre-Luc Posted July 28, 2015 Author Share Posted July 28, 2015 I really want to push the usability side of this one. If you've found other shortcomings of ForgotPassword that would like to be improved, let me know. We used to use ProcessForgotPassword too but it's not working that great for us. Link to comment Share on other sites More sharing options...
Pierre-Luc Posted July 28, 2015 Author Share Posted July 28, 2015 @Macrura -- Did the url break in PW's reset module or mine? Also what messaging client (if any)? Link to comment Share on other sites More sharing options...
Macrura Posted July 28, 2015 Share Posted July 28, 2015 sorry - i meant PW reset has hardcoded message that sends plaintext; the url was breaking in Outlook for Mac, i think some newer email clients don't break the URL 1 Link to comment Share on other sites More sharing options...
Pierre-Luc Posted July 28, 2015 Author Share Posted July 28, 2015 Alright! I haven't tested the URL so far in a lot of mail clients so was kinda wondering. ——— Just posted a new release. v.0.0.3 changes https://github.com/plauclair/PasswordReset/commit/95a61b91ccca276b6c9eef1b3884af0f98250c58 Link to comment Share on other sites More sharing options...
mr-fan Posted July 29, 2015 Share Posted July 29, 2015 Great module...i like it. Tested here with two different Mail Clients and RoundCube online....all works like a charm. Put this in the module directory if you are ready....very good extension. Best regards mr-fan Link to comment Share on other sites More sharing options...
jacmaes Posted July 29, 2015 Share Posted July 29, 2015 Hi Pierre-Luc, Looks great! is there a check for a minimum number of characters? Link to comment Share on other sites More sharing options...
Pierre-Luc Posted July 29, 2015 Author Share Posted July 29, 2015 Thanks guys. @mr-fan it needs to be production ready this week so for sure it's going to be there pretty soon. @jacmaes yes currently there is a length check in the module properties. There will be a strength check at some point, but probably not at v.1. 1 Link to comment Share on other sites More sharing options...
Pierre-Luc Posted July 29, 2015 Author Share Posted July 29, 2015 Just pushed v.0.0.4 Fixes : https://github.com/plauclair/PasswordReset/commit/fa9cdcbea7425c8efc8eb316573df89870f2c802 Link to comment Share on other sites More sharing options...
Pierre-Luc Posted July 29, 2015 Author Share Posted July 29, 2015 … and v.0.0.5 Changes : https://github.com/plauclair/PasswordReset/commit/cd1b8041906eaf31848e55a0973af6b2362a93ac Link to comment Share on other sites More sharing options...
Pierre-Luc Posted September 13, 2015 Author Share Posted September 13, 2015 Hey I've been wondering if any of you who are currently using this in production or testing have any feedback? I want to do a proper release soon, and I'd love your input! Cheers! 1 Link to comment Share on other sites More sharing options...
Beluga Posted November 6, 2015 Share Posted November 6, 2015 Hey I've been wondering if any of you who are currently using this in production or testing have any feedback? I want to do a proper release soon, and I'd love your input! Cheers! I just put this to use on a site, because I realized the core password reset module does not have good user experience. It is too much to expect a user to keep their tab open after clicking submit or that they have to do the reset on the same device. So thanks for this traditional reset module! Link to comment Share on other sites More sharing options...
Pierre-Luc Posted November 6, 2015 Author Share Posted November 6, 2015 My pleasure Beluga. Link to comment Share on other sites More sharing options...
Robin S Posted May 3, 2016 Share Posted May 3, 2016 Thanks for this module, Pierre-Luc. I get a couple of errors when using it. When assigning the template "password-reset" to a page or attempting to edit a page with the template assigned I get: ProcessWire: ProcessPageEdit: Unknown template Edit: this error seems to be caused by a conflict between Password Reset and the Batch Child Editor module. After uninstalling Batch Child Editor the error message is not shown. And when viewing a page with this template assigned on the frontend I get: Notice: Trying to get property of non-object in ...\site\modules\PasswordReset\PasswordReset.module on line 32 Looking at line 32 maybe this error is because my site is not multi-language? Link to comment Share on other sites More sharing options...
adrian Posted May 3, 2016 Share Posted May 3, 2016 The unknown template error is due to a bug in the settings for the password-reset template. 67 password-reset 121 0 0 {"childTemplates":[120] There is no template with an id of 120 in my system. 120 refers to the fieldgroup for the password-reset-request template, rather than the id of that template itself. This is the culprit: https://github.com/plauclair/PasswordReset/blob/master/PasswordReset.module#L279 That said, I have added an additional check to BatchChildEditor so the error won't occur. I am currently working on some other changes to BCE so I won't commit this fix just yet, but you can do it yourself by changing: https://github.com/adrianbj/BatchChildEditor/blob/master/BatchChildEditor.module#L711 if(!$this->isAllowedTemplateAdd($templateId, $pp)) continue; to: if(!$t->id || !$this->isAllowedTemplateAdd($t, $pp)) continue; 1 Link to comment Share on other sites More sharing options...
Pierre-Luc Posted May 11, 2016 Author Share Posted May 11, 2016 Thanks Adrian, I'll look if there are other side effects and better ways to fix this one and commit something by this weekend. Link to comment Share on other sites More sharing options...
adrian Posted May 11, 2016 Share Posted May 11, 2016 I think all you need to do is change: $passwordResetTemplate->childTemplates = [$requestFieldgroup->id]; to: $passwordResetTemplate->childTemplates = [$requestTemplate->id]; but maybe there is something more to it that I have overlooked. Link to comment Share on other sites More sharing options...
Pierre-Luc Posted May 11, 2016 Author Share Posted May 11, 2016 Most probably. When you said it was referencing the fieldset I knew something else was fishy. By the way I know from the GitHub stats that a few of you use the module. Is it working as expected, do you think I should provide a default stylesheet? I'd really like some feedback on the direction to go with. Link to comment Share on other sites More sharing options...
Ivan Gretsky Posted November 17, 2016 Share Posted November 17, 2016 @Pierre-Luc Thank you for your module! Please keep it going. Here are my wishes for the development roadmap. Make it possible to fully customize the markup. Maybe with some kind of template overrides or something. I think there is no need for anything default, rather an easy way to make it your own. Ajax would be a good option. A bit more docs would help. I did experience some bugs when using it with installation having users with multiple templates. It took some time to figure out that I should configure the passwordResetUser field to use multiple templates. And there should be a 1.0.0 version in the module directory, of course. I did check it on 3.x - seems to work fine. 1 Link to comment Share on other sites More sharing options...
jacmaes Posted December 17, 2016 Share Posted December 17, 2016 I'm getting the following error on PW 3.045: Error: Uncaught Error: Call to a member function of() on boolean in /home/my-domain/public_html/site/modules/PasswordReset/PasswordReset.module:187 Stack trace: #0 /home/my-domain/public_html/site/modules/PasswordReset/PasswordReset.module(235): PasswordReset->resetPassword() #1 /home/my-domain/public_html/site/templates/password-reset.php(3): PasswordReset->controller() #2 /home/my-domain/public_html/wire/core/TemplateFile.php(268): require('/home/my-domain...') #3 /home/my-domain/public_html/wire/core/Wire.php(374): ProcessWire\TemplateFile->___render() #4 /home/my-domain/public_html/wire/core/WireHooks.php(682): ProcessWire\Wire->_callMethod('___render', Array) #5 /home/my-domain/public_html/wire/core/Wire.php(402): ProcessWire\WireHooks->runHooks(Object(ProcessWire\TemplateFile), 'render', Array) #6 /home/my-domain/public_html/wire/modules/PageRender.module(514): ProcessWire\Wire->__call('render', Array) (línea 187 de /home/my-domain/public_html/site/modules/PasswordReset/PasswordReset.module) Link to comment Share on other sites More sharing options...
modifiedcontent Posted November 8, 2020 Share Posted November 8, 2020 I have to come up with a 'forgot password' feature for my custom login/register process. I hoped I could just use the core module ProcessForgotPassword, but see it hasn't been updated since 2012 and it is not clear to me how to use it. How do I add ProcessForgotPassword to my login form/template? The module is installed in the backend, but then what? Are there instructions anywhere? Is @Pierre-Luc's module safe to use for the latest Processwire versions? What/where is the latest version? Is anyone else using it with improvements? Are there other solutions/modules? I'll probably work on this next weekend and post more findings. Link to comment Share on other sites More sharing options...
netcarver Posted November 9, 2020 Share Posted November 9, 2020 Folks, Pete and I have a solution for the most commonly encountered password reset/login feature requests encountered across various large-scale membership sites we've worked on, including feedback from a site with ~40k users. We are polishing it up at the moment and will announce more in the next couple of days. 9 Link to comment Share on other sites More sharing options...
modifiedcontent Posted November 21, 2020 Share Posted November 21, 2020 @netcarver, any news on your solution? 1 Link to comment Share on other sites More sharing options...
netcarver Posted November 22, 2020 Share Posted November 22, 2020 Hi @modifiedcontent, yes, it's now all packaged up as a module and currently in testing. I posted about it here. Please be aware, as mentioned in that post, Pete and I intend to release this as a commercial module sometime this week. 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