Set cost type to integer

datibbaw · fabpot · commit 54f7d9c8986b · 2013-09-27T09:20:28.000+02:00
This seems to be a remnant of when the code dealt with crypt() directly. The password_hash() function expects the cost option to be an LVAL (it does a type conversion for strings).
diff --git a/Encoder/BCryptPasswordEncoder.php b/Encoder/BCryptPasswordEncoder.php
@@ -42,7 +42,7 @@ public function __construct($cost)
             throw new \InvalidArgumentException('Cost must be in the range of 4-31.');
         }
 
-        $this->cost = sprintf('%02d', $cost);
+        $this->cost = $cost;
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,7 @@ public function __construct($cost)`
`42`	`42`	`throw new \InvalidArgumentException('Cost must be in the range of 4-31.');`
`43`	`43`	`}`
`44`	`44`
`45`		`- $this->cost = sprintf('%02d', $cost);`
	`45`	`+ $this->cost = $cost;`
`46`	`46`	`}`
`47`	`47`
`48`	`48`	`/**`