-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Remove reading agency and route brandingUrl from GTFS data #6183
Remove reading agency and route brandingUrl from GTFS data #6183
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev-2.x #6183 +/- ##
=============================================
+ Coverage 69.92% 69.94% +0.02%
- Complexity 17730 17741 +11
=============================================
Files 1996 1997 +1
Lines 75416 75435 +19
Branches 7717 7722 +5
=============================================
+ Hits 52734 52763 +29
- Misses 20000 20004 +4
+ Partials 2682 2668 -14 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We seem to also include brandingUrl from GTFS routes.txt which is also some non-standard field. Should we consider removing that support as well? Seems like at least that's reusing same branding type as netex data.
application/src/main/java/org/opentripplanner/transit/model/organization/AgencyBuilder.java
Show resolved
Hide resolved
I think yes. I didn't notice that, but now that I look at it, it would make sense to remove that at the same time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you can remove the initialization of BRANDING_URL for route in this class so we don't have to update this test class if the brandingUrl gets removed from the OBA library at some point.
…way removes it as nonstandard
Summary
Remove the property brandingUrl from the internal data model for agency and route.
Do not read, but still accept, the nonstandard extension brandingUrl for those GTFS files.
Unit tests
Updated the tests to no longer check for brandingUrl.
Documentation
No change.