https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40597

--- Comment #1 from Emmi Takkinen <[email protected]> ---
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.
_______________________________________________
Koha-bugs mailing list -- [email protected]
To unsubscribe send an email to [email protected]
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to