From f674b7c9b094749fcef770d1ae03640b69b1ec83 Mon Sep 17 00:00:00 2001
From: Ivan Kohler <ivan@freeside.biz>
Date: Tue, 28 Aug 2012 02:05:55 -0700
Subject: [PATCH] remove validation from national_id old format per customer,
 RT#18543

---
 FS/FS/cust_main/NationalID.pm            | 12 ++++++++----
 httemplate/edit/cust_main/birthdate.html | 12 ++++++------
 httemplate/view/cust_main/misc.html      | 12 ++++++------
 3 files changed, 20 insertions(+), 16 deletions(-)

diff --git a/FS/FS/cust_main/NationalID.pm b/FS/FS/cust_main/NationalID.pm
index cc8d55685..a742b7637 100644
--- a/FS/FS/cust_main/NationalID.pm
+++ b/FS/FS/cust_main/NationalID.pm
@@ -36,11 +36,15 @@ sub set_national_id_from_cgi {
       } elsif ( $cgi->param('national_id2') =~ /\S/ ) {
         my $oldic = $cgi->param('national_id2');
         $oldic =~ s/\s//g;
-        if ( $oldic =~ /^\w\d{9}$/ ) {
+
+        # can you please remove validation for "Old IC/Passport:" field, customer
+        # will have other field format like, RF/123456, I/5234234 ...
+        #if ( $oldic =~ /^\w\d{9}$/ ) {
           $self->national_id($oldic);
-        } else {
-          $error ||= "Illegal Old IC/Passport: ". $cgi->param('national_id2');
-        }
+        #} else {
+        #  $error ||= "Illegal Old IC/Passport: ". $cgi->param('national_id2');
+        #}
+
       } else {
         $error ||= 'Either NRIC or Old IC/Passport is required';
       }
diff --git a/httemplate/edit/cust_main/birthdate.html b/httemplate/edit/cust_main/birthdate.html
index 5af819bdc..e1adbd3bd 100644
--- a/httemplate/edit/cust_main/birthdate.html
+++ b/httemplate/edit/cust_main/birthdate.html
@@ -6,13 +6,13 @@
 %  if ( $id_country ) {
 %   if ( $id_country eq 'MY' ) {
 %     my($old, $nric) = ( '', '');
-%     if ( $cust_main->national_id =~ /^\w\d{9}$/ ) {
-%       $old = $cust_main->national_id;
-%     } elsif ( $cust_main->national_id =~ /^\d{6}\-\d{2}\-\d{4}$/ ) {
+%     if ( $cust_main->national_id =~ /^\d{6}\-\d{2}\-\d{4}$/ ) {
 %       $nric = $cust_main->national_id;
-%     } else {
-%       warn "unknown national_id format";
-        <INPUT TYPE="hidden" NAME="national_id0" VALUE="<% $cust_main->national_id |h %>">
+%     } else { # elsif ( $cust_main->national_id =~ /^\w\d{9}$/ ) {
+%       $old = $cust_main->national_id;
+%     #} else {
+%     #  warn "unknown national_id format";
+%#       <INPUT TYPE="hidden" NAME="national_id0" VALUE="<% $cust_main->national_id |h %>">
 %     }
 
       <% include( '/elements/tr-input-text.html',
diff --git a/httemplate/view/cust_main/misc.html b/httemplate/view/cust_main/misc.html
index 9713b2469..263c266e7 100644
--- a/httemplate/view/cust_main/misc.html
+++ b/httemplate/view/cust_main/misc.html
@@ -107,13 +107,13 @@
 %   if ( $id_country eq 'MY' ) {
       <TR>
 %     my($old, $nric) = ( '', '');
-%     if ( $cust_main->national_id =~ /^\w\d{9}$/ ) {
-          <TD ALIGN="right"><% mt('Old IC/Passport') |h %></TD>
-%     } elsif ( $cust_main->national_id =~ /^\d{6}\-\d{2}\-\d{4}$/ ) {
+%     if ( $cust_main->national_id =~ /^\d{6}\-\d{2}\-\d{4}$/ ) {
           <TD ALIGN="right"><% mt('NRIC') |h %></TD>
-%     } else {
-%       warn "unknown national_id format";
-          <TD ALIGN="right"></TD>
+%     } else { # elsif ( $cust_main->national_id =~ /^\w\d{9}$/ ) {
+          <TD ALIGN="right"><% mt('Old IC/Passport') |h %></TD>
+%     #} else {
+%     #  warn "unknown national_id format";
+%#         <TD ALIGN="right"></TD>
 %     }
         <TD BGCOLOR="#ffffff"><% $cust_main->national_id |h %></TD>
       </TR>
-- 
2.11.0