The following is a bit silly:
$logger->setEventItem('foo', $foo);
$logger->setEventItem('bar', $bar);
$logger->setEventItem('baz', $baz);
$logger->err('Error message');
Better I think would be two alternatives:
$logger->foo = $foo;
$logger->bar = $bar;
$logger->baz = $baz;
$logger->err('Error message');
And:
$extras = array('foo' => $foo, 'bar' => $bar, 'baz' => $baz);
$logger->err('Error message', $extras);
As far as I can tell this would be backwards compatible.
Thoughts?
-Matt
