Skip to content

Catch invalid bind and visibility rules#1128

Merged
ryancooley merged 2 commits into4.1-developfrom
bugfix/FOUR-4849
Dec 17, 2021
Merged

Catch invalid bind and visibility rules#1128
ryancooley merged 2 commits into4.1-developfrom
bugfix/FOUR-4849

Conversation

@caleeli
Copy link
Copy Markdown
Collaborator

@caleeli caleeli commented Dec 15, 2021

Fix: https://processmaker.atlassian.net/browse/FOUR-4849

Issue:

  • Exception Cannot read properties of null on visibility rules, breaks all the rules
  • Bind control to property of null, break all the screen.

Fix:

  • Try/Catch visibility rules for each rule
  • Add a computed property (getter/setter) to proxy the field names

How to test

  • Import the attached screen
  • Select/Unselect the person Select List
  • When person is not selected (value=null) it breaks the screen

FOUR-4849.zip

@ryancooley ryancooley merged commit 03940a2 into 4.1-develop Dec 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants