hey folks, short general question: On the docs page for the WebProfiler module[

U010DNAA3QW

hey folks, short general question: On the docs page for the WebProfiler module[1], it says:

composer require spryker/web-profiler --dev

and:

To be able to use Web Profiler, add \Spryker\Zed\WebProfiler\Communication\Plugin\Application\WebProfilerApplicationPluginof thespryker-shop/web-profiler-widgetmodule to \Pyz\Zed\Application\ApplicationDependencyProvider::getApplicationPlugins().

So, the package is being installed with --dev, but it should be added normally to the ApplicationPlugins... Now my question: Does this mean we always should deploy Spryker with composer install --dev also on production systems? Because else the getApplicationPlugins() will contain a fatal error (class not found). Sounds a bit strange to me. How would I solve this?

U013EK8RD7A

IMHO environment condition or plugin existence check (simple class exists I guess) should be added

schoenfeld

composer install --dev should not be executed on prod.
Maybe this is helping as a reference https://github.com/spryker-shop/suite/blob/master/src/Pyz/Yves/ShopApplication/ShopApplicationDependencyProvider.php#L319

U013EK8RD7A

instead of

it should be something like

 $pluginStack = [
     // standard plugins
 ];

 if (profiler exists) {
    $pluginStack[] = new ProfilerPugin()
 }

 return $pluginStack;

U010DNAA3QW

🤯

U010DNAA3QW

sometimes it can be this easy 😄

U010DNAA3QW

thanks guys!

U010DNAA3QW

maybe the Documentation should be updated with this small idea, just a small suggestion 🙂