Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Script function .NET object‘s method call problem in Jint v3.0.0-beta-2051 #14346

Closed
hyzx86 opened this issue Sep 16, 2023 · 4 comments · Fixed by #14369
Closed

Script function .NET object‘s method call problem in Jint v3.0.0-beta-2051 #14346

hyzx86 opened this issue Sep 16, 2023 · 4 comments · Fixed by #14369
Labels
Milestone

Comments

@hyzx86
Copy link
Contributor

hyzx86 commented Sep 16, 2023

noticed that I was using the latest version of Jint, but I found a problem with method calls on.net objects

The following script works fine in v3.0.0-beta-2049, but 2051 shows that the method does not exist
useDb() will return a.net object and subsequent chain calls will all be.NET methods

var count = useDb().selectTypedQuery("UserProfile")
.where("UserName='Admin'")
.count();
return count!=0

So far I have rolled back the reference version of OC to 1.8.0-preview-17635

The relevant issues in Jint's repo, I guess they should be related

sebastienros/jint#1626

@hishamco
Copy link
Member

/cc @lahma

hyzx86 added a commit to hyzx86/OrchardCore that referenced this issue Sep 18, 2023
@hyzx86
Copy link
Contributor Author

hyzx86 commented Sep 19, 2023

@hishamco updated in #14369

@hyzx86
Copy link
Contributor Author

hyzx86 commented Sep 21, 2023

@hishamco pls reopen this issue , the problem still exists~
sebastienros/jint#1637

and sebastienros/jint#1638

@hishamco hishamco reopened this Sep 21, 2023
@sebastienros sebastienros added this to the 1.x milestone Sep 28, 2023
@sebastienros
Copy link
Member

Waiting for a new release of Jint

@hyzx86 hyzx86 closed this as completed Oct 22, 2023
@MikeAlhayek MikeAlhayek modified the milestones: 1.x, 1.8 Jan 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants