Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
17 Feb 2012 at 18:33 UTC
Updated:
31 Aug 2012 at 10:25 UTC
Forgot your password is a simple module that improves the user experience for the default Request New Password tab on the user login page.
Project page: http://drupal.org/sandbox/kruser/1444262
Git: git clone --branch 7.x-1.x kruser@git.drupal.org:sandbox/kruser/1444262.git forgot_your_password
For Drupal 7
http://ventral.org/pareview/httpgitdrupalorgsandboxkruser1444262git
Comments
Comment #1
svenaas commentedThis looks helpful, and I haven't found with the code (which is straightforward, uses t() where I expect it to, etc.). I also confirmed that PAReview isn't reporting any issues.
Comment #2
ankitchauhan commentedhi kruser
This project is bit short, At least 120 lines of code or 5 function should be there.
See the link http://groups.drupal.org/node/195848
Comment #3
misc commentedHi kruser, and welcome with your application.
You need to update your project page with more information and the information above according to our guidelines, see http://drupal.org/node/1187664 - the git repo should not be your private one, it should be git clone http://git.drupal.org/sandbox/kruser/1444262.git forgot_your_password. Are there similar modules on drupal.org?
A manual review:
Have you considered in submitting a patch to core, instead of doing a module?
Personally I think that the text should be configured in the admin interface instead of stored as strings.
Comment #4
misc commentedComment #5
klausiClosing due to lack of activity. Feel free to reopen if you are still working on this application.
Comment #5.0
klausiwrong branch