Fix: Add parameter type declarations

FakerPHP · Sep 4, 2023 · 736e319 · 736e319
1 parent 1f83b68
commit 736e319
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 26 deletions.
diff --git a/src/Faker/Calculator/Ean.php b/src/Faker/Calculator/Ean.php
@@ -17,11 +17,9 @@ class Ean
      *
      * @see https://en.wikipedia.org/wiki/International_Article_Number
      *
-     * @param string $digits
-     *
      * @return int
      */
-    public static function checksum($digits)
+    public static function checksum(string $digits)
     {
         $sequence = (strlen($digits) + 1) === 8 ? [3, 1] : [1, 3];
         $sums = 0;
@@ -41,7 +39,7 @@ public static function checksum($digits)
      *
      * @return bool
      */
-    public static function isValid($ean)
+    public static function isValid(string $ean)
     {
         if (!preg_match(self::PATTERN, $ean)) {
             return false;

diff --git a/src/Faker/Calculator/Iban.php b/src/Faker/Calculator/Iban.php
@@ -7,11 +7,9 @@ class Iban
     /**
      * Generates IBAN Checksum
      *
-     * @param string $iban
-     *
      * @return string Checksum (numeric string)
      */
-    public static function checksum($iban)
+    public static function checksum(string $iban)
     {
         // Move first four digits to end and set checksum to '00'
         $checkString = substr($iban, 4) . substr($iban, 0, 2) . '00';
@@ -34,11 +32,9 @@ static function (array $matches): string {
     /**
      * Converts letter to number
      *
-     * @param string $char
-     *
      * @return int
      */
-    public static function alphaToNumber($char)
+    public static function alphaToNumber(string $char)
     {
         return ord($char) - 55;
     }
@@ -50,7 +46,7 @@ public static function alphaToNumber($char)
      *
      * @return int
      */
-    public static function mod97($number)
+    public static function mod97(string $number)
     {
         $checksum = (int) $number[0];
 
@@ -64,11 +60,9 @@ public static function mod97($number)
     /**
      * Checks whether an IBAN has a valid checksum
      *
-     * @param string $iban
-     *
      * @return bool
      */
-    public static function isValid($iban)
+    public static function isValid(string $iban)
     {
         return self::checksum($iban) === substr($iban, 2, 2);
     }

diff --git a/src/Faker/Calculator/Luhn.php b/src/Faker/Calculator/Luhn.php
@@ -13,11 +13,9 @@
 class Luhn
 {
     /**
-     * @param string $number
-     *
      * @return int
      */
-    private static function checksum($number)
+    private static function checksum(string $number)
     {
         $number = (string) $number;
         $length = strlen($number);
@@ -35,11 +33,9 @@ private static function checksum($number)
     }
 
     /**
-     * @param string $partialNumber
-     *
      * @return string
      */
-    public static function computeCheckDigit($partialNumber)
+    public static function computeCheckDigit(string $partialNumber)
     {
         $checkDigit = self::checksum($partialNumber . '0');
 
@@ -53,23 +49,19 @@ public static function computeCheckDigit($partialNumber)
     /**
      * Checks whether a number (partial number + check digit) is Luhn compliant
      *
-     * @param string $number
-     *
      * @return bool
      */
-    public static function isValid($number)
+    public static function isValid(string $number)
     {
         return self::checksum($number) === 0;
     }
 
     /**
      * Generate a Luhn compliant number.
      *
-     * @param string $partialValue
-     *
      * @return string
      */
-    public static function generateLuhnNumber($partialValue)
+    public static function generateLuhnNumber(string $partialValue)
     {
         if (!preg_match('/^\d+$/', $partialValue)) {
             throw new \InvalidArgumentException('Argument should be an integer.');