minor fixes

diff --git a/ipf/orm/validator/country.php b/ipf/orm/validator/country.php
index 2536f1511b3da85adbba03ae999e89cd992e2a00..966ede8edcc422cce061777dc3a7493e42b77c58 100644 (file)
--- a/ipf/orm/validator/country.php
+++ b/ipf/orm/validator/country.php
@@ -257,4 +257,5 @@ class IPF_ORM_Validator_Country
         $value = strtolower($value);
         return isset(self::$countries[$value]);
     }
-}
\ No newline at end of file
+}
+

diff --git a/ipf/orm/validator/creditcard.php b/ipf/orm/validator/creditcard.php
index f4a59f600dac4ad24c4e37db11842f87ffef8d0e..62609ac97aedbc358968a8f20be33ae3c945ea8b 100644 (file)
--- a/ipf/orm/validator/creditcard.php
+++ b/ipf/orm/validator/creditcard.php
@@ -4,7 +4,6 @@ class IPF_ORM_Validator_Creditcard
 {                                                         
     public function validate($value)
     {
-        $cardType = "";
         $card_regexes = array(
             "/^4\d{12}(\d\d\d){0,1}$/"      => 'visa',
             "/^5[12345]\d{14}$/"            => 'mastercard',
@@ -13,15 +12,17 @@ class IPF_ORM_Validator_Creditcard
             "/^30[012345]\d{11}$/"          => 'diners',
             "/^3[68]\d{12}$/"               => 'diners',
         );
+
+        $cardType = '';
         foreach ($card_regexes as $regex => $type) {
             if (preg_match($regex, $value)) {
                  $cardType = $type;
                  break;
             }
         }
-        if ( ! $cardType) {
+        if (!$cardType)
             return false;
-        }
+
         /* mod 10 checksum algorithm */
         $revcode = strrev($value);
         $checksum = 0;
@@ -36,10 +37,7 @@ class IPF_ORM_Validator_Creditcard
                  $checksum += 1;
             }
         }
-        if ($checksum % 10 == 0) {
-            return true;
-        } else {
-            return false;
-        }
+        return $checksum % 10 == 0;
     }
-}
\ No newline at end of file
+}
+