Skip to content

TicketingFacilityEnumeration to ScopeOfTicketEnumeration#243

Merged
skinkie merged 6 commits into
masterfrom
skinkie-scopeofticketenumeration
Apr 14, 2022
Merged

TicketingFacilityEnumeration to ScopeOfTicketEnumeration#243
skinkie merged 6 commits into
masterfrom
skinkie-scopeofticketenumeration

Conversation

@skinkie
Copy link
Copy Markdown
Contributor

@skinkie skinkie commented Sep 23, 2021

Fix #235

Copy link
Copy Markdown
Contributor

@syversenkr syversenkr left a comment

Choose a reason for hiding this comment

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

  1. Not backwards compatible, obviously, but being a bug this fix is necessary. Do we need a deprecation/grace period?

  2. I believe the fix on line 1289 should be reverted, because
    <xsd:element name="TicketingFacility" type="TicketingFacilityEnumeration" default="unknown">
    should remain of TicketingFacilityEnumeration, as this is the "Types of TICKETING available" (i.e. "Allowed values for TicketingFacility")

However, the ScopeOfTicketList should be of type ScopeOfTicketListOfEnumerations (not "TicketingFacilityListOfEnumerations")
I.e. change line line 1277 to:
<xsd:element name="ScopeOfTicketList" type="ScopeOfTicketListOfEnumerations">

For elaboration of practical implications, see also comments on issue #235

@ue71603 ue71603 requested a review from syversenkr April 14, 2022 12:26
@ue71603 ue71603 dismissed syversenkr’s stale review April 14, 2022 12:31

Commit did the requested change., but somehow I can't accept the change. Probably because it is Stefan's PR

Comment thread xsd/netex_framework/netex_reusableComponents/netex_facility_version.xsd Outdated
Comment thread xsd/netex_framework/netex_reusableComponents/netex_facility_version.xsd Outdated
Comment thread xsd/netex_framework/netex_reusableComponents/netex_facility_version.xsd Outdated
Comment thread xsd/netex_framework/netex_reusableComponents/netex_facility_version.xsd Outdated
@skinkie skinkie merged commit 333a40a into master Apr 14, 2022
erlendnils1 pushed a commit to entur/NeTEx that referenced this pull request Jun 29, 2023
…cosystem#243)

* TicketingFacilityEnumeration to ScopeOfTicketEnumeration

Fix TransmodelEcosystem#235

Co-authored-by: Matthias Günter <matthias.guenter@sbb.ch>
erlendnils1 pushed a commit to entur/NeTEx that referenced this pull request Oct 31, 2023
…cosystem#243)

* TicketingFacilityEnumeration to ScopeOfTicketEnumeration

Fix TransmodelEcosystem#235

Co-authored-by: Matthias Günter <matthias.guenter@sbb.ch>
@skinkie skinkie deleted the skinkie-scopeofticketenumeration branch November 19, 2024 10:27
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.

ScopeOfTicketListOfEnumerations - wrong item type

4 participants