Bug 14570: Make it possible to add multiple guarantors to a record
authorKyle M Hall <kyle@bywatersolutions.com>
Mon, 2 May 2016 15:00:39 +0000 (15:00 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Tue, 20 Aug 2019 15:04:48 +0000 (16:04 +0100)
commit109334102fbb29c60fa9a7a762c5e68f4b2ce0be
tree1db301083e4bcc6d6c084449f49b1cf0c0e389e1
parenta5828bc7dccdfbc574b30a7f39c822d4a4cab8fe
Bug 14570: Make it possible to add multiple guarantors to a record

This patch adds the ability to set an unlimited number of guarantors
for a given patron. As before, each guarantor may be linked to another
Koha patron, and all the behavior that applies to a given guarantor
remains the same.

Test Plan:
 1) Apply this patch
 2) Run updatedatabase.pl
 3) Find some patrons with guarantors, verify the still have their guarantor
 4) Test adding and removing guarantors on a patron record, both Koha users and not
 5) Verify the "Add child" button works
 6) Verify NoIssuesChargeGuarantees still works
 7) Verify tools/cleanborrowers.pl will not delete a guarantor
 8) Verify the guarantors are displayed on moremember.pl
 9) Verify the guarantor is removed by members/update-child.pl
10) Verify the guarantor is removed by misc/cronjobs/j2a.pl
11) Verify import patrons converts guarantor_id, relationship, contactfirstname,
  and contactsurname into a guarantor
12) prove t/Patron.t
13) prove t/db_dependent/Circulation.t
14) prove t/db_dependent/Circulation/NoIssuesChargeGuarantees.t
15) prove t/db_dependent/Items.t
16) prove t/db_dependent/Koha/Patrons.t
17) prove t/db_dependent/Members.t
18) prove t/db_dependent/Patron/Relationships.t

Signed-off-by: Kim Peine <kmpeine@gmail.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Agustin Moyano <agustinmoyano@theke.io>
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
41 files changed:
C4/Auth_with_ldap.pm
C4/Circulation.pm
C4/Members.pm
C4/SIP/ILS/Patron.pm
Koha/Object.pm
Koha/Patron.pm
Koha/Patron/Relationship.pm [new file with mode: 0644]
Koha/Patron/Relationships.pm [new file with mode: 0644]
Koha/Patrons.pm
Koha/Patrons/Import.pm
Koha/REST/V1/Auth.pm
Koha/Schema/Result/Borrower.pm
Koha/Schema/Result/BorrowerRelationship.pm [new file with mode: 0644]
api/v1/swagger/definitions/patron.json
circ/circulation.pl
installer/data/mysql/atomicupdate/bug_14570.perl [new file with mode: 0644]
installer/data/mysql/kohastructure.sql
koha-tmpl/intranet-tmpl/prog/en/columns.def
koha-tmpl/intranet-tmpl/prog/en/includes/members-toolbar.inc
koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember-brief.tt
koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
koha-tmpl/intranet-tmpl/prog/js/members.js
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-memberentry.tt
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-privacy.tt
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
members/deletemem.pl
members/memberentry.pl
members/moremember.pl
members/update-child.pl
misc/cronjobs/j2a.pl
opac/opac-memberentry.pl
opac/opac-user.pl
t/Patron.t
t/db_dependent/Circulation/NoIssuesChargeGuarantees.t
t/db_dependent/Items.t
t/db_dependent/Koha/Patrons.t
t/db_dependent/Members.t
t/db_dependent/Patron/Relationships.t [new file with mode: 0755]
tools/import_borrowers.pl