Skip to content

Commit 1afc6ac

Browse files
authored
Merge pull request #209 from melnikoved/fix-slackbot-handler
bug #208 Slackbot handler does not work
2 parents 1b39941 + 32de375 commit 1afc6ac

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

DependencyInjection/Configuration.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -380,7 +380,7 @@ public function getConfigTreeBuilder()
380380
->scalarNode('include_extra')->defaultFalse()->end() // slack & slackwebhook
381381
->scalarNode('icon_emoji')->defaultNull()->end() // slack & slackwebhook
382382
->scalarNode('webhook_url')->end() // slackwebhook
383-
->scalarNode('slack_team')->end() // slackbot
383+
->scalarNode('team')->end() // slackbot
384384
->scalarNode('notify')->defaultFalse()->end() // hipchat
385385
->scalarNode('nickname')->defaultValue('Monolog')->end() // hipchat
386386
->scalarNode('token')->end() // pushover & hipchat & loggly & logentries & flowdock & rollbar & slack & slackbot
@@ -722,8 +722,8 @@ public function getConfigTreeBuilder()
722722
->thenInvalid('The webhook_url have to be specified to use a SlackWebhookHandler')
723723
->end()
724724
->validate()
725-
->ifTrue(function ($v) { return 'slackbot' === $v['type'] && (empty($v['stack_team']) || empty($v['token']) || empty($v['channel'])); })
726-
->thenInvalid('The stack_team, token and channel have to be specified to use a SlackbotHandler')
725+
->ifTrue(function ($v) { return 'slackbot' === $v['type'] && (empty($v['team']) || empty($v['token']) || empty($v['channel'])); })
726+
->thenInvalid('The team, token and channel have to be specified to use a SlackbotHandler')
727727
->end()
728728
->validate()
729729
->ifTrue(function ($v) { return 'cube' === $v['type'] && empty($v['url']); })

DependencyInjection/MonologExtension.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -549,9 +549,9 @@ private function buildHandler(ContainerBuilder $container, $name, array $handler
549549

550550
case 'slackbot':
551551
$definition->setArguments(array(
552-
$handler['slack_team'],
552+
$handler['team'],
553553
$handler['token'],
554-
$handler['channel'],
554+
urlencode($handler['channel']),
555555
$handler['level'],
556556
$handler['bubble'],
557557
));

0 commit comments

Comments
 (0)