iOS LocationService thread safety architecture changes.

Author: David Britch

src/Mobile/eShopOnContainers/eShopOnContainers.iOS/Services/LocationServiceImplementation.cs

@@ -15,32 +15,24 @@ namespace eShopOnContainers.iOS.Services
 {
     public class LocationServiceImplementation : ILocationServiceImplementation
     {
-        public double DesiredAccuracy { get; set; }
-        public bool IsGeolocationAvailable => true;
-        public bool IsGeolocationEnabled
-        {
-            get
-            {
-                var status = CLLocationManager.Status;
-                return CLLocationManager.LocationServicesEnabled;
-            }
-        }
+        Lazy<IPermissionsService> _permissionsService;
 
         public LocationServiceImplementation()
         {
             DesiredAccuracy = 100;
+            _permissionsService = new Lazy<IPermissionsService>(() => new PermissionsService(), LazyThreadSafetyMode.PublicationOnly);
         }
 
         #region Internal Implementation
 
         async Task<bool> CheckPermissions(Permission permission)
         {
-            var status = await PermissionsService.Current.CheckPermissionStatusAsync(permission);
+            var status = await _permissionsService.Value.CheckPermissionStatusAsync(permission);
             if (status != PermissionStatus.Granted)
             {
                 Console.WriteLine("Currently do not have Location permissions, requesting permissions");
 
-                var request = await PermissionsService.Current.RequestPermissionsAsync(permission);
+                var request = await _permissionsService.Value.RequestPermissionsAsync(permission);
                 if (request[permission] != PermissionStatus.Granted)
                 {
                     Console.WriteLine("Location permission denied, can not get positions async.");
@@ -61,6 +53,17 @@ CLLocationManager GetManager()
 
         #region ILocationServiceImplementation
 
+        public double DesiredAccuracy { get; set; }
+        public bool IsGeolocationAvailable => true;
+        public bool IsGeolocationEnabled
+        {
+            get
+            {
+                var status = CLLocationManager.Status;
+                return CLLocationManager.LocationServicesEnabled;
+            }
+        }
+
         public async Task<Position> GetPositionAsync(TimeSpan? timeout, CancellationToken? cancelToken = null)
         {
             var permission = Permission.LocationWhenInUse;

src/Mobile/eShopOnContainers/eShopOnContainers.iOS/Services/PermissionsService.cs

@@ -13,20 +13,6 @@ public class PermissionsService : IPermissionsService
     {
         CLLocationManager _locationManager;
 
-        static Lazy<IPermissionsService> implementation = new Lazy<IPermissionsService>(CreatePermissions, System.Threading.LazyThreadSafetyMode.PublicationOnly);
-        static IPermissionsService CreatePermissions()
-        {
-            return new PermissionsService();
-        }
-
-        public static IPermissionsService Current
-        {
-            get
-            {
-                return implementation.Value;
-            }
-        }
-
         #region Internal Implementation
 
         PermissionStatus GetLocationPermissionStatus(Permission permission)