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 batch contact/activity update with radio options #16855

Merged
merged 1 commit into from
Mar 19, 2020

Conversation

ray-wright
Copy link
Contributor

To account for radio option values that contain a space, wrap the value name in quotes.

Overview

When updating multiple contacts, activities, etc the 'copy value from first row' is problematic with radio options.

Before

If the column is a radio field and the value in the first row contains a space, they copy from first row errors and does fails to set the value down the list. Console error shows:

Uncaught Error: Syntax error, unrecognized expression: [value=Value with space]

After

With the additional quotes around the value, you can copy the radio value from the first row and apply to the entire list.

Comments

This only affected the radio values, other field type values with spaces updated without issues.

Wrap radio value in single quotes to account for options with spaces.
@civibot
Copy link

civibot bot commented Mar 19, 2020

(Standard links)

@civibot civibot bot added the master label Mar 19, 2020
@colemanw
Copy link
Member

Thanks for the fix!

@colemanw
Copy link
Member

Unrelated test fail

@colemanw colemanw merged commit 53aee42 into civicrm:master Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants