Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Feesql 扩展中添加 Searchs 会导致异常 #3462

Closed
1 task done
j4587698 opened this issue May 9, 2024 · 1 comment · Fixed by #3504
Closed
1 task done

Feesql 扩展中添加 Searchs 会导致异常 #3462

j4587698 opened this issue May 9, 2024 · 1 comment · Fixed by #3504
Assignees
Labels
bug Something isn't working
Milestone

Comments

@j4587698
Copy link
Collaborator

j4587698 commented May 9, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

目前的Freesql扩展使用搜索时会报sql错误。
检查发现在ToDynamicFilter方法中仅判断了if (actions.Filters != null)
SearchFilterActionGetFilterConditions并没有给Filters 赋值
就会导致产生一个key和value都为null的条件,freesql转为sql时就会产生一个(())的条件,sql异常

Expected Behavior

搜索栏输入任意内容
image

此时searches里有一条数据
image

这时候SearchFilterAction中由于没有加Filter,所以actions.Filters是null
image

这时候item里内容都为null,所以会产生一个(())的条件
image

Steps To Reproduce

No response

Exceptions (if any)

No response

.NET Version

No response

Anything else?

No response

@j4587698 j4587698 added the bug Something isn't working label May 9, 2024
Copy link

bb-auto bot commented May 9, 2024

@j4587698 Thank you for filing this issue. In order for us to investigate this issue, please provide a minimalistic repro project that illustrates the problem.

@bb-auto bb-auto bot added this to the v8.5.0 milestone May 9, 2024
@ArgoZhang ArgoZhang changed the title bug(): Feesql扩展中添加Searchs会导致异常 Feesql 扩展中添加 Searchs 会导致异常 May 13, 2024
@dotnetcore dotnetcore locked and limited conversation to collaborators May 14, 2024
@ArgoZhang ArgoZhang converted this issue into discussion #3500 May 14, 2024

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants