Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(jexl): don't consider the value of hidden fields in JEXL expr. #198

Merged
merged 1 commit into from
May 6, 2019
Merged

fix(jexl): don't consider the value of hidden fields in JEXL expr. #198

merged 1 commit into from
May 6, 2019

Commits on May 4, 2019

  1. fix(jexl): don't consider the value of hidden fields in JEXL expr. 

    This fixes some subtle issue that occur when once field depends on
multiple other fields:

1) The field's JEXL expression only doesn't have to be calculated, if
_all_ fields it depends on are hidden (not just one!)

2) If some of the fields it depends on are hidden, those fields should
behave as if they were unanswered, even if the have an answer in the
database.
    @czosel
    czosel committed May 4, 2019
    Configuration menu
    Copy the full SHA
    9c9350d View commit details
    Browse the repository at this point in the history