Set transit-group-priority for requests without via points

application/src/main/java/org/opentripplanner/routing/algorithm/raptoradapter/transit/mappers/RaptorRequestMapper.java

@@ -199,11 +199,17 @@ private RaptorRequest<T> doMap() {
   }
 
   private boolean hasPassThroughOnly() {
-    return request.getViaLocations().stream().allMatch(ViaLocation::isPassThroughLocation);
+    return (
+      request.isViaSearch() &&
+      request.getViaLocations().stream().allMatch(ViaLocation::isPassThroughLocation)
+    );
   }
 
   private boolean hasViaLocationsOnly() {
-    return request.getViaLocations().stream().noneMatch(ViaLocation::isPassThroughLocation);
+    return (
+      request.isViaSearch() &&
+      request.getViaLocations().stream().noneMatch(ViaLocation::isPassThroughLocation)
+    );
   }
 
   private boolean hasViaLocationsAndPassThroughLocations() {

application/src/test/java/org/opentripplanner/routing/algorithm/raptoradapter/transit/mappers/RaptorRequestMapperTest.java

@@ -1,6 +1,7 @@
 package org.opentripplanner.routing.algorithm.raptoradapter.transit.mappers;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.time.Duration;
@@ -90,6 +91,37 @@ void testPassThroughPoints() {
     );
   }
 
+  @Test
+  void testTransitGroupPriority() {
+    var req = new RouteRequest();
+
+    // Set relax transit-group-priority
+    req.withPreferences(p ->
+      p.withTransit(t -> t.withRelaxTransitGroupPriority(CostLinearFunction.of("30m + 1.2t")))
+    );
+
+    var result = map(req);
+
+    assertFalse(result.multiCriteria().transitPriorityCalculator().isEmpty());
+  }
+
+  @Test
+  void testVisitViaAllowsTransitGroupPriority() {
+    var req = new RouteRequest();
+
+    // Set visit-via and relax transit-group-priority
+    req.setViaLocations(
+      List.of(new VisitViaLocation("Via A", null, List.of(STOP_A.getId()), List.of()))
+    );
+    req.withPreferences(p ->
+      p.withTransit(t -> t.withRelaxTransitGroupPriority(CostLinearFunction.of("30m + 1.2t")))
+    );
+
+    var result = map(req);
+
+    assertFalse(result.multiCriteria().transitPriorityCalculator().isEmpty());
+  }
+
   @Test
   void testPassThroughPointsTurnTransitGroupPriorityOff() {
     var req = new RouteRequest();