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

Json binding should use the field local name #389

Closed
tefra opened this issue Jan 20, 2021 · 1 comment · Fixed by #390
Closed

Json binding should use the field local name #389

tefra opened this issue Jan 20, 2021 · 1 comment · Fixed by #390

Comments

@tefra
Copy link
Owner

tefra commented Jan 20, 2021

The parser/serializer always the field definition names instead of the metadata names like their xml counterparts.
tefra added a commit that referenced this issue Jan 20, 2021
@tefra
JSON Binding use fields local name
ce5ef22 
Closes #389
@tefra tefra changed the title Json binding use the fields metadata name Json binding should use the field local name Jan 20, 2021
@tefra tefra mentioned this issue Jan 20, 2021
Merged
tefra added a commit that referenced this issue Jan 20, 2021
@tefra
JSON Binding use fields local name
c860fbf 
Closes #389
tefra added a commit to tefra/xsdata-w3c-tests that referenced this issue Jan 20, 2021
@tefra
tefra/xsdata#389
a9edf5a
tefra added a commit to tefra/xsdata-samples that referenced this issue Jan 20, 2021
@tefra
tefra/xsdata#389
d797f8d
tefra added a commit to tefra/xsdata-samples that referenced this issue Jan 20, 2021
@tefra
tefra/xsdata#389
72b6538
tefra added a commit to tefra/xsdata-w3c-tests that referenced this issue Jan 20, 2021
@tefra
tefra/xsdata#389
0e4191c
@delfick
Copy link

delfick commented Nov 7, 2022 •
edited
Loading

Hello, is it possible to make this optional? I went to upgrade this dependency in a project that relies on the json output having different key names and afaict it's not possible to make it output var.name instead of var.local_name without just copying next_value and changing that yield.

Specifically here, https://github.com/tefra/xsdata/blob/v22.11/xsdata/formats/dataclass/serializers/json.py#L104

@tefra tefra mentioned this issue Nov 9, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

JSON Binding use fields local name tefra/xsdata
2 participants
@delfick @tefra