Closed (fixed)
Project:
Conditional Fields
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
6 Jul 2017 at 19:31 UTC
Updated:
24 Jul 2018 at 21:55 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
cslevy commentedHere is my patch which fixes the issue.
Comment #3
cslevy commentedComment #4
vitalyos commentedhi, doesn't work for me.
whenewer checkbox is checked or not, target field is always required.

Comment #5
geek-merlinI'd say #4 shows the WS is resolved, so the patch fixes the issue but maybe leaves followup issues.
Patch needs reroll.
Rerolled patch flying in.
Comment #6
geek-merlinTested the patch on a separat box and it fixes the WS. I can not reproduce #4, i get a validation message exactly when it should.
Comment #7
geek-merlinComment #9
geek-merlinTested and used in production, so committing.