Просмотр исходного кода

Merge pull request #706 from acrobat/amqp_handler_fix

Removed substr call in amqp routing key generator
Jordi Boggiano 9 лет назад
Родитель
Сommit
3ce314cc92
2 измененных файлов с 3 добавлено и 8 удалено
  1. 1 6
      src/Monolog/Handler/AmqpHandler.php
  2. 2 2
      tests/Monolog/Handler/AmqpHandlerTest.php

+ 1 - 6
src/Monolog/Handler/AmqpHandler.php

@@ -114,12 +114,7 @@ class AmqpHandler extends AbstractProcessingHandler
      */
     private function getRoutingKey(array $record)
     {
-        $routingKey = sprintf(
-            '%s.%s',
-            // TODO 2.0 remove substr call
-            substr($record['level_name'], 0, 4),
-            $record['channel']
-        );
+        $routingKey = sprintf('%s.%s', $record['level_name'], $record['channel']);
 
         return strtolower($routingKey);
     }

+ 2 - 2
tests/Monolog/Handler/AmqpHandlerTest.php

@@ -61,7 +61,7 @@ class AmqpHandlerTest extends TestCase
                 'channel' => 'test',
                 'extra' => array(),
             ),
-            'warn.test',
+            'warning.test',
             0,
             array(
                 'delivery_mode' => 2,
@@ -111,7 +111,7 @@ class AmqpHandlerTest extends TestCase
                 'extra' => array(),
             ),
             'log',
-            'warn.test',
+            'warning.test',
             false,
             false,
             null,