rfc:date.timezone_warning_removal

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revision Previous revision
Next revision
Previous revision
rfc:date.timezone_warning_removal [2015/02/15 17:26]
bwoebi typo
rfc:date.timezone_warning_removal [2017/09/22 13:28] (current)
Line 1: Line 1:
 ====== PHP RFC: Remove the date.timezone warning ====== ====== PHP RFC: Remove the date.timezone warning ======
-  * Version: 1.1+  * Version: 1.2
   * Date: 2015-01-27   * Date: 2015-01-27
   * Author: Bob Weinand, bobwei9@hotmail.com   * Author: Bob Weinand, bobwei9@hotmail.com
-  * Status: Under Discussion+  * Status: Implemented
   * First Published at: https://wiki.php.net/rfc/date.timezone_warning_removal   * First Published at: https://wiki.php.net/rfc/date.timezone_warning_removal
  
Line 9: Line 9:
 PHP should be usable without configuring it through either an ini file or defining settings on the command line. Currently this is not possible as the date.timezone is required to be set, despite it defaulting to the best option. PHP should be usable without configuring it through either an ini file or defining settings on the command line. Currently this is not possible as the date.timezone is required to be set, despite it defaulting to the best option.
  
-"date.timezone" is the only setting that is requied to be set. Every other setting, including those that have security implications default to sensible values silently; defaulting to sensible values does not appear to cause any problem for users. +"date.timezone" is the only setting that is required to be set. Every other setting, including those that have security implications default to sensible values silently; defaulting to sensible values does not appear to cause any problem for users. 
  
 The error message becomes entirely tedious when working with anything other than a single installed version of PHP. The error message becomes entirely tedious when working with anything other than a single installed version of PHP.
Line 31: Line 31:
 ===== Feedback ===== ===== Feedback =====
 ==== "Won't this make life more difficult for end users?" ==== ==== "Won't this make life more difficult for end users?" ====
-No. For users who install PHP through a distribution, the ini file is created by the package maintainer and it includes the "date.timezone" setting.+No. For users who install PHP through a distribution, the ini file is created by the package maintainer. They should set it to the systems timezone. (Some don't, but that is the fault of the distributions.)
  
 For end users who compile from source they can either modify the ini file that is used by PHP in the compile step or they could add it themselves when PHP is built e.g. For Debian they could easily stick the following in a post-install hook to fix it: For end users who compile from source they can either modify the ini file that is used by PHP in the compile step or they could add it themselves when PHP is built e.g. For Debian they could easily stick the following in a post-install hook to fix it:
Line 43: Line 43:
   * Those who use multiple versions of PHP or otherwise use PHP without going through the install process. This RFC will be beneficial to these users.   * Those who use multiple versions of PHP or otherwise use PHP without going through the install process. This RFC will be beneficial to these users.
   * People who have accidentally deleted their ini files from their production system. These users have bigger issues than this RFC can address.   * People who have accidentally deleted their ini files from their production system. These users have bigger issues than this RFC can address.
 +
 +Still, it is weird, to have that one setting emit a warning, but other much more important settings don't. The ini settings have defaults and they should be used.
  
 ==== "Users are idiots and can't be trusted" ==== ==== "Users are idiots and can't be trusted" ====
Line 52: Line 54:
  
 ===== Proposed Voting Choices ===== ===== Proposed Voting Choices =====
-Remove this warning (50%+1 vote): +Remove this warning (50%+1 vote) - Vote began 2015-02-16 and ended 2015-02-24. 
-  * Yes + 
-  * No+<doodle title="Should the warning about a not set date.timezone ini setting be removed in master?" auth="bwoebi" voteType="single" closed="true"> 
 +   * Yes 
 +   * No 
 +</doodle>
  
 ===== Patch ===== ===== Patch =====
Line 70: Line 75:
   * 1.0: Initial RFC (2015-01-27)   * 1.0: Initial RFC (2015-01-27)
   * 1.1: More neutral wording, discussion of feedback (2015-02-15)   * 1.1: More neutral wording, discussion of feedback (2015-02-15)
 +  * 1.2: Put into vote (2015-02-16)
rfc/date.timezone_warning_removal.1424021189.txt.gz · Last modified: 2017/09/22 13:28 (external edit)