Skip to content

sdk: Disallow invalid types in MultiLanguage objects - #431

Merged
s-heppner merged 9 commits into
eclipse-basyx:developfrom
ZANDx1:MultiLangTyping
Jan 9, 2026
Merged

sdk: Disallow invalid types in MultiLanguage objects#431
s-heppner merged 9 commits into
eclipse-basyx:developfrom
ZANDx1:MultiLangTyping

Conversation

@ZANDx1

@ZANDx1 ZANDx1 commented Nov 6, 2025

Copy link
Copy Markdown
Contributor

Previously, multilingual fields (display_name, description,
MultiLanguageProperty.value) could be assigned invalid types, leading to inconsistent
and sometimes invalid JSON/XML serialization.
As a result, this behavior could cause interoperability issues.

The AAS specification requires these fields to be of type
LangStringSet / MultiLanguageTextType. This requirement is now
enforced to ensure spec-compliant serialization.

Fixes #416

Comment thread sdk/basyx/aas/model/base.py Outdated
Comment thread sdk/basyx/aas/model/base.py Outdated
Comment thread sdk/basyx/aas/model/base.py Outdated
Comment thread sdk/basyx/aas/model/base.py Outdated
Comment thread sdk/basyx/aas/model/base.py Outdated
Comment thread sdk/basyx/aas/model/base.py Outdated
@zrgt

zrgt commented Nov 25, 2025

Copy link
Copy Markdown
Contributor

@ZANDx1 please implement the same logic for MultiLanguageProperty.value

@zrgt zrgt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ZANDx1 Please adjust the PR description

@s-heppner s-heppner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@s-heppner
s-heppner merged commit b600264 into eclipse-basyx:develop Jan 9, 2026
15 checks passed
@s-heppner s-heppner changed the title Bug Fix serialization sdk: Disallow invalid types in MultiLanguage objects Jan 9, 2026
@s-heppner s-heppner mentioned this pull request Jul 16, 2026
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

Successfully merging this pull request may close these issues.

3 participants