- Notifications
You must be signed in to change notification settings - Fork 9
Open
Description
Found in declare* Methods.
When using codeception amqp with the bunny implementation array results in an runtime error (because bunny also has a bug) and passing AMQPTable (which works after patching it) results in an PHP Type Error.
Here is the patch I used which works:
Index: src/Codeception/Module/AMQP.php IDEA additional info: Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP <+>UTF-8 =================================================================== diff --git a/src/Codeception/Module/AMQP.php b/src/Codeception/Module/AMQP.php --- a/src/Codeception/Module/AMQP.php +++ b/src/Codeception/Module/AMQP.php (date 1759839586848) @@ -13,6 +13,7 @@ use PhpAmqpLib\Connection\AMQPStreamConnection; use PhpAmqpLib\Exception\AMQPProtocolChannelException; use PhpAmqpLib\Message\AMQPMessage; +use PhpAmqpLib\Wire\AMQPTable; /** * This module interacts with message broker software that implements @@ -166,7 +167,7 @@ bool $auto_delete = true, bool $internal = false, bool $nowait = false, - ?array $arguments = null, + array|AMQPTable|null $arguments = null, ?int $ticket = null ) { return $this->getChannel()->exchange_declare( @@ -203,7 +204,7 @@ bool $exclusive = false, bool $auto_delete = true, bool $nowait = false, - ?array $arguments = null, + array|AMQPTable|null $arguments = null, ?int $ticket = null ): ?array { return $this->getChannel()->queue_declare( @@ -239,7 +240,7 @@ string $exchange, string $routing_key = '', bool $nowait = false, - ?array $arguments = null, + array|AMQPTable|null $arguments = null, ?int $ticket = null ) { return $this->getChannel()->queue_bind( Metadata
Metadata
Assignees
Labels
No labels