[Koha-bugs] [Bug 31458] Remove Koha::Patron::Debarment::AddDebarment and use $patron->add_restriction in preference

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

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

--- Comment #9 from Tomás Cohen Arazi <tomascohen at gmail.com> ---
(In reply to Martin Renvoize from comment #8)
> OK, So Kyle suggests we can/should remove the debarred and debarredcomment
> fields from the borrowers table.
> 
> We do however return a restricted boolean field in the current API.. I think
> it makes a fair bit of sense to continue to return this piece of primary
> information about the borrower there.. but it opens up the question.. should
> we actually remove the field from the table now and thus require a join for
> API queries or should I go ahead of convert the current date field to a
> proper boolean that's maintained by add_restriction and friends?

I did it a boolean so we can remove those heh.

The idea is that we would just embed the restriction objects, each of which
should include its own information.

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