Explicitly disable turn costs for /isochrone so there is no error if the encoder supports turn costs

Author: easbar

web-bundle/src/main/java/com/graphhopper/resources/IsochroneResource.java

@@ -76,6 +76,9 @@ public Response doGet(
         RouteResource.initHints(hintsMap, uriInfo.getQueryParameters());
         hintsMap.putObject(Parameters.CH.DISABLE, true);
         hintsMap.putObject(Parameters.Landmark.DISABLE, true);
+        // make sure to explicitly disable turn costs when resolving the profile, otherwise it might be true depending
+        // on the encoder
+        hintsMap.putObject(Parameters.Routing.EDGE_BASED, false);
         ProfileConfig profile = graphHopper.resolveProfile(hintsMap);
         if (profile.isTurnCosts()) {
             throw new IllegalArgumentException("Isochrone calculation does not support turn costs yet");

web-bundle/src/main/java/com/graphhopper/resources/SPTResource.java

@@ -64,6 +64,9 @@ public Response doGet(
         RouteResource.initHints(hintsMap, uriInfo.getQueryParameters());
         hintsMap.putObject(Parameters.CH.DISABLE, true);
         hintsMap.putObject(Parameters.Landmark.DISABLE, true);
+        // make sure to explicitly disable turn costs when resolving the profile, otherwise it might be true depending
+        // on the encoder
+        hintsMap.putObject(Parameters.Routing.EDGE_BASED, false);
         ProfileConfig profile = graphHopper.resolveProfile(hintsMap);
         if (profile.isTurnCosts()) {
             throw new IllegalArgumentException("SPT calculation does not support turn costs yet");

web/src/test/java/com/graphhopper/http/resources/IsochroneResourceTest.java

@@ -21,7 +21,6 @@
 import com.fasterxml.jackson.databind.JsonNode;
 import com.graphhopper.http.GraphHopperApplication;
 import com.graphhopper.http.util.GraphHopperServerTestConfiguration;
-import static com.graphhopper.http.util.TestUtils.clientTarget;
 import com.graphhopper.json.geo.JsonFeatureCollection;
 import com.graphhopper.util.Helper;
 import io.dropwizard.testing.junit.DropwizardAppRule;
@@ -35,6 +34,7 @@
 import javax.ws.rs.core.Response;
 import java.io.File;
 
+import static com.graphhopper.http.util.TestUtils.clientTarget;
 import static com.graphhopper.util.Parameters.Routing.BLOCK_AREA;
 import static junit.framework.TestCase.assertFalse;
 import static junit.framework.TestCase.assertTrue;
@@ -47,7 +47,8 @@ public class IsochroneResourceTest {
 
     static {
         config.getGraphHopperConfiguration().
-                putObject("graph.flag_encoders", "car").
+                // isochrone does not support turn costs yet, but use it anyway to make sure this is handled correctly
+                        putObject("graph.flag_encoders", "car|turn_costs=true").
                 putObject("datareader.file", "../core/files/andorra.osm.pbf").
                 putObject("graph.location", DIR);
     }