Sergei Predvoditelev
Sergei Predvoditelev
Currently, we can use `GridView` method `keepPageOnSort()` to configure it. By default page don't keep on sort changing.
Note. When passed data reader with limit to grid view with allowed sorting, need throw logic exception.
> Still need a check. If data reader has limit then sorting should be disabled. Maybe it's better to throw logic exception, so developer should change own code?
> Grid without sorting does make sense so I don't think throwing an exception is a good idea. Throw exception when data reader has limit and sorting enabled in grid...
It remains to implement: - [ ] TYPE_UPK - [ ] TYPE_UBIGPK
If this functionality is needed, then need to create issue in Yii DB.
Suggest don't change iterable data reader, but add comment to `ReadableDataInterface::read()` about keys (keys can have any value, you should not rely on them somehow).
May be add this feature in PHP 8.2 only? It will to allow check readonly class in `delete()` method and throw exception in this case.
Rector workflow make commit and now fail on pull request from forks. Can you try change `/.github/workflow/rector.yml` in your PR to this code: ```yml on: pull_request: paths-ignore: - 'docs/**' -...
But it's should be available in PR to origin repository