Access control modifiers fixed + a potential retain cycle addressed.

1Weather/Coordinators/TodayCoordinator.swift

@@ -7,26 +7,26 @@
 
 
 import UIKit
 import UIKit
 
 
-class TodayCoordinator:Coordinator {
-    //Private
+class TodayCoordinator: Coordinator {
+    // MARK: - Private
     private let navigationController = UINavigationController(nibName: nil, bundle: nil)
     private let navigationController = UINavigationController(nibName: nil, bundle: nil)
     private var tabBarController:UITabBarController?
     private var tabBarController:UITabBarController?
     
     
-    //Public
-    var childCoordinators = [Coordinator]()
-    var parentCoordinator: Coordinator?
+    // MARK: - Public
+    public var childCoordinators = [Coordinator]()
+    public var parentCoordinator: Coordinator?
     
     
-    init(tabBarController:UITabBarController) {
+    public init(tabBarController:UITabBarController) {
         self.tabBarController = tabBarController
         self.tabBarController = tabBarController
     }
     }
     
     
-    func start() {
+    public func start() {
         let todayViewController = TodayViewController(coordinator: self)
         let todayViewController = TodayViewController(coordinator: self)
         navigationController.viewControllers = [todayViewController]
         navigationController.viewControllers = [todayViewController]
         tabBarController?.add(viewController: navigationController)
         tabBarController?.add(viewController: navigationController)
     }
     }
     
     
-    func openLocationsSearch() {
+    public func openLocationsSearch() {
         let searchCoordinator = LocationSearchCoordinator(parentViewController: navigationController)
         let searchCoordinator = LocationSearchCoordinator(parentViewController: navigationController)
         searchCoordinator.parentCoordinator = self
         searchCoordinator.parentCoordinator = self
         searchCoordinator.start()
         searchCoordinator.start()

1Weather/Model/LocationManager.swift

@@ -21,7 +21,7 @@ public class LocationManager {
         
         
     private let weatherUpdateSource: WeatherSource
     private let weatherUpdateSource: WeatherSource
     private let healthSource: HealthSource
     private let healthSource: HealthSource
-    private let nwsAlertsManager: NWSAlertsManager
+    public let nwsAlertsManager: NWSAlertsManager
     private let storage: Storage
     private let storage: Storage
     private var defaultLocation = Location(deviceLocation: false,
     private var defaultLocation = Location(deviceLocation: false,
                                            coordinates: .init(latitude: 37.3230, longitude: -122.0322), // Cupertino
                                            coordinates: .init(latitude: 37.3230, longitude: -122.0322), // Cupertino

1Weather/UI/View controllers/Today/TodayViewController.swift

@@ -29,6 +29,7 @@ class TodayViewController: UIViewController {
         super.init(nibName: nil, bundle: nil)
         super.init(nibName: nil, bundle: nil)
     }
     }
     
     
+    @available(*, unavailable)
     required init?(coder: NSCoder) {
     required init?(coder: NSCoder) {
         fatalError("init(coder:) has not been implemented")
         fatalError("init(coder:) has not been implemented")
     }
     }
@@ -65,8 +66,8 @@ private extension TodayViewController {
     func prepareViewController() {
     func prepareViewController() {
         view.backgroundColor = ThemeManager.currentTheme.baseBackgroundColor
         view.backgroundColor = ThemeManager.currentTheme.baseBackgroundColor
         
         
-        localizationObserver = NotificationCenter.default.addObserver(forName: .localizationChange, object: nil, queue: .main, using: { _ in
-            self.tableView.reloadData()
+        localizationObserver = NotificationCenter.default.addObserver(forName: .localizationChange, object: nil, queue: .main, using: { [weak self] _ in
+            self?.tableView.reloadData()
         })
         })
     }
     }