refactor: address PR comments

see #100

Author: evansiroky

Diff for: lib/components/app/responsive-webapp.js

@@ -194,6 +194,9 @@ class RouterWrapper extends Component {
             <Route
               path='/print'
               component={(routerProps) => {
+                // combine the router props with the other props that get
+                // passed to the exported component. This way it's possible for
+                // the PrintLayout component to receive the custom icons prop.
                 const props = {...this.props, ...routerProps}
                 return <PrintLayout {...props} />
               }}

Diff for: lib/components/narrative/line-itin/access-leg-body.js

@@ -163,7 +163,7 @@ class AccessLegSummary extends Component {
         <div>
           {getLegModeLabel(leg)}
           {' '}
-          {leg.distance && <span> {distanceString(leg.distance)}</span>}
+          {leg.distance > 0 && <span> {distanceString(leg.distance)}</span>}
           {` to ${getPlaceName(leg.to, config.companies)}`}
         </div>
       </div>

Diff for: lib/util/itinerary.js

@@ -379,7 +379,7 @@ export function toSentenceCase (str) {
  */
 export function getLegIcon (leg, customIcons) {
   // check if a custom function exists for determining the icon for a leg
-  if (typeof customIcons.customIconForLeg === 'function') {
+  if (customIcons && typeof customIcons.customIconForLeg === 'function') {
     // function exits, get the icon string lookup. It's possible for there to be
     // a custom function that only returns a string for when a leg meets the
     // criteria of the custom function
@@ -406,7 +406,7 @@ export function getLegIcon (leg, customIcons) {
  * Get the configured company object for the given network string if the company
  * has been defined in the provided companies array config.
  */
-export function getCompanyForNetwork (networkString, companies = []) {
+function getCompanyForNetwork (networkString, companies = []) {
   const company = companies.find(co => co.id === networkString)
   if (!company) {
     console.warn(`No company found in config.yml that matches rented vehicle network: ${networkString}`, companies)