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

--- Comment #18 from Jonathan Druart <[email protected]> 
---
Comment on attachment 109210
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109210
Bug 19382: Add ability to block guarantees based on fees owed by guarantor and
other guarantees

Review of attachment 109210:
 --> 
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=19382&attachment=109210)
-----------------------------------------------------------------

::: t/db_dependent/Koha/Patron.t
@@ +153,5 @@
> +    is( $child_1->relationships_debt({ only_this_guarantor => 0, 
> include_guarantors => 0, include_this_patron => 0 }), 10, 'Family debt is 
> correct' );
> +    is( $child_1->relationships_debt({ only_this_guarantor => 0, 
> include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is 
> correct' );
> +    is( $child_1->relationships_debt({ only_this_guarantor => 0, 
> include_guarantors => 1, include_this_patron => 0 }), 10, 'Family debt is 
> correct' );
> +    is( $child_1->relationships_debt({ only_this_guarantor => 0, 
> include_guarantors => 1, include_this_patron => 1 }), 10, 'Family debt is 
> correct' );
> +    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
[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