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

--- Comment #14 from Julian Maurice <[email protected]> ---
(In reply to David Cook from comment #13)
> I've noticed a lot of Koha code directly returning the output of a method
> call, and it drives me crazy. We should do more data validation. It'll mean
> fewer bugs and easier maintenance.

What is there to check in this particular situation ? _new_from_dbic either
dies or return a blessed object. If it dies then Koha::Patron::guarantor should
die too, if it returns a blessed object, then there is nothing to check.

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
_______________________________________________
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