Improved

Author: 0x5bfa

src/Files.App.Storage/Storables/WindowsStorage/WindowsStorableHelpers.Shell.cs

@@ -48,8 +48,7 @@ public unsafe static HRESULT GetPropertyValue<TValue>(this IWindowsStorable stor
 
 		public unsafe static bool HasShellAttributes(this IWindowsStorable storable, SFGAO_FLAGS attributes)
 		{
-			return storable.ThisPtr->GetAttributes(SFGAO_FLAGS.SFGAO_FOLDER, out var returnedAttributes).Succeeded &&
-				returnedAttributes == attributes;
+			return storable.ThisPtr->GetAttributes(attributes, out var dwRetAttributes).Succeeded && dwRetAttributes == attributes;
 		}
 
 		public unsafe static string GetDisplayName(this IWindowsStorable storable, SIGDN options = SIGDN.SIGDN_FILESYSPATH)

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

@@ -206,9 +206,9 @@ public override async Task ExecutePinToSidebarCommand(WidgetCardItem? item)
 			{
 				unsafe
 				{
-					using ComPtr<IShellItem> pShellItem = default;
-					hr = pAgileReference.Get()->Resolve(IID.IID_IShellItem, (void**)pShellItem.GetAddressOf());
-					var windowsFile = new WindowsFile(pShellItem.Get());
+					IShellItem* pShellItem = null;
+					hr = pAgileReference.Get()->Resolve(IID.IID_IShellItem, (void**)&pShellItem);
+					using var windowsFile = new WindowsFile(pShellItem);
 
 					// NOTE: "pintohome" is an undocumented verb, which calls an undocumented COM class, windows.storage.dll!CPinToFrequentExecute : public IExecuteCommand, ...
 					return windowsFile.TryInvokeContextMenuVerb("pintohome");
@@ -242,9 +242,9 @@ public override async Task ExecuteUnpinFromSidebarCommand(WidgetCardItem? item)
 			{
 				unsafe
 				{
-					using ComPtr<IShellItem> pShellItem = default;
-					hr = pAgileReference.Get()->Resolve(IID.IID_IShellItem, (void**)pShellItem.GetAddressOf());
-					var windowsFile = new WindowsFile(pShellItem.Get());
+					IShellItem* pShellItem = null;
+					hr = pAgileReference.Get()->Resolve(IID.IID_IShellItem, (void**)&pShellItem);
+					using var windowsFile = new WindowsFile(pShellItem);
 
 					// NOTE: "unpinfromhome" is an undocumented verb, which calls an undocumented COM class, windows.storage.dll!CRemoveFromFrequentPlacesExecute : public IExecuteCommand, ...
 					// NOTE: "remove" is for some shell folders where the "unpinfromhome" may not work