diff --git a/src/main/java/org/opentripplanner/routing/core/State.java b/src/main/java/org/opentripplanner/routing/core/State.java index 1d931bfd211..690c9f136b5 100644 --- a/src/main/java/org/opentripplanner/routing/core/State.java +++ b/src/main/java/org/opentripplanner/routing/core/State.java @@ -1,8 +1,10 @@ package org.opentripplanner.routing.core; -import java.util.*; - -import org.opentripplanner.model.Station; +import java.util.ArrayList; +import java.util.Collection; +import java.util.Date; +import java.util.Objects; +import java.util.Set; import org.opentripplanner.routing.algorithm.astar.NegativeWeightException; import org.opentripplanner.routing.api.request.RoutingRequest; import org.opentripplanner.routing.edgetype.StreetEdge; @@ -38,9 +40,6 @@ public class State implements Cloneable { // we should DEFINITELY rename this variable and the associated methods. public double walkDistance; - // keep track of transit stations entered to prevent out-of-system transfers - public List transitStationList = new ArrayList(); - /* CONSTRUCTORS */ /** diff --git a/src/main/java/org/opentripplanner/routing/core/StateEditor.java b/src/main/java/org/opentripplanner/routing/core/StateEditor.java index 01aa6c239b3..10458669cd1 100644 --- a/src/main/java/org/opentripplanner/routing/core/StateEditor.java +++ b/src/main/java/org/opentripplanner/routing/core/StateEditor.java @@ -1,13 +1,11 @@ package org.opentripplanner.routing.core; -import org.opentripplanner.model.Station; import org.opentripplanner.routing.graph.Edge; import org.opentripplanner.routing.graph.Vertex; import org.opentripplanner.routing.api.request.RoutingRequest; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import java.util.Collections; import java.util.Set; /** @@ -395,16 +393,4 @@ public void setBikeRentalNetwork(Set networks) { public State getBackState() { return child.getBackState(); } - - // record that station was entered - public void markTransitStation(Station station) { - if (station != null) - child.transitStationList.add(station); - } - - // how many stations have been entered - public Set getTransitStationsEntered() { - return Set.copyOf(child.transitStationList); - } - } diff --git a/src/main/java/org/opentripplanner/routing/edgetype/StreetTransitEntityLink.java b/src/main/java/org/opentripplanner/routing/edgetype/StreetTransitEntityLink.java index e3825959746..d94ac97b373 100644 --- a/src/main/java/org/opentripplanner/routing/edgetype/StreetTransitEntityLink.java +++ b/src/main/java/org/opentripplanner/routing/edgetype/StreetTransitEntityLink.java @@ -138,16 +138,6 @@ else if (s0.isBikeRentingFromStation() && !(s0.mayKeepRentedBicycleAtDestination int streetToStopTime = getStreetToStopTime(); s1.incrementTimeInSeconds(streetToStopTime); s1.incrementWeight(STEL_TRAVERSE_COST + streetToStopTime); - if (this instanceof StreetTransitEntranceLink) { - // if we are entering transit mark it, but allow us to leave! - if (isLeavingStreetNetwork(req)) { - if (s1.getTransitStationsEntered().size() > 1) { - // ban this entrance, we've already taken an entrance - return null; - } - s1.markTransitStation(getTransitEntityVertex().getStationElement().getParentStation()); - } - } return s1.makeState(); }