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

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

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.

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.
Owner

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.

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.
Author

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

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

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)
Owner

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.

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.
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: joomla/Component-Builder#78