Separate walk from non-transit time

src/main/java/org/opentripplanner/apis/gtfs/datafetchers/ItineraryImpl.java

@@ -94,12 +94,12 @@ public DataFetcher<Long> waitingTime() {
 
   @Override
   public DataFetcher<Double> walkDistance() {
-    return environment -> getSource(environment).getNonTransitDistanceMeters();
+    return environment -> getSource(environment).walkDistanceMeters();
   }
 
   @Override
   public DataFetcher<Long> walkTime() {
-    return environment -> (long) getSource(environment).getNonTransitDuration().toSeconds();
+    return environment -> (long) getSource(environment).walkDuration().toSeconds();
   }
 
   @Override

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

@@ -158,7 +158,7 @@ public static GraphQLObjectType create(
           .name("walkDistance")
           .deprecate("Replaced by `streetDistance`.")
           .type(Scalars.GraphQLFloat)
-          .dataFetcher(env -> itinerary(env).getNonTransitDistanceMeters())
+          .dataFetcher(env -> itinerary(env).walkDistanceMeters())
           .build()
       )
       .field(

src/main/java/org/opentripplanner/model/plan/ItinerariesCalculateLegTotals.java

@@ -13,6 +13,8 @@ class ItinerariesCalculateLegTotals {
   int nTransitLegs = 0;
   Duration nonTransitDuration = Duration.ZERO;
   double nonTransitDistanceMeters = 0.0;
+  Duration walkDuration = Duration.ZERO;
+  double walkDistanceMeters = 0.0;
   Duration waitingDuration = Duration.ZERO;
   boolean walkOnly = true;
   boolean streetOnly = true;
@@ -42,9 +44,14 @@ private void calculate(List<Leg> legs) {
         if (!leg.isInterlinedWithPreviousLeg()) {
           ++nTransitLegs;
         }
-      } else if (leg.isStreetLeg()) {
+      } else if (leg instanceof StreetLeg streetLeg){
         nonTransitDuration = nonTransitDuration.plus(dt);
         nonTransitDistanceMeters += leg.getDistanceMeters();
+
+        if(streetLeg.isWalkingLeg()){
+          walkDuration = walkDuration.plus(streetLeg.getDuration());
+          walkDistanceMeters = walkDistanceMeters + streetLeg.getDistanceMeters();
+        }
       } else if (leg instanceof UnknownTransitPathLeg unknownTransitPathLeg) {
         nTransitLegs += unknownTransitPathLeg.getNumberOfTransfers() + 1;
       }

src/main/java/org/opentripplanner/model/plan/Itinerary.java

@@ -39,6 +39,9 @@ public class Itinerary implements ItinerarySortKey {
   private final boolean walkOnly;
   private final boolean streetOnly;
   private final Duration nonTransitDuration;
+  private final Duration walkDuration;
+  private final double walkDistanceMeters;
+
 
   /* mutable primitive properties */
   private Double elevationLost = 0.0;
@@ -75,6 +78,8 @@ public Itinerary(List<Leg> legs) {
     this.transitDuration = totals.transitDuration;
     this.nonTransitDuration = totals.nonTransitDuration;
     this.nonTransitDistanceMeters = DoubleUtils.roundTo2Decimals(totals.nonTransitDistanceMeters);
+    this.walkDuration = totals.walkDuration;
+    this.walkDistanceMeters = totals.walkDistanceMeters;
     this.waitingDuration = totals.waitingDuration;
     this.walkOnly = totals.walkOnly;
     this.streetOnly = totals.streetOnly;
@@ -663,4 +668,18 @@ public void setEmissionsPerPerson(Emissions emissionsPerPerson) {
   public Emissions getEmissionsPerPerson() {
     return this.emissionsPerPerson;
   }
+
+  /**
+   * How much walking this itinerary contains, in meters.
+   */
+  public double walkDistanceMeters() {
+      return walkDistanceMeters;
+  }
+
+  /**
+   * How long the walking is contained in this itinerary.
+   */
+  public Duration walkDuration() {
+    return walkDuration;
+  }
 }

src/test/resources/org/opentripplanner/apis/gtfs/expectations/plan-extended.json

@@ -11,6 +11,8 @@
             "co2" : 123.0
           },
           "numberOfTransfers" : 1,
+          "walkDistance" : 28.0,
+          "walkTime" : 20,
           "legs" : [
             {
               "mode" : "WALK",

src/test/resources/org/opentripplanner/apis/gtfs/queries/plan-extended.graphql

@@ -24,6 +24,8 @@
                 co2
             }
             numberOfTransfers
+            walkDistance
+            walkTime
             legs {
                 mode
                 from {