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.

Sign up for GitHub

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

Jump to bottom

New contact affiliation metadata in Search API includes parentheses #6570

Closed
jggautier opened this issue Jan 24, 2020 · 2 comments · Fixed by #6619
Closed

New contact affiliation metadata in Search API includes parentheses #6570

jggautier opened this issue Jan 24, 2020 · 2 comments · Fixed by #6619
Labels
Feature: API

Comments

@jggautier
Copy link
Contributor

In Dataverse 4.19, when Search API results include datasets with Contact Affiliation metadata, the affiliation value is wrapped in parenthesis:

Screen Shot 2020-01-23 at 8 23 58 PM

Was this by design? It reminds me of #5521
@pdurbin
Copy link
Member

pdurbin commented Jan 24, 2020

It's funny, nothing jumps out at me in pull request #6441 about why the parentheses would be added. It's definitely worth investigating further though. Very strange.

@djbrooke
Copy link
Contributor

djbrooke commented Feb 5, 2020

  • this comes from the datasetfield type format field (@sekmiller please correct me :))
  • Get the field value, not the display value

@djbrooke djbrooke added Small and removed Small labels Feb 5, 2020
@pdurbin pdurbin self-assigned this Feb 6, 2020
pdurbin added a commit that referenced this issue Feb 6, 2020
@pdurbin
remove parentheses from dataset contact affiliation #6570
d54206f
@pdurbin pdurbin mentioned this issue Feb 6, 2020
Merged
@pdurbin pdurbin removed their assignment Feb 6, 2020
pdurbin added a commit that referenced this issue Feb 6, 2020
@pdurbin
reduce code duplication #6570
e3a25fa
kcondon added a commit that referenced this issue Feb 6, 2020
@kcondon
Merge pull request #6619 from IQSS/6570-search-api
c4e6065 
remove parentheses from dataset contact affiliation #6570
@jggautier jggautier mentioned this issue Feb 24, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature: API
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

remove parentheses from dataset contact affiliation #6570 IQSS/dataverse
3 participants
@pdurbin @djbrooke @jggautier