Feature Request - Dynamic method & property support for script object registered from .Net
- Dominant language
- C#
- Stars
- 10.3k
- Forks
- 2.9k
- Avg merge
- 5d 22h
- Merged PRs (30d)
- 4
Description
I have a third party lib, contains hundreds of class, each class concain 10 ~ 100+ method & property.
I try to use this in CefSharp browser, but it is unwise to create .Net wrapper for every class.
The best way is create a simple wrapper class, contain base class info,
in script, just simply use `ob.methodA(); ob.propertyA = 'a';`
in wrapper .Net class , just catch the access event, then find and run or show error like "method/propert not found".
Then i try extend class from System.Dynamic, but it does not support.
> // CefSharp/Internals/JavascriptObjectRepository.cs , line 240
> // method Register
> var type = value.GetType();
> if (type.IsPrimitive || type.BaseType.Namespace.StartsWith("System."))
> {
> throw new ArgumentException("Registering of .Net framework built in types is not supported, " +
> "create your own Object and proxy the calls if you need to access a Window/Form/Control.", "value");
> }
i don't known why, mayby security issue?
i read source code to find a way to use dynamic method or property
finally, i think there is no way to do this.
> // CefSharp/Internals/JavascriptObjectRepository.cs
> // method AnalyseObjectForBinding
> // line 670 - 699
> // it cache the method to List
>// CefSharp/Internals/JavascriptObjectRepository.cs
> // method TryCallMethod
> // line 301 - 305
> // simply check List, if not found, throw error
also there can be an ugly way to use , wrapper class create proxy method
and use in script like `ob.proxyCall('methodA', []); ob.proxyGet('propertyA')`
it's too silly.
build custom cefsharp is hard and heavy to me, so i create this issue
suggest some way to support Dynamic method & property for script object registered from .Net
here is my solution:
[1] create interface to declear .Net class need Dynamic method & property support
> // CefSharp/IDynamicMethod.cs
> interface IDynamicMethod {
> MethodInfo resolveDynamicMethod(string name);
> };
>
> // CefSharp/IDynamicProperty.cs
> interface IDynamicProperty{
> PropertyInfo resolveDynamicProperty(string name);
> };
[2] modify `CefSharp/Internals/JavascriptObjectRepository.cs`
> // method `TryCallMethod` & `TryCallMethodAsync`
> // if method not found
> // if obj implement IDynamicMethod , try get method from obj.resolveDynamicMethod
> // do some validation & check (such as security issue) then wrap .Net MethodInfo to JavascriptMethod
>
> // method `TryGetProperty` & `TrySetProperty`
> // if property not found
> // if obj implement IDynamicProperty, try get property from obj.resolveDynamicProperty
> // do some validation & check (such as security issue) then wrap .Net PropertyInfo to JavascriptProperty
then we done, it's simply change few lines.
may be can cache dynamic wrappered JavascriptMethod or JavascriptProperty
but the better way is let user decide and write extra code to cache MethodInfo and PropertyInfo.
now if user require Dynamic Method or Property
just implment IDynamicMethod or IDynamicProperty,it‘s simple , elegent and comfortable.
Contributor guide
Research direction
Start with CefSharp/Internals/JavascriptObjectRepository.cs, especially Register, AnalyseObjectForBinding, TryCallMethod, TryCallMethodAsync, TryGetProperty, and TrySetProperty. Review how methods and properties are currently discovered, cached, validated, and reported when missing. Done would require a decided design for dynamic method and property resolution, including the security and validation behavior; no tests are named in the issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- desktop
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100