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

 ID:                 60179
 User updated by:    dbetz at df dot eu
 Reported by:        dbetz at df dot eu
 Summary:            wrong output with php_flag boolean
 Status:             Assigned
 Type:               Bug
 Package:            FPM related
 Operating System:   Gentoo Linux
 PHP Version:        5.3.8
 Assigned To:        fat
 Block user comment: N
 Private report:     N

 New Comment:

Hello,

yes, the option is set correct, but the return value of ini_get() is different 
than whith php-cgi.

Most apps like typo3, joomla, wordpress, eg. are checking in the installer, if 
php.ini settings are correct.
These setting are checked thru ini_get.

When ini_get(session.auto_start) is not 0 but "Off", then the check fails.

Here is the sample code from typo3 installer:

                if (ini_get('session.auto_start')) {
                        $sessionCreationError = 'Error: session.auto-start
is enabled.<br />';
                        $sessionCreationError .= 'The PHP option
session.auto-start is enabled. Disable this option in php.ini or
.htaccess:<br />';

I hope you understand now my problem.


Previous Comments:
------------------------------------------------------------------------
[2011-10-31 14:22:10] f...@php.net

hum ... not sure to understand your problem afterall :)

without the patch:
If I set php_flag[session.auto_start]=On
ini_get("session.auto_start") returns "On"
and session are created for each page (auto start is on as asked)

If I set php_flag[session.auto_start]=Off
ini_get("session.auto_start") returns "Off"
and session are NOT created for each page (auto start is off as asked)

For me this works as expected.
Do you have the same behavior ?

------------------------------------------------------------------------
[2011-10-31 13:52:48] dbetz at df dot eu

Hi fat,

i have tested the latest snapshot with the patch and have changed the config to
php_flag[session.auto_start] = Off

But the problem is still here.

Looking into fpm_conf.c at: static char *fpm_conf_set_array(... :
        if (convert_to_bool) {
                char *err = fpm_conf_set_boolean(value, &subconf, 0);
                if (err) return err;
                kv->value = strdup(b ? "On" : "Off");
        }

It looks like kv->value is either string "On" or "Off" or am i wrong ?

------------------------------------------------------------------------
[2011-10-31 13:18:55] f...@php.net

1- On wich version did you apply the patch ? Please test the patch on the 
lastest 5.3.x snapshot.

2- you should not surrender the parameter with double quote:

php_flag["session.auto_start"] = Off

should be

php_flag[session.auto_start] = Off


Can you get rid of the bug. On my side, I have the last 5.3.x snapshot and it 
works great. I can't reproduce your problem.

------------------------------------------------------------------------
[2011-10-31 13:11:01] dbetz at df dot eu

Hello,

sorry, the problem still exists.
session.auto_start is : Off
Some code here .....

With cli:
session.auto_start is : 0

------------------------------------------------------------------------
[2011-10-31 13:02:09] f...@php.net

Can you please test the patch attached ?

If you can test it quickly I can integrate it before 5.3.9 is going RC1.

thx
++ fat

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


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=60179


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

Reply via email to