[BUG] Explicit interface implementation issue

[ahdung]

Version: 5.0.0.4776-preview1

If you need this dll, I can provide.

[siegfriedpammer]

Yes, please provide the DLL. Seems like there is something missing in the metadata tables.

Thank you!

[ahdung]

I thank you, you sir are so great.

at DevComponents.DotNetBar.PanelEx last content.
FastReport.Bars.zip

[siegfriedpammer]

Thank you very much!

It's just like I suspected: There is no info in the Property, PropertyMap and MethodSemantics tables for these methods. I think, the only thing we could do is strip invalid modifiers from the declarations when generating C# code.

[ahdung]

@siegfriedpammer I think you can do more thing, like check it by Interface, Ixxx has Enabled property, class shouldn't has Ixxx.get_Enabled().

[yyjdelete]

See an reproduce for this, and it can be generated with roslyn.
Seems roslyn will generated those proxy method for interface when

case 1: none of ancestor impl the interface, and the class(ClassA1A) and the ancestor(ClassA2A) which the impl methods come from different assembly.

case 2: the impl methods(ClassA1C or ClassA1D) is defined in subclass of first ancestor which impl the interface (ClassA1E) but not override(directly use newslot or override an newslot) the methods in it.

Not sure if there is any document for this, but if those generated methods is removed manually:
For case 1: get an System.TypeLoadException: Method "xxx" is not implemented for runtime, and Error: Class implements interface but not method, Class: '' Interface: '' Missing method for ILVerify.
And for case 2 matches the wrong method.

//In assembly1
    public class ClassA1A : ClassA2A, IA
    {
    }
    public class ClassA1B : ClassA1C, IA
    {
    }
    public class ClassA1C : ClassA1D
    {
        public new int A { get; set; } = 1;
    }
    public class ClassA1D : ClassA1E
    {
        public new void C() { Console.WriteLine("ClassA1D"); }
    }
    public class ClassA1E : IA
    {
        public int A { get; set; }
        public event Action B;//also for events
        public void C() { Console.WriteLine("ClassA1E"); }//normal methods still works
    }

    //In any assembly
    public interface IA
    {
        int A { get; set; }
        event Action B;
        void C();
    }

//In another assembly2
public class ClassA2A
{
    public int A { get; set; }
    public event Action B;
    public void C() { Console.WriteLine("A2A"); }
}

---

Updated:
This will also generate an proxy method if it's not virtual, and remove it lead to missing method.
https://sharplab.io/#v2:CYLg1APgAgTAjAWAFBQMwAICWA7ALgUwCcAzAQwGN90BJAYXQG9l0Ws90BZLgChzd3QB9AJQBuZAF9kadLHT0QNekyStZGHAK4de2fkOHoAvAD4h4pFNWsgA
Normal method works well, but it can also be an indexer property.

    public interface IC
    {
        int MMM(in int _);
    }
    public class C : IC
    {
        public int MMM(in int x) => x;
    }

[siegfriedpammer]

Thank you very much for providing these reproduction steps!

[siegfriedpammer]

See Roslyn source code for the rules used by the C# compiler:

https://github.com/dotnet/roslyn/blob/fb028b7c198c2f726067bb8627cb014f1154a541/src/Compilers/CSharp/Portable/Symbols/Source/SourceMemberContainerSymbol_ImplementationChecks.cs#L1466-L1523