Apply suggestions from code review

Co-authored-by: Leonard Ehrenfried <[email protected]>

src/ext/java/org/opentripplanner/ext/flex/FlexRouter.java

@@ -180,7 +180,7 @@ private List<FlexServiceDate> createFlexServiceDates(
   /**
    * This class work as an adaptor around OTP services. This allows us to pass in this instance
    * and not the implementations (graph, transitService, flexIndex). We can easily mock this in
-   * unit-tests. This also serves as documentation of witch services the flex access/egress
+   * unit-tests. This also serves as documentation of which services the flex access/egress
    * generation logic needs.
    */
   private class CallbackAdapter implements FlexAccessEgressCallbackAdapter {

src/ext/java/org/opentripplanner/ext/flex/edgetype/FlexTripEdge.java

@@ -15,7 +15,7 @@
 import org.opentripplanner.transit.model.site.StopLocation;
 
 /**
- * Flex trips are not connected to the graph.
+ * Flex trips edges are not connected to the graph.
  */
 public class FlexTripEdge extends Edge {
 

src/ext/java/org/opentripplanner/ext/flex/template/AbstractFlexTemplate.java

@@ -186,7 +186,7 @@ private FlexAccessEgress createFlexAccessEgress(
   ) {
     var flexEdge = getFlexEdge(flexVertex, transferStop);
 
-    // Drop none routable and very short(<10s) trips
+    // Drop non-routable and very short(<10s) trips
     if (flexEdge == null || flexEdge.getTimeInSeconds() < MIN_FLEX_TRIP_DURATION_SECONDS) {
       return null;
     }

src/ext/java/org/opentripplanner/ext/flex/trip/UnscheduledTrip.java

@@ -255,7 +255,7 @@ public int findAlightIndex(StopLocation toStop) {
   @Override
   public FlexPathCalculator decorateFlexPathCalculator(FlexPathCalculator defaultCalculator) {
     // Get the correct {@link FlexPathCalculator} depending on the {@code timePenalty}.
-    // Ff the modifier doesn't actually modify, we return the regular calculator.
+    // If the modifier doesn't actually modify, we return the regular calculator.
     if (timePenalty.modifies()) {
       return new TimePenaltyCalculator(defaultCalculator, timePenalty);
     } else {

src/main/java/org/opentripplanner/apis/transmodel/model/plan/TripQuery.java

@@ -52,7 +52,7 @@ public static GraphQLFieldDefinition create(
           .name("dateTime")
           .description(
             "The date and time for the earliest time the user is willing to start the journey " +
-            "(if `false`or not set) or the latest acceptable time of arriving " +
+            "(if `false` or not set) or the latest acceptable time of arriving " +
             "(`true`). Defaults to now"
           )
           .type(gqlUtil.dateTimeScalar)

src/main/java/org/opentripplanner/routing/api/request/RouteRequest.java

@@ -117,7 +117,7 @@ public void withPreferences(Consumer<RoutingPreferences.Builder> body) {
   /**
    * The booking time is used to exclude services which are not bookable at the
    * requested booking time. If a service is bookable at this time or later, the service
-   * is included. Currently, OTP only supports this for FLEX access and egress.
+   * is included. This apply to FLEX access, egress and direct services.
    */
   public Instant bookingTime() {
     return bookingTime;