use strict;
use vars qw( @ISA $conf $start $stop );
+use FS::UID qw( dbh );
use FS::Record qw( qsearchs );
use FS::svc_acct;
use FS::port;
$error = $self->check;
return $error if $error;
- return "a session on that port is already open!"
- if qsearchs('session', { 'portnum' => $self->portnum, 'logout' => '' } );
+ my $oldAutoCommit = $FS::UID::AutoCommit;
+ local $FS::UID::AutoCommit = 0;
+ my $dbh = dbh;
+
+ if ( qsearchs('session', { 'portnum' => $self->portnum, 'logout' => '' } ) ) {
+ $dbh->rollback if $oldAutoCommit;
+ return "a session on that port is already open!";
+ }
$self->setfield('login', time()) unless $self->getfield('login');
$error = $self->SUPER::insert;
- return $error if $error;
+ if ( $error ) {
+ $dbh->rollback if $oldAutoCommit;
+ return $error;
+ }
$self->nas_heartbeat($self->getfield('login'));
#kcuy
my( $ip, $nasip, $nasfqdn ) = ( $port->ip, $nas->nasip, $nas->nasfqdn );
system( eval qq("$start") ) if $start;
-
+
+ $dbh->commit or die $dbh->errstr if $oldAutoCommit;
'';
}
local $SIG{TSTP} = 'IGNORE';
local $SIG{PIPE} = 'IGNORE';
+ my $oldAutoCommit = $FS::UID::AutoCommit;
+ local $FS::UID::AutoCommit = 0;
+ my $dbh = dbh;
+
$error = $self->check;
- return $error if $error;
+ if ( $error ) {
+ $dbh->rollback if $oldAutoCommit;
+ return $error;
+ }
$self->setfield('logout', time()) unless $self->getfield('logout');
$error = $self->SUPER::replace($old);
- return $error if $error;
+ if ( $error ) {
+ $dbh->rollback if $oldAutoCommit;
+ return $error;
+ }
$self->nas_heartbeat($self->getfield('logout'));
my $nas = qsearchs('nas',{'nasnum'=>$port->nasnum});
#kcuy
my( $ip, $nasip, $nasfqdn ) = ( $port->ip, $nas->nasip, $nas->nasfqdn );
- session( eval qq("$stop") ) if $stop;
+ system( eval qq("$stop") ) if $stop;
+
+ $dbh->commit or die $dbh->errstr if $oldAutoCommit;
'';
}
return $error if $error;
return "Unknown svcnum"
unless qsearchs('svc_acct', { 'svcnum' => $self->svcnum } );
- '';
+ $self->SUPER::check;
}
=item nas_heartbeat
=back
-=head1 VERSION
-
-$Id: session.pm,v 1.4 2000-12-08 22:22:31 ivan Exp $
-
=head1 BUGS
Maybe you shouldn't be able to insert a session if there's currently an open