mb_convert_encoding() Definition Needs to Check for Existing Definitions

tchwork / utf8

Portable and performant UTF-8, Unicode and Grapheme Clusters for PHP

Apache License 2.0

627 stars 50 forks source link

mb_convert_encoding() Definition Needs to Check for Existing Definitions #54

Closed pihish closed 8 years ago

pihish commented 8 years ago

mb_convert_encoding() method is currently being defined in the global scope. If a definition of it has already been defined by another library, it needs to not try to redefine it.

Original conflict found between Patchwork and symphony/polyfill-mbstring when running composer update.

nicolas-grekas commented 8 years ago

Will be fixed in v1.3.0 see #55 Meanwhile, please ensure that you allow v1.3 in your composer.json file to get this next version.

nicolas-grekas commented 8 years ago

Fixed in v1.1.31, v1.2.6 and v1.3.0