anyone knows where did `WebProfilerConstants` class go? it wasn't really marked deprecated, but then

UK5DS29L2

anyone knows where did WebProfilerConstants class go? it wasn't really marked deprecated, but then boom and it's gone

Valerii Trots

spryker/webprofiler was removed during your last composer update

Valerii Trots

Had today the same.

Unknown

do we know about the reason? 🤔

Valerii Trots

So either composer require spryker/webprofiler or I don't know.

Valerii Trots

No, @UJN2JRU4F, such question doesn't bother me as I'm not a developer. 😄

Unknown

😜

UK5DS29L2

well thing just wrecked my deploy, hence the question 🙂 it's still referenced in configs

Unknown

i have a project i started on tuesday and it still contains the class. must be very recent

Valerii Trots

I'll ask internally then.

Valerii Trots

@UK5DS29L2 is it suite or b2c\b2b demoshops?

UK5DS29L2

suite

UK5DS29L2

ah

UK5DS29L2

https://github.com/spryker-shop/suite/commit/cf2156f2f196be7d6b641492793f26064e942ae2

Unknown

how did this impact you? 🤔

Valerii Trots

it blocks deployment

Unknown

no i mean, how did he get suite changes into that project?

Valerii Trots

as it throws exception about missing class in configs

UK5DS29L2

composer update?

Valerii Trots

it's not suite changes
it's just composer update

Unknown

@UK5DS29L2 Spryker docker env?

Unknown

how old is the project?

UK5DS29L2

no docker

UK5DS29L2

by "how old" you mean when it was started?

UK5DS29L2

https://github.com/spryker-shop/suite/blob/master/config/Shared/config_default-development.php#L34
considering this is still here, I don't think it matters

UK5DS29L2

well the core problem why it does not install that class is because we use composer.lock (that's why we have it, right?) and unless you do composer update --dev then there's no way to get the require-dev installed

UK5DS29L2

so we can either:

• start keeping two composer locks for each env
• disable webprofiler at all (because I'd rather have code tested on dev on same package set that I want to push on production later)
• stop moving things between require and require-dev without earlier notice about it's deprecation 🙂

UK5DS29L2

temporarily removed WebProfilerConstants from development config to make the deploy work

UK5DS29L2

BUT I believe you should also update
https://github.com/spryker-shop/suite/blob/master/src/Pyz/Zed/Twig/TwigDependencyProvider.php
if it's only going to work in dev environment

UM53ZHLTF

Not a beauty, but atm we are using if (Environment::isDevelopment()) { $providers[] = new WebProfilerTwigLoaderPlugin(); } as a workaround. So on staging production we don't have this dependency, so it won't get loaded

UM53ZHLTF

but be aware its marked as deprecated now.