use FS::cust_main;
use FS::cust_pay_void;
-@ISA = qw(FS::Record FS::cust_main_Mixin FS::payinfo_Mixin );
+@ISA = qw( FS::Record FS::cust_main_Mixin FS::payinfo_Mixin );
$DEBUG = 0;
# " already exists";
# }
- $self->otaker(getotaker);
-
$self->SUPER::check;
}
sub payby_name {
my $self = shift;
- FS::payby->shortname( $self->payby );
+ if ( $self->payby eq 'BILL' ) { #kludge
+ 'Check';
+ } else {
+ FS::payby->shortname( $self->payby );
+ }
}
=item gatewaynum
#not the most efficient, but hey, it only has to run once
- my $count_sql =
- "SELECT COUNT(*) FROM cust_pay WHERE otaker IS NULL OR otaker = ''";
+ my $where = "WHERE ( otaker IS NULL OR otaker = '' OR otaker = 'ivan' ) ".
+ " AND 0 < ( SELECT COUNT(*) FROM cust_main ".
+ " WHERE cust_main.custnum = cust_pay.custnum ) ";
+
+ my $count_sql = "SELECT COUNT(*) FROM cust_pay $where";
my $sth = dbh->prepare($count_sql) or die dbh->errstr;
$sth->execute or die $sth->errstr;
my $count = 0;
my $lastprog = 0;
- while (1) {
- my $cust_pay = qsearchs( {
+ my @cust_pay = qsearch( {
'table' => 'cust_pay',
'hashref' => {},
- 'extra_sql' => "WHERE otaker IS NULL OR otaker = ''",
- 'order_by' => 'ORDER BY paynum LIMIT 1',
- } );
+ 'extra_sql' => $where,
+ 'order_by' => 'ORDER BY paynum',
+ } );
- return unless $cust_pay;
+ foreach my $cust_pay (@cust_pay) {
my $h_cust_pay = $cust_pay->h_search('insert');
if ( $h_cust_pay ) {
+ next if $cust_pay->otaker eq $h_cust_pay->history_user;
$cust_pay->otaker($h_cust_pay->history_user);
} else {
$cust_pay->otaker('legacy');
}
+
+ delete $FS::payby::hash{'COMP'}->{cust_pay}; #quelle kludge
my $error = $cust_pay->replace;
- die $error if $error;
+
+ if ( $error ) {
+ warn " *** WARNING: Error updaating order taker for payment paynum".
+ $cust_pay->paynun. ": $error\n";
+ next;
+ }
+
+ $FS::payby::hash{'COMP'}->{cust_pay} = ''; #restore it
$count++;
if ( $DEBUG > 1 && $lastprog + 30 < time ) {