summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorivan <ivan>2010-07-28 02:54:46 +0000
committerivan <ivan>2010-07-28 02:54:46 +0000
commit58c9445c8c514996203fef3486f2a9a864087155 (patch)
tree63b6f5928d66a8d1dca5ca8b69e5603dc90edd30
parentcd22a863e9cf3290a5fc83f32b572d0dac8da4e4 (diff)
fix unnecessary warnings on upgrade of remote Pg RADIUS db, RT#9178
-rw-r--r--FS/FS/Upgrade.pm12
1 files changed, 8 insertions, 4 deletions
diff --git a/FS/FS/Upgrade.pm b/FS/FS/Upgrade.pm
index b5c8ff169..72761b49e 100644
--- a/FS/FS/Upgrade.pm
+++ b/FS/FS/Upgrade.pm
@@ -170,7 +170,8 @@ sub upgrade_sqlradius {
my $str2time = str2time_sql( $dbh->{Driver}->{Name} );
my $group = "UserName";
$group .= ",Realm"
- if ( ref($part_export) =~ /withdomain/ );
+ if ref($part_export) =~ /withdomain/
+ || $dbh->{Driver}->{Name} =~ /^Pg/; #hmm
my $sth_alter = $dbh->prepare(
"ALTER TABLE radacct ADD COLUMN FreesideStatus varchar(32) NULL"
@@ -183,7 +184,10 @@ sub upgrade_sqlradius {
$sth_update->execute or die $errmsg.$sth_update->errstr;
} else {
my $error = $sth_alter->errstr;
- warn $errmsg.$error unless $error =~ /Duplicate column name/i;
+ warn $errmsg.$error
+ unless $error =~ /Duplicate column name/i #mysql
+ || $error =~ /already exists/i; #Pg
+;
}
} else {
my $error = $dbh->errstr;
@@ -197,8 +201,8 @@ sub upgrade_sqlradius {
unless ( $sth_index->execute ) {
my $error = $sth_index->errstr;
warn $errmsg.$error
- unless $error =~ /Duplicate key name/i #mysql
- || $error =~ /relation "freesidestatus" already exists/i; #Pg
+ unless $error =~ /Duplicate key name/i #mysql
+ || $error =~ /already exists/i; #Pg
}
} else {
my $error = $dbh->errstr;