Skip to content

Conversation

@haeckerbaer
Copy link
Collaborator

@haeckerbaer haeckerbaer changed the title SituationExchane - add optional list of keywords in PassengerInformationAction SituationExchange - add optional list of keywords in PassengerInformationAction Jun 11, 2024
@TuThoThai TuThoThai added the SX For everything SIRI-SX label Mar 21, 2025
@haeckerbaer
Copy link
Collaborator Author

Is related to the discussion about replacing nmtoken in IDs. Alternatively we change the type to xsd:string.

@ue71603
Copy link
Collaborator

ue71603 commented Sep 24, 2025

If we want to move away from NMToken we need to inform people. Using additional characters may break legacy systems.

@TuThoThai TuThoThai added this to the later milestone Sep 24, 2025
@haeckerbaer haeckerbaer modified the milestones: later, v2.3 Sep 24, 2025
@TuThoThai TuThoThai added Needs CEN documentation Update These require CEN documentation update to match XSD & examples and removed needs documentation update labels Sep 24, 2025
@haeckerbaer
Copy link
Collaborator Author

@ue71603 See my comment above about compatibility. I would like to add an example but need to ask on UmS side first as i'm not the initiator. By the way, the initial proposal from UmS didn't include special symbols as mentioned above (hence it's a new requirement from SG7)!

@ue71603
Copy link
Collaborator

ue71603 commented Oct 27, 2025

@haeckerbaer Just telling. If UTF-8 is allowed for identifiers, then very strange id need to be consumable by the it systems.

ue71603
ue71603 previously approved these changes Oct 27, 2025
@haeckerbaer
Copy link
Collaborator Author

@ue71603 what we could also do is xsd:list with <xsd:restriction base="xsd:string"> and <xsd:pattern value="[A-Za-z0-9._:-#]+"/> plus whatever characters we need. Would that be better? The question is what characters we have to add

@ue71603
Copy link
Collaborator

ue71603 commented Oct 28, 2025

I guess then we will have to change all the time. I informed NAPCore X 2.4. They should think about it also a bit.

@haeckerbaer
Copy link
Collaborator Author

10.11.2025: SG7 decided to discuss it in the SX task force since there might be a need to also integrate into ActionData

@haeckerbaer
Copy link
Collaborator Author

As per TaskForce decision on 24.11: moved new keywords element to ParameterisedActionStructure (from PassengerInformationActionStructure) so that it is also available in all of the "classical" actions (such as PushedAction).

@haeckerbaer haeckerbaer changed the base branch from integration to 2.3 December 19, 2025 11:33
<xsd:element name="Keywords" minOccurs="0">
<xsd:annotation>
<xsd:documentation>Arbitrary application specific classifiers.</xsd:documentation>
<xsd:documentation>A whitespace-separated list of arbitrary application specific classifiers, tags or message categories for filtering or display, for example as icons on a situation map. Can be overwritten in an individual PassengerInformationAction.</xsd:documentation>
Copy link
Collaborator

Choose a reason for hiding this comment

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

PassengerInformationAction to be changed to ParametrisedActionStructure

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

i replaced PassengerInformationAction with "action". is that Ok? I wouldn't reference abstract xsd structures but rather elements actually present in a "finished" xml. However, this would require a reference of multiple elements.. I chose the conceptual "action" for this reason but maybe ParametrisedActionStructure is still better suited. What do you think?

@TuThoThai
Copy link
Collaborator

As per group decision of 19 December 2025: agreement to have Keywords in ParametrisedActionStructure and keep the type as NMTOKENS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Needs CEN documentation Update These require CEN documentation update to match XSD & examples SX For everything SIRI-SX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants