1 package FS::cust_payby;
2 use base qw( FS::payinfo_Mixin FS::cust_main_Mixin FS::Record );
5 use Scalar::Util qw( blessed );
6 use Digest::SHA qw( sha512_base64 );
7 use Business::CreditCard qw( validate cardtype );
9 use FS::Msgcat qw( gettext );
10 use FS::Misc qw( card_types );
11 use FS::Record; #qw( qsearch qsearchs );
16 our @encrypted_fields = ('payinfo', 'paycvv');
17 sub nohistory_fields { ('payinfo', 'paycvv'); }
19 our $ignore_expired_card = 0;
20 our $ignore_banned_card = 0;
21 our $ignore_invalid_card = 0;
22 our $ignore_cardtype = 0;
25 install_callback FS::UID sub {
27 #yes, need it for stuff below (prolly should be cached)
28 $ignore_invalid_card = $conf->exists('allow_invalid_cards');
33 FS::cust_payby - Object methods for cust_payby records
39 $record = new FS::cust_payby \%hash;
40 $record = new FS::cust_payby { 'column' => 'value' };
42 $error = $record->insert;
44 $error = $new_record->replace($old_record);
46 $error = $record->delete;
48 $error = $record->check;
52 An FS::cust_payby object represents customer stored payment information.
53 FS::cust_payby inherits from FS::Record. The following fields are currently
120 The credit card type (deduced from the card number).
130 Creates a new record. To add the record to the database, see L<"insert">.
132 Note that this stores the hash reference, not a distinct copy of the hash it
133 points to. You can ask the object for a copy with the I<hash> method.
137 # the new method can be inherited from FS::Record, if a table method is defined
139 sub table { 'cust_payby'; }
143 Adds this record to the database. If there is an error, returns the error,
144 otherwise returns false.
151 local $SIG{HUP} = 'IGNORE';
152 local $SIG{INT} = 'IGNORE';
153 local $SIG{QUIT} = 'IGNORE';
154 local $SIG{TERM} = 'IGNORE';
155 local $SIG{TSTP} = 'IGNORE';
156 local $SIG{PIPE} = 'IGNORE';
158 my $oldAutoCommit = $FS::UID::AutoCommit;
159 local $FS::UID::AutoCommit = 0;
162 my $error = $self->check_payinfo_cardtype if $self->payby =~/^(CARD|DCRD)$/;
163 $self->SUPER::insert unless $error;
166 $dbh->rollback if $oldAutoCommit;
170 if ( $self->payby =~ /^(CARD|CHEK)$/ ) {
171 # new auto card/check info, want to retry realtime_ invoice events
172 # (new customer? that's okay, they won't have any)
173 my $error = $self->cust_main->retry_realtime;
175 $dbh->rollback if $oldAutoCommit;
180 $dbh->commit or die $dbh->errstr if $oldAutoCommit;
187 Delete this record from the database.
189 =item replace OLD_RECORD
191 Replaces the OLD_RECORD with this one in the database. If there is an error,
192 returns the error, otherwise returns false.
199 my $old = ( blessed($_[0]) && $_[0]->isa('FS::Record') )
201 : $self->replace_old;
203 if ( $self->payby =~ /^(CARD|DCRD)$/
204 && ( $self->payinfo =~ /xx/
205 || $self->payinfo =~ /^\s*N\/A\s+\(tokenized\)\s*$/
210 $self->payinfo($old->payinfo);
212 } elsif ( $self->payby =~ /^(CHEK|DCHK)$/ && $self->payinfo =~ /xx/ ) {
213 #fix for #3085 "edit of customer's routing code only surprisingly causes
214 #nothing to happen...
215 # this probably won't do the right thing when we don't have the
216 # public key (can't actually get the real $old->payinfo)
217 my($new_account, $new_aba) = split('@', $self->payinfo);
218 my($old_account, $old_aba) = split('@', $old->payinfo);
219 $new_account = $old_account if $new_account =~ /xx/;
220 $new_aba = $old_aba if $new_aba =~ /xx/;
221 $self->payinfo($new_account.'@'.$new_aba);
224 # only unmask paycvv if payinfo stayed the same
225 if ( $self->payby =~ /^(CARD|DCRD)$/ and $self->paycvv =~ /^\s*[\*x]+\s*$/ ) {
226 if ( $old->payinfo eq $self->payinfo
227 && $old->paymask eq $self->paymask
229 $self->paycvv($old->paycvv);
235 local($ignore_expired_card) = 1
236 if $old->payby =~ /^(CARD|DCRD)$/
237 && $self->payby =~ /^(CARD|DCRD)$/
238 && ( $old->payinfo eq $self->payinfo || $old->paymask eq $self->paymask );
240 local($ignore_banned_card) = 1
241 if ( $old->payby =~ /^(CARD|DCRD)$/ && $self->payby =~ /^(CARD|DCRD)$/
242 || $old->payby =~ /^(CHEK|DCHK)$/ && $self->payby =~ /^(CHEK|DCHK)$/ )
243 && ( $old->payinfo eq $self->payinfo || $old->paymask eq $self->paymask );
245 if ( $self->payby =~ /^(CARD|DCRD)$/
246 && $old->payinfo ne $self->payinfo
247 && $old->paymask ne $self->paymask )
249 my $error = $self->check_payinfo_cardtype;
250 return $error if $error;
252 if ( $conf->exists('business-onlinepayment-verification') ) {
253 $error = $self->verify;
255 $error = $self->tokenize;
257 return $error if $error;
261 local $SIG{HUP} = 'IGNORE';
262 local $SIG{INT} = 'IGNORE';
263 local $SIG{QUIT} = 'IGNORE';
264 local $SIG{TERM} = 'IGNORE';
265 local $SIG{TSTP} = 'IGNORE';
266 local $SIG{PIPE} = 'IGNORE';
268 my $oldAutoCommit = $FS::UID::AutoCommit;
269 local $FS::UID::AutoCommit = 0;
272 my $error = $self->SUPER::replace($old);
274 $dbh->rollback if $oldAutoCommit;
278 if ( $self->payby =~ /^(CARD|CHEK)$/
279 && ( ( $self->get('payinfo') ne $old->get('payinfo')
282 || grep { $self->get($_) ne $old->get($_) } qw(paydate payname)
287 # card/check/lec info has changed, want to retry realtime_ invoice events
288 my $error = $self->cust_main->retry_realtime;
290 $dbh->rollback if $oldAutoCommit;
295 $dbh->commit or die $dbh->errstr if $oldAutoCommit;
302 Checks all fields to make sure this is a valid record. If there is
303 an error, returns the error, otherwise returns false. Called by the insert
312 $self->ut_numbern('custpaybynum')
313 || $self->ut_foreign_key('custnum', 'cust_main', 'custnum')
314 || $self->ut_numbern('weight')
315 #encrypted #|| $self->ut_textn('payinfo')
316 #encrypted #|| $self->ut_textn('paycvv')
317 # || $self->ut_textn('paymask') #XXX something
318 || $self->ut_numbern('paystart_month')
319 || $self->ut_numbern('paystart_year')
320 || $self->ut_numbern('payissue')
321 # || $self->ut_textn('payname') #XXX something
322 || $self->ut_alphan('paystate')
323 || $self->ut_textn('paytype')
324 || $self->ut_ipn('payip')
326 return $error if $error;
330 FS::payby->can_payby($self->table, $self->payby)
331 or return "Illegal payby: ". $self->payby;
333 # If it is encrypted and the private key is not availaible then we can't
334 # check the credit card.
335 my $check_payinfo = ! $self->is_encrypted($self->payinfo);
337 # Need some kind of global flag to accept invalid cards, for testing
339 #XXX if ( !$import && $check_payinfo && $self->payby =~ /^(CARD|DCRD)$/ ) {
341 # In this block: detect card type; reject credit card / account numbers that
342 # are impossible or banned; reject other payment features (date, CVV length)
343 # that are inappropriate for the card type.
344 # However, if the payinfo is encrypted then just detect card type and assume
345 # the other checks were already done.
347 if ( !$ignore_invalid_card &&
348 $check_payinfo && $self->payby =~ /^(CARD|DCRD)$/ ) {
350 my $payinfo = $self->payinfo;
352 $payinfo =~ /^(\d{13,19}|\d{8,9})$/
353 or return gettext('invalid_card'); #. ": ". $self->payinfo;
355 $self->payinfo($payinfo);
357 or return gettext('invalid_card'); # . ": ". $self->payinfo;
359 # see parallel checks in check_payinfo_cardtype & payinfo_Mixin::payinfo_check
360 my $cardtype = $self->paycardtype;
361 if ( $self->tokenized ) {
362 $self->set('is_tokenized', 'Y'); #so we don't try to do it again
363 if ( $self->paymask =~ /^\d+x/ ) {
364 $cardtype = cardtype($self->paymask);
366 #return "paycardtype required ".
367 # "(can't derive from a token and no paymask w/prefix provided)"
371 $cardtype = cardtype($self->payinfo);
374 return gettext('unknown_card_type') if $cardtype eq "Unknown";
376 $self->set('paycardtype', $cardtype);
378 unless ( $ignore_banned_card ) {
379 my $ban = FS::banned_pay->ban_search( %{ $self->_banned_pay_hashref } );
381 if ( $ban->bantype eq 'warn' ) {
382 #or others depending on value of $ban->reason ?
383 return '_duplicate_card'.
384 ': disabled from'. time2str('%a %h %o at %r', $ban->_date).
385 ' until '. time2str('%a %h %o at %r', $ban->_end_date).
386 ' (ban# '. $ban->bannum. ')'
387 unless $self->override_ban_warn;
389 return 'Banned credit card: banned on '.
390 time2str('%a %h %o at %r', $ban->_date).
391 ' by '. $ban->otaker.
392 ' (ban# '. $ban->bannum. ')';
397 if (length($self->paycvv) && !$self->is_encrypted($self->paycvv)) {
398 if ( $cardtype eq 'American Express card' ) {
399 $self->paycvv =~ /^(\d{4})$/
400 or return "CVV2 (CID) for American Express cards is four digits.";
403 $self->paycvv =~ /^(\d{3})$/
404 or return "CVV2 (CVC2/CID) is three digits.";
411 if ( $cardtype =~ /^(Switch|Solo)$/i ) {
413 return "Start date or issue number is required for $cardtype cards"
414 unless $self->paystart_month && $self->paystart_year or $self->payissue;
416 return "Start month must be between 1 and 12"
417 if $self->paystart_month
418 and $self->paystart_month < 1 || $self->paystart_month > 12;
420 return "Start year must be 1990 or later"
421 if $self->paystart_year
422 and $self->paystart_year < 1990;
424 return "Issue number must be beween 1 and 99"
426 and $self->payissue < 1 || $self->payissue > 99;
429 $self->paystart_month('');
430 $self->paystart_year('');
434 } elsif ( !$ignore_invalid_card &&
435 $check_payinfo && $self->payby =~ /^(CHEK|DCHK)$/ ) {
437 my $payinfo = $self->payinfo;
438 $payinfo =~ s/[^\d\@\.]//g;
439 if ( $conf->config('echeck-country') eq 'CA' ) {
440 $payinfo =~ /^(\d+)\@(\d{5})\.(\d{3})$/
441 or return 'invalid echeck account@branch.bank';
442 $payinfo = "$1\@$2.$3";
443 } elsif ( $conf->config('echeck-country') eq 'US' ) {
444 $payinfo =~ /^(\d+)\@(\d{9})$/ or return 'invalid echeck account@aba';
447 $payinfo =~ /^(\d+)\@(\d+)$/ or return 'invalid echeck account@routing';
450 $self->payinfo($payinfo);
453 unless ( $ignore_banned_card ) {
454 my $ban = FS::banned_pay->ban_search( %{ $self->_banned_pay_hashref } );
456 if ( $ban->bantype eq 'warn' ) {
457 #or others depending on value of $ban->reason ?
458 return '_duplicate_ach' unless $self->override_ban_warn;
460 return 'Banned ACH account: banned on '.
461 time2str('%a %h %o at %r', $ban->_date).
462 ' by '. $ban->otaker.
463 ' (ban# '. $ban->bannum. ')';
468 } elsif ( $self->payby =~ /^CARD|DCRD$/ and $self->paymask ) {
469 # either ignoring invalid cards, or we can't decrypt the payinfo, but
470 # try to detect the card type anyway. this never returns failure, so
471 # the contract of $ignore_invalid_cards is maintained.
472 $self->set('paycardtype', cardtype($self->paymask));
474 $self->set('paycardtype', '');
477 # } elsif ( $self->payby eq 'PREPAY' ) {
479 # my $payinfo = $self->payinfo;
480 # $payinfo =~ s/\W//g; #anything else would just confuse things
481 # $self->payinfo($payinfo);
482 # $error = $self->ut_alpha('payinfo');
483 # return "Illegal prepayment identifier: ". $self->payinfo if $error;
484 # return "Unknown prepayment identifier"
485 # unless qsearchs('prepay_credit', { 'identifier' => $self->payinfo } );
488 if ( $self->payby =~ /^(CHEK|DCHK)$/ ) {
492 } elsif ( $self->payby =~ /^(CARD|DCRD)$/ ) {
494 # shouldn't payinfo_check do this?
495 # (except we don't ever call payinfo_check from here)
496 return "Expiration date required"
497 if $self->paydate eq '' || $self->paydate eq '-';
500 if ( $self->paydate =~ /^(\d{1,2})[\/\-](\d{2}(\d{2})?)$/ ) {
501 ( $m, $y ) = ( $1, length($2) == 4 ? $2 : "20$2" );
502 } elsif ( $self->paydate =~ /^19(\d{2})[\/\-](\d{1,2})[\/\-]\d+$/ ) {
503 ( $m, $y ) = ( $2, "19$1" );
504 } elsif ( $self->paydate =~ /^(20)?(\d{2})[\/\-](\d{1,2})[\/\-]\d+$/ ) {
505 ( $m, $y ) = ( $3, "20$2" );
507 return "Illegal expiration date: ". $self->paydate;
509 $m = sprintf('%02d',$m);
510 $self->paydate("$y-$m-01");
511 my($nowm,$nowy)=(localtime(time))[4,5]; $nowm++; $nowy+=1900;
512 return gettext('expired_card')
515 !$ignore_expired_card
516 && ( $y<$nowy || ( $y==$nowy && $1<$nowm ) );
520 if ( $self->payname eq '' && $self->payby !~ /^(CHEK|DCHK)$/ &&
521 ( ! $conf->exists('require_cardname')
522 || $self->payby !~ /^(CARD|DCRD)$/ )
524 $self->payname( $self->first. " ". $self->getfield('last') );
527 if ( $self->payby =~ /^(CHEK|DCHK)$/ ) {
528 $self->payname =~ /^([\w \,\.\-\']*)$/
529 or return gettext('illegal_name'). " payname: ". $self->payname;
532 $self->payname =~ /^([\w \,\.\-\'\&]*)$/
533 or return gettext('illegal_name'). " payname: ". $self->payname;
539 if ( ! $self->custpaybynum ) {
540 if ($conf->exists('business-onlinepayment-verification')) {
541 $error = $self->verify;
543 $error = $self->tokenize;
545 return $error if $error;
548 $error = $self->ut_daten('paydate');
549 return $error if $error;
554 sub check_payinfo_cardtype {
557 return '' if $ignore_cardtype;
559 return '' unless $self->payby =~ /^(CARD|CHEK)$/;
561 my $payinfo = $self->payinfo;
564 # see parallel checks in cust_payby::check & payinfo_Mixin::payinfo_check
565 if ( $self->tokenized($payinfo) ) {
566 $self->set('is_tokenized', 'Y'); #so we don't try to do it again
567 if ( $self->paymask =~ /^\d+x/ ) {
568 $self->set('paycardtype', cardtype($self->paymask));
570 $self->set('paycardtype', '');
571 #return "paycardtype required ".
572 # "(can't derive from a token and no paymask w/prefix provided)";
577 my %bop_card_types = map { $_=>1 } values %{ card_types() };
578 my $cardtype = cardtype($payinfo);
579 $self->set('paycardtype', $cardtype);
581 return "$cardtype not accepted" unless $bop_card_types{$cardtype};
587 sub _banned_pay_hashref {
598 'payby' => $payby2ban{$self->payby},
599 'payinfo' => $self->payinfo,
600 #don't ever *search* on reason! #'reason' =>
604 sub _new_banned_pay_hashref {
606 my $hr = $self->_banned_pay_hashref;
607 $hr->{payinfo_hash} = 'SHA512';
608 $hr->{payinfo} = sha512_base64($hr->{payinfo});
612 =item paydate_mon_year
614 Returns a two element list consisting of the paydate month and year.
618 sub paydate_mon_year {
621 my $date = $self->paydate; # || '12-2037';
623 #false laziness w/elements/select-month_year.html
624 if ( $date =~ /^(\d{4})-(\d{1,2})-\d{1,2}$/ ) { #PostgreSQL date format
626 } elsif ( $date =~ /^(\d{1,2})-(\d{1,2}-)?(\d{4}$)/ ) {
629 warn "unrecognized expiration date format: $date";
637 Returns a one line text label for this payment type.
654 my $name = $self->payby =~ /^(CARD|DCRD)$/
655 && $self->paycardtype || FS::payby->shortname($self->payby);
657 ( $self->payby =~ /^(CARD|CHEK)$/ ? $weight{$self->weight}. ' automatic '
660 "$name: ". $self->paymask.
661 ( $self->payby =~ /^(CARD|DCRD)$/
662 ? ' Exp '. join('/', $self->paydate_mon_year)
670 Runs a L<realtime_bop|FS::cust_main::Billing_Realtime::realtime_bop> transaction on this card
675 my( $self, %opt ) = @_;
677 $self->cust_main->realtime_bop({
679 'cust_payby' => $self,
686 Runs a L<realtime_tokenize|FS::cust_main::Billing_Realtime::realtime_tokenize> transaction on this card
692 return '' unless $self->payby =~ /^(CARD|DCRD)$/;
694 $self->cust_main->realtime_tokenize({
695 'cust_payby' => $self,
702 Runs a L<realtime_verify_bop|FS::cust_main::Billing_Realtime/realtime_verify_bop> transaction on this card
708 return '' unless $self->payby =~ /^(CARD|DCRD)$/;
710 $self->cust_main->realtime_verify_bop({
711 'cust_payby' => $self,
718 Returns a list of valid values for the paytype field (bank account type for
719 electronic check payment).
726 ('', 'Personal checking', 'Personal savings', 'Business checking', 'Business savings');
729 =item cgi_cust_payby_fields
731 Returns the field names used in the web interface (including some pseudo-fields).
735 sub cgi_cust_payby_fields {
737 [qw( payby payinfo paydate_month paydate_year paycvv payname weight
738 payinfo1 payinfo2 payinfo3 paytype paystate payname_CHEK )];
741 =item cgi_hash_callback HASHREF OLD
743 Subroutine (not a class or object method). Processes a hash reference
744 of web interface contet (transfers the data from pseudo-fields to real fields).
746 If OLD object is passed, also preserves locationnum, paystart_month, paystart_year,
747 payissue and payip. If the new field is blank but the old is not, the old field
752 sub cgi_hash_callback {
760 # the payby selector gives the choice of CARD or CHEK (or others, but
761 # those are the ones with auto and on-demand versions). if the user didn't
762 # choose a weight, then they mean DCRD/DCHK.
763 $hashref->{payby} = $noauto{$hashref->{payby}}
764 if ! $hashref->{weight} && exists $noauto{$hashref->{payby}};
766 if ( $hashref->{payby} =~ /^(CHEK|DCHK)$/ ) {
768 unless ( grep $hashref->{$_}, qw(payinfo1 payinfo2 payinfo3 payname_CHEK)) {
773 $hashref->{payinfo} = $hashref->{payinfo1}. '@';
774 $hashref->{payinfo} .= $hashref->{payinfo3}.'.'
775 if $conf->config('echeck-country') eq 'CA';
776 $hashref->{payinfo} .= $hashref->{'payinfo2'};
778 $hashref->{payname} = $hashref->{'payname_CHEK'};
780 } elsif ( $hashref->{payby} =~ /^(CARD|DCRD)$/ ) {
782 unless ( grep $hashref->{$_}, qw( payinfo paycvv payname ) ) {
789 $hashref->{paydate}= $hashref->{paydate_month}. '-'. $hashref->{paydate_year};
792 foreach my $field ( qw(locationnum paystart_month paystart_year payissue payip) ) {
793 next if $hashref->{$field};
794 next unless $old->get($field);
795 $hashref->{$field} = $old->get($field);
805 Returns a qsearch hash expression to search for parameters specified in HASHREF.
824 my ($class, $params) = @_;
829 # initialize these to prevent warnings
831 'paydate_year' => '',
839 if ( $params->{'payby'} ) {
841 my @payby = ref( $params->{'payby'} )
842 ? @{ $params->{'payby'} }
843 : ( $params->{'payby'} );
845 @payby = grep /^([A-Z]{4})$/, @payby;
846 my $in_payby = 'IN(' . join(',', map {"'$_'"} @payby) . ')';
847 push @where, "cust_payby.payby $in_payby"
852 # paydate_year / paydate_month
855 if ( $params->{'paydate_year'} =~ /^(\d{4})$/ ) {
857 $params->{'paydate_month'} =~ /^(\d\d?)$/
858 or die "paydate_year without paydate_month?";
862 'cust_payby.paydate IS NOT NULL',
863 "cust_payby.paydate != ''",
864 "CAST(cust_payby.paydate AS timestamp) < CAST('$year-$month-01' AS timestamp )"
868 # setup queries, subs, etc. for the search
871 $orderby ||= 'ORDER BY custnum';
873 # here is the agent virtualization
875 $FS::CurrentUser::CurrentUser->agentnums_sql(table => 'cust_main');
877 my $extra_sql = scalar(@where) ? ' WHERE '. join(' AND ', @where) : '';
879 my $addl_from = ' LEFT JOIN cust_main USING ( custnum ) ';
880 # always make address fields available in results
881 for my $pre ('bill_', 'ship_') {
883 ' LEFT JOIN cust_location AS '.$pre.'location '.
884 'ON (cust_main.'.$pre.'locationnum = '.$pre.'location.locationnum) ';
886 # always make referral available in results
887 # (maybe we should be using FS::UI::Web::join_cust_main instead?)
888 $addl_from .= ' LEFT JOIN (select refnum, referral from part_referral) AS part_referral_x ON (cust_main.refnum = part_referral_x.refnum) ';
890 my $count_query = "SELECT COUNT(*) FROM cust_payby $addl_from $extra_sql";
892 my @select = ( 'cust_payby.*',
893 #'cust_main.custnum',
894 # there's a good chance that we'll need these
895 'cust_main.bill_locationnum',
896 'cust_main.ship_locationnum',
897 FS::UI::Web::cust_sql_fields($params->{'cust_fields'}),
900 my $select = join(', ', @select);
903 'table' => 'cust_payby',
905 'addl_from' => $addl_from,
907 'extra_sql' => $extra_sql,
908 'order_by' => $orderby,
909 'count_query' => $count_query,
917 =item count_autobill_cards
919 Returns the number of unexpired cards configured for autobill
923 sub count_autobill_cards {
926 AND payby IN ('CARD','DCRD')
927 AND paydate > '".DateTime->now->ymd."'
931 =item count_autobill_checks
933 Returns the number of check accounts configured for autobill
937 sub count_autobill_checks {
940 AND payby IN ('CHEK','DCHEK')
947 local $ignore_banned_card = 1;
948 local $ignore_expired_card = 1;
949 local $ignore_invalid_card = 1;
950 $class->upgrade_set_cardtype;
951 $class->_upgrade_data_paydate_edgebug;
955 =item _upgrade_data_paydate_edgebug
957 Correct bad data injected into payment expire date column by Edge browser bug
959 The month and year values may have an extra character injected into form POST
960 data by Edge browser. It was possible for some bad month values to slip
961 past data validation.
963 If the stored value was out of range, it was causing payments screen to crash.
964 We can detect and fix this by dropping the second digit.
966 If the stored value is is 11 or 12, it's possible the user inputted a 1. In
967 this case, the payment method will fail to authorize, but the record will
968 not cause crashdumps for being out of range.
970 In short, check for any expiration month > 12, and drop the extra digit
974 sub _upgrade_data_paydate_edgebug {
975 my $journal_label = 'cust_payby_paydate_edgebug';
976 return if FS::upgrade_journal->is_done( $journal_label );
978 my $oldAutoCommit = $FS::UID::AutoCommit;
979 local $FS::UID::AutoCommit = 0;
983 cust_payby => { paydate => { op => '!=', value => '' }}
986 next unless $row->ut_daten('paydate');
988 # paydate column stored in database has failed date validation
989 my $bad_paydate = $row->paydate;
991 my @date = split /[\-\/]/, $bad_paydate;
992 @date = @date[2,0,1] if $date[2] > 1900;
994 # Only autocorrecting when month > 12 - notify operator
995 unless ( $date[1] > 12 ) {
997 'Unable to correct bad paydate stored in cust_payby row '.
998 'custpaybynum(%s) custnum(%s) paydate(%s)',
1005 $date[1] = substr( $date[1], 0, 1 );
1006 $row->paydate( join('-', @date ));
1008 if ( my $error = $row->replace ) {
1010 'Failed to autocorrect bad paydate stored in cust_payby row '.
1011 'custpaybynum(%s) custnum(%s) paydate(%s) - error: %s',
1020 'Autocorrected bad paydate stored in cust_payby row '.
1021 "custpaybynum(%s) custnum(%s) old-paydate(%s) new-paydate(%s)\n",
1030 FS::upgrade_journal->set_done( $journal_label );
1031 dbh->commit unless $oldAutoCommit;
1038 L<FS::Record>, schema.html from the base documentation.