BUG : checkboxes sub-elements come by twos

[jfquestiaux]

There seems to be a bug with the latest GitHub (187 as of dec 10) with the checkboxes element : all the sub-elements come by pairs and you can't delete the second one, so if you have two checkboxes, you are fine but if you need one or three, you can't save the element since you can't have empty sub-elements.

It is actually harder to describe than to show, so here is a screencast to show the problem : http://screencast.com/t/eHPKvXaEIbR

 

[rob]

yup - seems that the js was being fired twice, I've fixed that and also tidied up the presentation of the subptions

 

[jfquestiaux]

Yes the "doubling" is fixed and the new presentation is great.
Unforunately, there seems to be a JS error that crept in :

Erreur : ReferenceError: FbCheckBox is not defined
Fichier Source : http://www.betterliving.be/j3/index.php?option=com_fabrik&view=form&formid=5&Itemid=135
Ligne : 97

I had this error also with a Yes/No element on this form - deleted it gave me this error on the checkbox element

 

[cheesegrits]

Rob is working on this one now, as per our group Skype convo. Something to do with require.js and nested requires.

-- hugh

 

[jfquestiaux]

Well, after today's Git update (226), here is the situation :

With the browser cache cleared, if I come from a regular J! page (an article for instance), there is no error.
If I come from a Fabrik page (a page with a list or a form for instance), the error is back. Emptying the browser cache makes it go away though (before the last fix, that was not the case).

 

[jfquestiaux]

After update to 254, the error is still there.

 

[cheesegrits]

Can you point me at one of the pages exhibiting this problem?

-- hugh

 

[jfquestiaux]

The "doubling" of sub-elements is back after GitHub 273.

 

[jfquestiaux]

This seems to be fixed in GitHub 277.