ID:               33940
 Updated by:       [EMAIL PROTECTED]
 Reported By:      david dot tulloh at infaze dot com dot au
-Status:           Open
+Status:           Closed
 Bug Type:         Arrays related
 Operating System: *
 PHP Version:      5CVS-2005-08-02
-Assigned To:      
+Assigned To:      dmitry
 New Comment:

Your expectation is wrong because array_map() receives arrays arguments
by value. To make array_map() work as you expected you should pass
arrays by reference directly.
However there is anothe bug in this case. array_map() shouldn't modify
sorce array if it isn't called by reference.

This is fixed in CVS HEAD (6.0), PHP_5_1m PHP_5_0 and PHP_4_4.

See test case for more details: ext/standard/tests/array/bug33940.phpt


Previous Comments:
------------------------------------------------------------------------

[2005-08-01 05:05:50] david dot tulloh at infaze dot com dot au

Description:
------------
array_map fails to work recursively.
It does not pass by reference in the inner array_map call.

Changing the line to
$ret = array_map('ref_map', &$item);
provides the expected result but throws a Call-time pass-by-reference
warning.

Reproduce code:
---------------
<?php
function ref_map(&$item) {
    if(!is_array($item)) {
        $item = 1;
        return 2;
    } else {
        $ret = array_map('ref_map', $item);
        echo 'Inner return: '; print_r($ret);
        echo 'Inner item: '; print_r($item);
        return $ret;
    }
}

$a = array(array(0), 0);
$ret = array_map('ref_map', $a);
echo 'Array: '; print_r($a);
echo 'Return: '; print_r($ret);

?>


Expected result:
----------------
Inner return: Array
(
    [0] => 2
)
Inner item: Array
(
    [0] => 1
)
Array: Array
(
    [0] => Array
        (
            [0] => 1
        )

    [1] => 1
)
Return: Array
(
    [0] => Array
        (
            [0] => 2
        )

    [1] => 2
)


Actual result:
--------------
Inner return: Array
(
    [0] => 2
)
Inner item: Array
(
    [0] => 0
)
Array: Array
(
    [0] => Array
        (
            [0] => 0
        )

    [1] => 1
)
Return: Array
(
    [0] => Array
        (
            [0] => 2
        )

    [1] => 2
)



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


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

Reply via email to