Bug 26191

Summary: Relocate track_login call in Auth.pm (see 22543)
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: AuthenticationAssignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: aleisha, dpavlin, jonathan.druart+koha, lucas, martin.renvoize, victor
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.11.00, 20.05.06, 19.11.12
Bug Depends on: 22543    
Bug Blocks:    
Attachments: Bug 26191: Relocate track_login call in Auth.pm
Bug 26191: Relocate track_login call in Auth.pm

Description Marcel de Rooy 2020-08-11 11:21:23 UTC
From bug 22543 comment30:

(In reply to Jonathan Druart from comment #29)
> 2. This code has been put before track_login_daily, is that expected?

No. The track call should be done before.
Comment 1 Marcel de Rooy 2020-08-11 11:30:40 UTC
Created attachment 108052 [details] [review]
Bug 26191: Relocate track_login call in Auth.pm

This is a leftover from bug 22543.
Trivial move.

Test plan:
Do not apply this patch.
Pick a user that has not yet logged in today.
Only login via the opac and immediately check if borrowers.datelastseen did not change.
Apply this patch, restart, flush etc.
Only login via the opac and verify again rightaway (no further opac actions).
Now datelastseen should have been changed already.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 2 Marcel de Rooy 2020-10-02 09:44:39 UTC
Still applies, taking liberty to self signoff before we can also rebase this one liner ;)
Comment 3 Martin Renvoize 2020-10-05 15:14:20 UTC
Created attachment 111243 [details] [review]
Bug 26191: Relocate track_login call in Auth.pm

This is a leftover from bug 22543.
Trivial move.

Test plan:
Do not apply this patch.
Pick a user that has not yet logged in today.
Only login via the opac and immediately check if borrowers.datelastseen did not change.
Apply this patch, restart, flush etc.
Only login via the opac and verify again rightaway (no further opac actions).
Now datelastseen should have been changed already.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Martin Renvoize 2020-10-05 15:14:45 UTC
Passing QA, patch is clear and simple and works as expected.
Comment 5 Jonathan Druart 2020-10-08 09:38:01 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 6 Lucas Gass 2020-11-09 18:32:47 UTC
backported to 20.05.x for 20.05.06
Comment 7 Aleisha Amohia 2020-11-17 03:24:53 UTC
backported to 19.11.x for 19.11.12
Comment 8 Victor Grousset/tuxayo 2020-11-17 09:13:28 UTC
Missing dependencies for 19.05.x, shouldn't affect it.