projects
/
freeside.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
allow cust_pay.otaker upgrade to proceed even if there are some old crufty records...
[freeside.git]
/
FS
/
FS
/
cust_pay.pm
diff --git
a/FS/FS/cust_pay.pm
b/FS/FS/cust_pay.pm
index
90a14ea
..
de14610
100644
(file)
--- a/
FS/FS/cust_pay.pm
+++ b/
FS/FS/cust_pay.pm
@@
-432,8
+432,6
@@
sub check {
# " already exists";
# }
# " already exists";
# }
- $self->otaker(getotaker);
-
$self->SUPER::check;
}
$self->SUPER::check;
}
@@
-675,8
+673,11
@@
sub _upgrade_data { #class method
#not the most efficient, but hey, it only has to run once
#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 $sth = dbh->prepare($count_sql) or die dbh->errstr;
$sth->execute or die $sth->errstr;
@@
-691,7
+692,7
@@
sub _upgrade_data { #class method
my $cust_pay = qsearchs( {
'table' => 'cust_pay',
'hashref' => {},
my $cust_pay = qsearchs( {
'table' => 'cust_pay',
'hashref' => {},
- 'extra_sql' =>
"WHERE otaker IS NULL OR otaker = ''"
,
+ 'extra_sql' =>
$where
,
'order_by' => 'ORDER BY paynum LIMIT 1',
} );
'order_by' => 'ORDER BY paynum LIMIT 1',
} );
@@
-706,7
+707,15
@@
sub _upgrade_data { #class method
delete $FS::payby::hash{'COMP'}->{cust_pay}; #quelle kludge
my $error = $cust_pay->replace;
delete $FS::payby::hash{'COMP'}->{cust_pay}; #quelle kludge
my $error = $cust_pay->replace;
+
+ #infinite...
+ #if ( $error ) {
+ # warn " *** WARNING: Error updaating order taker for payment paynum".
+ # $cust_pay->paynun. ": $error\n";
+ # next;
+ #}
die $error if $error;
die $error if $error;
+
$FS::payby::hash{'COMP'}->{cust_pay} = ''; #restore it
$count++;
$FS::payby::hash{'COMP'}->{cust_pay} = ''; #restore it
$count++;