?

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
25 Nov 2014

This fixes: #5192

See 82d8459 for the fix against staging.

Steps to reproduce the issue

Download and extract xampp 5.6.3 (they change the versioning to reflect the PHP Version)
Download and extract joomla 2.5.x (github branche or stable)
try to install

Expected result

Install works (after fixing this it still don't work but i will open a new issue for that; see: #5193 )

Actual result

Deprecated: iconv_set_encoding(): Use of iconv.internal_encoding is deprecated in JRoot\libraries\joomla\string\string.php on line 28
Deprecated: iconv_set_encoding(): Use of iconv.input_encoding is deprecated in JRoot\libraries\joomla\string\string.php on line 29
Deprecated: iconv_set_encoding(): Use of iconv.output_encoding is deprecated in JRoot\libraries\joomla\string\string.php on line 30

screen shot 2014-11-25 at 01 20 50

System information (as much as possible)

Default xampp Konfiguration with:

Apache 2.4.10, MySQL 5.6.21, PHP 5.6.3, phpMyAdmin 4.2.11, OpenSSL 1.0.1, XAMPP Control Panel 3.2.1, Webalizer 2.23-04, Mercury Mail Transport System 4.63, FileZilla FTP Server 0.9.41, Tomcat 7.0.56 (with mod_proxy_ajp as connector), Strawberry Perl 7.0.56 Portable

Additional comments

I think the Joomla Core should work with the default xampp config as it is often used for "testing" and planing joomla if can't be done on the production site not only by experts mostly by end users to pre-test Joomla.

avatar zero-24 zero-24 - open - 25 Nov 2014
avatar jissues-bot jissues-bot - change - 25 Nov 2014
Labels Added: ?
avatar zero-24 zero-24 - change - 25 Nov 2014
Category Installation Libraries
avatar zero-24 zero-24 - change - 25 Nov 2014
Milestone Added:
avatar infograf768
infograf768 - comment - 25 Nov 2014

Should not
+if (version_compare(PHP_VERSION, '5.6', '>='))
be INSIDE the
if (function_exists('iconv') || ((!strtoupper(substr(PHP_OS, 0, 3)) === 'WIN' && dl('iconv.so'))))
conditional?

avatar infograf768
infograf768 - comment - 25 Nov 2014

see /libraries/vendor/joomla/string/src/String.php

avatar zero-24
zero-24 - comment - 25 Nov 2014
avatar infograf768
infograf768 - comment - 25 Nov 2014

I would keep the original conditional
if (function_exists('iconv') || ((!strtoupper(substr(PHP_OS, 0, 3)) === 'WIN' && dl('iconv.so'))))

What you think?

avatar Bakual
Bakual - comment - 25 Nov 2014

I think the Joomla Core should work with the default xampp config as it is often used for "testing" and planing joomla if can't be done on the production site not only by experts mostly by end users to pre-test Joomla.

To be blunt, I don't expect many testing being done on 2.5 anymore. Last release is only a week or two away. :smile:

avatar zero-24
zero-24 - comment - 25 Nov 2014

I would keep the original conditional

fixed @infograf768 zero-24@7023efb

avatar infograf768
infograf768 - comment - 26 Nov 2014

Does not cost much to get in. ☺ Merging. Thanks.

avatar infograf768 infograf768 - change - 26 Nov 2014
Milestone Added:
avatar infograf768 infograf768 - reference | ebead82 - 26 Nov 14
avatar infograf768 infograf768 - merge - 26 Nov 2014
avatar infograf768 infograf768 - close - 26 Nov 2014
avatar infograf768 infograf768 - close - 26 Nov 2014
avatar infograf768 infograf768 - change - 26 Nov 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-11-26 09:12:09
avatar zero-24 zero-24 - head_ref_deleted - 3 Dec 2014

Add a Comment

Login with GitHub to post a comment