use Carp qw( croak ); #confess );
use FS::Record qw( qsearch qsearchs );
use FS::access_user;
+use FS::UID qw( dbh );
sub otaker {
my $self = shift;
if ( !$access_user && $otaker =~ /^(.+), (.+)$/ ) { #same as below..
my($lastname, $firstname) = ($1, $2);
$otaker = lc($firstname.$lastname);
+ $otaker =~ s/ //g;
$access_user = qsearchs('access_user', { 'first' => $firstname,
'last' => $lastname } )
|| qsearchs('access_user', { 'username' => $otaker } );
$otaker; #not sure return is used anywhere, but just in case
} else { #get
if ( $self->usernum ) {
- $self->access_user->username;
+ # avoid a common failure mode: this should work even when the table
+ # isn't foreign-keyed to access_user
+ my $access_user = FS::access_user->by_key($self->usernum);
+ if (!$access_user) {
+ croak "otaker called on ".$self->table."#".
+ $self->get($self->primary_key).
+ " but user does not exist";
+ }
+ return $access_user->username;
} elsif ( length($self->get('otaker')) ) {
$self->get('otaker');
} else {
}
}
-sub access_user {
- my $self = shift;
- qsearchs('access_user', { 'usernum' => $self->usernum } );
-}
-
sub _upgrade_otaker {
my $class = shift;
my $table = $class->table;
if ( $username =~ /^(.+), (.+)$/ ) {
($lastname, $firstname) = ($1, $2);
$username = lc($firstname.$lastname);
+ $username =~ s/ //g;
}
my $access_user = new FS::access_user {
'username' => $username,
my $error = $record->replace;
die $error if $error;
}
+
+ if ( $table eq 'cust_attachment' ) {
+ warn " committing (cust_attachment) \n";
+ dbh->commit or die dbh->errstr;
+ }
}