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

Link to this comparison view

Next revision
Previous revision
rfc:parameter-no-type-variance [2016/11/21 09:27]
kelunik created
rfc:parameter-no-type-variance [2017/09/22 13:28] (current)
Line 1: Line 1:
-====== PHP RFC: Your Title Here ====== +====== PHP RFC: Parameter Type Widening ====== 
-  * Version: 0.1 +  * Version: 0.3 
-  * Date: 2016-11-21 +  * Date: 2017-01-03 
-  * Author: Niklas Keller <me@kelunik.com> +  * Authors: Niklas Keller <me@kelunik.com> 
-  * Status: Draft+  * Status: Implemented (7.2)
   * First Published at: http://wiki.php.net/rfc/parameter-no-type-variance   * First Published at: http://wiki.php.net/rfc/parameter-no-type-variance
-===== Introduction ===== 
-PHP doesn't currently allow variance for parameters as checking these for compatibility isn't possible on compile time. This limitation is caused by autoloading and doesn't allow widening the accepted parameters. 
 ===== Proposal ===== ===== Proposal =====
-This RFC proposes to allow ommiting the type entirely in a subclass, as dropping all parameter constraints is always valid according to the LSP principle.+PHP currently doesn'allow any variance of parameter types when checking whether a method implemented in a class is compatible with the method defined either in the parent class or interface.
-===== Backward Incompatible Changes ===== +This RFC proposes to allow omitting a type entirely in a subclass, as dropping all parameter constraints is always valid according to the contravariance rule.
-===== Proposed PHP Version(s) ===== +Implementing this RFC would allow libraries to be upgraded to use type declarations in their method signaturesCurrently adding types to a method of a class in a library would break any code that extends that class.
-Next minor version, currently 7.2.+
-===== RFC Impact ===== +This would provide an easier upgrade path for libraries to start using scalar types, to replace manual checks being done inside the methods, without requiring an update for all sub-classes.
-==== To SAPIs ==== +
-==== To Existing Extensions ==== +Another example of this being useful is ''DateTime::createFromFormat()''. This method is already documented to accept only a class of type ''DateTimeZone'' as the 3rd parameter in the manual, but the type declaration is not actually present in the implementation.
-==== To Opcache ==== +A change was made to the ''DateTime::createFromFormat()'' to add the ''DateTimeZone'' type to the 3rd parameter. However, this change had to be reverted, as all classes extending ''DateTime'' currently don't have this type declaration, so they started throwing a method signature mismatch warning. This RFC would allow the ''DateTimeZone'' type to be added to the 3rd parameter, without breaking code that extends the ''DateTime'' class.
-==== New Constants ==== 
-==== php.ini Defaults ==== +==== Example ====
-===== Open Issues =====+<code php> 
 +class ArrayClass { 
 +  public function foo(array $foo) { /* ... */ } 
 +// This RFC proposes allowing the type to be widened to be untyped aka any 
 +// type can be passed as the parameter. 
 +// Any type restrictions can be done via user code in the method body. 
 +class EverythingClass extends ArrayClass { 
 +  public function foo($foo) { /* ... */ } 
 +**Current Result** 
 +''Warning: Declaration of EverythingClass::foo($foo) should be compatible with ArrayClass::foo(array $foo) in %s on line 18'' 
 +**Proposed Result** 
 +Compiles without a warning. 
 +===== Backward Incompatible Changes =====
 None. None.
 +===== Proposed PHP Version(s) =====
 +Next version, currently 7.2.
 ===== Future Scope ===== ===== Future Scope =====
-In the future there might be support for contra-variance / co-variance for parameters / return types.+Unfortunately "true" contravariance for class types isn't part of this RFC, as implementing that is far more difficult, and would require additional rules about autoloading and/or class compilation, which might only be acceptable at a major release.
-===== Proposed Voting Choices =====+===== Voting =====
 Requires a 2/3 majority. Requires a 2/3 majority.
-===== Patches and Tests ===== +<doodle title="Parameter Type Widening" auth="kelunik" voteType="single" closed="true"> 
-TBD.+   * Yes 
 +   * No 
-===== Implementation ===== +Voting started on 9th of January but has been stopped due to the mailing list failures. Voting has been resumed on 11th of January and has ended on 31th of January.
-===== References ===== +===== Patches and Tests =====
-===== Rejected Features ===== +  * https://github.com/php/php-src/pull/2265 
 +===== Implementation ===== 
 +  - Merged into PHP 7.2 
 +  - https://github.com/php/php-src/commit/2edc3cf8d2dd75501bf1049cf1b3ca57f11f1234
rfc/parameter-no-type-variance.1479720427.txt.gz · Last modified: 2017/09/22 13:28 (external edit)