GH-1881 - Fix interface method attributes when generating proxies

Author: PleasantD

src/NHibernate.Test/NHSpecificTest/GH1881/TestFixture.cs

@@ -0,0 +1,33 @@
+using System;
+using System.Linq;
+using System.Reflection;
+using NHibernate.Collection;
+using NHibernate.Collection.Generic;
+using NHibernate.Proxy.DynamicProxy;
+using NUnit.Framework;
+
+namespace NHibernate.Test.NHSpecificTest.GH1881
+{
+	[TestFixture]
+	[Obsolete]
+	public class TestFixture
+	{
+		[Test]
+		public void InterfacesShouldBeImplementedExplicitlyOnProxies()
+		{
+			var proxy = new ProxyFactory().CreateProxy(typeof(PersistentGenericBag<object>), null, typeof(ILazyInitializedCollection));
+			Assert.That(proxy, Is.Not.Null);
+
+			foreach (var method in proxy.GetType().GetMethods().Where(m => m.DeclaringType == typeof(ILazyInitializedCollection)))
+			{
+				// These attributes are what .NET uses for explicitly implemented interface methods
+				Assert.That(method.Attributes, Is.EqualTo(MethodAttributes.Private |
+				                                          MethodAttributes.Final |
+				                                          MethodAttributes.Virtual |
+				                                          MethodAttributes.HideBySig |
+				                                          MethodAttributes.NewSlot |
+				                                          MethodAttributes.SpecialName));
+			}
+		}
+	}
+}

src/NHibernate/Proxy/DynamicProxy/ProxyFactory.cs

@@ -118,13 +118,13 @@ private TypeInfo CreateUncachedProxyType(System.Type baseType, IReadOnlyCollecti
 
 			// Provide a custom implementation of ISerializable
 			// instead of redirecting it back to the interceptor
-			foreach (MethodInfo method in ProxyBuilderHelper.GetProxiableMethods(baseType, interfaces).Where(method => method.DeclaringType != typeof(ISerializable)))
+			foreach (MethodInfo method in ProxyBuilderHelper.GetProxiableMethods(parentType, interfaces).Where(method => method.DeclaringType != typeof(ISerializable)))
 			{
 				ProxyMethodBuilder.CreateProxiedMethod(interceptorField, method, typeBuilder);
 			}
 
 			// Make the proxy serializable
-			AddSerializationSupport(baseType, baseInterfaces, typeBuilder, interceptorField, defaultConstructor);
+			AddSerializationSupport(parentType, baseInterfaces, typeBuilder, interceptorField, defaultConstructor);
 			TypeInfo proxyType = typeBuilder.CreateTypeInfo();
 
 			ProxyAssemblyBuilder.Save(assemblyBuilder);

src/NHibernate/Proxy/NHibernateProxyBuilder.cs

@@ -394,7 +394,7 @@ private static void EmitCallBaseIfLazyInitializerIsNull(
 			IL.Emit(OpCodes.Bne_Un, skipBaseCall);
 
 			IL.Emit(OpCodes.Ldarg_0);
-			EmitCallMethod(IL, OpCodes.Call, method);
+			EmitCallMethod(IL, method.DeclaringType.IsInterface ? OpCodes.Callvirt : OpCodes.Call, method);
 			IL.Emit(OpCodes.Ret);
 
 			IL.MarkLabel(skipBaseCall);

src/NHibernate/Proxy/ProxyBuilderHelper.cs

@@ -137,7 +137,24 @@ internal static MethodBuilder GetObjectDataMethodBuilder(TypeBuilder typeBuilder
 		internal static MethodBuilder GenerateMethodSignature(string name, MethodInfo method, TypeBuilder typeBuilder)
 		{
 			//TODO: Should we use attributes of base method?
-			var methodAttributes = MethodAttributes.Public | MethodAttributes.HideBySig | MethodAttributes.Virtual;
+			MethodAttributes methodAttributes;
+			if (method.DeclaringType.IsInterface)
+			{
+				// These are the attributes used for an explicit interface method implementation in .NET.
+				methodAttributes =
+					MethodAttributes.Private |
+					MethodAttributes.Final |
+					MethodAttributes.Virtual |
+					MethodAttributes.HideBySig |
+					MethodAttributes.NewSlot |
+					MethodAttributes.SpecialName;
+				// .NET uses an expanded name for explicit interface implementation methods.
+				name = typeBuilder.FullName + "." + method.DeclaringType.FullName + "." + name;
+			}
+			else
+			{
+				methodAttributes = MethodAttributes.Public | MethodAttributes.HideBySig | MethodAttributes.Virtual;
+			}
 
 			if (method.IsSpecialName)
 				methodAttributes |= MethodAttributes.SpecialName;