Egn::is_foreigner() method: Making it more precise.

Signed-off-by:Ivan Tcholakov <[email protected]>

Author: ivantcholakov

Egn.php

@@ -48,34 +48,34 @@ class Egn {
     protected static $EGN_WEIGHTS = array(2, 4, 8, 5, 10, 9, 7, 3, 6);
 
     protected static $EGN_REGIONS = array(
-        'Благоевград' =>        43,  /* от 000 до 043 */ 
-        'Бургас' =>             93,  /* от 044 до 093 */ 
-        'Варна' =>              139, /* от 094 до 139 */ 
-        'Велико Търново' =>     169, /* от 140 до 169 */ 
-        'Видин' =>              183, /* от 170 до 183 */ 
-        'Враца' =>              217, /* от 184 до 217 */ 
-        'Габрово' =>            233, /* от 218 до 233 */ 
-        'Кърджали' =>           281, /* от 234 до 281 */ 
-        'Кюстендил' =>          301, /* от 282 до 301 */ 
-        'Ловеч' =>              319, /* от 302 до 319 */ 
-        'Монтана' =>            341, /* от 320 до 341 */ 
-        'Пазарджик' =>          377, /* от 342 до 377 */ 
-        'Перник' =>             395, /* от 378 до 395 */ 
-        'Плевен' =>             435, /* от 396 до 435 */ 
-        'Пловдив' =>            501, /* от 436 до 501 */ 
-        'Разград' =>            527, /* от 502 до 527 */ 
-        'Русе' =>               555, /* от 528 до 555 */ 
-        'Силистра' =>           575, /* от 556 до 575 */ 
-        'Сливен' =>             601, /* от 576 до 601 */ 
-        'Смолян' =>             623, /* от 602 до 623 */ 
-        'София - град' =>       721, /* от 624 до 721 */ 
-        'София - окръг' =>      751, /* от 722 до 751 */ 
-        'Стара Загора' =>       789, /* от 752 до 789 */ 
-        'Добрич (Толбухин)' =>  821, /* от 790 до 821 */ 
-        'Търговище' =>          843, /* от 822 до 843 */ 
-        'Хасково' =>            871, /* от 844 до 871 */ 
-        'Шумен' =>              903, /* от 872 до 903 */ 
-        'Ямбол' =>              925, /* от 904 до 925 */ 
+        'Благоевград' =>        43,  /* от 000 до 043 */
+        'Бургас' =>             93,  /* от 044 до 093 */
+        'Варна' =>              139, /* от 094 до 139 */
+        'Велико Търново' =>     169, /* от 140 до 169 */
+        'Видин' =>              183, /* от 170 до 183 */
+        'Враца' =>              217, /* от 184 до 217 */
+        'Габрово' =>            233, /* от 218 до 233 */
+        'Кърджали' =>           281, /* от 234 до 281 */
+        'Кюстендил' =>          301, /* от 282 до 301 */
+        'Ловеч' =>              319, /* от 302 до 319 */
+        'Монтана' =>            341, /* от 320 до 341 */
+        'Пазарджик' =>          377, /* от 342 до 377 */
+        'Перник' =>             395, /* от 378 до 395 */
+        'Плевен' =>             435, /* от 396 до 435 */
+        'Пловдив' =>            501, /* от 436 до 501 */
+        'Разград' =>            527, /* от 502 до 527 */
+        'Русе' =>               555, /* от 528 до 555 */
+        'Силистра' =>           575, /* от 556 до 575 */
+        'Сливен' =>             601, /* от 576 до 601 */
+        'Смолян' =>             623, /* от 602 до 623 */
+        'София - град' =>       721, /* от 624 до 721 */
+        'София - окръг' =>      751, /* от 722 до 751 */
+        'Стара Загора' =>       789, /* от 752 до 789 */
+        'Добрич (Толбухин)' =>  821, /* от 790 до 821 */
+        'Търговище' =>          843, /* от 822 до 843 */
+        'Хасково' =>            871, /* от 844 до 871 */
+        'Шумен' =>              903, /* от 872 до 903 */
+        'Ямбол' =>              925, /* от 904 до 925 */
         'Друг/Неизвестен' =>    999, /* от 926 до 999 - Такъв регион понякога се ползва при
                                                         родени преди 1900, за родени в чужбина
                                                         или ако в даден регион се родят повече
@@ -367,17 +367,38 @@ public static function get_birthday($string) {
 
     /* Is it a personal number of foreigner? A simple check. */
     public static function is_foreigner($string) {
- 
-        if (strlen($string) > 3) {
 
-            if (substr($string, 0, 2) == '10') {
+        if (!ctype_digit($string)) {
+            return false;
+        }
+
+        if (strlen($string) == 10) {
+
+            $pnf_weights = array(21, 19, 17, 13, 11, 9, 7, 3, 1);
+
+            $checksum = substr($string, 9, 1);
+            $pnfsum = 0;
+
+            for ($i = 0; $i < 9; $i++) {
+                $pnfsum += substr($string, $i, 1) * $pnf_weights[$i];
+            }
+
+            $valid_checksum = $pnfsum % 10;
+
+            if ($checksum == $valid_checksum) {
                 return true;
             }
 
-            // Added by Ivan Tcholakov, 09-SEP-2011.
+            return false;
+        }
+
+        if (strlen($string) == 11) {
+
             if (substr($string, 0, 3) == '229') {
                 return true;
             }
+
+            return false;
         }
 
         return false;