Assign AllCommand subcommands in the constructor (fix uninitialized typed property on CakePHP 5.4)

[dereuromark]

Fixes Cluster B of cakephp/cakephp#19491.

quite a heavy change though that most likely still leaves other third party commands affected.

Problem

ControllerAllCommand, ModelAllCommand and TemplateAllCommand declare their wrapped subcommand as a non-nullable typed property and assign it in initialize():

protected ControllerCommand $controllerCommand;

public function initialize(): void
{
    parent::initialize();
    $this->controllerCommand = new ControllerCommand();
}

On CakePHP 5.4, BaseCommand::run() builds the option parser before calling initialize(). buildOptionParser() reads the subcommand, which is still uninitialized at that point:

Typed property Bake\Command\ControllerAllCommand::$controllerCommand
must not be accessed before initialization

This surfaced as errors in ControllerAllCommandTest::testExecute and ModelAllCommandTest::testExecute. TemplateAllCommand never reads its subcommand in buildOptionParser(), so its breakage was latent.

Fix

Assign the subcommand in the constructor, so it is available regardless of the run/initialize ordering:

public function __construct(?CommandFactoryInterface $factory = null)
{
    parent::__construct($factory);

    $this->controllerCommand = new ControllerCommand();
}

The initialize() override is dropped; the inherited BakeCommand::initialize() (table-locator fallback) still runs. All three commands are kept consistent.

Tests

Added a testGetOptionParserBeforeInitialize to each command test that calls getOptionParser() directly on a freshly constructed command - the exact pre-initialize() path the runner takes. These reproduce the reported error on the old code and pass with the fix, independent of the installed CakePHP version.