Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix autofilling when using multiple autofill fields #7487

Closed
wants to merge 5 commits into from

Conversation

hluchas
Copy link
Contributor

@hluchas hluchas commented Apr 30, 2019

Please be sure you are submitting this against the staging branch.

Q A
Bug fix? Y
New feature?
Automated tests included? Y
Related user documentation PR URL
Related developer documentation PR URL
Issues addressed (#s or URLs)
BC breaks?
Deprecations?

Description: When using multiple checkbox groups or selects, only first one is autofilled.

Steps to reproduce the bug:

  1. Create multiple Boolean or Select custom fields
  2. Create form with checkbox groups mapping to those custom fields, using "Use assigned contact/company field's list choices. = Yes" in the Properties Tab
  3. Add form to a landing page, then submit form with at least one of those checkboxes checked
  4. Refresh landing page that the form is on

Steps to test this PR:

  1. Load up this PR
  2. Use reproduction steps

@escopecz escopecz added bug Issues or PR's relating to bugs ready-to-test PR's that are ready to test labels May 1, 2019
@npracht npracht added this to the 2.15.2 milestone May 2, 2019
@escopecz
Copy link
Sponsor Member

escopecz commented May 7, 2019

@kuzmany @hluchas is this duplicate of #7347?

@kuzmany
Copy link
Member

kuzmany commented May 7, 2019

@escopecz yes it is, our #7347 fix radio + checkbox + some boolean unexpected autofill behaviour with it

@hluchas
Copy link
Contributor Author

hluchas commented May 7, 2019

Funny, but OK. I would like to merge my PR naturally and because I've better division for alias _'.$alias.'_'., but @kuzmany has more fixed in it.

I don't know.

@kuzmany kuzmany added pending-test-confirmation PR's that require one test before they can be merged and removed ready-to-test PR's that are ready to test labels May 12, 2019
@kuzmany kuzmany added duplicate and removed pending-test-confirmation PR's that require one test before they can be merged labels May 19, 2019
@kuzmany
Copy link
Member

kuzmany commented May 19, 2019

We're going to close 2.15.2 release, then I decide to close this PR favor to #7347
Better regex division could be submitted in another PR.

@kuzmany kuzmany closed this May 19, 2019
@escopecz escopecz removed this from the 2.15.2 milestone May 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues or PR's relating to bugs duplicate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants