Skip to content

1.1.x: Add exists method to PaginatorTrait with tests#2843

Merged
Expurple merged 1 commit intoSeaQL:1.1.xfrom
Expurple:1.1.x-exists
Dec 7, 2025
Merged

1.1.x: Add exists method to PaginatorTrait with tests#2843
Expurple merged 1 commit intoSeaQL:1.1.xfrom
Expurple:1.1.x-exists

Conversation

@Expurple
Copy link
Copy Markdown
Member

@Expurple Expurple commented Dec 7, 2025

Rebase #2623 from master into 1.1.x.

This is a nice backwards-compatible feature, I'd like to have it in 1.1.x and adopt in advance

* Add exists method to PaginatorTrait and corresponding tests

* Refactor import order and simplify exists tests for better readability

* Fix placed_at field to use ChronoDateTime for consistency in seed data

* Remove performance comparison test for exists vs count methods from exists_tests

* Refactor paginator query to use Expr::exists

* Revert "Refactor paginator query to use Expr::exists"

This reverts commit ef0521f.

* Remove unnecessary database backend builder in paginator query
@Expurple
Copy link
Copy Markdown
Member Author

Expurple commented Dec 7, 2025

Clippy complaints come from the 1.1.x base, ignore those

@Expurple Expurple merged commit 2e856f9 into SeaQL:1.1.x Dec 7, 2025
35 of 36 checks passed
Sign up for free to 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