[data grid] Maximum update depth exceeded. Use controlled rowSelectionModel with disabled rowSelection

[headironc]

Steps to reproduce

Steps:

1. Open this link to live example: (demo)
2. Click the checkbox not disabled
3. Check the console, found infinite rowSelectionModel changed

Current behavior

Click the avalible checkbox, and the rowSelectionModel is always in the status of changing.

Expected behavior

It shouldn do nothing.

Context

No response

Your environment

npx @mui/envinfo

  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

  System:
    OS: macOS 15.1.1
  Binaries:
    Node: 22.11.0 - ~/.nvm/versions/node/v22.11.0/bin/node
    npm: 10.9.0 - ~/.nvm/versions/node/v22.11.0/bin/npm
    pnpm: 9.15.1 - ~/.nvm/versions/node/v22.11.0/bin/pnpm
  Browsers:
    Chrome: 131.0.6778.205
    Edge: Not Found
    Safari: 18.1.1
  npmPackages:
    @emotion/react: ^11.14.0 => 11.14.0
    @emotion/styled: ^11.14.0 => 11.14.0
    @mui/base: 5.0.0-beta.67 => 5.0.0-beta.67
    @mui/lab: 6.0.0-beta.20 => 6.0.0-beta.20
    @mui/material: ^6.2.1 => 6.2.1
    @mui/material-nextjs: ^6.2.1 => 6.2.1
    @mui/x-data-grid-pro: ^7.23.3 => 7.23.3
    @mui/x-license: 7.23.2 => 7.23.2
    @types/react: ^19.0.2 => 19.0.2
    react: ^19.0.0 => 19.0.0
    react-dom: ^19.0.0 => 19.0.0
    typescript: ^5.7.2 => 5.7.2

Search keywords: rowSelectionModel,rowSelection

Order ID: 85082

[michelengelen]

Hey @headironc ... this is definitely unintentional. Although I don't really understand why you would set rowSelection={false} and still try to control it with onRowSelection?

We should not fire the callback when selection changes, so it is a bug. Adding it to the board!

[headironc]

Hey @headironc ... this is definitely unintentional. Although I don't really understand why you would set rowSelection={false} and still try to control it with onRowSelection?

We should not fire the callback when selection changes, so it is a bug. Adding it to the board!

I'm going to disable row selection during initialization, if certain conditions are met to allow partial row selection. And then i found the select all button is still avalible.