Zend\Log\Formatter\Xml improvement to handle extra data array

[dkemper]

I found out, that the \Zend\Log\Formatter\Xml doesn´t support "extra" information, when its a extra data is an array. I made an example to show my little change.

Hope its useful.

Example-Code:

$stream = new \Zend\Log\Writer\Stream('data/log/application.log');
$stream->setFormatter(new \Zend\Log\Formatter\Xml());
$logger = new \Zend\Log\Logger();
$logger->addProcessor(new \Zend\Log\Processor\Backtrace());
$logger->addWriter($stream);

$logger->info('wrong stuff');

before:

<logEntry>
   <timestamp>2014-04-14T23:40:17+02:00</timestamp>
   <priority>6</priority>
   <priorityName>INFO</priorityName>
   <message>wrong stuff</message>
</logEntry>

after:

<logEntry>
  <timestamp>2014-04-14T23:40:17+02:00</timestamp>
  <priority>6</priority>
  <priorityName>INFO</priorityName>
  <message>wrong stuff</message>
  <extra>
      <file>/home/project/module/App/src/App/Controller/ConsoleController.php</file>        <line>51</line>
      <class>App\Controller\ConsoleController</class>
      <function>loggerTestAction</function>
  </extra>
</logEntry>

[weierophinney]

What about nested arrays? If it's going to support arrays, may as well go the whole way! (hint: use recursion)

[dkemper]

I made a initial version for nested array handling. I didn't find anything in the library. So i hope this version is a least something we can discuss about. Nameing etc.

[samsonasik]

an xml, not a xml

[ezimuel]

@dkemper can you provide a unit tests for a simple scenario (single array) and nested arrays?

[Ocramius]

@dkemper manually merged @bf2d0d3308863185a1f70b8df8c3a9fddf06c5d8, thanks!