Fix work with queue arguments in queue_declare

[manakao]

To work with queue arguments, we need to wrap arguments by \PhpAmqpLib\Wire\AMQPTable.
Exception will be:

PhpAmqpLib\Exception\AMQPOutOfRangeException

File:
vendor/php-amqplib/php-amqplib/PhpAmqpLib/Wire/AMQPAbstractCollection.php:417
Message:
AMQP-rabbit doesn't define data of type []

[thomasvargiu]

Thanks @manakao.
In the master branch the minimum required version of php-amqplib is ^2.7 and I could merge it, but I think it's not a problem of this library.

As you can see here the php-amplib library should handle it.

Can you provide some examples where you have errors?
Can you write a test for it?

[thomasvargiu]

Small changes

[manakao]

Can you provide some examples where you have errors?

When i want to use custom table options such as

'queue'                     => [
    'name'      => 'notification_delay_60',
    'arguments' => [
        'x-dead-letter-exchange' => 'notification',
        'x-message-ttl' => 60000, // 60 * 1000 ms
        'x-expires' => 70000,  // 70 * 1000 ms
    ],
],

[thomasvargiu]

@manakao can you write a test? Do you need help for it?

[manakao]

@manakao can you write a test? Do you need help for it?

Yes, i write TestSetupFabric for BaseAmqp

[thomasvargiu]

Nice work! Can you resolve my note? Then I'll merge it and tag a new release.

[thomasvargiu]

It's not a test, and I think we can remove this method because I don't see any usage.

[manakao]

It's not a test, and I think we can remove this method because I don't see any usage.

Are you mean testSetChannel or getMethod ? If last it is already deleted.

[thomasvargiu]

You're right! My mistake! Thanks.

[thomasvargiu]

Thank you @manakao. I tagged version 3.0.1

[thomasvargiu]

@manakao I'm having problems with these changes. Are you using it?

[thomasvargiu]

Reverted in 3.0.2