Additional "issue", re: Issue #75 #78

New Issue

Closed

opened 2017-05-06 18:47:00 +00:00 by mwweb · 4 comments

mwweb commented

2017-05-06 18:47:00 +00:00

(Migrated from github.com)

In moving through a few more parts, since the upgrade to Joomla! 3.7.0, it looks like another multi-select box that "broke" is on the admin views -> conditions. The Match isn't loading with different options as it used to. I haven't checked anywhere else as yet to see if there are any other issues.

Llewellyn commented

2017-05-07 11:16:09 +00:00

Okay, thanks. Joomla made a change in the id structure of each field in a repeatable field. Painful when they do these things, but okay I fixed that and lets see if there is more before I push the update.

mwweb commented

2017-05-07 15:50:32 +00:00

(Migrated from github.com)

Since joomla made a change like that, is there something that jcb users need to change in repeatable fields that they may use?

Llewellyn commented

2017-05-07 15:59:53 +00:00

Only if they attached JavaScript to the ids of each repeatable field. What changed?

id="jform_fields_field_name-1" became id="1-jform_fields_field_name"

So they moved the counter of the field.

Everything JCB builds still works, but if you added custom JavaScript to the repeatable fields you may also have this problem.

Only if they attached JavaScript to the ids of each repeatable field. What changed? `id="jform_fields_field_name-1"` became `id="1-jform_fields_field_name"` So they moved the counter of the field. Everything JCB builds still works, but if you added custom JavaScript to the repeatable fields you may also have this problem.

ro-ot commented

2017-05-07 16:29:38 +00:00

(Migrated from github.com)

They made the change so you could move the fields around and so change the order of the fields. So it was needed and could not be avoided. Repeatable fields should be treated as beta as you all know I suppose and it still has many bugs, or so I think. So we should keep an eyes on this.