From:             espaiz at gmail dot com
Operating system: Debian 4.0.3
PHP version:      5.2.6
PHP Bug Type:     Arrays related
Bug description:  unset in foreach breaks recrusion

Description:
------------
Made a simple recrusion function, sorting id by parentid, unsetting sorted
values to shorten time of next recrusions. Unsetting inside foreach with
deep arrays does work(!), but with simple array unset somehow stops the
recrusion (first cicle will never finish its loop, the last loop will be
made by latest child).

Tested with PHP 5.2.6 on Debian-40r3-amd64 and Windows Vista 32


Reproduce code:
---------------
<?php
function recrusion($array, $return, $pid=0, $level=0){
        foreach($array as $id=>$parent){
                if( $parent===$pid ){
                        $return[$id]= $level;
                        unset($array[$id]); /* remove this to get correct 
results */
                        recrusion(&$array, &$return, $id, $level+1);
                }
        }
}
$return= array();
$array= array(1=>0,2=>1,3=>2,4=>3,5=>2,6=>2,7=>6,8=>6,9=>0,10=>0);
recrusion($array, &$return);
var_dump( $return );
?>

Expected result:
----------------
array(10) { [1]=> int(0) [2]=> int(1) [3]=> int(2) [4]=> int(3) [5]=>
int(2) [6]=> int(2) [7]=> int(3) [8]=> int(3) [9]=> int(0) [10]=> int(0) }

Actual result:
--------------
array(6) { [1]=> int(0) [2]=> int(1) [3]=> int(2) [4]=> int(3) [9]=>
int(0) [10]=> int(0) } 

-- 
Edit bug report at http://bugs.php.net/?id=45385&edit=1
-- 
Try a CVS snapshot (PHP 5.2): 
http://bugs.php.net/fix.php?id=45385&r=trysnapshot52
Try a CVS snapshot (PHP 5.3): 
http://bugs.php.net/fix.php?id=45385&r=trysnapshot53
Try a CVS snapshot (PHP 6.0): 
http://bugs.php.net/fix.php?id=45385&r=trysnapshot60
Fixed in CVS:                 http://bugs.php.net/fix.php?id=45385&r=fixedcvs
Fixed in release:             
http://bugs.php.net/fix.php?id=45385&r=alreadyfixed
Need backtrace:               http://bugs.php.net/fix.php?id=45385&r=needtrace
Need Reproduce Script:        http://bugs.php.net/fix.php?id=45385&r=needscript
Try newer version:            http://bugs.php.net/fix.php?id=45385&r=oldversion
Not developer issue:          http://bugs.php.net/fix.php?id=45385&r=support
Expected behavior:            http://bugs.php.net/fix.php?id=45385&r=notwrong
Not enough info:              
http://bugs.php.net/fix.php?id=45385&r=notenoughinfo
Submitted twice:              
http://bugs.php.net/fix.php?id=45385&r=submittedtwice
register_globals:             http://bugs.php.net/fix.php?id=45385&r=globals
PHP 4 support discontinued:   http://bugs.php.net/fix.php?id=45385&r=php4
Daylight Savings:             http://bugs.php.net/fix.php?id=45385&r=dst
IIS Stability:                http://bugs.php.net/fix.php?id=45385&r=isapi
Install GNU Sed:              http://bugs.php.net/fix.php?id=45385&r=gnused
Floating point limitations:   http://bugs.php.net/fix.php?id=45385&r=float
No Zend Extensions:           http://bugs.php.net/fix.php?id=45385&r=nozend
MySQL Configuration Error:    http://bugs.php.net/fix.php?id=45385&r=mysqlcfg

Reply via email to