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

Nick Clemens (kidclamp) <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #1 from Nick Clemens (kidclamp) <[email protected]> ---
AddReturn does some stuff, eventually calls Item->store

In Koha::Item->store we call _set_found_trigger 

_set_found_trigger  adds a 'lost_charge' message to the item if we have found a
previous lost or overdue lost accountline and the rule is set to charge a new
fine

This gets passed back to AddReturn which looks for the last issue of the item,
and charges a new fine from the previous due_date until today

I believe what we need to do is:
1 - Verify the lost charge found in _set_found_trigger is the same patron we
are charging now
2 - Verify the issue was not returned since the issue

Current thoughts:
Store the borrowernumber in the item_lost entry in statistics when marking an
item lost - then we can verify that current lost is attributed to patron

Pass the patroid from the lost fine found in _set_found_trigger back to
AddReturn and verify the patron is the same as the one with the issue

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to