Skip to content

Allow composing multiple filters(PropertyFilter, PropertyPreFilter, L…#3877

Merged
wenshao merged 3 commits intoalibaba:mainfrom
jujn:fix_3874
Nov 26, 2025
Merged

Allow composing multiple filters(PropertyFilter, PropertyPreFilter, L…#3877
wenshao merged 3 commits intoalibaba:mainfrom
jujn:fix_3874

Conversation

@jujn
Copy link
Copy Markdown
Collaborator

@jujn jujn commented Nov 19, 2025

…abelFilter), for issue #3874

What this PR does / why we need it?

Summary of your change

Please indicate you've done the following:

  • Made sure tests are passing and test coverage is added if needed.
  • Made sure commit message follow the rule of Conventional Commits specification.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed.

@wenshao
Copy link
Copy Markdown
Member

wenshao commented Nov 25, 2025

这个CompositeXXXFilter放到package com.alibaba.fastjson2.filter下面，并且不声明为public，避免用户传入的CompositeFilter，然后修改了用户传入的Filter。

@jujn
Copy link
Copy Markdown
Collaborator Author

jujn commented Nov 26, 2025

这个CompositeXXXFilter放到package com.alibaba.fastjson2.filter下面，并且不声明为public，避免用户传入的CompositeFilter，然后修改了用户传入的Filter。

欧克，已修改

@wenshao wenshao merged commit 0e08a1f into alibaba:main Nov 26, 2025
21 checks passed
wenshao pushed a commit that referenced this pull request Dec 26, 2025
@jujn @wenshao
Allow composing multiple filters(PropertyFilter, PropertyPreFilter, L… (
309e2c8 
#3877)

* Allow composing multiple filters(PropertyFilter, PropertyPreFilter, LabelFilter), for issue #3874

* update

* adjust class position
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jujn @wenshao