Bug 23170 - in j2a.pl when a patron passes to an adult the name and surname of Guarantor stay on account
Summary: in j2a.pl when a patron passes to an adult the name and surname of Guarantor ...
Status: CLOSED INVALID
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor (vote)
Assignee: Christophe Croullebois
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-06-20 10:08 UTC by Christophe Croullebois
Modified: 2020-06-04 20:32 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
add anonymize option to j2a.pl (5.96 KB, patch)
2019-06-20 14:09 UTC, Christophe Croullebois
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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.