Skip to content

Commit 2852785

Browse files
authored
Merge pull request #1 from andrew-demb/patch-1
Fix return type for Connection::connect()
2 parents 484734f + 13cf9fd commit 2852785

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/Decorator/JaegerConnectionDecorator.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,10 @@ public function __construct(Connection $connection, TracerInterface $tracer)
2626
parent::__construct($connection);
2727
}
2828

29-
public function connect()
29+
public function connect(): bool
3030
{
3131
if ($this->isConnected()) {
32-
return;
32+
return false;
3333
}
3434
$span = $this->tracer
3535
->start('dbal.connect')
@@ -38,7 +38,7 @@ public function connect()
3838
->addTag(new DbalAutoCommitTag($this->isAutoCommit()))
3939
->addTag(new DbalNestingLevelTag($this->getTransactionNestingLevel()));
4040
try {
41-
parent::connect();
41+
return parent::connect();
4242
} catch (\Exception $e) {
4343
$span->addTag(new DbalErrorCodeTag($e->getCode()))
4444
->addTag(new ErrorTag());

0 commit comments

Comments
 (0)