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 tabular not properly dealiasing items #18186
Conversation
@rijkvanzanten Going back to the discussion we had on Discord, your idea of being able to apply the translations display to any field wouldn't work at the moment as the display needs to configure which fields it should load, but the problem here is that you can only specify fields relative to your field so you can't load fields on the same level meaning, the display wouldn't be able to access the What I did now is to remove the hacky logic so we can fix this for now and figure out a proper way to support translations display on none translations fields, meaning that for now we will only show it as an array like so: |
* Fix tabular not properly dealiasing items * run linter * remove translations edgecase * Fix linter warnings * Remove unused var --------- Co-authored-by: rijkvanzanten <rijkvanzanten@me.com>
The problem was that when merging the item returned from the api to replace alaised fields with their original names, I didn't take into account that the field without an alias could override the dealiased field, thus resulting in the wrong item.
closes #18181