tonyair/silverstripe-userforms
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-userforms.git synced 2024-10-22 17:05:42 +02:00
Code Issues Projects Releases Wiki Activity

Fix logic for radio buttons

Browse Source 
parents only works if you have a single radio group with conditionals.
closest is what we want.
This commit is contained in:
Derek Kaijser 2017-03-02 11:58:39 +13:00 committed by GitHub
parent e0d9399654
commit 3d8f5d1406
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
code/model/UserDefinedForm.php
View File

@ -517,7 +517,7 @@ class UserDefinedForm_Controller extends Page_Controller
$expression = '$(this).prop("checked")'; $expression = '$(this).prop("checked")';
} elseif ($radioField) { } elseif ($radioField) {
// We cannot simply get the value of the radio group, we need to find the checked option first. // We cannot simply get the value of the radio group, we need to find the checked option first.
$expression = '$(this).parents(".field, .control-group").find("input:checked").val()=="'. $rule->FieldValue .'"'; $expression = '$(this).closest(".field, .control-group").find("input:checked").val()=="'. $rule->FieldValue .'"';
} else { } else {
$expression = '$(this).val() == "'. $rule->FieldValue .'"'; $expression = '$(this).val() == "'. $rule->FieldValue .'"';
} }