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

Unknown option "employee" #255

Open leup opened 2 years ago

leup commented 2 years ago

PrestaShop version(s)

1.7.

Php version

7.2

Fop Console version

dev

Operating system

linux

command

No response

Describe the bug

Hello everyone,

I may have misused the cli command but I did not succeed to use the employee option as defined here:

https://github.com/friends-of-presta/fop_console/blob/73e57c9b0cea7c356d1e574ad1ed65a7a78731ee/src/Command.php#L41

I tried -em, --em, --employee to no avail.

What I found was that the option seems to be configured too late in the process. The "option not found" was emitted before entering the initialize method.

So I had to move it to configure() as such:

    protected function configure()
    {
        $this->addOption('employee', null, InputOption::VALUE_REQUIRED, 'Specify employee context (id).', null);

        parent::configure();
    }

(I also removed the shortcut because at first I thought it was the issue)

Did I misused the option or is this a real bug ?

leup commented 2 years ago

Anyone ? :)

SebSept commented 2 years ago

@friends-of-presta/fop-console-contributors anyone ?

tom-combet commented 2 years ago

Hi, I can check it next week, not before, sorry.

tom-combet commented 1 year ago

Very sorry for the late answer, I replied here.