BUG: GridFieldOrderableRows interaction with GridFieldPaginator

[mfendeksilverstripe]

BUG: GridFieldOrderableRows interaction with GridFieldPaginator

Impacted version: 3.5.0

Re-ordering items across pages produces unexpected results.

Steps to reproduce

• taxonomy group <- many many through (with Sort on through object) -> taxonomy term (also has a Sort column)
• generate a taxonomy group with 200 taxonomy terms
• navigate to a page (for example page 4)
• drag an items to be reordered (for example item number 50) and drop it on the previous page icon in the paginator

Expected result

Item number 50 will be moved to the previous page while leaving all the items in the same place

Actual result

The list gets completely shifted with first 44 items leaving their place on the first page.

Screenshots

Notes

I haven't done too much investigation into this but it's possible that the random sort order is coming from the Sort column on the taxonomy terms. Maybe the re-order action gets confused on which DB column needs to be used when using the paginator reorder.

Normal reorder action within a single page works fine though.

PRs

• FIX #355 Sorting in paginated lists #418

[johannesx75]

I think I've found the problem. The code that tried to figure out the new sort did not respect the original sort of the DataList. I've created a pull request for the 4.x branch: #418