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

get county fields populated via api call #20309

Merged
merged 1 commit into from
May 17, 2021

Conversation

jmcclelland
Copy link
Contributor

Ensure that county fields are properly populated when
calling the Profile.getfields api.

Overview

The Profile.getfields with api_action=submit API helps re-use CiviCRM profiles in other contexts by providing details on all the fields in the profile. Unfortunately, county is left out! This change ensures that the county field is included.

Before

When calling the API: Profile.getfields api_action=submit get_options=all profile_id=NNN on a profile that includes the county field, the county field is not fully populated.

After

After this change, the county field is populated.

Comments

This is a tiny change that I hope won't have any nasty repercussions. I know that api3 is being left in the dust, but this teeny fix really helps the remoteform extension live a little longer.

Ensure that county fields are properly populated when
calling the Profile.getfields api.
@civibot
Copy link

civibot bot commented May 14, 2021

(Standard links)

@civibot civibot bot added the master label May 14, 2021
jmcclelland added a commit to progressivetech/net.ourpowerbase.remoteform that referenced this pull request May 14, 2021
Note: this depends on this change to core being merged:

civicrm/civicrm-core#20309
@eileenmcnaughton
Copy link
Contributor

@jmcclelland so I guess this is OK - my only worry would be performance but that would only be an issue if county is in the profile?

@jmcclelland
Copy link
Contributor Author

@eileenmcnaughton Yes, that's my impression. If county is not in the profile, it should have no effect and no performance hit. The only surprise should be for people who had county in their profile and figured out some work around to make it work with this API call. But, I suspect that will be just about nobody.

@eileenmcnaughton
Copy link
Contributor

OK

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants