From: Ivan Kohler <ivan@freeside.biz>
Date: Sat, 12 Jan 2013 05:57:44 +0000 (-0800)
Subject: svc_phone.sim_imsi was not for 2.3 branch, reverting, RT#20768
X-Git-Url: http://git.freeside.biz/gitweb/?a=commitdiff_plain;h=85a657f7c32fc5548ca17f7f5024955c87c07f43;p=freeside.git

svc_phone.sim_imsi was not for 2.3 branch, reverting, RT#20768
---

diff --git a/FS/FS/Schema.pm b/FS/FS/Schema.pm
index c434e8a2f..03a6c2748 100644
--- a/FS/FS/Schema.pm
+++ b/FS/FS/Schema.pm
@@ -3271,7 +3271,6 @@ sub tables_hashref {
         'svcnum',       'int',         '',      '', '', '', 
         'countrycode',  'varchar',     '',       3, '', '', 
         'phonenum',     'varchar',     '',      15, '', '',  #12 ?
-        'sim_imsi',     'varchar', 'NULL',      15, '', '',
         'pin',          'varchar', 'NULL', $char_d, '', '',
         'sip_password', 'varchar', 'NULL', $char_d, '', '',
         'phone_name',   'varchar', 'NULL', $char_d, '', '',
diff --git a/FS/FS/svc_phone.pm b/FS/FS/svc_phone.pm
index bb5757332..118748ea2 100644
--- a/FS/FS/svc_phone.pm
+++ b/FS/FS/svc_phone.pm
@@ -22,11 +22,10 @@ $DEBUG = 0;
 @pw_set = ( 'a'..'k', 'm','n', 'p-z', 'A'..'N', 'P'..'Z' , '2'..'9' );
 
 #ask FS::UID to run this stuff for us later
-FS::UID->install_callback( sub { 
+$FS::UID::callback{'FS::svc_acct'} = sub { 
   $conf = new FS::Conf;
   $phone_name_max = $conf->config('svc_phone-phone_name-max_length');
-}
-);
+};
 
 =head1 NAME
 
@@ -68,10 +67,6 @@ primary key
 
 =item phonenum
 
-=item sim_imsi
-
-SIM IMSI (http://en.wikipedia.org/wiki/International_mobile_subscriber_identity)
-
 =item sip_password
 
 =item pin
@@ -151,7 +146,6 @@ sub table_info {
                             disable_select => 1,
                           },
         'phonenum'     => 'Phone number',
-        'sim_imsi'     => 'IMSI', #http://en.wikipedia.org/wiki/International_mobile_subscriber_identity
         'pin'          => { label => 'Voicemail PIN', #'Personal Identification Number',
                             type  => 'text',
                             disable_inventory => 1,
@@ -470,7 +464,6 @@ sub check {
     $self->ut_numbern('svcnum')
     || $self->ut_numbern('countrycode')
     || $self->$phonenum_check_method('phonenum')
-    || $self->ut_numbern('sim_imsi')
     || $self->ut_anything('sip_password')
     || $self->ut_numbern('pin')
     || $self->ut_textn('phone_name')
@@ -491,10 +484,6 @@ sub check {
   ;
   return $error if $error;
 
-  return 'Illegal IMSI (not 14-15 digits)' #shorter?
-    if length($self->sim_imsi)
-    && ( length($self->sim_imsi) < 14 || length($self->sim_imsi) > 15 );
-
     # LNP data validation
     return 'Cannot set LNP fields: no LNP in progress'
 	if ( ($self->lnp_desired_due_date || $self->lnp_due_date 
diff --git a/httemplate/edit/svc_phone.cgi b/httemplate/edit/svc_phone.cgi
index 8ee71b82d..9647b6887 100644
--- a/httemplate/edit/svc_phone.cgi
+++ b/httemplate/edit/svc_phone.cgi
@@ -28,11 +28,6 @@ my $begin_callback = sub {
                 type     => 'select-did',
                 label    => 'Phone number',
                 multiple => $bulk,
-              },
-              { field     => 'sim_imsi',
-                type      => 'text',
-                size      => 15,
-                maxlength => 15,
               };
 
   push @$fields, { field => 'domsvc',
diff --git a/httemplate/view/svc_phone.cgi b/httemplate/view/svc_phone.cgi
index 0eba09383..40c8d850b 100644
--- a/httemplate/view/svc_phone.cgi
+++ b/httemplate/view/svc_phone.cgi
@@ -16,7 +16,7 @@ my %labels = map { $_ =>  ( ref($fields->{$_})
                          );
                  } keys %$fields;
 
-my @fields = qw( countrycode phonenum sim_imsi );
+my @fields = qw( countrycode phonenum );
 push @fields, 'domain' if $conf->exists('svc_phone-domain');
 push @fields, qw( pbx_title sip_password pin phone_name forwarddst email );