@opentripplanner/core-utils 6.0.1-alpha.1 → 7.0.0-alpha.3

package/src/itinerary.ts

@@ -7,33 +7,37 @@ import {
   Itinerary,
   LatLngArray,
   Leg,
-  Step
+  Money,
+  Step,
+  TncFare
 } from "@opentripplanner/types";
 import turfAlong from "@turf/along";
 
+/*
 import {
-  calculateFares,
-  getLegModeLabel,
-  getModeForPlace,
-  getPlaceName,
-  getStepDirection,
-  getStepInstructions,
-  getStepStreetName,
-  getTimeZoneOffset,
-  getTransitFare
+  // calculateFares,
+  // getLegModeLabel,
+  // getModeForPlace,
+  // getPlaceName,
+  // getStepDirection,
+  // getStepInstructions,
+  // getStepStreetName,
+  // getTimeZoneOffset,
+  // getTransitFare
 } from "./deprecated";
 
 export {
-  calculateFares,
-  getLegModeLabel,
-  getModeForPlace,
-  getPlaceName,
-  getStepDirection,
-  getStepInstructions,
-  getStepStreetName,
-  getTimeZoneOffset,
-  getTransitFare
+  // calculateFares,
+  // getLegModeLabel,
+  // getModeForPlace,
+  // getPlaceName,
+  // getStepDirection,
+  // getStepInstructions,
+  // getStepStreetName,
+  // getTimeZoneOffset,
+  // getTransitFare
 };
+*/
 
 // All OTP transit modes
 export const transitModes = [
@@ -445,12 +449,53 @@ export function calculatePhysicalActivity(
   };
 }
 
-export function calculateTncFares(itinerary) {
-  // TODO: don't rely on deprecated methods!
-  // At the moment this is safe as none of these exported variables contain strings
-  const { maxTNCFare, minTNCFare, tncCurrencyCode } = calculateFares(
-    itinerary,
-    true
-  );
-  return { maxTNCFare, minTNCFare, tncCurrencyCode };
+/**
+ * For an itinerary, calculates the TNC fares and returns an object with
+ * these values and currency info.
+ * It is assumed that the same currency is used for all TNC legs.
+ */
+export function calculateTncFares(itinerary: Itinerary): TncFare {
+  let minTNCFare = 0;
+  let maxTNCFare = 0;
+  let currencyCode;
+  itinerary.legs.forEach(({ hailedCar, mode, tncData }) => {
+    if (mode === "CAR" && hailedCar && tncData) {
+      const { currency, maxCost, minCost } = tncData;
+      minTNCFare += minCost;
+      maxTNCFare += maxCost;
+      // Assumes a single currency for entire itinerary.
+      currencyCode = currency;
+    }
+  });
+
+  return {
+    currencyCode,
+    maxTNCFare,
+    minTNCFare
+  };
+}
+
+/**
+ * For a given fare component (either total fare or component parts), returns
+ * an object with the fare value (in cents).
+ */
+export function getTransitFare(
+  fareComponent: Money
+): {
+  currencyCode: string;
+  transitFare: number;
+} {
+  // Default values (if fare component is not valid).
+  let transitFare = 0;
+  let currencyCode = "USD";
+  if (fareComponent) {
+    // Assign values without declaration.
+    // See: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Destructuring_assignment#assignment_without_declaration
+    ({ currencyCode } = fareComponent.currency);
+    transitFare = fareComponent.cents;
+  }
+  return {
+    currencyCode,
+    transitFare
+  };
 }

package/src/map.ts

@@ -1,14 +1,16 @@
 import { LatLngArray, Location, UserPosition } from "@opentripplanner/types";
 import { toSentenceCase } from "./itinerary";
 
+/*
 import {
-  coordsToString,
-  getDetailText,
-  latlngToString,
-  logDeprecationWarning
+  // coordsToString,
+  // getDetailText //,
+  // latlngToString,
+  // logDeprecationWarning
 } from "./deprecated";
 
-export { coordsToString, getDetailText, latlngToString };
+export { coordsToString, getDetailText , latlngToString };
+*/
 
 export function currentPositionToLocation(
   currentPosition: UserPosition
@@ -26,6 +28,12 @@ export function currentPositionToLocation(
   };
 }
 
+// TRICKY: It is used in query.js and in the context of
+// otp-rr actions where the intl context is not available.
+export function coordsToString(coords: number[]): string {
+  return coords.length && coords.map(c => (+c).toFixed(5)).join(", ");
+}
+
 export function stringToCoords(str: string): number[] {
   return (str && str.split(",").map(c => +c)) || [];
 }
@@ -40,23 +48,10 @@ export function constructLocation(latlng: {
   };
 }
 
-export function formatStoredPlaceName(
-  location: Location,
-  withDetails = true
-): string {
-  if (withDetails) {
-    logDeprecationWarning("the formatStoredPlaceName withDetails parameter");
-  }
-
-  let displayName =
-    location.type === "home" || location.type === "work"
-      ? toSentenceCase(location.type)
-      : location.name;
-  if (withDetails) {
-    const detailText = getDetailText(location);
-    if (detailText) displayName += ` (${detailText})`;
-  }
-  return displayName;
+export function formatStoredPlaceName(location: Location): string {
+  return location.type === "home" || location.type === "work"
+    ? toSentenceCase(location.type)
+    : location.name;
 }
 
 export function matchLatLon(location1: Location, location2: Location): boolean {

package/src/query.js

@@ -1,20 +1,20 @@
-import moment from "moment";
+import { format, isMatch, parse } from "date-fns";
 import getGeocoder from "@opentripplanner/geocoder/lib";
 import qs from "qs";
 
 import { getTransitModes, hasCar, isAccessMode } from "./itinerary";
-import { stringToCoords } from "./map";
+import { coordsToString, stringToCoords } from "./map";
 import queryParams from "./query-params";
 import {
   getCurrentTime,
   getCurrentDate,
-  OTP_API_DATE_FORMAT,
-  OTP_API_TIME_FORMAT
+  OTP_API_TIME_FORMAT,
+  OTP_API_DATE_FORMAT_DATE_FNS
 } from "./time";
 
-import { coordsToString, summarizeQuery } from "./deprecated";
+// import { coordsToString, summarizeQuery } from "./deprecated";
 
-export { summarizeQuery };
+// export { summarizeQuery };
 
 /* The list of default parameters considered in the settings panel */
 
@@ -315,9 +315,15 @@ export function planParamsToQuery(params) {
         break;
       case "time":
         {
-          const parsedTime = moment(params.time, TIME_FORMATS);
-          query.time = parsedTime.isValid()
-            ? parsedTime.format(OTP_API_TIME_FORMAT)
+          // Match one of the supported time formats
+          const matchedTimeFormat = TIME_FORMATS.find(timeFormat =>
+            isMatch(params.time, timeFormat)
+          );
+          query.time = matchedTimeFormat
+            ? format(
+                parse(params.time, matchedTimeFormat, new Date()),
+                OTP_API_TIME_FORMAT
+              )
             : getCurrentTime();
         }
         break;
@@ -421,8 +427,8 @@ export function getRoutingParams(config, currentQuery, ignoreRealtimeUpdates) {
     }
 
     // check date/time validity; ignore both if either is invalid
-    const dateValid = moment(params.date, OTP_API_DATE_FORMAT).isValid();
-    const timeValid = moment(params.time, OTP_API_TIME_FORMAT).isValid();
+    const dateValid = isMatch(params.date, OTP_API_DATE_FORMAT_DATE_FNS);
+    const timeValid = isMatch(params.time, OTP_API_TIME_FORMAT);
 
     if (!dateValid || !timeValid) {
       delete params.time;
@@ -447,11 +453,8 @@ export function getRoutingParams(config, currentQuery, ignoreRealtimeUpdates) {
     // Additional processing specific to PROFILE mode
   } else {
     // check start and end time validity; ignore both if either is invalid
-    const startTimeValid = moment(
-      params.startTime,
-      OTP_API_TIME_FORMAT
-    ).isValid();
-    const endTimeValid = moment(params.endTime, OTP_API_TIME_FORMAT).isValid();
+    const startTimeValid = isMatch(params.startTime, OTP_API_TIME_FORMAT);
+    const endTimeValid = isMatch(params.endTime, OTP_API_TIME_FORMAT);
 
     if (!startTimeValid || !endTimeValid) {
       delete params.startTimeValid;

package/src/time.ts

@@ -1,21 +1,7 @@
 import { Config } from "@opentripplanner/types";
-import {
-  startOfDay,
-  add,
-  format,
-  formatDuration as dateFnsFormatDuration
-} from "date-fns";
+import { startOfDay, add, format } from "date-fns";
 import { utcToZonedTime } from "date-fns-tz";
 
-/* eslint-disable import/no-cycle */
-import {
-  formatTime,
-  formatDurationWithSeconds,
-  formatDuration
-} from "./deprecated-with-types";
-
-export { formatTime, formatDuration, formatDurationWithSeconds };
-
 // special constants for making sure the following date format is always sent to
 // OTP regardless of whatever the user has configured as the display format
 export const OTP_API_DATE_FORMAT = "YYYY-MM-DD";
@@ -24,73 +10,22 @@ export const OTP_API_DATE_FORMAT = "YYYY-MM-DD";
 export const OTP_API_DATE_FORMAT_DATE_FNS = "yyyy-MM-dd";
 export const OTP_API_TIME_FORMAT = "HH:mm";
 
-/**
- * To ease the transition away from moment.js, this method uses date-fns to format durations
- * the way moment.js did.
- * @param {number}  seconds     The number of seconds to format
- * @param {boolean} showSeconds Whether to render seconds or not
- * @param {boolean} localize    If true, will create output like moment.js using date-fns locale.
- * Otherwise, uses date-fns default
- * @returns                   Formatted duration
- */
-export function formatDurationLikeMoment(
-  seconds: number,
-  showSeconds: boolean,
-  localize: { enabled: boolean; code: string } = {
-    enabled: true,
-    code: "en-US"
-  }
-): string {
-  // date-fns doesn't do this automatically
-  if ((!showSeconds && seconds < 60) || seconds === 0) {
-    return "0 min";
-  }
-
-  const hours = Math.floor(seconds / 3600);
-  const minutes = Math.floor((seconds - hours * 3600) / 60);
-  const secondsLeftOver = showSeconds
-    ? seconds - hours * 3600 - minutes * 60
-    : 0;
-  const specLookup = {
-    xHours: "hr",
-    xMinutes: "min",
-    xSeconds: "sec"
-  };
-  const locale = localize
-    ? {
-        // Maintain backwards compatibility when called with localize=true
-        code: localize?.code || "en-US",
-        formatDistance: (spec, val) => {
-          return `${val} ${specLookup[spec]}`;
-        }
-      }
-    : undefined;
-
-  return dateFnsFormatDuration(
-    {
-      hours,
-      minutes,
-      seconds: secondsLeftOver
-    },
-    {
-      format: ["hours", "minutes", "seconds"],
-      locale
-    }
-  );
-}
-
 /**
  * Breaks up a duration in seconds into hours, minutes, and seconds.
  * @param {number} seconds The number of seconds to break up
  * @returns an object with fields with the corresponding, hours, minutes, seconds.
  */
-export function toHoursMinutesSeconds(seconds) {
-  const hours = Math.floor(seconds / 3600);
-  const minutes = Math.floor((seconds - hours * 3600) / 60);
+export function toHoursMinutesSeconds(
+  seconds: number
+): {
+  hours: number;
+  minutes: number;
+  seconds: number;
+} {
   return {
-    hours,
-    minutes,
-    seconds: seconds - hours * 3600 - minutes * 60
+    hours: Math.floor(seconds / 3600),
+    minutes: Math.floor(seconds / 60) % 60,
+    seconds: seconds % 60
   };
 }