xafardero / generate-iban

src/Bban/SpainBban.php

Doc Comments +8 added lines, -8 removed lines

@@ -121,7 +121,7 @@ 
     }
 
     /**
-     * @param $bankCode
+     * @param string $bankCode
      *
      * @throws InvalidArgumentException
      */
@@ -133,7 +133,7 @@ 
     }
 
     /**
-     * @param $branchCode
+     * @param string $branchCode
      *
      * @throws InvalidArgumentException
      */
@@ -145,7 +145,7 @@ 
     }
 
     /**
-     * @param $checkDigits
+     * @param string $checkDigits
      *
      * @throws InvalidArgumentException
      */
@@ -157,7 +157,7 @@ 
     }
 
     /**
-     * @param $accountNumber
+     * @param string $accountNumber
      *
      * @throws InvalidArgumentException
      */
@@ -169,10 +169,10 @@ 
     }
 
     /**
-     * @param $bankCode
-     * @param $branchCode
-     * @param $checkDigits
-     * @param $accountNumber
+     * @param string $bankCode
+     * @param string $branchCode
+     * @param string $checkDigits
+     * @param string $accountNumber
      *
      * @throws InvalidArgumentException
      */

Spacing +7 added lines, -7 removed lines

@@ -68,7 +68,7 @@ 
      */
     public static function fromString($bban)
     {
-        if (! preg_match('/^[\d]{20}$/', $bban)) {
+        if (!preg_match('/^[\d]{20}$/', $bban)) {
             throw new InvalidArgumentException('Bban should be 20 numbers');
         }
 
@@ -117,7 +117,7 @@ 
      */
     public function __toString()
     {
-        return $this->bankCode . $this->branchCode . $this->checkDigits . $this->accountNumber;
+        return $this->bankCode.$this->branchCode.$this->checkDigits.$this->accountNumber;
     }
 
     /**
@@ -127,7 +127,7 @@ 
      */
     private static function validateBankCodeFormat($bankCode)
     {
-        if (! preg_match('/^[\d]{4}$/', $bankCode)) {
+        if (!preg_match('/^[\d]{4}$/', $bankCode)) {
             throw new InvalidArgumentException('Bank code should be 4 numbers');
         }
     }
@@ -139,7 +139,7 @@ 
      */
     private static function validateBranchCodeFormat($branchCode)
     {
-        if (! preg_match('/^[\d]{4}$/', $branchCode)) {
+        if (!preg_match('/^[\d]{4}$/', $branchCode)) {
             throw new InvalidArgumentException('Branch code should be 4 numbers');
         }
     }
@@ -151,7 +151,7 @@ 
      */
     private static function validateCheckDigitsFormat($checkDigits)
     {
-        if (! preg_match('/^[\d]{2}$/', $checkDigits)) {
+        if (!preg_match('/^[\d]{2}$/', $checkDigits)) {
             throw new InvalidArgumentException('Check digits should be 4 numbers');
         }
     }
@@ -163,7 +163,7 @@ 
      */
     private static function validateAccountNumberFormat($accountNumber)
     {
-        if (! preg_match('/^[\d]{10}$/', $accountNumber)) {
+        if (!preg_match('/^[\d]{10}$/', $accountNumber)) {
             throw new InvalidArgumentException('Account number should be 10 numbers');
         }
     }
@@ -184,7 +184,7 @@ 
     ) {
         $dc = '';
         $validations = [6, 3, 7, 9, 10, 5, 8, 4, 2, 1];
-        foreach ([$bankCode . $branchCode, $accountNumber] as $string) {
+        foreach ([$bankCode.$branchCode, $accountNumber] as $string) {
             $suma = 0;
             for ($i = 0, $len = strlen($string); $i < $len; $i++) {
                 $suma += $validations[$i] * substr($string, $len - $i - 1, 1);

src/Iban.php

Doc Comments +2 added lines, -2 removed lines

@@ -144,7 +144,7 @@ 
     }
 
     /**
-     * @param $countryCode
+     * @param string $countryCode
      *
      * @throws InvalidArgumentException
      */
@@ -156,7 +156,7 @@ 
     }
 
     /**
-     * @param $checkDigits
+     * @param string $checkDigits
      *
      * @throws InvalidArgumentException
      */

Spacing +6 added lines, -6 removed lines

@@ -59,7 +59,7 @@ 
      */
     public static function fromString($iban)
     {
-        if (! preg_match('/^[0-9A-Z]{16,34}$/', $iban)) {
+        if (!preg_match('/^[0-9A-Z]{16,34}$/', $iban)) {
             throw new InvalidArgumentException('Iban should be between 16 and 34 characters');
         }
 
@@ -67,7 +67,7 @@ 
         $checkDigits = substr($iban, 2, 2);
         $bbanString = substr($iban, 4);
 
-        if (! array_key_exists($countryCode, self::$countriesSupported)) {
+        if (!array_key_exists($countryCode, self::$countriesSupported)) {
             throw new InvalidArgumentException(
                 sprintf(
                     'The country code %s is not supported',
@@ -140,7 +140,7 @@ 
     {
         $bbanString = $this->bban;
 
-        return $this->countryCode . $this->checkDigits . $bbanString;
+        return $this->countryCode.$this->checkDigits.$bbanString;
     }
 
     /**
@@ -150,7 +150,7 @@ 
      */
     private static function validateCountryCodeFormat($countryCode)
     {
-        if (! preg_match('/^[A-Z]{2}$/', $countryCode)) {
+        if (!preg_match('/^[A-Z]{2}$/', $countryCode)) {
             throw new InvalidArgumentException('The country code should be 2 letters');
         }
     }
@@ -162,7 +162,7 @@ 
      */
     private static function validateCheckDigitsFormat($checkDigits)
     {
-        if (! preg_match('/^[\d]{2}$/', $checkDigits)) {
+        if (!preg_match('/^[\d]{2}$/', $checkDigits)) {
             throw new InvalidArgumentException('The IBAN checksum must be 2 numeric characters');
         }
     }
@@ -179,7 +179,7 @@ 
         $checkDigits,
         BbanInterface $bban
     ) {
-        $rearranged = strval($bban) . $countryCode . $checkDigits;
+        $rearranged = strval($bban).$countryCode.$checkDigits;
         $digitsList = str_split($rearranged);
 
         $digitsList = array_map(['self', 'digitToInt'], $digitsList);