Skip to content

Parse NeTEx fare zones from a FareFrame#4563

Merged
hannesj merged 4 commits into
opentripplanner:dev-2.xfrom
entur:otp2_netex_fare_zones
Nov 8, 2022
Merged

Parse NeTEx fare zones from a FareFrame#4563
hannesj merged 4 commits into
opentripplanner:dev-2.xfrom
entur:otp2_netex_fare_zones

Conversation

@hannesj

@hannesj hannesj commented Nov 2, 2022

Copy link
Copy Markdown
Contributor

Summary

It is possible to define TariffZones in two ways, in a SiteFrame as TariffZones or in a FareFrame as its superclass, FareZone. Currently only the first way is implemented in OTP. This adds a parser for the FareFrame to read FareZones

@hannesj hannesj added !New Feature A functional feature targeting the end user. +NeTEx This issue is related to the Netex model/import. labels Nov 2, 2022
@hannesj hannesj added this to the 2.3 milestone Nov 2, 2022
@hannesj
hannesj requested review from t2gran and vpaturet November 2, 2022 10:03
@hannesj
hannesj requested a review from a team as a code owner November 2, 2022 10:03
.withIgnoreFareFrame(
config
.of("ignoreFareFrame")
.since(V2_2)

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.

Since 2.2 has been released we need a new enum value for 2.3

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated

@hannesj
hannesj force-pushed the otp2_netex_fare_zones branch from 9dbd09c to 1bd7edc Compare November 3, 2022 09:03
@hannesj
hannesj merged commit 3f8af65 into opentripplanner:dev-2.x Nov 8, 2022
@hannesj
hannesj deleted the otp2_netex_fare_zones branch November 8, 2022 15:05
t2gran pushed a commit that referenced this pull request Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

+NeTEx This issue is related to the Netex model/import. !New Feature A functional feature targeting the end user.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants