Skip to content

iconv deprecation warning/error in PHP 5.6.2 #780

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
nyov opened this issue Dec 4, 2014 · 2 comments
Closed

iconv deprecation warning/error in PHP 5.6.2 #780

nyov opened this issue Dec 4, 2014 · 2 comments

Comments

@nyov
Copy link

nyov commented Dec 4, 2014

The warning (?) makes the magento error page display so I consider it an error; after enabling display of php error messages in pub/errors/local.xml, I got this, in a few variations:

There has been an error processing your request

Deprecated Functionality: iconv_set_encoding(): Use of iconv.internal_encoding is deprecated in [magento]/lib/internal/Zend/Validate/Hostname.php on line 521

Error log record number: blah

I commented out two instances of iconv_set_encoding each, in lib/internal/Zend/Validate/Hostname.php and lib/internal/Zend/Validator/Hostname.php.

This makes the pages display. But from a quick git grep in the source, there are some more instances of it inside lib/internal/Zend which might bring doom at some point.

PHP version of debian jessie/testing

PHP 5.6.2-1 (cli) (built: Oct 17 2014 17:15:37) 
Copyright (c) 1997-2014 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2014 Zend Technologies
    with Zend OPcache v7.0.4-dev, Copyright (c) 1999-2014, by Zend Technologies
@buskamuza
Copy link
Contributor

@nyov , please, check next Magento version (should come soon) - it will use Zend Farmework 1.12.9, which is compatible with PHP 5.6.

@nyov
Copy link
Author

nyov commented Dec 6, 2014

Thank you! And nice work on moving ZF1 into an external dependency.
Verified to work out of the box now, with alpha107.

@nyov nyov closed this as completed Dec 6, 2014
mmansoor-magento pushed a commit that referenced this issue Jan 30, 2017
Bug fixes:
* MAGETWO-62783: Wrong address template used for order e-mails
fe-lix- pushed a commit to fe-lix-/magento2 that referenced this issue Apr 6, 2018
…le-product-indexer

MSI#457 Configurable Product Indexer
magento-engcom-team added a commit that referenced this issue Jul 30, 2019
… CountryCodeEnum #780

 - Merge Pull Request magento/graphql-ce#780 from magento/graphql-ce:editorial-CountryCodeEnum
 - Merged commits:
   1. e9a3a09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants