Disable "shadowing" of methods?

[HLWeil]

Given the following class

type Person(name : string) =
    
    inherit DynamicObj()

    let mutable name = name

    member this.Name
        with get() = name
        and set(value) = name <- value

    member this.Walk () = printfn "%s is walking" name

You can currently set a dynamic property with the same name as an existing method.

let p = Person("John")

p.SetValue("Walk", "Shadowing")

p.Walk() // John is walking

p.GetValue("Walk") // val it: obj = "Shadowing"

Should this work, or fail? @kMutagene

[kMutagene]

Can we even transpile a lookup of the methods or is than non-transpilable reflection?

[HLWeil]

Yes this is transpilable as far as I know. In principle it could the same logic as the Systen.ICloneable.Clone() lookup you implemented for the DeepCopy method. E.g.

DynamicObj/src/DynamicObj/FableJS.fs

Line 163 in 073da2d

[<Emit("""$0["System.ICloneable.Clone"] != undefined && (typeof $0["System.ICloneable.Clone"]) === 'function'""")>]

[kMutagene]

On another Note, this currently removes the method in transpiled Code right? I think this is expected behaviour in js/py right? So when we prevent this it gets less idiomatic in those languages