Closed (fixed)
Project:
Password Policy
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Jul 2012 at 06:11 UTC
Updated:
5 Jul 2014 at 23:30 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
erikwebb commentedPlease give this a test.
Comment #2
P3t3r commentedNeither fix works for me (7.x-1.1 on core 7.15). Still getting redirected to the non-password part.
Comment #3
erikwebb commented@P3t3r - What is the exact URL that you are being redirected to? Is it
/useror/user/###/edit?Comment #4
P3t3r commentedI'm getting redirected to
MYSITE/user/55/edit?pass-reset-token=MhM7TUMSN0Eak5-jCuRZaFHSqehI4awUW-_O_r336wII noticed that there has appeared a new wild entry "Password tab" in the root (!) of my admin menu, but the setting there doesn't seem to do anything for this problem (probably it's only for after the pasword is effectively set).
Comment #5
P3t3r commentedJust installed a brand new Drupal 7.15 with default settings and only password tab and mail system (to receive the registration mail) as modules.
Even after the fix you present above, the problem persists: the login link in the registration looks like
http://m.y.i.p/test/user/reset/5/1344623528/upLdFUqjXZyuhm7q_zCfpjNgpBXhX68hYCMRHvFSz6Eand I end up at page
http://m.y.i.p/test/user/5/edit?pass-reset-token=PalM7vNljHEsKKlystackDcySE7A-OH8aMt43NztTZMin the browser bar (in IE 7 if it's relevant).
At the edit tab thus. Not at the password tab.
Comment #6
mloigeret commentedI tried applying this patch to the stable module 7.x-1.1 and it did not work.
I applied the patch password_policy-update_password_change_tab_d7-1330502-23.patch (from http://drupal.org/node/1330502 #23) first and then I could apply the present patch.
But now I get the error Notice : Undefined index: pass in password_policy_password_tab() (line 41 in ....mysite/sites/all/modules/password_policy/contrib/password_tab/password_policy_password_tab.pages.inc).
When I validate the form it seems that there is still a need for the old password, which I am not supposed to have because the user requests to reset his password.
I am using Drupal core 7.15
Am I missing something ?
Comment #7
mloigeret commentedEasier to reproduce... I get the same error message (i.e. Notice : Undefined index: pass in password_policy_password_tab() (line 41 dans .....mysite/sites/all/modules/password_policy/contrib/password_tab/password_policy_password_tab.pages.inc). if I apply this patch over dev version 7.x-1.x-dev .
Comment #8
erikwebb commentedSince Password tab is now committed with the 7.x-1.2 release, this may need some new testing.
Comment #9
chaloum commentedif you change line 80 in the password_policy_tab.module from
$path .= '/password';
to
$path=str_replace("/edit","/password",$path);
It should work for you
Comment #10
erikwebb commentedCleaner is always better.
Comment #11
erikwebb commentedNew and improved!
Comment #12
villette commentedI have updated the patch #11 to work with the versions from 7.x-1.4
Comment #14
jacktonkin commented12: password_policy-incorrect_redirect_after_reset-1680146-12.patch queued for re-testing.
Comment #16
aohrvetpv commentedNew patch, same code as #12. Reproduced problem and confirmed this fixes.
Comment #18
aohrvetpv commentedD6 doesn't have
hook_drupal_goto_alter(), so this will require a different approach for 6.x, assuming the bug exists there.Comment #19
aohrvetpv commentedComment #20
aohrvetpv commentedDoes not affect 6.x-1.x. This bug was due to attempting to redirect to the 6.x-1.x Password Tab password page path (
user/X/edit/password) instead of the proper 7.x-1.x path (user/X/password).