Just I was testing this module, it's not working for the text fields which set "unlimited" under "Allowed number of values" in field setting.

Also, not easy to configure for all the fields in website. We should provide for admins to configure in easy way, because instead of asking field machine name, it's required field id which is actually dynamic one.

Comments

reddyweb created an issue. See original summary.

goyalsushant’s picture

Assigned: Unassigned » goyalsushant
izmeez’s picture

Agreed. The settings are kinda hidden, not a proper admin interface and issues with various input fields, eg search input field works with barik theme but not others.

goyalsushant’s picture

The issue for unlimited fields is their dynamic IDs. It can be handled using regex for defining field ID.

Attached is the patch for fixing the same.
Also, as mentioned by @izmeez I have tested it out on various fields and themes and it's working now.
Kindly check.

goyalsushant’s picture

Assigned: goyalsushant » Unassigned
Status: Active » Needs review
Issue tags: -bug +fixed
izmeez’s picture

I am removing the issue tag "fixed" as it is not committed to the module and that tag makes things confusing.

There is a lot of overlap between this patch and the patch in #3201151: Input Reset Does not work with ajax. Each patch takes a different approach and the patch in this issue addresses the extra issue of multi-valued fields. I am not sure if this patch should be broken into two with one addressing the ajax issue and the other the multi-valued fields. Both issues will be marked as related for now.

I have only done some limited testing with this patch and the one in the other issue.

rajveergangwar’s picture

Hi Everyone,

Can someone help patch review and mark it and working fine. So i can commit it to module