https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40597 --- Comment #1 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- Created attachment 207108 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=207108&action=edit Bug 40597: Log changes to borrower relationships Currently adding or deleting relationships between patron guarantor and their guarantee are not logged in Koha. Changes for non-patron guarantor are logged because they are made to borrowers table. This patch adds new logging systempreference BorrowerRelationshipsLog. It uses borrower_relationships.id as an object value and logs are viewable from Log viewer tool. To test from Koha staff interface: 1. Apply this patch, run updatedatabase.pl and restart services. 2. Enable systempreference BorrowerRelationshipsLog. 3. Create a new patron that is guarantee and add guarantor for them. 4. Navigate to Tools->Log viewer. => Note that there is now option for Borrower relationships module. 5. Search from logs for borrower relationships. => Confirm relationship created in step 3. is displayed correctly. 6. Make sure syspref ChildNeedsGuarantor is disbaled and edit guarantee patron and remove their guarantor. => Confirm removing guarantor is displayed correctly in Log viewer tool. 7. Add new guarantor for guarantee. 8. Change guarantees patron categorys upper age limit so that their age is no longer within that limit. 9. Use "Update child to adult patron" from patron details or check out page to change guarantees patron category. => Confirm removing guarantor is displayed correctly in Log viewer tool. To test update_patrons_category.pl cronjob: 1. Create new guarantee patron, add guarantor for them. 2. Change guarantees patron categorys upper age limit again so that their age is no longer within that limit. 3. Run cronjob update_patrons_category.pl from console: perl update_patrons_category.pl --too_old --from [guarantee category] --to [patron category] -v -c => Confirm removing guarantor is displayed correctly in Log viewer tool. Also prove that tests t/db_dependent/Koha/Patron.t and t/db_dependent/Patrons.t pass. Sponsored-by: Koha-Suomi Oy -- You are receiving this mail because: You are watching all bug changes.