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

--- Comment #23 from Jonathan Druart <jonathan.dru...@bugs.koha-community.org> 
---
Yes indeed.
My idea was to make the tests more readable, and this:

+            $amount = 0 if ( $patron->id == $child_2->id && ( $i == 0 || $i ==
2 || $i == 4 || $i == 5 || $i == 6 ) );

is something I'd like to avoid :)


What about the previous question?
(In reply to Jonathan Druart from comment #18)
> > +    is( $child_1->relationships_debt({ only_this_guarantor => 1, 
> > include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is 
> > correct' );
> 
> Is this correct? $child_1 is not a guarantor so I don't think it should
> follow "only_this_guarantor". I would have expected 10 here, can you confirm?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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