X-Git-Url: http://git.freeside.biz/gitweb/?a=blobdiff_plain;f=fs_signup%2Ffs_signup_server;h=70045e63b5003d55b76577a1408017ef0113bc7d;hb=a63b2729682586d8860290576e9307629424dbe0;hp=e7b372a4e0813e8ab3ad1777feefd61bd4c2127f;hpb=1f93febd8c42aa344867d9f46601075ecf949ec4;p=freeside.git

diff --git a/fs_signup/fs_signup_server b/fs_signup/fs_signup_server
index e7b372a4e..70045e63b 100755
--- a/fs_signup/fs_signup_server
+++ b/fs_signup/fs_signup_server
@@ -142,9 +142,12 @@ while (1) {
 
     my @invoicing_list = split( /\s*\,\s*/, $signup_data->{'invoicing_list'} );
 
+    $signup_data->{'pkgpart'} =~ /^(\d+)$/ or '' =~ /^()$/;
+    my $pkgpart = $1;
+
     my $part_pkg =
-      qsearchs( 'part_pkg', { 'pkgpart' => $signup_data->{'pkgpart'} } )
-        or $error ||= "WARNING: unknown pkgpart ". $signup_data->{pkgpart};
+      qsearchs( 'part_pkg', { 'pkgpart' => $pkgpart } )
+        or $error ||= "WARNING: unknown pkgpart: $pkgpart";
     my $svcpart = $part_pkg->svcpart unless $error;
 
     my $cust_pkg = new FS::cust_pkg ( {
@@ -188,11 +191,10 @@ while (1) {
         #should check list for errors...
         #$cust_main->suspend;
         $cust_main->cancel;
+        $error = '_decline';
       }
     }
 
-    $error ||= '_decline';
-
     warn "[fs_signup_server] Sending results...\n" if $Debug;
     print $writer $error, "\n";