Differences

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

--- rfc:deprecate-bareword-strings [2017/03/05 22:35] – extra heading to draw the eye imsop
+++ rfc:deprecate-bareword-strings [2017/09/22 13:28] – external edit 127.0.0.1
@@ Line 3: / Line 3: @@
   * Date: 2017-03-05
   * Author: Rowan Collins rowan.collins@gmail.com
-  * Status: In Discussion
+  * Status: Accepted
   * First Published at: http://wiki.php.net/rfc/deprecate-bareword-strings
@@ Line 119: / Line 119: @@
-===== Proposed Voting Choices =====
+===== Voting =====
-A single yes/no vote requiring a 2/3 majority.
+Voting opened on 2017-03-08, and will close on 2017-03-22 at 22:00 UTC
-In PHP 7.2, replace the "Undefined constant" ''E_NOTICE'' with an ''E_WARNING'' stating that the fallback is deprecated; and in PHP 8.0, remove the fallback and replace the ''E_WARNING'' with an ''Error''.
+The vote requires a 2/3 majority to accept the proposal.
+Voting is on the following proposal:
-===== Patches and Tests =====
+  - In PHP 7.2, raise the severity of the message "Use of undefined constant" from E_NOTICE to E_WARNING, and change the text of the message to ''Use of undefined constant %s; assumed '%s' (this will throw an error in a future version of PHP)''
+  - In PHP 8.0, remove the fallback, and replace the ''E_WARNING'' with a thrown ''Error'' with message ''Use of undefined constant %s''
+<doodle title="Raise severity of undefined constants to E_WARNING in 7.2, and Error in 8.0?" auth="imsop" voteType="single" closed="true">
+   * Yes
+   * No
+</doodle>
-None yet.
 ===== Implementation =====
   * Pull Request implementing the change itself: https://github.com/php/php-src/pull/2404
+  * Merged into master: https://github.com/php/php-src/commit/1b565f1393f82e0ce0c94806cc7f52c6d9c5e87d
   * Pull Request to the Language Spec documenting the new behaviour: https://github.com/php/php-langspec/pull/193

Differences

Page Tools