[Table] Rename onChangeRowsPerPage, onChangePage

[eladmotola]

Breaking changes

• [Table] Rename onChangeRowsPerPage to onRowsPerPageChange and onChangePage to onPageChange due to API consistency.

  <TablePagination
  - onChangeRowsPerPage={()=>{}}
  - onChangePage={()=>{}}
  + onRowsPerPageChange={()=>{}}
  + onPageChange={()=>{}}

---

Part of #20012

[mui-pr-bot]

Details of bundle changes

Generated by 🚫 dangerJS against b344c04

[eps1lon]

@eladmotola Thanks!

[stevemk14ebr]

Hi, this change broke some stuff. The documentation here is not valid now: https://material-ui.com/api/table-pagination/ and when this prop is not provided the pagination throws an error as these events are undefined. Combined with the documentation being incorrect this makes the simple act of upgrading to the latest material-ui version cause an error: with is not a function throwing internally in the material-ui source.

[eps1lon]

material-ui.com/api/table-pagination

That is the documentation for the latest stable release i.e. v4. This PR targetted v5 for which the documentation can be found at https://next.material-ui.com/api/table-pagination

[mbrookes]

Hey @eladmotola! 👋 Do you think you might be able to add a deprecation warning for this on the master branch?

You can probably find a suitable example to follow in #20012 Deprecations section – PR # prefixed with "Dep:".

Thanks!

[eladmotola]

Hey @mbrookes !
I'm sorry but I'm busy at the moment so I dont think I will have the time to get to it.

[mbrookes]

No worries, and thanks for your contribution so far! 🙏 We'll try to pick this up before v5, but if you get a chance in the meantime, feel free to jump in. (Would hate to have to revert this! 😅)