use vars qw(@ISA);
@ISA = qw( FS::payinfo_Mixin FS::Record );
-=had1 DESCRIPTION
+=head1 DESCRIPTION
This is a mixin class for records that contain payinfo.
=cut
sub paymask {
- my($self,$paymask)=@_;
+ my($self, $paymask) = @_;
-
- if ($paymask ne '') {
- # I hate this little bit of magic... I don't expect it to cause a problem, but who knows...
- # If the payinfo is passed in masked then ignore it and set it based on the payinfo
- # The only guy that should call this in this way is... $self->payinfo
+ if ( defined($paymask) && $paymask ne '' ) {
+ # I hate this little bit of magic... I don't expect it to cause a problem,
+ # but who knows... If the payinfo is passed in masked then ignore it and
+ # set it based on the payinfo. The only guy that should call this in this
+ # way is... $self->payinfo
$self->setfield('paymask', $self->mask_payinfo());
+
} else {
+
$paymask=$self->getfield('paymask');
if (!defined($paymask) || $paymask eq '') {
- # Generate it if it's blank - Note that we're not going to set it - just generate
+ # Generate it if it's blank - Note that we're not going to set it - just
+ # generate
$paymask = $self->mask_payinfo();
}
+
}
+
return $paymask;
}
my $self = shift;
FS::payby->can_payby($self->table, $self->payby)
- or return "Illegal payby";
+ or return "Illegal payby: ". $self->payby;
if ( $self->payby eq 'CARD' ) {
my $payinfo = $self->payinfo;