Skip to content

Commit ec1f61f

Browse files
authored
Merge pull request #113 from firegento/develop
Publish release for JsonStream fixes.
2 parents 102b79c + b1165e3 commit ec1f61f

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/app/code/community/FireGento/Logger/Model/JsonStream.php

+3-3
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
* @package FireGento_Logger
2626
* @author FireGento Team <team@firegento.com>
2727
*/
28-
class FireGento_Logger_Model_Stream extends Zend_Log_Writer_Stream
28+
class FireGento_Logger_Model_JsonStream extends Zend_Log_Writer_Stream
2929
{
3030
/**
3131
* @var bool
@@ -73,10 +73,10 @@ protected function _write($event)
7373
$event = Mage::helper('firegento_logger')->getEventObjectFromArray($event);
7474
Mage::helper('firegento_logger')->addEventMetadata($event, NULL, $this->_enableBacktrace);
7575
$eventData = $event->getEventDataArray();
76-
$eventData = array_filter($eventData, 'is_null');
76+
$eventData = array_filter($eventData, function ($var) { return $var !== null; });
7777
$line = @json_encode($eventData);
7878

79-
if (false === @fwrite($this->_stream, $line)) {
79+
if (false === @fwrite($this->_stream, $line . PHP_EOL)) {
8080
//require_once 'Zend/Log/Exception.php';
8181
throw new Zend_Log_Exception("Unable to write to stream");
8282
}

0 commit comments

Comments
 (0)