[Koha-bugs] [Bug 22393] Remove last remaining manualinvoice use

[bugzilla-daemon at bugs.koha-community.org]

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

--- Comment #22 from Martin Renvoize <martin.renvoize at ptfs-europe.com> ---
The fallback search for deleted_items works well, but the subsequent
accountline creation fails due to a foreign key relationship from
accountlines.itemnumber to items.itemnumber.. as the item is now in the
delete_items table the key does not exist in items. (We have an 'on delete set
null' condition for said key relationship, which feels incorrect to me).


> > 
> > > 6/ Add a manual invoice with type 'LOST' and a valid barcode, but not
> > > one that will match a checkout for your user. Note an error is displayed
> > 
> > This succeeds with no problems, and I think that is right. If a patron comes
> > in, takes a book off the shelf and destroys it they may want to won up. Or a
> > parent taking responsibility for a child. There are many cases you may want
> > to charge a lost with no checkout
> 
> Agreed, this should be allowed.. I think I just miswrote the plan there.

I'm not so sure here.. we have internal logic for refunding lost and found
items etc which rely on consistently set issue_id's.. perhaps a 'Damaged paid
for' might be more helpful.. hmm.

-- 
You are receiving this mail because:
You are watching all bug changes.