Skip to content

throw exception if property is null for the doctrine filters - #7681

Merged
soyuka merged 2 commits into
api-platform:mainfrom
aaa2000:feat-throw-exception-if-property-is-null
Jan 22, 2026
Merged

throw exception if property is null for the doctrine filters#7681
soyuka merged 2 commits into
api-platform:mainfrom
aaa2000:feat-throw-exception-if-property-is-null

Conversation

@aaa2000

Copy link
Copy Markdown
Contributor
QA
Branch?main for features
LicenseMIT

The parameter property is required for some filters; in this case, this PR throws an exception
@see#7668 (comment)

Do I need to add unit tests?

@soyuka

Copy link
Copy Markdown
Member

Should we target 4.2 instead?

@aaa2000

Copy link
Copy Markdown
ContributorAuthor

The AbstractUuidFilter is not on the main branch,. If you want, I can do two pull requests: the first one on the main branch with the first commit, and the second one on the 4.2 branch with the second commit.

@soyuka
soyuka merged commit 149fe24 into api-platform:mainJan 22, 2026
149 checks passed
@soyuka

Copy link
Copy Markdown
Member

thanks I'll cherry-pick

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@aaa2000@soyuka