Edit report at https://bugs.php.net/bug.php?id=55157&edit=1

 ID:                 55157
 Comment by:         gergo at gergoerdosi dot com
 Reported by:        olav at fwt dot no
 Summary:            ArrayIterator::offsetUnset(); does not work
                     correctly
 Status:             Assigned
 Type:               Bug
 Package:            SPL related
 Operating System:   openSUSE 11.4/Debian 5
 PHP Version:        Irrelevant
 Assigned To:        colder
 Block user comment: N
 Private report:     N

 New Comment:

Please ignore my comment above. The offsetUnset() method unsets elements using 
their index. Changing the code to this gives the expected result (0, 4):

$items = new ArrayObject(array(0, 1, 2, 3, 4));

foreach($items->getArrayCopy() as $item) {
        if(in_array($item, array(1, 2, 3))) {
                $items->offsetUnset($item);
        }
}

var_dump($items->getArrayCopy());


Previous Comments:
------------------------------------------------------------------------
[2011-07-19 15:09:34] gergo at gergoerdosi dot com

I have the same problem, used this code:

$items = new ArrayObject(array(1, 2, 3, 4, 5));

foreach($items as $item) {
        if(in_array($item, array(2, 3, 4))) {
                $items->offsetUnset($item);
        }
}

var_dump($items->getArrayCopy());

Expected result: array(1, 5), actual result: array(1, 3, 5).

------------------------------------------------------------------------
[2011-07-07 07:18:36] j...@php.net

Sorry, I should clarify that the code above gives:

array(1) { [1]=> int(1) }

Whereas you'd probably expect:

array(0) { }

------------------------------------------------------------------------
[2011-07-07 07:17:12] j...@php.net

Just to narrow the scope of the bug a bit:

$a = range( 0,9 );
$b = new ArrayIterator( $a );
 
foreach($b as $k=>$v) {
    $b->offsetUnset($k);
}

var_dump($b->getArrayCopy()); // why is 1 still around?

------------------------------------------------------------------------
[2011-07-07 06:52:24] olav at fwt dot no

Description:
------------
ArrayIterator always skips the second element in the array when calling 
offsetUnset(); on it while looping through.

Using the key from iterator and unsetting in the actual ArrayObject works as 
expected.

Running php 5.3.5 on openSUSE 11.4
Replicated same bug on Debian 5 running PHP 5.2.6-1+lenny9.

php5 is installed as binary on both systems, SPL is builtin package.


Test script:
---------------
<?php

// Create a array range from 0 to 9
$a = new ArrayObject( range( 0,9 ) );
$b = new ArrayIterator( $a );
 
for ( $b->rewind(); $b->valid(); $b->next() )
{
    echo "#{$b->current()} - \r\n";
    $b->offsetUnset( $b->key() );
}
?>

Expected result:
----------------
Expected a list of from 0 to 9 echoed out.

Actual result:
--------------
Lists out 0 and then 2-9 leaving index 1 untouched in the original ArrayObject.



------------------------------------------------------------------------



-- 
Edit this bug report at https://bugs.php.net/bug.php?id=55157&edit=1

Reply via email to