laminas / laminas-stdlib

SPL extensions, array utilities, error handlers, and more
https://docs.laminas.dev/laminas-stdlib/
BSD 3-Clause "New" or "Revised" License
190 stars 40 forks source link

grapheme_strpos() expects parameter 1 to be string, bool given #27

Closed javabudd closed 3 years ago

javabudd commented 3 years ago

BC Break Report

Q A
Version 3.4.0

Summary

After upgrading to 3.4.0, our application starts failing with error messages related to parameter expectations in the grapheme_strpos function. It seems related to https://github.com/laminas/laminas-stdlib/commit/5a28d8c31a3ae236dd529a548a3ccf6fac43f45f

Previous behavior

Application is able to pass a boolean value into the strpos() method.

Current behavior

Application cannot pass a boolean value and an exception is thrown (shown below)

How to reproduce

Pass a boolean value into the strpos method.

TypeError: grapheme_strpos() expects parameter 1 to be string, bool given

/vendor/laminas/laminas-stdlib/src/StringWrapper/Intl.php:87
/vendor/laminas/laminas-i18n/src/Validator/IsFloat.php:243
/vendor/laminas/laminas-validator/src/ValidatorChain.php:245
/vendor/laminas/laminas-inputfilter/src/Input.php:433
/vendor/laminas/laminas-inputfilter/src/BaseInputFilter.php:274
/vendor/laminas/laminas-inputfilter/src/BaseInputFilter.php:228
javabudd commented 3 years ago

After some further investigation it look like this is better suited for the laminas-i18n/src/Validator repository. Closing.