Fix #136 -- Support dependent fields in formsets (#316)

mardukbp · web-flow · commit f78152762510 · 2025-03-15T17:50:37.000+01:00
* Find the closest common ancestor of two chained selects instead of assuming it is a form. * Try to find the dependent field by its name attribute using a two-fold search strategy: exact match otherwise ends with Special thanks to @rez0n
diff --git a/django_select2/static/django_select2/django_select2.js b/django_select2/static/django_select2/django_select2.js
@@ -26,9 +26,16 @@
 
 let dependentFields = $element.data('select2-dependent-fields')
 if (dependentFields){
+ const findElement = function (selector){
+ const result = $(selector, $element.closest(`:has(${selector})`))
+ if (result.length > 0) return result
+ else return null
+ }
 dependentFields = dependentFields.trim().split(/\s+/)
 $.each(dependentFields, function (i, dependentField){
- result[dependentField] = $('[name=' + dependentField + ']', $element.closest('form')).val()
+ const nameIs = `[name=${dependentField}]`
+ const nameEndsWith = `[name$=-${dependentField}]`
+ result[dependentField] = (findElement(nameIs) || findElement(nameEndsWith)).val()
 })
 }
 

-Original file line number
+Diff line change
 letdependentFields=$element.data('select2-dependent-fields')
 if(dependentFields){
 +constfindElement=function(selector){
 +constresult=$(selector,$element.closest(`:has(${selector})`))
 +if(result.length>0)returnresult
 +elsereturnnull
 +}
 dependentFields=dependentFields.trim().split(/\s+/)
 $.each(dependentFields,function(i,dependentField){
 -result[dependentField]=$('[name='+dependentField+']',$element.closest('form')).val()
 +constnameIs=`[name=${dependentField}]`
 +constnameEndsWith=`[name$=-${dependentField}]`
 +result[dependentField]=(findElement(nameIs)||findElement(nameEndsWith)).val()
 })
+}