[Koha-bugs] [Bug 29741] Add Koha::Patron->safe_to_delete
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Tue Dec 21 08:22:53 CET 2021
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29741
--- Comment #11 from Martin Renvoize <martin.renvoize at ptfs-europe.com> ---
(In reply to Tomás Cohen Arazi from comment #6)
> (In reply to Tomás Cohen Arazi from comment #5)
> > (In reply to Martin Renvoize from comment #4)
> > > Sorry dude, not 100% sure of the construct here... I feel like
> > > 'safe_to_delete' should return a Boolean given it's method name?
> > >
> > > Perhaps a boolean in scalar context and a Boolean + hash or array of errors
> > > as the second slot of an array in list context?
> >
> > Maybe a Koha::Validation object that validates to a bool, and carries
> > messages in it?
>
> I replicated Koha::Item's to limit the conflict surface, but...hey... Hehe
I should have read more code for context, happy to stick with this as there's a
precidence in Koha::Item.. I thought I'd seen something somewhere but couldn't
place it. In the transfers work I threw exceptions and allowed a force option
to be passed, but the use case isn't the same. I'd love to see us settle on
something consistent.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list