Bug #72992 [Csd->ReO]: mbstring.internal_encoding doesn't inherit default_charset

From: Date: Sat, 10 Sep 2016 02:01:39 +0000
Subject: Bug #72992 [Csd->ReO]: mbstring.internal_encoding doesn't inherit default_charset
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-203940@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=72992&edit=1

 ID:                 72992
 Updated by:         yohgaki@php.net
 Reported by:        zoeslam at gmail dot com
 Summary:            mbstring.internal_encoding doesn't inherit
                     default_charset
-Status:             Closed
+Status:             Re-Opened
 Type:               Bug
 Package:            mbstring related
 Operating System:   Ubuntu 16.04
 PHP Version:        7.0.10
 Assigned To:        yohgaki
 Block user comment: N
 Private report:     N

 New Comment:

I'll just commit INI propagation part of attached patch, too. Then you'll see modified
INIs.


Previous Comments:
------------------------------------------------------------------------
[2016-09-10 01:57:47] yohgaki@php.net

So you would like to have INI propagation as attached patch?

------------------------------------------------------------------------
[2016-09-09 11:37:59] zoeslam at gmail dot com

Hi, I recompiled from source with your commit, but nothing changed.

Could you explain me better your comment about INI settings?

If I read the manual at http://php.net/manual/en/mbstring.configuration.php#ini.mbstring.internal-encoding
I assume that, when every setting is left empty except "default_charset", MB will use
"default_charset" value for the internal encoding, isn't so?

If it is so, the bug is still present

------------------------------------------------------------------------
[2016-09-08 04:57:19] yohgaki@php.net

Automatic comment on behalf of yohgaki
Revision: http://git.php.net/?p=php-src.git;a=commit;h=8bbd0952e5bba88426bac1596dcc3bfa504dbe4e
Log: Fix Bug #72992 mbstring.internal_encoding doesn't inherit default_charset

------------------------------------------------------------------------
[2016-09-08 04:14:12] yohgaki@php.net

INI could be propagated, but it's not necessary. If higher precedence INI is set, lower
precedence INI is not set, higher is used.

I'll commit fix for logical error part only.

------------------------------------------------------------------------
[2016-09-08 04:12:21] yohgaki@php.net

The following patch has been added/updated:

Patch Name: bug72992.patch
Revision:   1473307940
URL:        https://bugs.php.net/patch-display.php?bug=72992&patch=bug72992.patch&revision=1473307940

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=72992


--
Edit this bug report at https://bugs.php.net/bug.php?id=72992&edit=1


Thread (19 messages)

« previous php.bugs (#203940) next »