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

handle SonataAdminBundle Pager::nbresults deprecation #501

Merged
merged 2 commits into from
Jan 10, 2021

Conversation

dmaicher
Copy link
Contributor

@dmaicher dmaicher commented Jan 9, 2021

Subject

I am targeting this branch, because its BC.

See
sonata-project/SonataAdminBundle#6732
sonata-project/SonataDoctrineORMAdminBundle#1257

Changelog

### Added
- implemented `Sonata\AdminBundle\Datagrid\PagerInterface::countResults()`

### Deprecated
- `Sonata\DoctrineMongoDBAdminBundle\Datagrid\Pager::computeNbResult()`
- `Sonata\DoctrineMongoDBAdminBundle\Datagrid\Pager::getNbResults()`
- `Sonata\DoctrineMongoDBAdminBundle\Datagrid\Pager::setNbResults()`

@dmaicher dmaicher marked this pull request as ready for review January 9, 2021 22:18
OskarStark
OskarStark previously approved these changes Jan 10, 2021
@VincentLanglet VincentLanglet merged commit 1796591 into sonata-project:3.x Jan 10, 2021
@VincentLanglet
Copy link
Member

Thanks

@dmaicher dmaicher deleted the nbresults_deprecation branch January 10, 2021 13:05
@franmomu franmomu added the minor label Jan 10, 2021
franmomu added a commit to franmomu/SonataDoctrineMongoDBAdminBundle that referenced this pull request Jan 10, 2021
VincentLanglet pushed a commit that referenced this pull request Jan 10, 2021
It was missing from #501
VincentLanglet pushed a commit that referenced this pull request Jan 16, 2021
* handle SonataAdminBundle Pager::nbresults deprecation (#501)

* handle SonataAdminBundle Pager::nbresults deprecation

* make setResultsCount() private

* Remove suggested package

The user should have ext-mongodb installed.

* Avoid triggering deprecations

These methods were deprecated in sonata-project/admin-bundle.

* Add upgrade note

It was missing from #501

* DevKit updates (#505)

* fix compatibility

* fix compatibility

* cs-fixer

* remove redundant cast to int

Co-authored-by: David Maicher <[email protected]>
Co-authored-by: Fran Moreno <[email protected]>
Co-authored-by: Sonata CI <[email protected]>
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 this pull request may close these issues.

4 participants