diff --git a/src/Monolog/Formatter/ElasticaFormatter.php b/src/Monolog/Formatter/ElasticaFormatter.php index 128e07ab..c6f3c8e1 100644 --- a/src/Monolog/Formatter/ElasticaFormatter.php +++ b/src/Monolog/Formatter/ElasticaFormatter.php @@ -65,7 +65,7 @@ class ElasticaFormatter extends NormalizerFormatter /** * Convert a log message into an Elastica Document - * @param array $record + * @param array $record * @return Document */ protected function getDocument(array $record): Document diff --git a/src/Monolog/Formatter/ElasticsearchFormatter.php b/src/Monolog/Formatter/ElasticsearchFormatter.php index 1667f2cc..84affef3 100644 --- a/src/Monolog/Formatter/ElasticsearchFormatter.php +++ b/src/Monolog/Formatter/ElasticsearchFormatter.php @@ -32,7 +32,7 @@ class ElasticsearchFormatter extends NormalizerFormatter /** * @param string $index Elasticsearch index name - * @param string $type Elasticsearch record type + * @param string $type Elasticsearch record type */ public function __construct(string $index, string $type) { diff --git a/src/Monolog/Handler/ElasticaHandler.php b/src/Monolog/Handler/ElasticaHandler.php index 4a444b85..78a8d1ea 100644 --- a/src/Monolog/Handler/ElasticaHandler.php +++ b/src/Monolog/Handler/ElasticaHandler.php @@ -81,6 +81,7 @@ class ElasticaHandler extends AbstractProcessingHandler if ($formatter instanceof ElasticaFormatter) { return parent::setFormatter($formatter); } + throw new \InvalidArgumentException('ElasticaHandler is only compatible with ElasticaFormatter'); } diff --git a/src/Monolog/Handler/ElasticsearchHandler.php b/src/Monolog/Handler/ElasticsearchHandler.php index 7d5d3a87..6d0021b9 100644 --- a/src/Monolog/Handler/ElasticsearchHandler.php +++ b/src/Monolog/Handler/ElasticsearchHandler.php @@ -89,6 +89,7 @@ class ElasticsearchHandler extends AbstractProcessingHandler if ($formatter instanceof ElasticsearchFormatter) { return parent::setFormatter($formatter); } + throw new InvalidArgumentException('ElasticsearchHandler is only compatible with ElasticsearchFormatter'); } @@ -122,7 +123,7 @@ class ElasticsearchHandler extends AbstractProcessingHandler /** * Use Elasticsearch bulk API to send list of documents * - * @param array $records + * @param array $records * @throws \RuntimeException */ protected function bulkSend(array $records): void diff --git a/src/Monolog/Handler/ZendMonitorHandler.php b/src/Monolog/Handler/ZendMonitorHandler.php index 0ced7079..236472ee 100644 --- a/src/Monolog/Handler/ZendMonitorHandler.php +++ b/src/Monolog/Handler/ZendMonitorHandler.php @@ -39,8 +39,8 @@ class ZendMonitorHandler extends AbstractProcessingHandler ]; /** - * @param string|int $level The minimum logging level at which this handler will be triggered. - * @param bool $bubble Whether the messages that are handled can bubble up the stack or not. + * @param string|int $level The minimum logging level at which this handler will be triggered. + * @param bool $bubble Whether the messages that are handled can bubble up the stack or not. * @throws MissingExtensionException */ public function __construct($level = Logger::DEBUG, bool $bubble = true) diff --git a/src/Monolog/Processor/ProcessorInterface.php b/src/Monolog/Processor/ProcessorInterface.php index 7e64d4df..c58639b8 100644 --- a/src/Monolog/Processor/ProcessorInterface.php +++ b/src/Monolog/Processor/ProcessorInterface.php @@ -1,4 +1,4 @@ -format('Y-m-d\TH:i:s.uP').'", "extra": {} -}', $formatter->format($record)); +}', + $formatter->format($record) + ); $formatter->setJsonPrettyPrint(false); $record = $this->getRecord(); diff --git a/tests/Monolog/Handler/ElasticaHandlerTest.php b/tests/Monolog/Handler/ElasticaHandlerTest.php index fb64e657..423adbe6 100644 --- a/tests/Monolog/Handler/ElasticaHandlerTest.php +++ b/tests/Monolog/Handler/ElasticaHandlerTest.php @@ -183,6 +183,7 @@ class ElasticaHandlerTest extends TestCase $client = new Client(); $handler = new ElasticaHandler($client, $this->options); + try { $handler->handleBatch([$msg]); } catch (\RuntimeException $e) { diff --git a/tests/Monolog/Handler/ElasticsearchHandlerTest.php b/tests/Monolog/Handler/ElasticsearchHandlerTest.php index 943a67ee..7895332b 100644 --- a/tests/Monolog/Handler/ElasticsearchHandlerTest.php +++ b/tests/Monolog/Handler/ElasticsearchHandlerTest.php @@ -80,7 +80,7 @@ class ElasticsearchHandlerTest extends TestCase ], ], $data, - ] + ], ]; // setup ES client mock @@ -229,7 +229,7 @@ class ElasticsearchHandlerTest extends TestCase /** * Return last created document id from ES response * - * @param array $info Elasticsearch last request info + * @param array $info Elasticsearch last request info * @return string|null */ protected function getCreatedDocId(array $info) @@ -255,7 +255,7 @@ class ElasticsearchHandlerTest extends TestCase $params = [ 'index' => $index, 'type' => $type, - 'id' => $documentId + 'id' => $documentId, ]; $data = $client->get($params); diff --git a/tests/Monolog/Handler/InsightOpsHandlerTest.php b/tests/Monolog/Handler/InsightOpsHandlerTest.php index 209858ae..db6b1e05 100644 --- a/tests/Monolog/Handler/InsightOpsHandlerTest.php +++ b/tests/Monolog/Handler/InsightOpsHandlerTest.php @@ -12,9 +12,9 @@ namespace Monolog\Handler; use Monolog\Test\TestCase; - use Monolog\Logger; +use Monolog\Logger; - /** +/** * @author Robert Kaufmann III * @author Gabriel Machado */ diff --git a/tests/Monolog/Handler/RavenHandlerTest.php b/tests/Monolog/Handler/RavenHandlerTest.php index 5ebfbcff..cff5f543 100644 --- a/tests/Monolog/Handler/RavenHandlerTest.php +++ b/tests/Monolog/Handler/RavenHandlerTest.php @@ -294,7 +294,7 @@ class RavenHandlerTest extends TestCase $this->assertSame('test', $breadcrumbs[1]['category']); $this->assertSame('info', $breadcrumbs[1]['level']); $this->assertSame('Step 2: prepare user redirect', $breadcrumbs[1]['message']); - + $handler->resetBreadcrumbs(); $handler->handle($this->getRecord(Logger::INFO, 'Hello!')); $this->assertEmpty($ravenClient->breadcrumbs->fetch()); diff --git a/tests/Monolog/LoggerTest.php b/tests/Monolog/LoggerTest.php index 7dce79ba..f94882db 100644 --- a/tests/Monolog/LoggerTest.php +++ b/tests/Monolog/LoggerTest.php @@ -666,7 +666,7 @@ class LoggerTest extends \PHPUnit\Framework\TestCase $assertBufferOfBufferHandlerEmpty = function () use ($getProperty, $bufferHandler, $that) { $that->assertEmpty($getProperty($bufferHandler, 'buffer')); }; - $assertBuffersEmpty = function() use ($assertBufferOfBufferHandlerEmpty, $getProperty, $fingersCrossedHandler, $that) { + $assertBuffersEmpty = function () use ($assertBufferOfBufferHandlerEmpty, $getProperty, $fingersCrossedHandler, $that) { $assertBufferOfBufferHandlerEmpty(); $that->assertEmpty($getProperty($fingersCrossedHandler, 'buffer')); }; diff --git a/tests/Monolog/SignalHandlerTest.php b/tests/Monolog/SignalHandlerTest.php index c2c145bb..5169705a 100644 --- a/tests/Monolog/SignalHandlerTest.php +++ b/tests/Monolog/SignalHandlerTest.php @@ -1,4 +1,4 @@ -signalHandlers[$signo] = pcntl_signal_get_handler($signo); } else { @@ -284,5 +284,4 @@ class SignalHandlerTest extends TestCase array(true, true, 1, 1), ); } - }