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

Conversation

Ocramius
Copy link
Member

@Ocramius Ocramius commented May 5, 2014

Fixes #6218

See https://wiki.php.net/rfc/default_encoding

PHP 5.6 and master, introduce new php.ini setting. Old iconv./mbstring. php.ini parameters will be removed for master PHP6. Use of iconv./mbstring. php.ini parameters raise E_DEPRECATED for 5.6 and up.

…precation

See https://wiki.php.net/rfc/default_encoding

 > PHP 5.6 and master, introduce new php.ini setting. Old iconv.*/mbstring.* php.ini parameters will be removed for master PHP6. Use of iconv.*/mbstring.* php.ini parameters raise E_DEPRECATED for 5.6 and up.
@Ocramius Ocramius added this to the 2.3.2 milestone May 5, 2014
@Maks3w
Copy link
Member

Maks3w commented May 5, 2014

This is not the good way. internal_encoding still having a special meaning on 5.6.

default_encoding < internal_encoding < iconv.internal_encoding

So set default_encoding does not ensure you are safe to use iconv with the expected encoding.

@Maks3w Maks3w removed the bug label May 5, 2014
@Maks3w
Copy link
Member

Maks3w commented May 5, 2014

By the way. This is not a bug. (there is not nothing broken)

@Ocramius
Copy link
Member Author

Ocramius commented May 5, 2014

@Maks3w the builds were throwing warnings without this

@Ocramius
Copy link
Member Author

Ocramius commented May 5, 2014

And I'd also say that we should always keep our distance from deprecated features

weierophinney added a commit that referenced this pull request May 7, 2014
…coding-deprecated-in-php-5.6

#6218 - applying hotfix for `iconv.internal_encoding` deprecation
weierophinney added a commit that referenced this pull request May 7, 2014
@weierophinney weierophinney merged commit 01fe153 into zendframework:master May 7, 2014
weierophinney added a commit that referenced this pull request May 7, 2014
@Ocramius Ocramius deleted the hotfix/#6218-iconv-internal-encoding-deprecated-in-php-5.6 branch May 7, 2014 15:24
weierophinney added a commit to zendframework/zend-xmlrpc that referenced this pull request May 15, 2015
…fix/zendframework/zendframework#6218-iconv-internal-encoding-deprecated-in-php-5.6

zendframework/zendframework#6218 - applying hotfix for `iconv.internal_encoding` deprecation
weierophinney added a commit to zendframework/zend-xmlrpc that referenced this pull request May 15, 2015
weierophinney added a commit to zendframework/zend-xmlrpc that referenced this pull request May 15, 2015
weierophinney added a commit to zendframework/zend-validator that referenced this pull request May 15, 2015
…fix/zendframework/zendframework#6218-iconv-internal-encoding-deprecated-in-php-5.6

zendframework/zendframework#6218 - applying hotfix for `iconv.internal_encoding` deprecation
weierophinney added a commit to zendframework/zend-validator that referenced this pull request May 15, 2015
weierophinney added a commit to zendframework/zend-validator that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

iconv.internal_encoding is deprecated
4 participants