Bug 23170

Summary: in j2a.pl when a patron passes to an adult the name and surname of Guarantor stay on account
Product: Koha Reporter: Christophe Croullebois <christophe.croullebois>
Component: PatronsAssignee: Christophe Croullebois <christophe.croullebois>
Status: CLOSED INVALID QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: christophe.croullebois, gmcharlt, jesse, kyle
Version: Main   
Hardware: All   
OS: All   
GIT URL: Initiative type: ---
Sponsorship status: --- Comma delimited list of Sponsors:
Crowdfunding goal: 0 Crowdfunding committed: 0
Crowdfunding contact: Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: add anonymize option to j2a.pl

Description Christophe Croullebois 2019-06-20 10:08:13 UTC
To test :

1° run the j2a.pl with at least 1 juvenile/child patron who have the age to be moved to adult
2° on Patron details you see 'Guarantor' still with name and surname
Comment 1 Katrin Fischer 2019-06-20 12:05:34 UTC
I am not sure if we can assume that it should be removed... maybe comments from libraries using this would be good or it could be an option to the script?
Comment 2 Christophe Croullebois 2019-06-20 12:09:42 UTC
Hi Katrin,

good idea, I will propose that in patch.
thx
Comment 3 Christophe Croullebois 2019-06-20 14:09:52 UTC
Created attachment 90849 [details] [review]
add anonymize option to j2a.pl

Test plan :
Without the patch

1° run the j2a.pl with at least 1 juvenile/child patron who have the age to be moved to adult
2° on Patron details you see 'Guarantor' still with name and surname

With the patch

1° run the j2a.pl with the option '-a' and with at least 1 juvenile/child patron who have the age to be moved to adult
2° on Patron details no more 'Guarantor' with name and surname

Thx
Comment 4 Jesse Maseto 2019-10-02 11:00:38 UTC
Patch doesn't apply. Merge conflicts.
Comment 5 Christophe Croullebois 2019-10-25 09:25:44 UTC
Hi Jesse, thx, but in fact the code has changed drastically.
So I think this this bug report is no longer relevant.
I close it.