Issue 443 - Addressing Review Comments

Author: Emily Toop

Prox/Prox/Utilities/GoogleDirectionsMatrixTravelTimesProvider.swift

@@ -120,7 +120,7 @@ class GoogleDirectionsMatrixTravelTimesProvider: TravelTimesProvider {
         let dataTask = sessionManager.dataTask(with: request) { response, object, error in
             if error != nil {
                 NSLog("Error fetching Google Distance Matrix request \(error)")
-                completion(nil)
+                return completion(nil)
             }
 
             guard let responseObject = object as? [String: Any],

Prox/Prox/Utilities/PlaceUtilities.swift

@@ -45,6 +45,7 @@ struct PlaceUtilities {
             }
             let sortedByTravelTime: [Place] = sortedTravelTimes.flatMap { time in
                 let placeIndex = sortedByDistance.index { place in
+                    // if the TravelTime does not have a place id, then we have to match TravelTimes to Places via it's coordinates instead
                     return time.destinationPlaceKey == place.id
                         || (time.destination?.latitude == place.latLong.latitude && time.destination?.longitude == place.latLong.longitude)
                 }