cascading dropdown in a repeating group - element doesn't populate when watch is set to default

[skyrun]

i have a std country->region cascading dropdown. i have set the country default to united states (value=US) which works. however the state list doesn't populate by default until i pick something else, then pick us again.

could be related to it being in a repeating group.

you can see it here:
https://testlocation.skyrun.com/index.php?option=com_fabrik&view=form&formid=138&rowid=1428&debug=1

add a new group and see the default united states but no states to pick

 

[cheesegrits]

Doesn't work at all for me, that date_enh.js (which I think is your custom date element) is erroring out during page load, so the CDD JS is never initializing.

I don't know if that's causing the problem you are seeing - probably not, as you said it's working if you select another country, which wouldn't work if you were seeing the same problem I am. But I can't debug it with the date error.

Line 62, "cannot read property value of null", where it does this.getDateField().value at the start of setUp(), which is returning null because the date_time element seems to be readonly, so has no date field.

-- hugh

 

[skyrun]

i'm not getting any console js errors. please try an ignognito/private window or a cache clear. i think you have some old code.

 

[cheesegrits]

The compressed file seems to be OK, but if I enable Debug JS, the uncompressed file errors out:

https://testlocation.skyrun.com/plugins/fabrik_element/date_enh/date_enh.js

... you can see getDateField() at the start of setUp(), about a page down, before the "if (this.options.editable) {".

And I can't debug JS without ... well, Debug JS.

Also, for some reason, &fabrikdebug=1 doesn't work, even if "Allow Debug" is set to "Yes".

-- hugh

 

[skyrun]

fixed.

 

[skyrun]

sorry if i confused. the .js you were seeing when fabrikdebug=1 is fixed so you can now work on the issue.

the issue is still there.

 

[cheesegrits]

Oh, I knew what you meant. And I'm working on the issue locally now. I could have sworn we had code to re-fire the CDD on group duplication, but I apparently not.

I'll update this thread when it's fixed.

-- hugh

 

[cheesegrits]

OK, should be fixed as of this commit:

https://github.com/Fabrik/fabrik/co...6f2bf52#diff-65e8ed9f5e7ff3f97e74353d40ecc705

My memory was correct, we do have a specific mechanism for re-building the CDD options when attached to a new duplicated group. But for reasons unknown, I commented it out a few years ago:

https://github.com/Fabrik/fabrik/co...2d95dfa#diff-65e8ed9f5e7ff3f97e74353d40ecc705

... in a totally unrelated commit. I vaguely remember testing some other issue with CDD's in repeats, can't remember what it was, but it was very corner case. Commenting out that line fixed that problem, but wasn't really a fix, and I think I intended to find a real fix but obviously got distracted and forgot, and that temporary testing change got committed with something else. So I've probably re-introduced some other subtle issue with CDD's in repeats, but for now, we should be good.

-- hugh