Duplicate command fop:shop:status

friends-of-presta / fop_console

Prestashop Module providing a set of shell/terminal commands for developers (PrestaShop 1.7.5+)

Academic Free License v3.0

84 stars 35 forks source link

Duplicate command fop:shop:status #254

Open Grezor opened 2 years ago

Grezor commented 2 years ago

PrestaShop version(s)

1.7.8.6

Php version

7.4

Fop Console version

dev

Operating system

windows

command

fop:shop:status

Describe the bug

Hello, This may not be a bug, just an information

when you type: php bin/console list fop

There are two times the command :

fop:shop-status Display shops statuses
fop:shop:status [fop:shop-status] Display shops statuses

see attached image

regards, Geoffrey / Grezor

nenes25 commented 2 years ago

Hello @Grezor,

It's not a bug, but a feature related with the alias of the commands. It's also the case for some others. The real question behind this issue is if this alias are still needed.

WDYT ping @jf-viguier @SebSept @tom-combet

Grezor commented 2 years ago

Hello @nenes25

Thanks for the feedback. Yes I had seen that some have other aliases. If you want I can delete the command: fop:shop-status.

to keep the same uniform => fop:shop:status @jf-viguier @SebSept @tom-combet

tom-combet commented 2 years ago

As @SebSept said, it's actually a good practice to keep backward compatibility in same major version.

Is it time to release a new major? I don't think so, maybe we can wait PS 8.0?