projects
/
freeside.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
RT 37817 - created new billing event condition, invoice has not been sent.
[freeside.git]
/
FS
/
FS
/
cust_credit_bill.pm
diff --git
a/FS/FS/cust_credit_bill.pm
b/FS/FS/cust_credit_bill.pm
index
bd76c2e
..
ad5c712
100644
(file)
--- a/
FS/FS/cust_credit_bill.pm
+++ b/
FS/FS/cust_credit_bill.pm
@@
-1,15
+1,9
@@
package FS::cust_credit_bill;
package FS::cust_credit_bill;
+use base qw( FS::cust_main_Mixin FS::cust_bill_ApplicationCommon );
use strict;
use strict;
-use vars qw( @ISA $conf );
-use FS::UID qw( getotaker );
-use FS::Record qw( qsearch qsearchs );
-use FS::cust_main;
-#use FS::cust_refund;
-use FS::cust_credit;
-use FS::cust_bill;
-
-@ISA = qw( FS::Record );
+use vars qw( $conf );
+use FS::UID;
#ask FS::UID to run this stuff for us later
FS::UID->install_callback( sub {
#ask FS::UID to run this stuff for us later
FS::UID->install_callback( sub {
@@
-38,8
+32,9
@@
FS::cust_credit_bill - Object methods for cust_credit_bill records
=head1 DESCRIPTION
An FS::cust_credit_bill object represents application of a credit (see
=head1 DESCRIPTION
An FS::cust_credit_bill object represents application of a credit (see
-L<FS::cust_credit>) to an invoice (see L<FS::cust_bill>). FS::cust_credit
-inherits from FS::Record. The following fields are currently supported:
+L<FS::cust_credit>) to an invoice (see L<FS::cust_bill>). FS::cust_credit_bill
+inherits from FS::cust_bill_ApplicationCommon and FS::Record. The following
+fields are currently supported:
=over 4
=over 4
@@
-69,26
+64,16
@@
see L<"insert">.
sub table { 'cust_credit_bill'; }
sub table { 'cust_credit_bill'; }
+sub _app_source_name { 'credit'; }
+sub _app_source_table { 'cust_credit'; }
+sub _app_lineitem_breakdown_table { 'cust_credit_bill_pkg'; }
+sub _app_part_pkg_weight_column { 'credit_weight'; }
+
=item insert
Adds this cust_credit_bill to the database ("Posts" all or part of a credit).
If there is an error, returns the error, otherwise returns false.
=item insert
Adds this cust_credit_bill to the database ("Posts" all or part of a credit).
If there is an error, returns the error, otherwise returns false.
-=cut
-
-sub insert {
- my $self = shift;
- my $error = $self->SUPER::insert(@_);
- return $error if $error;
-
- if ( $conf->exists('invoice_send_receipts') ) {
- my $send_error = $self->cust_bill->send;
- warn "Error sending receipt: $send_error\n" if $send_error;
- }
-
- '';
-}
-
=item delete
Currently unimplemented.
=item delete
Currently unimplemented.
@@
-99,6
+84,8
@@
sub delete {
my $self = shift;
return "Can't delete application for closed credit"
if $self->cust_credit->closed =~ /^Y/i;
my $self = shift;
return "Can't delete application for closed credit"
if $self->cust_credit->closed =~ /^Y/i;
+ return "Can't delete application for closed invoice"
+ if $self->cust_bill->closed =~ /^Y/i;
$self->SUPER::delete(@_);
}
$self->SUPER::delete(@_);
}
@@
-125,30
+112,23
@@
sub check {
my $error =
$self->ut_numbern('creditbillnum')
my $error =
$self->ut_numbern('creditbillnum')
- || $self->ut_
number(
'crednum')
- || $self->ut_
number('invnum'
)
+ || $self->ut_
foreign_key('crednum', 'cust_credit',
'crednum')
+ || $self->ut_
foreign_key('invnum', 'cust_bill', 'invnum'
)
|| $self->ut_numbern('_date')
|| $self->ut_money('amount')
|| $self->ut_numbern('_date')
|| $self->ut_money('amount')
+ || $self->ut_foreign_keyn('pkgnum', 'cust_pkg', 'pkgnum')
;
return $error if $error;
return "amount must be > 0" if $self->amount <= 0;
;
return $error if $error;
return "amount must be > 0" if $self->amount <= 0;
- return "Unknown credit"
- unless my $cust_credit =
- qsearchs( 'cust_credit', { 'crednum' => $self->crednum } );
-
- return "Unknown invoice"
- unless my $cust_bill =
- qsearchs( 'cust_bill', { 'invnum' => $self->invnum } );
-
$self->_date(time) unless $self->_date;
return "Cannot apply more than remaining value of credit"
$self->_date(time) unless $self->_date;
return "Cannot apply more than remaining value of credit"
- unless $self->amount <= $cust_credit->credited;
+ unless $self->amount <= $
self->
cust_credit->credited;
return "Cannot apply more than remaining value of invoice"
return "Cannot apply more than remaining value of invoice"
- unless $self->amount <= $cust_bill->owed;
+ unless $self->amount <= $
self->
cust_bill->owed;
$self->SUPER::check;
}
$self->SUPER::check;
}
@@
-157,33
+137,17
@@
sub check {
Returns the credit (see L<FS::cust_credit>)
Returns the credit (see L<FS::cust_credit>)
-=cut
-
-sub cust_credit {
- my $self = shift;
- qsearchs( 'cust_credit', { 'crednum' => $self->crednum } );
-}
-
-=item cust_bill
-
-Returns the invoice (see L<FS::cust_bill>)
-
-=cut
-
-sub cust_bill {
- my $self = shift;
- qsearchs( 'cust_bill', { 'invnum' => $self->invnum } );
-}
-
=back
=head1 BUGS
The delete method.
=back
=head1 BUGS
The delete method.
+This probably should have been called cust_bill_credit.
+
=head1 SEE ALSO
=head1 SEE ALSO
-L<FS::Record>, L<FS::cust_
refund>, L<FS::cust_
bill>, L<FS::cust_credit>,
+L<FS::Record>, L<FS::cust_bill>, L<FS::cust_credit>,
schema.html from the base documentation.
=cut
schema.html from the base documentation.
=cut