fix: avoid session flash during console install

[boboldehampsink]

Summary

• avoid starting a web session from the plugin install handler during console installs
• log the Pro edition requirement as a warning for console requests while keeping the existing CP flash for web requests

Context

When Craft installs plugins from project config in a console command, migration output may already have been sent before EVENT_AFTER_INSTALL_PLUGIN fires. Calling Craft::$app->session->setError() in that path can trigger session_name(): Session name cannot be changed after headers have already been sent and abort installation.

[jaymeh]

Hi @boboldehampsink ,

Thanks for your contribution here, I can't see any issues with the code you have committed. The only thing to note is that we use a Conventional Commits workflow, tied to our release cycle. We therefore require any PR's/Commits to be tacked with the relevant structure to build our changelog.

In your case just putting fix: in front of your commit message and PR would satisfy this requirement and we can get things merged.

You can find more about conventional commits here: https://www.conventionalcommits.org/en/v1.0.0/

[jaymeh]

Thanks again @boboldehampsink. This code is now published and available in release v4.3.3.