Can outdated command be forwarded as well?

[hostep]

Hi there

Thank you very much for this wonderful composer plugin, very happy with it so far!

I've noticed with the forward-command config, that only composer install and composer update are being forwarded to the bin directories.
But I sometimes like to run composer outdated as well against everything, and right now I need to do this one by one, which is a bit annoying.
Would be nice if this command can also be forwarded.

Thanks for considering this! 🙂

[theofidry]

That's a good idea. PRs are welcomed! I don't think it requires much more than updating

composer-bin-plugin/src/BamarniBinPlugin.php

Line 36 in 0f2f6f5

private const FORWARDED_COMMANDS = ['install', 'update'];

[hostep]

Ah cool, good to know. I'll see if I can find some time in the next few days/weeks to test this out and send a PR.
Thanks for the info!

[hostep]

Just gave it a little try, it's unfortunately a bit more complex. Because composer outdated is more of an alias to composer show --outdated

So adding outdated to the FORWARDED_COMMANDS const does not work. When I add show to it, it does not work with composer outdated, but does work with composer show --outdated.
And I can see that composer bin all outdated also does not work.

I haven't debugged further since I lack the time at the moment.

What do you think, should we try to make composer outdated also working somehow? It would be convenient in my opinion.

[theofidry]

I think we could adapt this condition:

composer-bin-plugin/src/BamarniBinPlugin.php

Line 134 in 0f2f6f5

&& in_array($commandName, self::FORWARDED_COMMANDS, true)

Instead of just checking the in_array we could check if it's the show command with the outdated flag for example. The check could also be extracted into a dedicated method to keep it understandable

[LastDragon-ru]

And I can see that composer bin all outdated also does not work.

Yep, for me it checks the first package only :(