Calling Developers!
We are reenergizing our code contribution process! Learn More

What are the Slack Archives?

It’s a history of our time together in the Slack Community! There’s a ton of knowledge in here, so feel free to search through the archives for a possible answer to your question.

Because this space is not active, you won’t be able to create a new post or comment here. If you have a question or want to start a discussion about something, head over to our categories and pick one to post in! You can always refer back to a post from Slack Archives if needed; just copy the link to use it as a reference..

just noticed that the Twig option `strict_variables` is implicitly disabled when debug mode is disab

Options
UKGT7RC7P
UKGT7RC7P Posts: 538 🧑🏻‍🚀 - Cadet
edited June 2020 in Slack General

just noticed that the Twig option strict_variables is implicitly disabled when debug mode is disabled. this means that any error in Twig error is silently ignored on production by default. In our case this meant that a payment option stopped working with no apparent reason or log message.

In general do people use strict_variables in production? To me it seems this is the better choice.

Comments

  • Unknown
    Options

    @UUZSG90UR recently had this issue

  • Unknown
    Options

    and can we continue this in ?

  • UKGT7RC7P
    UKGT7RC7P Posts: 538 🧑🏻‍🚀 - Cadet
    Options

    it's not really help. the solution is to configure strict_variables in the YVES_TWIG_OPTIONS if you want that

  • UKGT7RC7P
    UKGT7RC7P Posts: 538 🧑🏻‍🚀 - Cadet
    Options

    (you can override it)

  • UKGT7RC7P
    UKGT7RC7P Posts: 538 🧑🏻‍🚀 - Cadet
    Options

    just wondering if this should be the default behavior

  • UKGT7RC7P
    UKGT7RC7P Posts: 538 🧑🏻‍🚀 - Cadet
    Options

    I see this alot (not just in Spryker)

  • UKGT7RC7P
    UKGT7RC7P Posts: 538 🧑🏻‍🚀 - Cadet
    Options

    but ultimately it means that while you "might get away with it" you equally, may not. 😄

  • Unknown
    Options

    i agree

  • Unknown
    Options

    i mean, it should not make things fail. most of the time, at least

  • Unknown
    Options

    but should still be logged somehow

  • UKGT7RC7P
    UKGT7RC7P Posts: 538 🧑🏻‍🚀 - Cadet
    edited June 2020
    Options

    not sure, i think it should fail loudly and that this should be avoided by being defensive (and by testing the site frequently)