Skip to content

Commit d1b933e

Browse files
fix(ActiveGtfsPlusEditor): Pass feed version summary to child components.
1 parent e8eb7a1 commit d1b933e

File tree

4 files changed

+18
-18
lines changed

4 files changed

+18
-18
lines changed

lib/gtfsplus/actions/gtfsplus.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,9 @@ import {stopsAndRoutes} from '../../gtfs/util/graphql'
1010
import {getHeaders} from '../../common/util/util'
1111
import {VISIBILITY} from '../components/GtfsPlusTable'
1212
import {startJobMonitor} from '../../manager/actions/status'
13-
1413
import type {
1514
FeedVersion,
15+
FeedVersionSummary,
1616
GtfsRoute,
1717
GtfsStop,
1818
GtfsPlusValidation,
@@ -205,7 +205,7 @@ export function uploadGtfsPlusFeed (feedVersionId: string, file: File) {
205205
export function loadGtfsEntities (
206206
tableId: string,
207207
rows: Array<GTFSPlusEntity>,
208-
feedVersion: FeedVersion,
208+
feedVersionSummary: FeedVersionSummary,
209209
validation: GtfsPlusValidation
210210
) {
211211
return function (dispatch: dispatchFn, getState: getStateFn) {
@@ -227,7 +227,7 @@ export function loadGtfsEntities (
227227
return row
228228
})
229229
: rows
230-
const {namespace} = feedVersion
230+
const {namespace} = feedVersionSummary
231231
const {routesToLoad, stopsToLoad} = lookupMissingEntities(filteredRows, getState().gtfsplus.gtfsEntityLookup)
232232
if (routesToLoad.length === 0 && stopsToLoad.length === 0) {
233233
return

lib/gtfsplus/components/GtfsPlusEditor.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,14 +8,14 @@ import {Link} from 'react-router'
88
import ManagerPage from '../../common/components/ManagerPage'
99
import OptionButton from '../../common/components/OptionButton'
1010
import * as gtfsPlusActions from '../actions/gtfsplus'
11-
import GtfsPlusTable from './GtfsPlusTable'
1211
import {getGtfsPlusSpec} from '../../common/util/config'
1312
import * as feedsActions from '../../manager/actions/feeds'
14-
1513
import type {Props as ContainerProps} from '../containers/ActiveGtfsPlusEditor'
16-
import type {Feed, FeedVersion, GtfsPlusValidation, Project} from '../../types'
14+
import type {Feed, FeedVersionSummary, GtfsPlusValidation, Project} from '../../types'
1715
import type {ManagerUserState} from '../../types/reducers'
1816

17+
import GtfsPlusTable from './GtfsPlusTable'
18+
1919
export type Props = ContainerProps & {
2020
activeTableId: string,
2121
addGtfsPlusRow: typeof gtfsPlusActions.addGtfsPlusRow,
@@ -24,8 +24,8 @@ export type Props = ContainerProps & {
2424
downloadGtfsPlusFeed: typeof gtfsPlusActions.downloadGtfsPlusFeed,
2525
feedSource: Feed,
2626
feedSourceId: string,
27-
feedVersion: FeedVersion,
2827
feedVersionId: string,
28+
feedVersionSummary: FeedVersionSummary,
2929
fetchFeedSourceAndProject: typeof feedsActions.fetchFeedSourceAndProject,
3030
gtfsEntityLookup: Object,
3131
loadGtfsEntities: typeof gtfsPlusActions.loadGtfsEntities,
@@ -100,8 +100,8 @@ export default class GtfsPlusEditor extends Component<Props> {
100100
_getGtfsEntity = (type: string, id: string) => this.props.gtfsEntityLookup[`${type}_${id}`]
101101

102102
_newRowsDisplayed = (rows: Array<any>) => {
103-
const {activeTableId, feedVersion, loadGtfsEntities, validation} = this.props
104-
if (feedVersion) loadGtfsEntities(activeTableId, rows, feedVersion, validation)
103+
const {activeTableId, feedVersionSummary, loadGtfsEntities, validation} = this.props
104+
if (feedVersionSummary) loadGtfsEntities(activeTableId, rows, feedVersionSummary, validation)
105105
}
106106

107107
_selectTable = (activeTableId: string) => this.props.setActiveTable({activeTableId})
@@ -127,7 +127,7 @@ export default class GtfsPlusEditor extends Component<Props> {
127127
const {
128128
activeTableId,
129129
feedSource,
130-
feedVersion,
130+
feedVersionSummary,
131131
project,
132132
tableData,
133133
user,
@@ -157,7 +157,7 @@ export default class GtfsPlusEditor extends Component<Props> {
157157
<li><Link to='/'>Explore</Link></li>
158158
<li><Link to='/project'>Projects</Link></li>
159159
<li><Link to={`/project/${project.id}`}>{project.name}</Link></li>
160-
<li><Link to={`/feed/${feedSource.id}/version/${feedVersion.version}/gtfsplus`}>{feedSource.name}</Link></li>
160+
<li><Link to={`/feed/${feedSource.id}/version/${feedVersionSummary.version}/gtfsplus`}>{feedSource.name}</Link></li>
161161
<li className='active'>Edit GTFS+</li>
162162
</ul>
163163
</Col>

lib/gtfsplus/components/GtfsPlusField.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,8 @@ export default class GtfsPlusField extends Component<Props> {
4242
}
4343

4444
render () {
45-
const {currentValue, data: rowData, feedVersion, field, getGtfsEntity} = this.props
45+
const {currentValue, data: rowData, feedVersionSummary, field, getGtfsEntity} = this.props
46+
const namespace = feedVersionSummary && feedVersionSummary.namespace
4647
switch (field.inputType) {
4748
case 'TEXT':
4849
case 'GTFS_TRIP':
@@ -108,7 +109,7 @@ export default class GtfsPlusField extends Component<Props> {
108109
return (
109110
<GtfsSearch
110111
feeds={[]}
111-
namespace={feedVersion && feedVersion.namespace}
112+
namespace={namespace}
112113
limit={100}
113114
entities={['routes']}
114115
minimumInput={1}
@@ -124,7 +125,7 @@ export default class GtfsPlusField extends Component<Props> {
124125
return (
125126
<GtfsSearch
126127
feeds={[]}
127-
namespace={feedVersion && feedVersion.namespace}
128+
namespace={namespace}
128129
limit={100}
129130
entities={['stops']}
130131
clearable={false}

lib/gtfsplus/containers/ActiveGtfsPlusEditor.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ import {
1717
setVisibilityFilter
1818
} from '../actions/gtfsplus'
1919
import {getValidationIssuesForTable, getVisibleRows, getFilteredPageCount} from '../selectors'
20-
2120
import type {AppState, RouterProps} from '../../types/reducers'
2221

2322
export type Props = RouterProps
@@ -43,16 +42,16 @@ const mapStateToProps = (state: AppState, ownProps: Props) => {
4342
const feedSource = project && project.feedSources
4443
? project.feedSources.find(fs => fs.id === feedSourceId)
4544
: null
46-
const feedVersion = feedSource && feedSource.feedVersions
47-
? feedSource.feedVersions.find(v => v.id === feedVersionId)
45+
const feedVersionSummary = feedSource && feedSource.feedVersionSummaries
46+
? feedSource.feedVersionSummaries.find(v => v.id === feedVersionId)
4847
: null
4948
return {
5049
activeTableId,
5150
currentPage,
5251
feedSource,
5352
feedSourceId,
54-
feedVersion,
5553
feedVersionId,
54+
feedVersionSummary,
5655
gtfsEntityLookup,
5756
pageCount: getFilteredPageCount(state),
5857
project,

0 commit comments

Comments
 (0)