@@ -48,12+49,15 @@ public class RouteJob extends Job {
return Status.OK_STATUS;
}
} catch (RoutingException e) {
return Status.OK_STATUS;
}
} catch (RoutingException e) {
- LOGGER.error("Routing failed for waypoints {} and router {}", waypoints, router, e);
+ LOGGER.error("Routing failed for waypoints {} and router '{}'", waypoints, router, e);
callback.completeExceptionally(e);
callback.completeExceptionally(e);
+ // This results in UI feedback of the error.
+ // Perhaps later we get rid of this and let the UI code handle the notifications.
+ return new Status(IStatus.ERROR, Activator.PLUGIN_ID, e.getMessage());
}
}
LOGGER.info("No router could calculate route between waypoints {}. Available routers: {}", waypoints, routers);
}
}
LOGGER.info("No router could calculate route between waypoints {}. Available routers: {}", waypoints, routers);
- return Status.OK_STATUS;
+ return new Status(IStatus.ERROR, Activator.PLUGIN_ID, NLS.bind("No router could calculate route between waypoints {0}. Available routers: {1}", waypoints, routers));