From: ivan <ivan>
Date: Tue, 15 Jul 2008 23:25:08 +0000 (+0000)
Subject: where in the world is $DBI::errstr
X-Git-Tag: root_of_webpay_support~480
X-Git-Url: http://git.freeside.biz/gitweb/?a=commitdiff_plain;h=58d13b1770a2429111dec618b8b89996d9720bf4;p=freeside.git

where in the world is $DBI::errstr
---

diff --git a/FS/FS/Upgrade.pm b/FS/FS/Upgrade.pm
index 55eed8ea1..6794f4d9f 100644
--- a/FS/FS/Upgrade.pm
+++ b/FS/FS/Upgrade.pm
@@ -117,20 +117,18 @@ sub upgrade_sqlradius {
   my @part_export = FS::part_export::sqlradius->all_sqlradius_withaccounting();
 
   foreach my $part_export ( @part_export ) {
+
+    my $errmsg = 'Error adding FreesideStatus to '.
+                 $part_export->option('datasrc'). ': ';
+
     my $dbh = DBI->connect(
       ( map $part_export->option($_), qw ( datasrc username password ) ),
       { PrintError => 0, PrintWarn => 0 }
-    );
-
-    unless ($dbh) {
-      warn "can't connect to RADIUS database ".
-           $part_export->option('datasrc').  ": $DBI::errstr\n";
+    ) or do {
+      warn $errmsg.$DBI::errstr;
       next;
-    }
+    };
 
-    my $errmsg = 'Error adding FreesideStatus to '.
-                 $part_export->option('datasrc'). ': ';
-  
     my $str2time = str2time_sql( $dbh->{Driver}->{Name} );
     my $group = "UserName";
     $group .= ",Realm"