Search is closed properly.

1Weather/UI/View controllers/Locations/LocationViewController.swift

@@ -202,7 +202,7 @@ extension LocationViewController: LocationsViewModelDelegate {
         self.showAlert(withTitle: title, message: message)
     }
     
-    func viewModelDidSelectCity(model: LocationsViewModel, city: Location) {
+    func viewModelDidSelectCity(model: LocationsViewModel) {
         close()
     }
 }
@@ -353,8 +353,6 @@ extension LocationViewController: UITableViewDelegate {
 //                analytics(log: .ANALYTICS_FTUE_SEARCH_POPULAR)
 //            }
         }
-        self.close()
-        //TODO: should be done from ViewModel?
     }
 }
 

1Weather/UI/View controllers/Locations/LocationsViewModel.swift

@@ -12,7 +12,7 @@ public protocol LocationsViewModelDelegate:class {
     func viewModelDidChange(model:LocationsViewModel)
     func viewModelDisplayModeDidChange(model: LocationsViewModel)
     func viewModelError(model:LocationsViewModel, title:String?, message:String?)
-    func viewModelDidSelectCity(model:LocationsViewModel, city: Location)
+    func viewModelDidSelectCity(model:LocationsViewModel)
 }
 
 public enum LocationsViewModelDisplayMode {
@@ -192,8 +192,7 @@ public class LocationsViewModel {
             return
         }
         locationManager.addIfNeeded(partialLocation: city, selectLocation: true)
-        // TODO: close View Controller here, otherwise we can't choose already selected city.
-        // Currently it's done in the didSelectRow method in the VC.
+        self.delegate?.viewModelDidSelectCity(model: self)
     }
     
     func delete(city: PartialLocation) {
@@ -232,9 +231,6 @@ extension LocationsViewModel: LocationManagerDelegate {
     public func locationManager(_ locationManager: LocationManager, changedSelectedLocation newLocation: Location?) {
         DispatchQueue.main.async {
             self.delegate?.viewModelDidChange(model: self)
-            if let location = newLocation {
-                self.delegate?.viewModelDidSelectCity(model: self, city: location)
-            }
         }
     }