In .htaccess, several PHP configuration settings are made with the php_value directive. Some of these settings are boolean values, and PHP documentation (http://us2.php.net/configuration.changes) says they should be set with the php_flag directive, not php_value:

Note: Don't use php_value to set boolean values. php_flag should be used instead.

I have not seen any problem using the php_value directive, but for best compatibility it seems advisable to follow PHP's documentation.

The attached patch changes "php_value ... 0" to "php_flag ... off" for the boolean settings: magic_quotes_gpc, register_globals, session.auto_start, and mbstring.encoding_translation.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Robin Monks’s picture

FileSize
968 bytes

Free re-roll!

Robin

c960657’s picture

The re-rolled patch no longer uses "off" instead of "0". Was that intentional?

maartenvg’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
976 bytes

Review and tested.
I'm with the original patch, on|off is the preferred method (see mentioned documentation).

Attached is the "off" version of the patch in #1.

Given the small size of this patch, I reckon this is RTBC.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.