actually, I think previously behavior seems bug... current behavior is more reasonable, "it overwrite the old value".

@laruence: My intuition would say that extract() without EXTR_REFS should behave the same was as manually doing the assignments, i.e.:

    extract(['a' => 'b', 'c' => 'd']);
    // should be the same as writing
    $a = 'b';
    $c = 'd';

So in this case the equivalent code without extract() would be:

    $var = 'original value';
    $ref =& $var;
    $var = 'new value';

In which case the result that PHP 5.6 provides is correct.

If EXTR_REFS is specified then all assignments should happen with =&, which breaks references. In this case the current behavior matches the previous one already.

I agree with nikic. It should do the same as doing it manually.

okey, I fixed it, make it be consistent with 5.6's behavior

thanks

The fix for this bug has been committed.

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/.

 For Windows:

http://windows.php.net/snapshots/
 
Thank you for the report, and for helping us make PHP better.