diff options
author | ivan <ivan> | 2010-07-28 02:54:46 +0000 |
---|---|---|
committer | ivan <ivan> | 2010-07-28 02:54:46 +0000 |
commit | 58c9445c8c514996203fef3486f2a9a864087155 (patch) | |
tree | 63b6f5928d66a8d1dca5ca8b69e5603dc90edd30 | |
parent | cd22a863e9cf3290a5fc83f32b572d0dac8da4e4 (diff) |
fix unnecessary warnings on upgrade of remote Pg RADIUS db, RT#9178
-rw-r--r-- | FS/FS/Upgrade.pm | 12 |
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; |