Purge command broken

[innologi]

Hi,

The rabbitmq purge command is broken in v2.7.0 due to this change:
590e531#diff-b5ba74a19cc6229de08f6be0aba8420e579b6d0a2cd1433bfdc98bf18221118fR53-R60

The purge command inherits this from BaseConsumerCommand, but purge does not have a messages option, so it fails on the getOption() call from the change:

$ bin/console rabbitmq:purge default_consumer
13:02:04 CRITICAL [console] Error thrown while running command "rabbitmq:purge default_consumer". Message: "The "messages" option does not exist." ["exception" => Symfony\Component\Console\Exception\InvalidArgumentException
{ …}

,"command" => "rabbitmq:purge default_consumer","message" => "The "messages" option does not exist."]

The "messages" option does not exist.

Downgrading to v2.6.0 resolves the issue.

[mihaileu]

@andreea-anamaria do you have time for looking intro ?

[mihaileu]

fixed in #650

[michaelphillpotts]

Hi

For this fix in #650 is it necessary to include ext-pcntl in composer.json?

As I understand pcntl is not available on Windows platform and prevents installation.
But v2.7.1 does not require ext-pcntl which is what I am using as a work around.

[alister]

is it necessary to include ext-pcntl in composer.json?

This also stops me from upgrading. We don't have pcntl installed, and don't need it. The extension's use is entirely optional within the code itself.

[mihaileu]

@alister @michaelphillpotts please come with a PR
@Eloar @ramunasd do you see any issues if we put it as suggested package instead of required one ?

[Eloar]

@mihaileu @innologi this issue might be closed