How often password reset has been used ? update_password just contains: logaction( "MEMBERS", "CHANGE PASS", … ) So the log does not really tell you. The borrower_password_recovery has no autoincrement id; so no luck there too.. opac/opac-password-recovery.pl could perhaps send a different code to update_password ?
Dep 21178 for update_password vs set_password
Hi Marcel, I don't fully understand the problem you describe. Is this still valid?
(In reply to Katrin Fischer from comment #2) > Hi Marcel, I don't fully understand the problem you describe. Is this still > valid? Yes it is. Will send a patch.
Created attachment 158256 [details] [review] Bug 21431: Add action parameter to set_password Added to the call in opac-password-recovery.pl. This allows differentiating between password change and password reset when viewing the logs. Test plan: Enable BorrowersLog. Do a password recovery on OPAC. Check with log viewer for 'RESET PASS' action. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Created attachment 158257 [details] [review] Bug 21431: Add unit test Test plan: Run t/db_dependent/Koha/Patrons.t Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Created attachment 158258 [details] [review] Bug 21431: Adjust template for logviewer Test plan: Verify that you can filter by reset password. And that the uppercase code is replaced by a description. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Created attachment 158268 [details] [review] Bug 21431: Add action parameter to set_password Added to the call in opac-password-recovery.pl. This allows differentiating between password change and password reset when viewing the logs. Test plan: Enable BorrowersLog. Do a password recovery on OPAC. Check with log viewer for 'RESET PASS' action. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Created attachment 158269 [details] [review] Bug 21431: Add unit test Test plan: Run t/db_dependent/Koha/Patrons.t Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Created attachment 158270 [details] [review] Bug 21431: Adjust template for logviewer Test plan: Verify that you can filter by reset password. And that the uppercase code is replaced by a description. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
(In reply to Matt Blenkinsop from comment #9) > Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Thx Matt !
Created attachment 158660 [details] [review] Bug 21431: Add action parameter to set_password Added to the call in opac-password-recovery.pl. This allows differentiating between password change and password reset when viewing the logs. Test plan: Enable BorrowersLog. Do a password recovery on OPAC. Check with log viewer for 'RESET PASS' action. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 158661 [details] [review] Bug 21431: Add unit test Test plan: Run t/db_dependent/Koha/Patrons.t Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 158662 [details] [review] Bug 21431: Adjust template for logviewer Test plan: Verify that you can filter by reset password. And that the uppercase code is replaced by a description. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
All working as described.. QA scripts happy and tests passing. Passing QA
Pushed to master for 23.11. Nice work everyone, thanks!
Enhancement not pushed to 23.05.x
I believe we have a list of these in the manual (I see to remember translating these)