Seems like this thing never got finished. There aren't any tests for it either..

In original doc (http://www.php.net/~helly/php/ext/spl/), RegexIterator::REPLACE wait for "replacement" property, but it's doesn't exist in __construct and properties.

In ext/spl replacement exist as public property but not in __construct method, so I try to extends it like this

public function __construct($iterator, $regex, $mode = 0, $flags = 0, $preg_flags = 0)
{			
    parent::__construct($iterator, $regex, $mode, $flags, $preg_flags);
    $this->replacement = '$1';					
}

but nothing append to :(

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.

Hi, i saw the test file (bug50579.phpt), it's strange to put the replacement property on the iterator parameter insteed the Regexiterator instance, imagine i've this follow code :

class foo extends ArrayIterator {
	public function __construct( ) {
		parent::__construct(array(
		'test1'=>'test888', 
		'test2'=>'what?', 
		'test3'=>'test999',
		'replacement' => 'replacement'), ArrayIterator::ARRAY_AS_PROPS);
		$this->replacement = '[$1]';
	}
}

if a would read $h->replacement, i've '[$1]' insteed 'replacement'.

it's more logical to do :

$i = new RegexIterator($h, '/^test(.*)/', RegexIterator::REPLACE);
$i->replacement = '[$0]';

or maybe make a methode :

$i->setReplacement('[$0]');

You are right, I've fixed the property issue.

Thanks for reviewing/testing/reporting! :)