--- /Users/fm56/Desktop/sin/sin.module	2008-12-07 20:33:28.000000000 -0500
+++ /Volumes/Sagan/Jobs/pitchersMound/mirror/sites/all/modules/sin/sin.module	2009-01-19 22:43:13.000000000 -0500
@@ -148,14 +148,14 @@ function sin_field($op, &$node, $field, 
     case 'validate': // corresponds to hook phone_widget validate in sin-5.x
       foreach ($node_field as $delta => $item) {
         if ($item['value'] != '') {
-          if ($field['type'] == 'fr_sin' && !valid_fr_sin($item['value'])) {
-            form_set_error($field['field_name'],t('"%value" is not a valid French Social Insurance Numbers<br>Social Insurance Numbers should only contains numbers and be like ...',array('%value' => $item['value'])));
+          if ($field['type'] == 'fr_sin' && !valid_sin_number('fr', $item['value'])) {
+            form_set_error($field['field_name'],t('"%value" is not a valid French Social Insurance numbers.<br>Social Insurance numbers should only contain numbers and be like ...',array('%value' => $item['value'])));
           }
-          if ($field['type'] == 'ca_sin' && !valid_ca_sin($item['value'])) {
-            form_set_error($field['field_name'],t('"%value" is not a valid Canadian Social Insurance Numbers<br>Social Insurance Numbers should be like 999999999 ...',array('%value' => $item['value'])));
+          if ($field['type'] == 'ca_sin' && !valid_sin_number('ca', $item['value'])) {
+            form_set_error($field['field_name'],t('"%value" is not a valid Canadian Social Insurance number.<br>Social Insurance numbers should be like 999999999 ...',array('%value' => $item['value'])));
           }
-          if ($field['type'] == 'us_sin' && !valid_us_sin($item['value'])) {
-            form_set_error($field['field_name'],t('"%value" is not a valid US Social Insurance Numbers<br>Social Insurance Numbers should be like 999-99-9999 or 999999999 ...',array('%value' => $item['value'])));
+          if ($field['type'] == 'us_sin' && !valid_sin_number('us', $item['value'])) {
+            form_set_error($field['field_name'],t('"%value" is not a valid US Social Security number.<br>Social Insurance numbers should only contain numbers and be like 999-99-9999 or 999999999 ...',array('%value' => $item['value'])));
           }  
         }
       }
@@ -163,7 +163,7 @@ function sin_field($op, &$node, $field, 
 
     case 'presave': // corresponds to hook phone_widget 'process form values' in sin-5.x
       foreach ($node_field as $delta => $item) {
-        //format the phone number
+        //format the number
         if ($item['value'] != '') {
           if ($field['type'] == 'fr_sin') {
             $node_field[$delta]['value'] = format_sin_number('fr', $node_field[$delta]['value'], $field);
@@ -524,9 +524,9 @@ function theme_sin_formatter_default($el
  * @param string $sinnumber
  * @return boolean Returns boolean FALSE if the SIN number is not valid.  
  */
-function valid_sin_number($countrycode, $phonenumber) {
+function valid_sin_number($countrycode, $sinnumber) {
 
-  $countrycode = trim($countrycode); 
+  $countrycode = trim($countrycode);
   $sinnumber = trim($sinnumber);
 
   if ($countrycode == 'fr' 
@@ -540,7 +540,7 @@ function valid_sin_number($countrycode, 
   	module_load_include('inc', 'sin', 'sin.'. $countrycode);
   	
 	if (function_exists($valid_sin_function)) {
-	    return $valid_phone_function($sinnumber);
+	    return $valid_sin_function($sinnumber);
 	}
 	else  { 
 	    return false; 	
