From 15a02054fe0f91eda70003b8d1cf29497d19aa91 Mon Sep 17 00:00:00 2001 From: ivan Date: Sun, 1 Jun 2008 22:48:15 +0000 Subject: refactor payment history slightly, add refund receipts, have "unapplied" refunds show like other unapplied/open things, RT#3545 --- httemplate/search/elements/cust_pay_or_refund.html | 7 +- httemplate/view/cust_main/payment_history.html | 454 +++------------------ .../view/cust_main/payment_history/credit.html | 110 +++++ .../view/cust_main/payment_history/invoice.html | 34 ++ .../view/cust_main/payment_history/payment.html | 180 ++++++++ .../view/cust_main/payment_history/refund.html | 50 +++ .../cust_main/payment_history/voided_payment.html | 37 ++ httemplate/view/cust_refund.html | 141 +++++++ 8 files changed, 607 insertions(+), 406 deletions(-) create mode 100644 httemplate/view/cust_main/payment_history/credit.html create mode 100644 httemplate/view/cust_main/payment_history/invoice.html create mode 100644 httemplate/view/cust_main/payment_history/payment.html create mode 100644 httemplate/view/cust_main/payment_history/refund.html create mode 100644 httemplate/view/cust_main/payment_history/voided_payment.html create mode 100644 httemplate/view/cust_refund.html (limited to 'httemplate') diff --git a/httemplate/search/elements/cust_pay_or_refund.html b/httemplate/search/elements/cust_pay_or_refund.html index cd1e19ded..9457ae2eb 100755 --- a/httemplate/search/elements/cust_pay_or_refund.html +++ b/httemplate/search/elements/cust_pay_or_refund.html @@ -242,10 +242,9 @@ if ( $cgi->param('magic') ) { } my $link = ''; -if ( $thing eq 'pay' ) { #XXX no refund receipts yet - $link = [ "${p}view/cust_pay.html?paynum=", 'paynum' ] - if $FS::CurrentUser::CurrentUser->access_right('View invoices'); #XXX for now - #later# if $FS::CurrentUser::CurrentUser->access_right('View customer payments'); +if ( $FS::CurrentUser::CurrentUser->access_right('View invoices') #XXX for now + || $FS::CurrentUser::CurrentUser->access_right('View customer payments') ){ + $link = [ "${p}view/cust_$thing.html?${thing}num=", $thing.'num' ] } my $cust_link = sub { diff --git a/httemplate/view/cust_main/payment_history.html b/httemplate/view/cust_main/payment_history.html index 582fc8d7e..750e59d2f 100644 --- a/httemplate/view/cust_main/payment_history.html +++ b/httemplate/view/cust_main/payment_history.html @@ -84,408 +84,6 @@
% } -%#get payment history -%my @history = (); -% -%#invoices -%foreach my $cust_bill ($cust_main->cust_bill) { -% my $pre = ( $cust_bill->owed > 0 ) -% ? 'Open ' -% : ''; -% my $post = ( $cust_bill->owed > 0 ) ? '' : ''; -% my $invnum = $cust_bill->invnum; -% my $link = $curuser->access_right('View invoices') -% ? qq!! -% : ''; -% my $events = ''; -% if ( $cust_bill->num_cust_event -% && ( $curuser->access_right('Billing event reports') -% || $curuser->access_right('View customer billing events') -% ) -% ) { -% $events = -% qq!
( View invoice events )'; -% } -% push @history, { -% 'date' => $cust_bill->_date, -% 'desc' => $link. $pre. -% "Invoice #$invnum (Balance \$". $cust_bill->owed. ')'. -% $post. ( $link ? '' : '' ). $events, -% 'charge' => $cust_bill->charged, -% }; -%} -% -%#payments (some false laziness w/credits) -%foreach my $cust_pay ($cust_main->cust_pay) { -% -% my $payby = $cust_pay->payby; -% -% my $payinfo; -% if ( $payby eq 'CARD' ) { -% $payinfo = $cust_pay->paymask; -% } elsif ( $payby eq 'CHEK' ) { -% my( $account, $aba ) = split('@', $cust_pay->paymask ); -% $payinfo = "ABA $aba, Acct #$account"; -% } else { -% $payinfo = $cust_pay->payinfo; -% } -% my @cust_bill_pay = $cust_pay->cust_bill_pay; -% my @cust_pay_refund = $cust_pay->cust_pay_refund; -% -% my $target = "$payby$payinfo"; -% $payby =~ s/^BILL$/Check #/ if $payinfo; -% $payby =~ s/^CHEK$/Electronic check /; -% $payby =~ s/^PREP$/Prepaid card /; -% $payby =~ s/^CARD$/Credit card #/; -% $payby =~ s/^COMP$/Complimentary by /; -% $payby =~ s/^CASH$/Cash/; -% $payby =~ s/^WEST$/Western Union/; -% $payby =~ s/^MCRD$/Manual credit card/; -% $payby =~ s/^BILL$//; -% my $info = $payby ? "($payby$payinfo)" : ''; -% -% my( $pre, $post, $desc, $apply, $ext ) = ( '', '', '', '', '' ); -% if ( scalar(@cust_bill_pay) == 0 -% && scalar(@cust_pay_refund) == 0 ) { -% #completely unapplied -% $pre = 'Unapplied '; -% $post = ''; -% if ( $curuser->access_right('Apply payment') ) { -% $apply = ' ('. include( '/elements/popup_link.html', -% 'label' => 'apply', -% 'action' => "${p}edit/cust_bill_pay.cgi?". -% $cust_pay->paynum, -% 'actionlabel' => 'Apply payment', -% 'width' => 392, -% #default# 'height' => 336, -% ). -% ')'; -% } -% } elsif ( scalar(@cust_bill_pay) == 1 -% && scalar(@cust_pay_refund) == 0 -% && $cust_pay->unapplied == 0 ) { -% #applied to one invoice, the usual situation -% $desc = ' '. $cust_bill_pay[0]->applied_to_invoice; -% } elsif ( scalar(@cust_bill_pay) == 0 -% && scalar(@cust_pay_refund) == 1 -% && $cust_pay->unapplied == 0 ) { -% #applied to one refund -% $desc = ' refunded on '. time2str("%D", $cust_pay_refund[0]->_date); -% } else { -% #complicated -% $desc = '
'; -% foreach my $app ( sort { $a->_date <=> $b->_date } -% ( @cust_bill_pay, @cust_pay_refund ) ) { -% if ( $app->isa('FS::cust_bill_pay') ) { -% $desc .= '  '. -% '$'. $app->amount. -% ' '. $app->applied_to_invoice. -% '
'; -% #' on '. time2str("%D", $cust_bill_pay->_date). -% } elsif ( $app->isa('FS::cust_pay_refund') ) { -% $desc .= '  '. -% '$'. $app->amount. -% ' refunded on '. time2str("%D", $app->_date). -% '
'; -% } else { -% die "$app is not a FS::cust_bill_pay or FS::cust_pay_refund"; -% } -% } -% if ( $cust_pay->unapplied > 0 ) { -% $desc .= '  '. -% '$'. -% $cust_pay->unapplied. ' unapplied'; -% if ( $curuser->access_right('Apply payment') ) { -% $desc = ' ('. include( '/elements/popup_link.html', -% 'label' => 'apply', -% 'action' => "${p}edit/cust_bill_pay.cgi?". -% $cust_pay->paynum, -% 'actionlabel' => 'Apply payment', -% 'width' => 392, -% #default# 'height' => 336, -% ). -% ')'; -% } -% $desc .= '
'; -% } -% } -% -% my $view = -% ' ('. include('/elements/popup_link.html', -% 'label' => 'view receipt', -% 'action' => "${p}view/cust_pay.html?link=popup;paynum=". -% $cust_pay->paynum, -% 'actionlabel' => 'Payment Receipt', -% ). -% ')'; -% -% my $refund = ''; -% my $refund_days = $conf->config('card_refund-days') || 120; -% if ( $cust_pay->closed !~ /^Y/i -% && $cust_pay->payby =~ /^(CARD|CHEK)$/ -% && time-$cust_pay->_date < $refund_days*86400 -% && $cust_pay->unrefunded > 0 -% && $curuser->access_right('Refund payment') -% ) { -% $refund = qq! (refund)!; -% } -% -% my $void = ''; -% if ( $cust_pay->closed !~ /^Y/i -% && ( ( $cust_pay->payby eq 'CARD' -% && $curuser->access_right('Credit card void') -% ) -% || ( $cust_pay->payby eq 'CHEK' -% && $curuser->access_right('Echeck void') -% ) -% || ( $cust_pay->payby !~ /^(CARD|CHEK)$/ -% && $curuser->access_right('Regular void') -% ) -% ) -% ) -% { -% $void = qq! (void)!; -% } -% -% my $delete = ''; -% if ( $cust_pay->closed !~ /^Y/i -% && $conf->exists('deletepayments') -% && $curuser->access_right('Delete payment') -% ) -% { -% $delete = qq! (delete)!; -% } -% -% my $unapply = ''; -% if ( $cust_pay->closed !~ /^Y/i -% && scalar(@cust_bill_pay) -% && $curuser->access_right('Unapply payment') -% ) -% { -% $unapply = qq! (unapply)!; -% } -% -% my $otaker = $cust_pay->otaker; -% $otaker = 'auto billing' if $otaker eq 'fs_daily'; -% $otaker = 'customer self-service' if $otaker eq 'fs_selfservice'; -% -% push @history, { -% 'date' => $cust_pay->_date, -% 'desc' => $pre. "Payment$post by $otaker $info$desc". -% "$view$apply$refund$void$delete$unapply", -% 'payment' => $cust_pay->paid, -% 'target' => $target, -% }; -%} -% -%#voided payments -%foreach my $cust_pay_void ($cust_main->cust_pay_void) { -% -% my $payby = $cust_pay_void->payby; -% my $payinfo = $payby eq 'CARD' -% ? $cust_pay_void->paymask -% : $cust_pay_void->payinfo; -% -% $payby =~ s/^BILL$/Check #/ if $payinfo; -% $payby =~ s/^CHEK$/Electronic check /; -% $payby =~ s/^BILL$//; -% $payby =~ s/^(CARD|COMP)$/$1 /; -% my $info = $payby ? " ($payby$payinfo)" : ''; -% -% my $unvoid = ''; -% if ( $cust_pay_void->closed !~ /^Y/i -% && $curuser->access_right('Unvoid') -% ) -% { -% $unvoid = qq! (unvoid)!; -% } -% -% push @history, { -% 'date' => $cust_pay_void->_date, -% 'desc' => "Payment $info voided ". -% time2str("%D", $cust_pay_void->void_date). -% " by ". $cust_pay_void->otaker. ''. $unvoid, -% 'void_payment' => $cust_pay_void->paid, -% }; -% -%} -% -%#credits (some false laziness w/payments) -%foreach my $cust_credit ($cust_main->cust_credit) { -% -% my @cust_credit_bill = $cust_credit->cust_credit_bill; -% my @cust_credit_refund = $cust_credit->cust_credit_refund; -% -% my( $pre, $post, $desc, $apply, $ext ) = ( '', '', '', '', '' ); -% if ( scalar(@cust_credit_bill) == 0 -% && scalar(@cust_credit_refund) == 0 ) { -% #completely unapplied -% $pre = 'Unapplied '; -% $post = ''; -% if ( $curuser->access_right('Apply credit') ) { -% $apply = ' ('. include( '/elements/popup_link.html', -% 'label' => 'apply', -% 'action' => "${p}edit/cust_credit_bill.cgi?". -% $cust_credit->crednum, -% 'actionlabel' => 'Apply credit', -% 'width' => 392, -% #default# 'height' => 336, -% ). -% ')'; -% } -% } elsif ( scalar(@cust_credit_bill) == 1 -% && scalar(@cust_credit_refund) == 0 -% && $cust_credit->credited == 0 ) { -% #applied to one invoice, the usual situation -% $desc = ' '. $cust_credit_bill[0]->applied_to_invoice; -% } elsif ( scalar(@cust_credit_bill) == 0 -% && scalar(@cust_credit_refund) == 1 -% && $cust_credit->credited == 0 ) { -% #applied to one refund -% $desc = ' refunded on '. time2str("%D", $cust_credit_refund[0]->_date); -% } else { -% #complicated -% $desc = '
'; -% foreach my $app ( sort { $a->_date <=> $b->_date } -% ( @cust_credit_bill, @cust_credit_refund ) ) { -% if ( $app->isa('FS::cust_credit_bill') ) { -% $desc .= '  '. -% '$'. $app->amount. -% ' '. $app->applied_to_invoice. -% '
'; -% #' on '. time2str("%D", $app->_date). -% } elsif ( $app->isa('FS::cust_credit_refund') ) { -% $desc .= '  '. -% '$'. $app->amount. -% ' refunded on '. time2str("%D", $app->_date). -% '
'; -% } else { -% die "$app is not a FS::cust_credit_bill or a FS::cust_credit_refund"; -% } -% } -% if ( $cust_credit->credited > 0 ) { -% $desc .= '  $'. -% $cust_credit->credited. ' unapplied'; -% if ( $curuser->access_right('Apply credit') ) { -% $desc = ' ('. include( '/elements/popup_link.html', -% 'label' => 'apply', -% 'action' => "${p}edit/cust_credit_bill.cgi?". -% $cust_credit->crednum, -% 'actionlabel' => 'Apply credit', -% 'width' => 392, -% #default# 'height' => 336, -% ). -% ')'; -% } -% $desc .= '
'; -% } -% } -%# -% my $delete = ''; -% if ( $cust_credit->closed !~ /^Y/i -% -% #s'pose deleting a credit isn't bad like deleting a payment -% # and this needs to be generally available until we have credit voiding.. -% #&& $conf->exists('deletecredits') -% -% && $curuser->access_right('Delete credit') -% ) -% { -% $delete = qq! (!. -% qq!delete)!; -% } -% -% my $unapply = ''; -% if ( $cust_credit->closed !~ /^Y/i -% && scalar(@cust_credit_bill) -% && $curuser->access_right('Unapply credit') -% ) -% { -% $unapply = qq! (!. -% qq!unapply)!; -% } -% -% push @history, { -% 'date' => $cust_credit->_date, -% 'desc' => $pre. "Credit$post by ". $cust_credit->otaker. -% ( $cust_credit->reason -% ? ' ('. $cust_credit->reason. ')' -% : '' -% ). -% "$desc$apply$delete$unapply", -% 'credit' => $cust_credit->amount, -% }; -% -%} -% -%#refunds -%foreach my $cust_refund ($cust_main->cust_refund) { -% -% my $payby = $cust_refund->payby; -% my $payinfo = $payby eq 'CARD' -% ? $cust_refund->paymask -% : $cust_refund->payinfo; -% -% $payby =~ s/^BILL$/Check #/ if $payinfo; -% $payby =~ s/^CHEK$/Electronic check /; -% $payby =~ s/^(CARD|COMP)$/$1 /; -% -% my $delete = ''; -% if ( $cust_refund->closed !~ /^Y/i -% && $conf->exists('deleterefunds') -% && $curuser->access_right('Delete refund') -% ) -% { -% $delete = qq! (delete)!; -% } -% -% push @history, { -% 'date' => $cust_refund->_date, -% 'desc' => "Refund ($payby$payinfo) by ". $cust_refund->otaker. "
". -% $delete, -% 'refund' => $cust_refund->refund, -% }; -% -%} -% -% - <% include("/elements/table-grid.html") %> % my $bgcolor1 = '#eeeeee'; @@ -681,4 +279,56 @@ my %status = ( 'All' => '', ); +#get payment history +my @history = (); + +#invoices +foreach my $cust_bill ($cust_main->cust_bill) { + push @history, { + 'date' => $cust_bill->_date, + 'desc' => include('payment_history/invoice.html', $cust_bill), + 'charge' => $cust_bill->charged, + }; +} + +#payments (some false laziness w/credits) +foreach my $cust_pay ($cust_main->cust_pay) { + push @history, { + 'date' => $cust_pay->_date, + 'desc' => include('payment_history/payment.html', $cust_pay), + 'payment' => $cust_pay->paid, + #'target' => $target, #XXX + }; +} + +#voided payments +foreach my $cust_pay_void ($cust_main->cust_pay_void) { + push @history, { + 'date' => $cust_pay_void->_date, + 'desc' => include('payment_history/voided_payment.html', $cust_pay_void), + 'void_payment' => $cust_pay_void->paid, + }; + +} + +#credits (some false laziness w/payments) +foreach my $cust_credit ($cust_main->cust_credit) { + push @history, { + 'date' => $cust_credit->_date, + 'desc' => include('payment_history/credit.html', $cust_credit), + 'credit' => $cust_credit->amount, + }; + +} + +#refunds +foreach my $cust_refund ($cust_main->cust_refund) { + push @history, { + 'date' => $cust_refund->_date, + 'desc' => include('payment_history/refund.html', $cust_refund), + 'refund' => $cust_refund->refund, + }; + +} + diff --git a/httemplate/view/cust_main/payment_history/credit.html b/httemplate/view/cust_main/payment_history/credit.html new file mode 100644 index 000000000..bf769455b --- /dev/null +++ b/httemplate/view/cust_main/payment_history/credit.html @@ -0,0 +1,110 @@ +<% $pre %>Credit<% $post %> +by <% $cust_credit->otaker %><% "$reason$desc$apply$delete$unapply" %> +<%init> + +my( $cust_credit, %opt ) = @_; + +my $curuser = $FS::CurrentUser::CurrentUser; + +my @cust_credit_bill = $cust_credit->cust_credit_bill; +my @cust_credit_refund = $cust_credit->cust_credit_refund; + +my( $pre, $post, $desc, $apply, $ext ) = ( '', '', '', '', '' ); +if ( scalar(@cust_credit_bill) == 0 + && scalar(@cust_credit_refund) == 0 ) { + #completely unapplied + $pre = 'Unapplied '; + $post = ''; + if ( $curuser->access_right('Apply credit') ) { + $apply = ' ('. include( '/elements/popup_link.html', + 'label' => 'apply', + 'action' => "${p}edit/cust_credit_bill.cgi?". + $cust_credit->crednum, + 'actionlabel' => 'Apply credit', + 'width' => 392, + #default# 'height' => 336, + ). + ')'; + } +} elsif ( scalar(@cust_credit_bill) == 1 + && scalar(@cust_credit_refund) == 0 + && $cust_credit->credited == 0 ) { + #applied to one invoice, the usual situation + $desc = ' '. $cust_credit_bill[0]->applied_to_invoice; +} elsif ( scalar(@cust_credit_bill) == 0 + && scalar(@cust_credit_refund) == 1 + && $cust_credit->credited == 0 ) { + #applied to one refund + $desc = ' refunded on '. time2str("%D", $cust_credit_refund[0]->_date); +} else { + #complicated + $desc = '
'; + foreach my $app ( sort { $a->_date <=> $b->_date } + ( @cust_credit_bill, @cust_credit_refund ) ) { + if ( $app->isa('FS::cust_credit_bill') ) { + $desc .= '  '. + '$'. $app->amount. + ' '. $app->applied_to_invoice. + '
'; + #' on '. time2str("%D", $app->_date). + } elsif ( $app->isa('FS::cust_credit_refund') ) { + $desc .= '  '. + '$'. $app->amount. + ' refunded on '. time2str("%D", $app->_date). + '
'; + } else { + die "$app is not a FS::cust_credit_bill or a FS::cust_credit_refund"; + } + } + if ( $cust_credit->credited > 0 ) { + $desc .= '  $'. + $cust_credit->credited. ' unapplied'; + if ( $curuser->access_right('Apply credit') ) { + $desc = ' ('. include( '/elements/popup_link.html', + 'label' => 'apply', + 'action' => "${p}edit/cust_credit_bill.cgi?". + $cust_credit->crednum, + 'actionlabel' => 'Apply credit', + 'width' => 392, + #default# 'height' => 336, + ). + ')'; + } + $desc .= '
'; + } +} +# +my $delete = ''; +if ( $cust_credit->closed !~ /^Y/i + + #s'pose deleting a credit isn't bad like deleting a payment + # and this needs to be generally available until we have credit voiding.. + #&& $conf->exists('deletecredits') + + && $curuser->access_right('Delete credit') + ) +{ + $delete = qq! (!. + qq!delete)!; +} + +my $unapply = ''; +if ( $cust_credit->closed !~ /^Y/i + && scalar(@cust_credit_bill) + && $curuser->access_right('Unapply credit') + ) +{ + $unapply = qq! (!. + qq!unapply)!; +} + +my $reason = $cust_credit->reason + ? ' ('. $cust_credit->reason. ')' + : ''; + + + diff --git a/httemplate/view/cust_main/payment_history/invoice.html b/httemplate/view/cust_main/payment_history/invoice.html new file mode 100644 index 000000000..39c67396e --- /dev/null +++ b/httemplate/view/cust_main/payment_history/invoice.html @@ -0,0 +1,34 @@ +<% $link %><% $pre %>Invoice #<% $invnum %> +(Balance $ <% $cust_bill->owed %>)<% $post %><% $link ? '' : '' %><% $events %> +<%init> + +my( $cust_bill, %opt ) = @_; + +my $curuser = $FS::CurrentUser::CurrentUser; + +my($pre, $post) = ('', ''); +if ( $cust_bill->owed > 0 ) { + $pre = 'Open '; + $post = ''; +} + +my $invnum = $cust_bill->invnum; + +my $link = $curuser->access_right('View invoices') + ? qq!! + : ''; + +my $events = ''; +#1.9 +if ( $cust_bill->num_cust_event + && ( $curuser->access_right('Billing event reports') + || $curuser->access_right('View customer billing events') + ) + ) { + $events = + qq!
( View invoice events )'; +} +# + + diff --git a/httemplate/view/cust_main/payment_history/payment.html b/httemplate/view/cust_main/payment_history/payment.html new file mode 100644 index 000000000..96fca7f49 --- /dev/null +++ b/httemplate/view/cust_main/payment_history/payment.html @@ -0,0 +1,180 @@ +<% $pre %>Payment<% $post %> by <% $otaker %> +<% "$info$desc$view$apply$refund$void$delete$unapply" %> +<%init> + +my( $cust_pay, %opt ) = @_; + +my $conf = new FS::Conf; +my $curuser = $FS::CurrentUser::CurrentUser; + +my $payby = $cust_pay->payby; + +my $payinfo; +if ( $payby eq 'CARD' ) { + $payinfo = $cust_pay->paymask; +} elsif ( $payby eq 'CHEK' ) { + my( $account, $aba ) = split('@', $cust_pay->paymask ); + $payinfo = "ABA $aba, Acct #$account"; +} else { + $payinfo = $cust_pay->payinfo; +} +my @cust_bill_pay = $cust_pay->cust_bill_pay; +my @cust_pay_refund = $cust_pay->cust_pay_refund; + +my $target = "$payby$payinfo"; +$payby =~ s/^BILL$/Check #/ if $payinfo; +$payby =~ s/^CHEK$/Electronic check /; +$payby =~ s/^PREP$/Prepaid card /; +$payby =~ s/^CARD$/Credit card #/; +$payby =~ s/^COMP$/Complimentary by /; +$payby =~ s/^CASH$/Cash/; +$payby =~ s/^WEST$/Western Union/; +$payby =~ s/^MCRD$/Manual credit card/; +$payby =~ s/^BILL$//; +my $info = $payby ? "($payby$payinfo)" : ''; + +my( $pre, $post, $desc, $apply, $ext ) = ( '', '', '', '', '' ); +if ( scalar(@cust_bill_pay) == 0 + && scalar(@cust_pay_refund) == 0 ) { + #completely unapplied + $pre = 'Unapplied '; + $post = ''; + if ( $curuser->access_right('Apply payment') ) { + $apply = ' ('. include( '/elements/popup_link.html', + 'label' => 'apply', + 'action' => "${p}edit/cust_bill_pay.cgi?". + $cust_pay->paynum, + 'actionlabel' => 'Apply payment', + 'width' => 392, + #default# 'height' => 336, + ). + ')'; + } +} elsif ( scalar(@cust_bill_pay) == 1 + && scalar(@cust_pay_refund) == 0 + && $cust_pay->unapplied == 0 ) { + #applied to one invoice, the usual situation + $desc = ' '. $cust_bill_pay[0]->applied_to_invoice; +} elsif ( scalar(@cust_bill_pay) == 0 + && scalar(@cust_pay_refund) == 1 + && $cust_pay->unapplied == 0 ) { + #applied to one refund + $desc = ' refunded on '. time2str("%D", $cust_pay_refund[0]->_date); +} else { + #complicated + $desc = '
'; + foreach my $app ( sort { $a->_date <=> $b->_date } + ( @cust_bill_pay, @cust_pay_refund ) ) { + if ( $app->isa('FS::cust_bill_pay') ) { + $desc .= '  '. + '$'. $app->amount. + ' '. $app->applied_to_invoice. + '
'; + #' on '. time2str("%D", $cust_bill_pay->_date). + } elsif ( $app->isa('FS::cust_pay_refund') ) { + $desc .= '  '. + '$'. $app->amount. + ' refunded on '. time2str("%D", $app->_date). + '
'; + } else { + die "$app is not a FS::cust_bill_pay or FS::cust_pay_refund"; + } + } + if ( $cust_pay->unapplied > 0 ) { + $desc .= '  '. + '$'. + $cust_pay->unapplied. ' unapplied'; + if ( $curuser->access_right('Apply payment') ) { + $desc = ' ('. include( '/elements/popup_link.html', + 'label' => 'apply', + 'action' => "${p}edit/cust_bill_pay.cgi?". + $cust_pay->paynum, + 'actionlabel' => 'Apply payment', + 'width' => 392, + #default# 'height' => 336, + ). + ')'; + } + $desc .= '
'; + } +} + +my $view = + ' ('. include('/elements/popup_link.html', + 'label' => 'view receipt', + 'action' => "${p}view/cust_pay.html?link=popup;paynum=". + $cust_pay->paynum, + 'actionlabel' => 'Payment Receipt', + ). + ')'; + +my $refund = ''; +my $refund_days = $conf->config('card_refund-days') || 120; +if ( $cust_pay->closed !~ /^Y/i + && $cust_pay->payby =~ /^(CARD|CHEK)$/ + && time-$cust_pay->_date < $refund_days*86400 + && $cust_pay->unrefunded > 0 + && $curuser->access_right('Refund payment') +) { + $refund = qq! (refund)!; +} + +my $void = ''; +if ( $cust_pay->closed !~ /^Y/i + && ( ( $cust_pay->payby eq 'CARD' + && $curuser->access_right('Credit card void') + ) + || ( $cust_pay->payby eq 'CHEK' + && $curuser->access_right('Echeck void') + ) + || ( $cust_pay->payby !~ /^(CARD|CHEK)$/ + && $curuser->access_right('Regular void') + ) + ) + ) +{ + $void = qq! (void)!; +} + +my $delete = ''; +if ( $cust_pay->closed !~ /^Y/i + && $conf->exists('deletepayments') + && $curuser->access_right('Delete payment') + ) +{ + $delete = qq! (delete)!; +} + +my $unapply = ''; +if ( $cust_pay->closed !~ /^Y/i + && scalar(@cust_bill_pay) + && $curuser->access_right('Unapply payment') + ) +{ + $unapply = qq! (unapply)!; +} + +my $otaker = $cust_pay->otaker; +$otaker = 'auto billing' if $otaker eq 'fs_daily'; +$otaker = 'customer self-service' if $otaker eq 'fs_selfservice'; + + diff --git a/httemplate/view/cust_main/payment_history/refund.html b/httemplate/view/cust_main/payment_history/refund.html new file mode 100644 index 000000000..5f9bca339 --- /dev/null +++ b/httemplate/view/cust_main/payment_history/refund.html @@ -0,0 +1,50 @@ +<% $pre %>Refund<% $post %> +(<% $payby. $payinfo %>) +by <% $cust_refund->otaker %><% $view %><% $delete %> +<%init> + +my( $cust_refund, %opt ) = @_; + +my $conf = new FS::Conf; +my $curuser = $FS::CurrentUser::CurrentUser; + +my $payby = $cust_refund->payby; +my $payinfo = $payby eq 'CARD' + ? $cust_refund->paymask + : $cust_refund->payinfo; + +$payby =~ s/^BILL$/Check #/ if $payinfo; +$payby =~ s/^CHEK$/Electronic check /; +$payby =~ s/^(CARD|COMP)$/$1 /; + +my($pre, $post) = ('', ''); +if ( $cust_refund->unapplied > 0 ) { + $pre = 'Unapplied '; + $post = ''; +} + +my $view = + ' ('. include('/elements/popup_link.html', + 'label' => 'view receipt', + 'action' => "${p}view/cust_refund.html?link=popup;". + 'refundnum='. $cust_refund->refundnum, + 'actionlabel' => 'Payment Receipt', + ). + ')'; + + +my $delete = ''; +if ( $cust_refund->closed !~ /^Y/i + && $conf->exists('deleterefunds') + && $curuser->access_right('Delete refund') + ) +{ + $delete = qq! (delete)!; +} + + + diff --git a/httemplate/view/cust_main/payment_history/voided_payment.html b/httemplate/view/cust_main/payment_history/voided_payment.html new file mode 100644 index 000000000..9cbc47b66 --- /dev/null +++ b/httemplate/view/cust_main/payment_history/voided_payment.html @@ -0,0 +1,37 @@ +Payment <% $info %> +voided <% time2str("%D", $cust_pay_void->void_date) %> +by <% $cust_pay_void->otaker %><% $unvoid %> +<%init> + +my( $cust_pay_void, %opt ) = @_; + +my $curuser = $FS::CurrentUser::CurrentUser; + +my $payby = $cust_pay_void->payby; +my $payinfo = $payby eq 'CARD' + ? $cust_pay_void->paymask + : $cust_pay_void->payinfo; + +$payby =~ s/^BILL$/Check #/ if $payinfo; +$payby =~ s/^CHEK$/Electronic check /; +$payby =~ s/^BILL$//; +$payby =~ s/^(CARD|COMP)$/$1 /; +my $info = $payby ? " ($payby$payinfo)" : ''; + +my $unvoid = ''; +if ( $cust_pay_void->closed !~ /^Y/i + && $curuser->access_right('Unvoid') + ) +{ + $unvoid = qq! (unvoid)!; +} + + diff --git a/httemplate/view/cust_refund.html b/httemplate/view/cust_refund.html new file mode 100644 index 000000000..40f6ab630 --- /dev/null +++ b/httemplate/view/cust_refund.html @@ -0,0 +1,141 @@ +% if ( $link eq 'popup' ) { + + <% include('/elements/header-popup.html', "Refund Receipt" ) %> + +
Print

+ +% } elsif ( $link eq 'print' ) { + + <% include('/elements/header-popup.html', "Refund Receipt" ) %> + +% #it would be nice if the menubar could be hidden for print, but better to +% # have it available than not, otherwise the user winds up at a dead end + <% menubar( + "View this customer (#$custnum)" => "${p}view/cust_main.cgi?$custnum", + ) + %> +

+ +% } else { + + <% include('/elements/header.html', "Refund Receipt", menubar( + "View this customer (#$custnum)" => "${p}view/cust_main.cgi?$custnum", + 'Print receipt' => $pr_link, + )) + %> + +% } + +% unless ($link eq 'popup' ) { + <% include('/elements/small_custview.html', + $custnum, + scalar($conf->config('countrydefault')), + 1, #no balance + ) + %> +

+% } + +<% ntable("#cccccc", 2) %> + + + Refund# + <% $cust_refund->refundnum %> + + + + Date + <% time2str"%a %b %o, %Y %r", $cust_refund->_date %> + + + + Amount + <% $money_char. $cust_refund->refund %> + + + + Reason + <% $cust_refund->reason %> + + + + Refund method + <% $cust_refund->payby_name %> #<% $cust_refund->paymask %> + + +% if ( $cust_refund->payby =~ /^(CARD|CHEK|LECB)$/ && $cust_refund->paybatch ) { + + + Processor + <% $cust_refund->processor %> + + + + Authorization# + <% $cust_refund->authorization %> + + +% if ( $cust_refund->order_number ) { + + Order# + <% $cust_refund->order_number %> + +% } + +% } + + + +% if ( $link eq 'print' ) { + + + +% } + +% if ( $link =~ /^(popup|print)$/ ) { + + +% } else { + <% include('/elements/footer.html') %> +% } + +<%init> + +my $curuser = $FS::CurrentUser::CurrentUser; + +die "access denied" + unless $curuser->access_right('View invoices') #remove this in 1.9 EVENTUALLY + || $curuser->access_right('View customer payments'); + #'View customer refunds' ??? + + +$cgi->param('refundnum') =~ /^(\d+)$/ or die "no refundnum"; +my $refundnum = $1; + +my $link = ''; +if ( $cgi->param('link') =~ /^(\w+)$/ ) { + $link = $1; +} + +my $cust_refund = qsearchs({ + 'select' => 'cust_refund.*', + 'table' => 'cust_refund', + 'addl_from' => 'LEFT JOIN cust_main USING ( custnum )', + 'hashref' => { 'refundnum' => $refundnum }, + 'extra_sql' => ' AND '. $FS::CurrentUser::CurrentUser->agentnums_sql, +}); +die "Refund #$refundnum not found!" unless $cust_refund; + +my $pr_link = "${p}view/cust_refund.html?link=print;refundnum=$refundnum"; + +my $custnum = $cust_refund->custnum; + +my $conf = new FS::Conf; + +my $money_char = $conf->config('money_char') || '$'; + +tie my %payby, 'Tie::IxHash', FS::payby->payby2longname; + + -- cgit v1.2.1