Skip to content
This repository has been archived by the owner on Jan 8, 2021. It is now read-only.

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

Closed
pihish opened this issue Dec 11, 2015 · 2 comments
Closed

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

pihish opened this issue Dec 11, 2015 · 2 comments

Comments

@pihish
Copy link

pihish commented Dec 11, 2015

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
Copy link
Contributor

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
Copy link
Contributor

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nicolas-grekas @pihish