Fixed bug #530 in declaration of IPropertyStoreFactory.GetPropertyStore.

Author: dahall

PInvoke/Ole/PropSys/PropSys.Interfaces.cs

@@ -1502,7 +1502,7 @@ public interface IPropertyStoreFactory
 		/// </remarks>
 		// https://docs.microsoft.com/en-us/windows/desktop/api/propsys/nf-propsys-ipropertystorefactory-getpropertystore HRESULT
 		// GetPropertyStore( GETPROPERTYSTOREFLAGS flags, IUnknown *pUnkFactory, REFIID riid, void **ppv );
-		void GetPropertyStore(GETPROPERTYSTOREFLAGS flags, [Optional, MarshalAs(UnmanagedType.IUnknown)] ICreateObject? pUnkFactory, in Guid riid, out IPropertyStore ppv);
+		void GetPropertyStore(GETPROPERTYSTOREFLAGS flags, [Optional] ICreateObject? pUnkFactory, in Guid riid, out IPropertyStore ppv);
 
 		/// <summary>
 		/// <para>

UnitTests/PInvoke/Shell32/IShellFolderTests.cs

@@ -1,5 +1,7 @@
 using NUnit.Framework;
+using System.Linq;
 using Vanara.Collections;
+using static Vanara.PInvoke.PropSys;
 using static Vanara.PInvoke.Shell32;
 
 namespace Vanara.PInvoke.Tests;
@@ -47,4 +49,20 @@ public void EnumSearchesTest()
 		}
 
 	}
+
+	[Test]
+	public void Issue530Test()
+	{
+		var pFolder = (IShellFolder2)new MyDocuments();
+		var item = pFolder.EnumObjects().FirstOrDefault();
+		Assert.That(item, Is.Not.Null);
+
+		IPropertyStoreFactory? propertyStoreFactoryForChild = null;
+		Assert.That(() => propertyStoreFactoryForChild = pFolder.BindToObject<IPropertyStoreFactory>(item!), Throws.Nothing);
+		Assert.That(propertyStoreFactoryForChild, Is.Not.Null);
+
+		IPropertyStore? propertyStoreForChild = null;
+		Assert.That(() => propertyStoreFactoryForChild!.GetPropertyStore(GETPROPERTYSTOREFLAGS.GPS_DEFAULT, null, typeof(IPropertyStore).GUID, out propertyStoreForChild), Throws.Nothing);
+		Assert.That(propertyStoreForChild, Is.Not.Null);
+	}
 }