Skip to content

Commit d5168e6

Browse files
Address PR feedback
1 parent 7a75386 commit d5168e6

File tree

2 files changed

+5
-7
lines changed

2 files changed

+5
-7
lines changed

Diff for: example-config.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -350,7 +350,7 @@ routingTypes:
350350
itinerary:
351351
# Provides an array of valid mode combinations that returned itineraries will be filtered against
352352
# If left blank, all itineraries deemed valid by OTP will be returned
353-
acceptableValidModeCombos: [
353+
validModeCombinations: [
354354
['WALK'],
355355
['WALK', 'PERSONAL'],
356356
['WALK', 'TRANSIT', 'SHARED'],

Diff for: lib/actions/apiV2.js

+4-6
Original file line numberDiff line numberDiff line change
@@ -982,8 +982,7 @@ export function routingQuery(searchId = null, updateSearchInReducer) {
982982
{}
983983

984984
const strictModes = !!config?.itinerary?.strictItineraryFiltering
985-
const acceptableValidModeCombos =
986-
config?.itinerary?.acceptableValidModeCombos
985+
const validModeCombinations = config?.itinerary?.validModeCombinations
987986

988987
// Filter mode definitions based on active mode keys
989988
const activeModeButtons = config.modes?.modeButtons.filter((mb) =>
@@ -1119,15 +1118,14 @@ export function routingQuery(searchId = null, updateSearchInReducer) {
11191118
// If "acceptableValidModeCombos" is provided, filter out itineraries that do not match our list of valid mode combinations
11201119
// (e.g. "WALK" + "DRIVE")
11211120
// TODO: Remove this once we switch to planConnection API
1122-
if (acceptableValidModeCombos?.length > 0) {
1123-
console.log(!!acceptableValidModeCombos)
1121+
if (validModeCombinations?.length > 0) {
11241122
filteredItineraries = filteredItineraries.filter((itin) => {
11251123
const modeCombo = Array.from(
11261124
new Set(itin.legs.map((leg) => SIMPLIFICATIONS[leg.mode]))
11271125
)
1128-
return !!acceptableValidModeCombos.find(
1126+
return validModeCombinations.find(
11291127
(vc) =>
1130-
modeCombo.every((m) => vc.includes(m)) &&
1128+
modeCombo.length === vc.length &&
11311129
vc.every((m) => modeCombo.includes(m))
11321130
)
11331131
})

0 commit comments

Comments
 (0)