diff --git a/src/Symfony/Component/Notifier/Bridge/Slack/Block/SlackActionsBlock.php b/src/Symfony/Component/Notifier/Bridge/Slack/Block/SlackActionsBlock.php index 25da853677415..848b9bfd98f83 100644 --- a/src/Symfony/Component/Notifier/Bridge/Slack/Block/SlackActionsBlock.php +++ b/src/Symfony/Component/Notifier/Bridge/Slack/Block/SlackActionsBlock.php @@ -24,16 +24,26 @@ public function __construct() /** * @return $this */ - public function button(string $text, string $url, ?string $style = null): static + public function button(string $text, ?string $url = null, ?string $style = null, ?string $value = null): static { if (25 === \count($this->options['elements'] ?? [])) { throw new \LogicException('Maximum number of buttons should not exceed 25.'); } - $element = new SlackButtonBlockElement($text, $url, $style); + $element = new SlackButtonBlockElement($text, $url, $style, $value); $this->options['elements'][] = $element->toArray(); return $this; } + + /** + * @return $this + */ + public function id(string $id): static + { + $this->options['block_id'] = $id; + + return $this; + } } diff --git a/src/Symfony/Component/Notifier/Bridge/Slack/Block/SlackButtonBlockElement.php b/src/Symfony/Component/Notifier/Bridge/Slack/Block/SlackButtonBlockElement.php index ff83bb9d870a5..8b1eed19472cb 100644 --- a/src/Symfony/Component/Notifier/Bridge/Slack/Block/SlackButtonBlockElement.php +++ b/src/Symfony/Component/Notifier/Bridge/Slack/Block/SlackButtonBlockElement.php @@ -16,7 +16,7 @@ */ final class SlackButtonBlockElement extends AbstractSlackBlockElement { - public function __construct(string $text, string $url, ?string $style = null) + public function __construct(string $text, ?string $url = null, ?string $style = null, ?string $value = null) { $this->options = [ 'type' => 'button', @@ -24,12 +24,19 @@ public function __construct(string $text, string $url, ?string $style = null) 'type' => 'plain_text', 'text' => $text, ], - 'url' => $url, ]; + if ($url) { + $this->options['url'] = $url; + } + if ($style) { // primary or danger $this->options['style'] = $style; } + + if ($value) { + $this->options['value'] = $value; + } } } diff --git a/src/Symfony/Component/Notifier/Bridge/Slack/Tests/Block/SlackActionsBlockTest.php b/src/Symfony/Component/Notifier/Bridge/Slack/Tests/Block/SlackActionsBlockTest.php index 2a21a39133c1f..682a895bdffc0 100644 --- a/src/Symfony/Component/Notifier/Bridge/Slack/Tests/Block/SlackActionsBlockTest.php +++ b/src/Symfony/Component/Notifier/Bridge/Slack/Tests/Block/SlackActionsBlockTest.php @@ -19,8 +19,9 @@ final class SlackActionsBlockTest extends TestCase public function testCanBeInstantiated() { $actions = new SlackActionsBlock(); - $actions->button('first button text', 'https://example.org') + $actions->button('first button text', 'https://example.org', null, 'test-value') ->button('second button text', 'https://example.org/slack', 'danger') + ->button('third button text', null, null, 'test-value-3') ; $this->assertSame([ @@ -33,6 +34,7 @@ public function testCanBeInstantiated() 'text' => 'first button text', ], 'url' => 'https://example.org', + 'value' => 'test-value' ], [ 'type' => 'button', @@ -43,6 +45,14 @@ public function testCanBeInstantiated() 'url' => 'https://example.org/slack', 'style' => 'danger', ], + [ + 'type' => 'button', + 'text' => [ + 'type' => 'plain_text', + 'text' => 'third button text', + ], + 'value' => 'test-value-3', + ] ], ], $actions->toArray()); }