Merge pull request #553 from ibi-group/mtc-gtfsplus

Add GTFS+ validation issue details

Author: landonreed

lib/common/util/config.js

@@ -37,7 +37,7 @@ export function getGtfsPlusSpec (): Array<GtfsSpecTable> {
   const CONFIG: DataToolsConfig = window.DT_CONFIG
   const GTFS_PLUS_SPEC = CONFIG.specifications.gtfsplus
   if (!GTFS_PLUS_SPEC) throw new Error('GTFS+ yml configuration file is not defined!')
-  return GTFS_PLUS_SPEC
+  return GTFS_PLUS_SPEC.sort((table1, table2) => table1.name.localeCompare(table2.name))
 }
 
 /**

lib/gtfsplus/components/GtfsPlusVersionSummary.js

@@ -1,14 +1,24 @@
 // @flow
 
+import moment from 'moment'
 import React, {Component} from 'react'
-import {Panel, Row, Col, Table, Button, ButtonToolbar, Glyphicon, Alert} from 'react-bootstrap'
+import {
+  Alert,
+  Button,
+  ButtonToolbar,
+  Col,
+  Glyphicon,
+  Panel,
+  Row,
+  Table
+} from 'react-bootstrap'
 import {browserHistory, Link} from 'react-router'
-import moment from 'moment'
 
-import {getGtfsPlusSpec} from '../../common/util/config'
 import * as gtfsPlusActions from '../actions/gtfsplus'
 
+import {getGtfsPlusSpec} from '../../common/util/config'
 import type {Props as ContainerProps} from '../containers/ActiveGtfsPlusVersionSummary'
+import type {GtfsPlusValidationIssue} from '../../types'
 import type {GtfsPlusReducerState, ManagerUserState} from '../../types/reducers'
 
 type Props = ContainerProps & {
@@ -51,7 +61,7 @@ export default class GtfsPlusVersionSummary extends Component<Props, State> {
     return issuesForTable[tableId].length.toLocaleString()
   }
 
-  _getTableLevelIssues = (tableId: string) => {
+  _getTableLevelIssues = (tableId: string): ?Array<GtfsPlusValidationIssue> => {
     const {issuesForTable} = this.props
     if (!issuesForTable) return null
     if (!(tableId in issuesForTable)) return null
@@ -214,7 +224,7 @@ export default class GtfsPlusVersionSummary extends Component<Props, State> {
                               {tableLevelIssues.length} critical table issue(s):
                               <ul>
                                 {tableLevelIssues.map((issue, i) =>
-                                  <li key={i}>{issue.description}</li>)}
+                                  <li key={i}>{issue.fieldName}: {issue.description}</li>)}
                               </ul>
                             </small>
                             : null