use new GtfsLocation where previously GtfsLocationGroup was used

ibi-group · Jan 3, 2025 · 4dae31a · 4dae31a
1 parent ee6429c
commit 4dae31a
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 7 deletions.
diff --git a/lib/editor/actions/location.js b/lib/editor/actions/location.js
@@ -10,7 +10,7 @@ import { receivedNewEntity, savedGtfsEntity } from './active'
 
 export function saveLocationGroup (
   feedId: ?string,
-  locationGroup: GtfsLocationGroup,
+  locationGroup: GtfsLocation,
   refetch: ?boolean = true
 ) {
   return function (dispatch: dispatchFn, getState: getStateFn) {

diff --git a/lib/editor/components/pattern/EditShapePanel.js b/lib/editor/components/pattern/EditShapePanel.js
@@ -21,7 +21,7 @@ import {
   getPatternDistance,
   isValidStopControlPoint
 } from '../../util/map'
-import type {ControlPoint, LatLng, Pattern, GtfsStop, GtfsLocation, GtfsLocationGroup} from '../../../types'
+import type {ControlPoint, LatLng, Pattern, GtfsStop, GtfsLocation} from '../../../types'
 import type {EditSettingsUndoState} from '../../../types/reducers'
 
 import EditSettings from './EditSettings'
@@ -30,7 +30,7 @@ type Props = {
   activePattern: Pattern,
   controlPoints: Array<ControlPoint>,
   editSettings: EditSettingsUndoState,
-  locationGroups: Array<GtfsLocationGroup>,
+  locationGroups: Array<GtfsLocation>,
   locations: Array<GtfsLocation>,
   patternSegment: number,
   resetActiveGtfsEntity: typeof activeActions.resetActiveGtfsEntity,

diff --git a/lib/editor/selectors/index.js b/lib/editor/selectors/index.js
@@ -432,7 +432,7 @@ function addPatternStopsToShapePoints (
   patternStops: Array<PatternStop>,
   activePatternStops: Array<GtfsStop>,
   activePatternLocations: Array<GtfsLocation>,
-  activePatternLocationGroups: Array<GtfsLocationGroup>,
+  activePatternLocationGroups: Array<GtfsLocation>,
   projectStops: boolean = false,
   totalShapePointsDistTraveled: number
 ): {patternSegments: Array<Coordinates>, shapePoints: Array<ShapePoint>} {

diff --git a/lib/editor/util/map.js b/lib/editor/util/map.js
@@ -25,7 +25,6 @@ import type {
   GeoJsonLinestring,
   GeoJsonPoint,
   GtfsLocation,
-  GtfsLocationGroup,
   GtfsStop,
   LatLng,
   Pattern,
@@ -875,7 +874,7 @@ export function straightLineDistancesBetweenStopAnchors (
 export function generateControlPointsFromPatternStops (
   patternStops: Array<PatternStop>,
   stops: Array<GtfsStop>,
-  locationGroups: Array<GtfsLocationGroup>,
+  locationGroups: Array<GtfsLocation>,
   locations: Array<GtfsLocation>
 ): any { // FIXME: fix flow return type
   const controlPoints = []
@@ -915,7 +914,7 @@ export function generateControlPointsFromPatternStops (
   }
 }
 
-export function stopToPoint (stop: GtfsStop | GtfsLocation | GtfsLocationGroup) {
+export function stopToPoint (stop: GtfsStop | GtfsLocation) {
   if (!stop.stop_lon || !stop.stop_lat) {
     console.error('Serious issue: stopToPoint called without a point! Using 0, 0 coordinates. This is a very bad thing')
     return point([0, 0])

diff --git a/lib/types/index.js b/lib/types/index.js
@@ -824,6 +824,13 @@ export type GtfsLocation = {|
   zone_id?: string
 |}
 
+// export type GtfsLocationGroup = {
+//   id: number,
+//   location_group_id: string,
+//   location_group_name: ?string,
+//   stop_id: ?string[]
+// }
+
 export type StopWithFeed = GtfsStop & {child_stops?: Array<GtfsStop>, feed: Feed}
 
 export type LatLng = {