php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #48667 "could not implement" - NOT redefining functions
Submitted: 2009-06-23 22:14 UTC Modified: 2010-02-01 13:48 UTC
From: uramihsayibok at gmail dot com Assigned: colder (profile)
Status: Closed Package: SPL related
PHP Version: 5.2.10 OS: Windows XP
Private report: No CVE-ID: None
 [2009-06-23 22:14 UTC] uramihsayibok at gmail dot com
Description:
------------
When trying to make a class that implements IteratorAggregate and SPL's SeekableIterator the order of the interfaces in the list matters. One order (IA first SI second) works while the other (SI first IA second) produces a fatal error in Unknown.

THESE TWO INTERFACES ARE SEPARATE AND DISTINCT - NO FUNCTIONS IN COMMON!
Traversable -> Iterator -> SeekableIterator (seek,current,next,key,valid,rewind)
Traversable -> IteratorAggregate (getIterator)


I haven't been able to reproduce this with user-defined interfaces and I haven't gone looking to see if this happens with any other built-in interfaces.

Something to do with that fact that they redefine how iteration on a class works?

Reproduce code:
---------------
#1 (works)
abstract class Foo1 implements IteratorAggregate, SeekableIterator {}

#2 (does not work)
abstract class Foo2 implements SeekableIterator, IteratorAggregate {}


Expected result:
----------------
#1 and #2 should both work or both not work.
In this case, work, since neither have any functions in common to redefine.

Actual result:
--------------
#1 - okay
#2 - "Fatal error: Class Foo2 could not implement interface IteratorAggregate in Unknown on line 0"


Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2009-08-25 07:27 UTC] sjoerd@php.net
Reproducable and strange. Error is also not very descriptive.
 [2009-08-26 12:29 UTC] admin at somesite dot de
When i set display_errors to on and display_startup_errors to on i even get an error message window displaying the same error. according to the taskmanager, this window is created by csrss.exe .

i use Windows XP SP2 with xampp + PHP5.3
 [2010-02-01 13:45 UTC] svn@php.net
Automatic comment from SVN on behalf of colder
Revision: http://svn.php.net/viewvc/?view=revision&revision=294304
Log: Fix #48667 (Implementing Iterator and IteratorAggregate is now restricted in both orders)
 [2010-02-01 13:48 UTC] colder@php.net
This bug has been fixed in SVN.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.

Both orders are now restricted.

The problem is not that they do share any function, but they both 
overwrite the internal get_iterator handler used by foreach, for 
instance.
 [2010-02-05 00:37 UTC] svn@php.net
Automatic comment from SVN on behalf of pajoye
Revision: http://svn.php.net/viewvc/?view=revision&revision=294549
Log: - Fix #48667 (Implementing Iterator and IteratorAggregate is now restricted..
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Thu Nov 21 13:01:29 2024 UTC