From 1967430c40d3d0025efb4f2d6dc33df2171158bc Mon Sep 17 00:00:00 2001
From: Michal Maciejewski <michal.mac@gmail.com>
Date: Tue, 14 Nov 2017 11:37:02 +0100
Subject: [PATCH] bugfix: all jobs in initial routes are included into
 VRP.allJobs

---
 .../jsprit/core/problem/VehicleRoutingProblem.java     | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java
index 913c5eea..ce3f9cad 100644
--- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java
+++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java
@@ -81,7 +81,7 @@ public class VehicleRoutingProblem {
 
         private Map<String, Job> tentativeJobs = new LinkedHashMap<String, Job>();
 
-        private Set<String> jobsInInitialRoutes = new HashSet<String>();
+        private Set<Job> jobsInInitialRoutes = new HashSet<>();
 
         private Map<String, Coordinate> tentative_coordinates = new HashMap<String, Coordinate>();
 
@@ -308,7 +308,7 @@ public class VehicleRoutingProblem {
                 incActivityIndexCounter();
                 if (act instanceof TourActivity.JobActivity) {
                     Job job = ((TourActivity.JobActivity) act).getJob();
-                    jobsInInitialRoutes.add(job.getId());
+                    jobsInInitialRoutes.add(job);
                     addLocationToTentativeLocations(job);
                     registerJobAndActivity(abstractAct, job);
                 }
@@ -429,7 +429,7 @@ public class VehicleRoutingProblem {
                 transportCosts = new CrowFlyCosts(getLocations());
             }
             for (Job job : tentativeJobs.values()) {
-                if (!jobsInInitialRoutes.contains(job.getId())) {
+                if (!jobsInInitialRoutes.contains(job)) {
                     addJobToFinalJobMapAndCreateActivities(job);
                 }
             }
@@ -438,8 +438,8 @@ public class VehicleRoutingProblem {
             for (Job job : jobs.values()) {
                 ((AbstractJob)job).setIndex(jobIndexCounter++);
             }
-            for (String jobId : jobsInInitialRoutes) {
-                ((AbstractJob)tentativeJobs.get(jobId)).setIndex(jobIndexCounter++);
+            for (Job job : jobsInInitialRoutes) {
+                ((AbstractJob)job).setIndex(jobIndexCounter++);
             }
             
             boolean hasBreaks = addBreaksToActivityMap();