Req #77518 [NEW]: SeekableIterator::seek() should accept 'int' typehint as documented
| From: | claude dot pache at gmail dot com | Date: | Thu, 24 Jan 2019 19:11:01 +0000 |
| Subject: | Req #77518 [NEW]: SeekableIterator::seek() should accept 'int' typehint as documented | ||
| Groups: | php.bugs | ||
| Request: | Send a blank email to php-bugs+get-219179@lists.php.net to get a copy of this message | ||
From: claude dot pache at gmail dot com
Operating system:
PHP version: 7.3.1
Package: SPL related
Bug Type: Feature/Change Request
Bug description:SeekableIterator::seek() should accept 'int' typehint as documented
Description:
------------
This is related to Bug #71051, but since
https://wiki.php.net/rfc/parameter-no-type-variance
has been implemented, it should be fixable in PHP7.2+ without breaking
legacy code.
Per documentation, the signature of SeekableIterator::seek() is:
abstract public SeekableIterator::seek ( int $position ) : void
However, when implementing the interface, providing the typehint
âintâ triggers a compile error.
Test script:
---------------
class Foo implements SeekableIterator {
function current() { }
function key() { }
function next() { }
function rewind() { }
function valid() { }
function seek(int $position) { }
}
Expected result:
----------------
No error.
Actual result:
--------------
Fatal error: Declaration of Foo::seek(int $position) must be compatible
with SeekableIterator::seek($position)
--
Edit bug report at https://bugs.php.net/bug.php?id=77518&edit=1
--
Try a snapshot (PHP 5.4): https://bugs.php.net/fix.php?id=77518&r=trysnapshot54
Try a snapshot (PHP 5.5): https://bugs.php.net/fix.php?id=77518&r=trysnapshot55
Try a snapshot (trunk): https://bugs.php.net/fix.php?id=77518&r=trysnapshottrunk
Fixed in SVN: https://bugs.php.net/fix.php?id=77518&r=fixed
Fixed in release: https://bugs.php.net/fix.php?id=77518&r=alreadyfixed
Need backtrace: https://bugs.php.net/fix.php?id=77518&r=needtrace
Need Reproduce Script: https://bugs.php.net/fix.php?id=77518&r=needscript
Try newer version: https://bugs.php.net/fix.php?id=77518&r=oldversion
Not developer issue: https://bugs.php.net/fix.php?id=77518&r=support
Expected behavior: https://bugs.php.net/fix.php?id=77518&r=notwrong
Not enough info: https://bugs.php.net/fix.php?id=77518&r=notenoughinfo
Submitted twice: https://bugs.php.net/fix.php?id=77518&r=submittedtwice
register_globals: https://bugs.php.net/fix.php?id=77518&r=globals
PHP 4 support discontinued: https://bugs.php.net/fix.php?id=77518&r=php4
Daylight Savings: https://bugs.php.net/fix.php?id=77518&r=dst
IIS Stability: https://bugs.php.net/fix.php?id=77518&r=isapi
Install GNU Sed: https://bugs.php.net/fix.php?id=77518&r=gnused
Floating point limitations: https://bugs.php.net/fix.php?id=77518&r=float
No Zend Extensions: https://bugs.php.net/fix.php?id=77518&r=nozend
MySQL Configuration Error: https://bugs.php.net/fix.php?id=77518&r=mysqlcfg