Fix up 1

Author: 0x5bfa

src/Files.App/Services/Windows/WindowsQuickAccessService.cs

@@ -54,11 +54,11 @@ public async Task InitializeAsync()
 
 		public async Task<bool> UpdatePinnedFoldersAsync()
 		{
-			return await Task.Run(() =>
+			return await Task.Run(async () =>
 			{
 				try
 				{
-					List<LocationItem> items = [];
+					List<INavigationControlItem> items = [];
 					foreach (var path in GetPinnedFolders())
 						items.Add(await CreateItemOf(path));
 
@@ -75,13 +75,16 @@ public async Task<bool> UpdatePinnedFoldersAsync()
 
 					var eventArgs = GetChangedActionEventArgs(snapshot, items);
 					PinnedFoldersChanged?.Invoke(this, eventArgs);
+
+					return true;
 				}
 				catch
 				{
+					return false;
 				}
 			});
 
-			unsafe IEnumerable<string> GetPinnedFolders()
+			unsafe List<string> GetPinnedFolders()
 			{
 				HRESULT hr = default;
 
@@ -99,6 +102,7 @@ unsafe IEnumerable<string> GetPinnedFolders()
 
 				// Enumerate pinned folders
 				int index = 0;
+				List<string> paths = [];
 				using ComPtr<IShellItem> pShellItem = default;
 				while (pEnumShellItems.Get()->Next(1, pShellItem.GetAddressOf()) == HRESULT.S_OK)
 				{
@@ -114,10 +118,12 @@ unsafe IEnumerable<string> GetPinnedFolders()
 					var path = szDisplayName.ToString();
 					PInvoke.CoTaskMemFree(szDisplayName.Value);
 
-					yield return path;
+					paths.Add(path);
 
 					index++;
 				}
+
+				return paths;
 			}
 
 			async Task<LocationItem> CreateItemOf(string path)
@@ -181,7 +187,7 @@ async Task<LocationItem> CreateItemOf(string path)
 				return locationItem;
 			}
 
-			NotifyCollectionChangedEventArgs GetChangedActionEventArgs(IReadOnlyList<LocationItem> oldItems, IList<LocationItem> newItems)
+			NotifyCollectionChangedEventArgs GetChangedActionEventArgs(IReadOnlyList<INavigationControlItem> oldItems, IList<INavigationControlItem> newItems)
 			{
 				if (newItems.Count - oldItems.Count is 1)
 				{

src/Files.App/ViewModels/UserControls/Widgets/QuickAccessWidgetViewModel.cs

@@ -145,8 +145,8 @@ private async Task UpdateCollectionAsync(NotifyCollectionChangedEventArgs e)
 						{
 							if (e.NewItems is not null)
 							{
-								// e.NewItems.Cast<LocationItem>().Single()
-								var cardItem = new WidgetFolderCardItem();
+								var item = e.NewItems.Cast<LocationItem>().Single();
+								var cardItem = new WidgetFolderCardItem(item, SystemIO.Path.GetFileName(item.Text), true) { Path = item.Path };
 								AddItemToCollection(cardItem);
 							}
 						}
@@ -157,8 +157,8 @@ private async Task UpdateCollectionAsync(NotifyCollectionChangedEventArgs e)
 							{
 								Items.RemoveAt(e.OldStartingIndex);
 
-								// e.NewItems.Cast<LocationItem>().Single()
-								var cardItem = new WidgetFolderCardItem();
+								var item = e.NewItems.Cast<LocationItem>().Single();
+								var cardItem = new WidgetFolderCardItem(item, SystemIO.Path.GetFileName(item.Text), true) { Path = item.Path };
 								AddItemToCollection(cardItem);
 							}
 						}
@@ -178,7 +178,7 @@ private async Task UpdateCollectionAsync(NotifyCollectionChangedEventArgs e)
 								Items.Clear();
 								foreach (var item in items)
 								{
-									var cardItem = new WidgetFolderCardItem();
+									var cardItem = new WidgetFolderCardItem(item, SystemIO.Path.GetFileName(item.Text), true) { Path = item.Path };
 									AddItemToCollection(cardItem);
 								}
 							}