View | Details | Raw Unified | Return to bug 19382
Collapse All | Expand All

(-)a/C4/Circulation.pm (+15 lines)
Lines 796-801 sub CanBookBeIssued { Link Here
796
        }
796
        }
797
    }
797
    }
798
798
799
    # Check the debt of this patrons guarantors *and* the guarantees of those guarantors
800
    my $no_issues_charge_guarantors = C4::Context->preference("NoIssuesChargeGuarantorsWithGuarantees");
801
    $no_issues_charge_guarantors = undef unless looks_like_number( $no_issues_charge_guarantors );
802
    if ( defined $no_issues_charge_guarantors ) {
803
        my $guarantors_non_issues_charges += $patron->relationships_debt({ include_guarantors => 1, only_this_guaranor => 0, include_this_patron => 1 });
804
805
        if ( $guarantors_non_issues_charges > $no_issues_charge_guarantors && !$inprocess && !$allowfineoverride) {
806
            $issuingimpossible{DEBT_GUARANTORS} = $guarantors_non_issues_charges;
807
        } elsif ( $guarantors_non_issues_charges > $no_issues_charge_guarantors && !$inprocess && $allowfineoverride) {
808
            $needsconfirmation{DEBT_GUARANTORS} = $guarantors_non_issues_charges;
809
        } elsif ( $allfinesneedoverride && $guarantors_non_issues_charges > 0 && $guarantors_non_issues_charges <= $no_issues_charge_guarantors && !$inprocess ) {
810
            $needsconfirmation{DEBT_GUARANTORS} = $guarantors_non_issues_charges;
811
        }
812
    }
813
799
    if ( C4::Context->preference("IssuingInProcess") ) {
814
    if ( C4::Context->preference("IssuingInProcess") ) {
800
        if ( $non_issues_charges > $amountlimit && !$inprocess && !$allowfineoverride) {
815
        if ( $non_issues_charges > $amountlimit && !$inprocess && !$allowfineoverride) {
801
            $issuingimpossible{DEBT} = $non_issues_charges;
816
            $issuingimpossible{DEBT} = $non_issues_charges;
(-)a/C4/SIP/ILS/Patron.pm (-3 / +7 lines)
Lines 67-76 sub new { Link Here
67
    $dexpiry and $dexpiry =~ s/-//g;    # YYYYMMDD
67
    $dexpiry and $dexpiry =~ s/-//g;    # YYYYMMDD
68
68
69
    # Get fines and add fines for guarantees (depends on preference NoIssuesChargeGuarantees)
69
    # Get fines and add fines for guarantees (depends on preference NoIssuesChargeGuarantees)
70
    my $fines_amount = $flags->{CHARGES}->{amount};
70
    my $fines_amount = $flags->{CHARGES}->{amount}; #TODO Replace with $patron->account->non_issues_charges
71
    $fines_amount = ($fines_amount and $fines_amount > 0) ? $fines_amount : 0;
71
    $fines_amount = ($fines_amount and $fines_amount > 0) ? $fines_amount : 0;
72
    my $guarantees_fines_amount = $flags->{CHARGES_GUARANTEES} ? $flags->{CHARGES_GUARANTEES}->{amount} : 0;
72
    if ( C4::Context->preference('NoIssuesChargeGuarantorsWithGuarantees') ) {
73
    $fines_amount += $guarantees_fines_amount;
73
        $fines_amount += $patron->relationships_debt({ include_guarantors => 1, only_this_guaranor => 0, include_this_patron => 1 });
74
    } else {
75
        my $guarantees_fines_amount = $flags->{CHARGES_GUARANTEES} ? $flags->{CHARGES_GUARANTEES}->{amount} : 0; #TODO: Replace with $patron->relationships_debt
76
        $fines_amount += $guarantees_fines_amount;
77
    }
74
78
75
    my $fee_limit = _fee_limit();
79
    my $fee_limit = _fee_limit();
76
    my $fine_blocked = $fines_amount > $fee_limit;
80
    my $fine_blocked = $fines_amount > $fee_limit;
(-)a/Koha/Patron.pm (+45 lines)
Lines 470-475 sub guarantee_relationships { Link Here
470
    );
470
    );
471
}
471
}
472
472
473
=head3 relationships_debt
474
475
Returns the amount owed by the patron's guarantors *and* the other guarantees of those guarantors
476
477
=cut
478
479
sub relationships_debt {
480
    my ($self, $params) = @_;
481
482
    my $include_guarantors  = $params->{include_guarantors};
483
    my $only_this_guarantor = $params->{only_this_guaranor};
484
    my $include_this_patron = $params->{include_this_patron};
485
486
    my @guarantors;
487
    if ( $only_this_guarantor ) {
488
        @guarantors = $self->guarantor_relationships ? ( $self ) : ();
489
    } elsif ( $self->guarantor_relationships ) {
490
        # I am a guarantee, just get all my guarantors
491
        @guarantors = $self->guarantor_relationships->guarantors;
492
    } else {
493
        # I am a guarantor, I need to get all the guarantors of all my guarantees
494
        @guarantors = map { $_->guarantor_relationships->guarantors } $self->guarantee_relationships->guarantees;
495
    }
496
497
    my $non_issues_charges = 0;
498
    my $seen = $include_this_patron ? {} : { $self->id => 1 }; # For tracking members already added to the total
499
    foreach my $g (@guarantors) {
500
        $non_issues_charges += $g->account->non_issues_charges if $include_guarantors;
501
502
        # We've added what the guarantor owes, not added in that guarantor's guarantees as well
503
        my @guarantees = map { $_->guarantee } $g->guarantee_relationships();
504
        my $guarantees_non_issues_charges;
505
        foreach my $g (@guarantees) {
506
            next if $seen->{ $g->id };
507
            $guarantees_non_issues_charges += $g->account->non_issues_charges;
508
            # Mark this guarantee as seen so we don't double count a guarantee linked to multiple guarantors
509
            $seen->{ $g->id } = 1;
510
        }
511
512
        $non_issues_charges += $guarantees_non_issues_charges;
513
    }
514
515
    return $non_issues_charges;
516
}
517
473
=head3 housebound_profile
518
=head3 housebound_profile
474
519
475
Returns the HouseboundProfile associated with this patron.
520
Returns the HouseboundProfile associated with this patron.
(-)a/circ/circulation.pl (+6 lines)
Lines 402-407 if (@$barcodes) { Link Here
402
        }
402
        }
403
    }
403
    }
404
404
405
    if ( $error->{DEBT_GUARANTORS} ) {
406
        $template_params->{DEBT_GUARANTORS} = $error->{DEBT_GUARANTORS};
407
        $template_params->{IMPOSSIBLE} = 1;
408
        $blocker = 1;
409
    }
410
405
    if ( $error->{UNKNOWN_BARCODE} or not $onsite_checkout or not C4::Context->preference("OnSiteCheckoutsForce") ) {
411
    if ( $error->{UNKNOWN_BARCODE} or not $onsite_checkout or not C4::Context->preference("OnSiteCheckoutsForce") ) {
406
        delete $question->{'DEBT'} if ($debt_confirmed);
412
        delete $question->{'DEBT'} if ($debt_confirmed);
407
        foreach my $impossible ( keys %$error ) {
413
        foreach my $impossible ( keys %$error ) {
(-)a/installer/data/mysql/atomicupdate/bug_19283.perl (+10 lines)
Line 0 Link Here
1
$DBversion = 'XXX'; # will be replaced by the RM
2
if ( CheckVersion($DBversion) ) {
3
    $dbh->do(q{
4
        INSERT IGNORE INTO systempreferences ( `variable`, `value`, `options`, `explanation`, `type` ) VALUES
5
        ('NoIssuesChargeGuarantorsWithGuarantees','','','Define maximum amount withstanding before check outs are blocked including guarantors and their other guarantees','Integer');
6
    });
7
8
    print "Upgrade to $DBversion done (Bug 19382 - Add ability to block guarantees based on fees owed by guarantor and other guarantees)\n";
9
    SetVersion($DBversion);
10
}
(-)a/installer/data/mysql/sysprefs.sql (+1 lines)
Lines 325-330 INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, ` Link Here
325
('NewsAuthorDisplay','none','none|opac|staff|both','Display the author name for news items.','Choice'),
325
('NewsAuthorDisplay','none','none|opac|staff|both','Display the author name for news items.','Choice'),
326
('noissuescharge','5','','Define maximum amount withstanding before check outs are blocked','Integer'),
326
('noissuescharge','5','','Define maximum amount withstanding before check outs are blocked','Integer'),
327
('NoIssuesChargeGuarantees','','','Define maximum amount withstanding before check outs are blocked','Integer'),
327
('NoIssuesChargeGuarantees','','','Define maximum amount withstanding before check outs are blocked','Integer'),
328
('NoIssuesChargeGuarantorsWithGuarantees','','','Define maximum amount withstanding before check outs are blocked including guarantors and their other guarantees','Integer'),
328
('noItemTypeImages','0',NULL,'If ON, disables item-type images','YesNo'),
329
('noItemTypeImages','0',NULL,'If ON, disables item-type images','YesNo'),
329
('NoRenewalBeforePrecision','exact_time','date|exact_time','Calculate "No renewal before" based on date only or exact time of due date','Choice'),
330
('NoRenewalBeforePrecision','exact_time','date|exact_time','Calculate "No renewal before" based on date only or exact time of due date','Choice'),
330
('NotesBlacklist','',NULL,'List of notes fields that should not appear in the title notes/description separator of details','free'),
331
('NotesBlacklist','',NULL,'List of notes fields that should not appear in the title notes/description separator of details','free'),
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref (+5 lines)
Lines 344-349 Circulation: Link Here
344
            - pref: NoIssuesChargeGuarantees
344
            - pref: NoIssuesChargeGuarantees
345
              class: integer
345
              class: integer
346
            - '[% local_currency %] in fines.'
346
            - '[% local_currency %] in fines.'
347
        -
348
            - Prevent a patron from checking out if the patron has guarantors and those guarantor's guarantees owing in total more than
349
            - pref: NoIssuesChargeGuarantorsWithGuarantees
350
              class: integer
351
            - '[% local_currency %] in fines.'
347
        -
352
        -
348
            - pref: RentalsInNoissuesCharge
353
            - pref: RentalsInNoissuesCharge
349
              choices:
354
              choices:
(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt (+8 lines)
Lines 105-110 Link Here
105
                                    <li>The patron's guarantees collectively have a debt of [% DEBT_GUARANTEES | $Price %].</li>
105
                                    <li>The patron's guarantees collectively have a debt of [% DEBT_GUARANTEES | $Price %].</li>
106
                                [% END %]
106
                                [% END %]
107
107
108
                                [% IF ( DEBT_GUARANTORS ) %]
109
                                    <li>The patron's guarantors and their other guarantees collectively have a debt of [% DEBT_GUARANTORS | $Price %].</li>
110
                                [% END %]
111
108
                                [% IF ( RENTALCHARGE && RENTALCHARGE > 0 ) %]
112
                                [% IF ( RENTALCHARGE && RENTALCHARGE > 0 ) %]
109
                                    <li>Rental charge for this item: [% RENTALCHARGE | $Price %]</li>
113
                                    <li>Rental charge for this item: [% RENTALCHARGE | $Price %]</li>
110
                                [% END %]
114
                                [% END %]
Lines 309-314 Link Here
309
313
310
                            <!-- RESULT OF ISSUING REQUEST -->
314
                            <!-- RESULT OF ISSUING REQUEST -->
311
                            <ul>
315
                            <ul>
316
                                [% IF ( DEBT_GUARANTORS ) %]
317
                                    <li>The patron's guarantors and their other guarantees collectively have a debt of [% DEBT_GUARANTORS | $Price %].</li>
318
                                [% END %]
319
312
                                [% IF ( STATS ) %]
320
                                [% IF ( STATS ) %]
313
                                    <li>Local use recorded</li>
321
                                    <li>Local use recorded</li>
314
                                [% END %]
322
                                [% END %]
(-)a/opac/sco/sco-main.pl (-1 / +1 lines)
Lines 151-157 elsif ( $patron && ( $op eq 'checkout' || $op eq 'renew' ) ) { Link Here
151
    );
151
    );
152
    my $issue_error;
152
    my $issue_error;
153
    if ( $confirm_required = scalar keys %$needconfirm ) {
153
    if ( $confirm_required = scalar keys %$needconfirm ) {
154
        for my $error ( qw( UNKNOWN_BARCODE max_loans_allowed ISSUED_TO_ANOTHER NO_MORE_RENEWALS NOT_FOR_LOAN DEBT WTHDRAWN RESTRICTED RESERVED ITEMNOTSAMEBRANCH EXPIRED DEBARRED CARD_LOST GNA INVALID_DATE UNKNOWN_BARCODE TOO_MANY DEBT_GUARANTEES USERBLOCKEDOVERDUE PATRON_CANT PREVISSUE NOT_FOR_LOAN_FORCING ITEM_LOST) ) {
154
        for my $error ( qw( UNKNOWN_BARCODE max_loans_allowed ISSUED_TO_ANOTHER NO_MORE_RENEWALS NOT_FOR_LOAN DEBT WTHDRAWN RESTRICTED RESERVED ITEMNOTSAMEBRANCH EXPIRED DEBARRED CARD_LOST GNA INVALID_DATE UNKNOWN_BARCODE TOO_MANY DEBT_GUARANTEES DEBT_GUARANTORS USERBLOCKEDOVERDUE PATRON_CANT PREVISSUE NOT_FOR_LOAN_FORCING ITEM_LOST) ) {
155
            if ( $needconfirm->{$error} ) {
155
            if ( $needconfirm->{$error} ) {
156
                $issue_error = $error;
156
                $issue_error = $error;
157
                $confirmed = 0;
157
                $confirmed = 0;
(-)a/t/db_dependent/Koha/Patron.t (-2 / +205 lines)
Lines 19-25 Link Here
19
19
20
use Modern::Perl;
20
use Modern::Perl;
21
21
22
use Test::More tests => 2;
22
use Test::More tests => 3;
23
use Test::Exception;
23
use Test::Exception;
24
24
25
use Koha::Database;
25
use Koha::Database;
Lines 73-78 subtest 'add_guarantor() tests' => sub { Link Here
73
    $schema->storage->txn_rollback;
73
    $schema->storage->txn_rollback;
74
};
74
};
75
75
76
subtest 'relationships_debt() tests' => sub {
77
78
    plan tests => 168;
79
80
    $schema->storage->txn_begin;
81
82
    t::lib::Mocks::mock_preference( 'borrowerRelationship', 'parent' );
83
84
    my $parent_1 = $builder->build_object({ class => 'Koha::Patrons' });
85
    my $parent_2 = $builder->build_object({ class => 'Koha::Patrons' });
86
    my $child_1 = $builder->build_object({ class => 'Koha::Patrons' });
87
    my $child_2 = $builder->build_object({ class => 'Koha::Patrons' });
88
89
    $child_1->add_guarantor({ guarantor_id => $parent_1->borrowernumber, relationship => 'parent' });
90
    $child_1->add_guarantor({ guarantor_id => $parent_2->borrowernumber, relationship => 'parent' });
91
    $child_2->add_guarantor({ guarantor_id => $parent_1->borrowernumber, relationship => 'parent' });
92
    $child_2->add_guarantor({ guarantor_id => $parent_2->borrowernumber, relationship => 'parent' });
93
94
    is( $child_1->guarantor_relationships->guarantors->count, 2, 'Child 1 has correct number of guarantors' );
95
    is( $child_2->guarantor_relationships->guarantors->count, 2, 'Child 2 has correct number of guarantors' );
96
    is( $parent_1->guarantee_relationships->guarantees->count, 2, 'Parent 1 has correct number of guarantors' );
97
    is( $parent_2->guarantee_relationships->guarantees->count, 2, 'Parent 2 has correct number of guarantors' );
98
99
    # 3 params, count from 0 to 6 in binary ( 3 places ) to get the set of switches, then do that 4 times, one for each parent and child
100
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
101
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
102
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
103
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
104
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
105
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
106
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
107
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
108
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
109
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
110
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
111
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
112
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
113
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
114
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
115
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
116
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
117
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
118
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
119
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
120
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
121
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
122
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
123
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
124
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
125
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
126
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
127
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
128
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
129
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
130
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
131
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
132
133
    $child_2->account->add_debit({ type => 'ACCOUNT', amount => 10, interface => 'commandline' });
134
    is( $child_2->account->non_issues_charges, 10, 'Child 2 owes correct amount' );
135
136
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
137
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
138
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
139
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
140
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
141
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
142
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
143
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
144
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
145
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
146
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
147
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
148
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
149
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
150
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
151
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
152
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 10, 'Family debt is correct' );
153
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
154
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 10, 'Family debt is correct' );
155
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 10, 'Family debt is correct' );
156
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 10, 'Family debt is correct' );
157
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
158
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 10, 'Family debt is correct' );
159
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 10, 'Family debt is correct' );
160
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
161
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
162
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
163
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 10, 'Family debt is correct' );
164
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
165
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
166
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
167
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 10, 'Family debt is correct' );
168
169
    $parent_1->account->add_debit({ type => 'ACCOUNT', amount => 10, interface => 'commandline' });
170
    is( $parent_1->account->non_issues_charges, 10, 'Parent 1 owes correct amount' );
171
172
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
173
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
174
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
175
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
176
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
177
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
178
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
179
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
180
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
181
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
182
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
183
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
184
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
185
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
186
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
187
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
188
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 10, 'Family debt is correct' );
189
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
190
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 20, 'Family debt is correct' );
191
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 20, 'Family debt is correct' );
192
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 10, 'Family debt is correct' );
193
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
194
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 20, 'Family debt is correct' );
195
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 20, 'Family debt is correct' );
196
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
197
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
198
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 10, 'Family debt is correct' );
199
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 20, 'Family debt is correct' );
200
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
201
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
202
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 10, 'Family debt is correct' );
203
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 20, 'Family debt is correct' );
204
205
    $parent_2->account->add_debit({ type => 'ACCOUNT', amount => 10, interface => 'commandline' });
206
    is( $parent_2->account->non_issues_charges, 10, 'Parent 2 owes correct amount' );
207
208
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
209
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
210
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
211
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
212
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
213
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
214
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
215
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
216
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
217
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
218
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
219
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
220
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
221
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
222
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
223
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
224
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 10, 'Family debt is correct' );
225
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
226
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 30, 'Family debt is correct' );
227
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 30, 'Family debt is correct' );
228
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 10, 'Family debt is correct' );
229
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
230
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 30, 'Family debt is correct' );
231
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 30, 'Family debt is correct' );
232
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
233
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
234
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 20, 'Family debt is correct' );
235
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 30, 'Family debt is correct' );
236
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
237
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 10, 'Family debt is correct' );
238
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 20, 'Family debt is correct' );
239
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 30, 'Family debt is correct' );
240
241
    $child_1->account->add_debit({ type => 'ACCOUNT', amount => 10, interface => 'commandline' });
242
    is( $child_1->account->non_issues_charges, 10, 'Child 1 owes correct amount' );
243
244
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
245
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
246
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
247
    is( $parent_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
248
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
249
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
250
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
251
    is( $parent_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
252
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
253
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
254
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
255
    is( $parent_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
256
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 0, 'Family debt is correct' );
257
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 0, 'Family debt is correct' );
258
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 0, 'Family debt is correct' );
259
    is( $parent_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 0, 'Family debt is correct' );
260
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 10, 'Family debt is correct' );
261
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 20, 'Family debt is correct' );
262
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 30, 'Family debt is correct' );
263
    is( $child_1->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 40, 'Family debt is correct' );
264
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 10, 'Family debt is correct' );
265
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 20, 'Family debt is correct' );
266
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 30, 'Family debt is correct' );
267
    is( $child_1->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 40, 'Family debt is correct' );
268
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 0 }), 10, 'Family debt is correct' );
269
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 0, include_this_patron => 1 }), 20, 'Family debt is correct' );
270
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 0 }), 30, 'Family debt is correct' );
271
    is( $child_2->relationships_debt({ include_this_patron => 0, include_guarantors => 1, include_this_patron => 1 }), 40, 'Family debt is correct' );
272
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 0 }), 10, 'Family debt is correct' );
273
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 0, include_this_patron => 1 }), 20, 'Family debt is correct' );
274
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 0 }), 30, 'Family debt is correct' );
275
    is( $child_2->relationships_debt({ include_this_patron => 1, include_guarantors => 1, include_this_patron => 1 }), 40, 'Family debt is correct' );
276
277
    $schema->storage->txn_rollback;
278
};
279
76
subtest 'add_enrolment_fee_if_needed() tests' => sub {
280
subtest 'add_enrolment_fee_if_needed() tests' => sub {
77
281
78
    plan tests => 2;
282
    plan tests => 2;
79
- 

Return to bug 19382