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

property, suffixed with @lt isn't found even if it exists #999

Open
Tracked by #966
karina-klinkeviciute opened this issue Dec 1, 2024 · 1 comment
Open
Tracked by #966

Comments

@karina-klinkeviciute
Copy link
Contributor

karina-klinkeviciute commented Dec 1, 2024

Image
Image

ModelReferenceKeyNotFound: Model reference key 'surname@lt' not found in
"<spinta.components.Model(name='datasets/gov/rc/jangis_ws/n437_israsas/NaturalPerson')>".
Context:
component: spinta.types.datatype.ExternalRef
manifest: default
schema: Sheet1:18
dataset: datasets/gov/rc/jangis_ws/n437_israsas
resource: resource1
model: <spinta.components.Model(name='datasets/gov/rc/jangis_ws/n437_israsas/NaturalPerson')>
entity: None
property: natural_person
attribute: None
type: _external_ref
resource.backend: datasets/gov/rc/jangis_ws/n437_israsas/resource1
ref: surname@lt

This doesn't happen in a similar situation, but where property doesn't have @

This is specifically in the situation where the property with the language tag exists on another model, and we are referring to that model and list the properties in the square brackets. If we are using the property name without the language tag, it works.

For example, if model ModelName has property code.lt and we refer to it as ModelName[code], it works, but if we refer to it as ModelName[code@lt], it doesn't work (question is, should it work?).

JANGIS 437

Related

@karina-klinkeviciute karina-klinkeviciute changed the title property, suffixed with @ls isn't found even if it exists property, suffixed with @lt isn't found even if it exists Dec 1, 2024
@karina-klinkeviciute karina-klinkeviciute linked a pull request Dec 12, 2024 that will close this issue
@karina-klinkeviciute
Copy link
Contributor Author

karina-klinkeviciute commented Dec 12, 2024

works after merging #1004

@karina-klinkeviciute karina-klinkeviciute linked a pull request Dec 16, 2024 that will close this issue
@karina-klinkeviciute karina-klinkeviciute removed a link to a pull request Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant