Merge pull request #3 from Treblle/fix/type-fix

JustSteveKing · web-flow · commit 8115ccd50174 · 2024-04-19T13:12:15.000+01:00
Expanding type allowance on 'handleString' to be more forgiving, but …
diff --git a/src/Masking/FieldMasker.php b/src/Masking/FieldMasker.php
@@ -30,8 +30,12 @@ public function mask(array $data): array
         return $collector;
     }
 
-    private function handleString(string $key, string $value): string
+    private function handleString(string|int|float|bool $key, string $value): string
     {
+        if (! is_string($key)) {
+            $key = (string) $key;
+        }
+
         static $lowerFields = null;
         if ($lowerFields === null) {
             $lowerFields = array_map('strtolower', $this->fields);

Original file line number	Diff line number	Diff line change
`@@ -30,8 +30,12 @@ public function mask(array $data): array`
`30`	`30`	`return $collector;`
`31`	`31`	`}`
`32`	`32`
`33`		`- private function handleString(string $key, string $value): string`
	`33`	`+ private function handleString(string\|int\|float\|bool $key, string $value): string`
`34`	`34`	`{`
	`35`	`+ if (! is_string($key)) {`
	`36`	`+ $key = (string) $key;`
	`37`	`+ }`
	`38`	`+`
`35`	`39`	`static $lowerFields = null;`
`36`	`40`	`if ($lowerFields === null) {`
`37`	`41`	`$lowerFields = array_map('strtolower', $this->fields);`