? ?
avatar Fedik
Fedik
22 Aug 2015

Current Repeatable field have problem with storage format and force to use JSON ... also very tricky in logic :smile:

In my own work I use my own repeatable field, that have more clear logic to store the data, like:

array(
  array(/*row 1 values*/),
  array(/*row 2 values*/),
  ...
  array(/*row N values*/)
)

Field values is sent to the server respectively to the fields hierarchy (groups) , and do not use hidden field with JSON value (as existing repeatable field). This is allow to handle the "repeatable value" as any other value from the form, in backend Model.

Also it do not use that tricky modal popup, and allow to use different layouts (table or simple div elements) to render the repeatable form. Only cons (but not really cons), it require external XML for subform.

So, question, whether will be interesting to anyone if I try adapt it to Joomla core (for 3.5)?
Then we could deprecate existing Repeatable field.

otherwise just close this issue :smile:

some screenshots:
table:
screen 2015-08-22 17 51 05 1189x224
screen 2015-08-22 17 54 57 1246x174
div container:
screen 2015-08-22 17 52 52 1216x404

avatar Fedik Fedik - open - 22 Aug 2015
avatar zero-24 zero-24 - change - 23 Aug 2015
Labels Added: ? ?
avatar zero-24 zero-24 - change - 23 Aug 2015
Category JavaScript
avatar uzielweb
uzielweb - comment - 29 Aug 2015

I support this change. The actual repeatable is crappy.

avatar peterlose
peterlose - comment - 29 Aug 2015

This change would be awesome. I use the repeatable field quite a lot, and it really does suck :)

avatar dgt41
dgt41 - comment - 29 Aug 2015

:+1: from me as well

avatar roland-d
roland-d - comment - 29 Aug 2015

@wilsonge Any thoughts?

avatar wilsonge
wilsonge - comment - 29 Aug 2015

:+1: to rework

avatar Fedik
Fedik - comment - 29 Aug 2015

very well, then I will prepare the pull request soon

avatar Fedik
Fedik - comment - 6 Sep 2015

So there it is #7829
Please test.

avatar Fedik Fedik - change - 6 Sep 2015
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-09-06 11:37:08
Closed_By Fedik
avatar Fedik Fedik - close - 6 Sep 2015
avatar Fedik Fedik - close - 6 Sep 2015

Add a Comment

Login with GitHub to post a comment