Removed LoadAsync call to Basemap before loading MapView. (#582)

Refactor async methods to sync in BasemapGallery.
Esri · Jun 26, 2024 · ae34e71 · ae34e71
1 parent d7dd108
commit ae34e71
Showing 1 changed file with 8 additions and 9 deletions.
diff --git a/src/Toolkit/Toolkit/UI/Controls/BasemapGallery/BasemapGalleryController.cs b/src/Toolkit/Toolkit/UI/Controls/BasemapGallery/BasemapGalleryController.cs
@@ -143,7 +143,7 @@ private void HandleAvailableBasemapsChanged()
                 AvailableBasemaps?.ToList()?.ForEach(bmgi => bmgi.NotifySpatialReferenceChanged(GeoModel));
 
                 // Update selection.
-                _ = UpdateSelectionForGeoModelBasemap();
+                UpdateSelectionForGeoModelBasemap();
             }
             finally
             {
@@ -160,25 +160,25 @@ private void HandleAvailableBasemapsCollectionChanged(object? sender, NotifyColl
                 case NotifyCollectionChangedAction.Replace:
                 case NotifyCollectionChangedAction.Reset:
                     e.NewItems?.OfType<BasemapGalleryItem>().ToList().ForEach(bmgi => bmgi.NotifySpatialReferenceChanged(GeoModel));
-                    _ = UpdateSelectionForGeoModelBasemap();
+                    UpdateSelectionForGeoModelBasemap();
                     break;
                 case NotifyCollectionChangedAction.Remove:
-                    _ = UpdateSelectionForGeoModelBasemap();
+                    UpdateSelectionForGeoModelBasemap();
                     break;
             }
         }
 
         private void HandleGeoModelChanged()
         {
             AvailableBasemaps?.ToList().ForEach(item => item.NotifySpatialReferenceChanged(GeoModel));
-            _ = UpdateSelectionForGeoModelBasemap();
+            UpdateSelectionForGeoModelBasemap();
         }
 
         private void HandleGeoModelPropertyChanged(object? sender, PropertyChangedEventArgs e)
         {
             if (e.PropertyName == nameof(GeoModel.Basemap) && !_ignoreEventsFlag)
             {
-                _ = UpdateSelectionForGeoModelBasemap();
+                UpdateSelectionForGeoModelBasemap();
             }
             else if (e.PropertyName == nameof(GeoModel.SpatialReference))
             {
@@ -244,11 +244,11 @@ public async Task LoadFromDefaultPortal()
             }
         }
 
-        private async Task UpdateSelectionForGeoModelBasemap()
+        private void UpdateSelectionForGeoModelBasemap()
         {
             if (GeoModel?.Basemap is Basemap inputBasemap)
             {
-                if (await BasemapIsActuallyNotABasemap(inputBasemap))
+                if (BasemapIsActuallyNotABasemap(inputBasemap))
                 {
                     SelectedBasemap = null;
                 }
@@ -270,9 +270,8 @@ private async Task UpdateSelectionForGeoModelBasemap()
         /// <summary>
         /// Maps and scenes start with an empty basemap that should not be shown in the UI.
         /// </summary>
-        private static async Task<bool> BasemapIsActuallyNotABasemap(Basemap input)
+        private static bool BasemapIsActuallyNotABasemap(Basemap input)
         {
-            await input.LoadAsync();
             if (!input.BaseLayers.Any() && !input.ReferenceLayers.Any())
             {
                 return true;