[data grid] When changing rows and columns simultaneously, new column valueGetter will be invoked on previous rows.

[MVelez94]

Steps to reproduce

Link to live example

Steps:

1. Run the example, and open your console.
2. Click the button.
3. See that, at first, the previous row is logged in the incoming valueGetter

Current behavior

Simultaneous update of rows and columns causes some columns valueGetters to operate on the previous set of rows, if said columns share the same field.

Expected behavior

I think what it should happen is that no column valueGetter should be called on previous data if it has changed. It looks like both a leak and a source of errors.

Context

I am fetching from an endpoint which provides me with the columns and the rows, so I set these upon successful fetch on a table that has preloaded data.

Your environment

npx @mui/envinfo

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

Search keywords: data-grid x-data-grid column rows state valueGetter

[MVelez94]

I can work on this, if allowed and issue proceeds.

[michelengelen]

Hey @MVelez94 ... thanks for picking this up. If you need assistance please feel free to reach out!

[MVelez94]

Noted! Will start working on it now.

…

On Mon, Feb 26, 2024, 10:54 AM Michel Engelen ***@***.***> wrote: Hey @MVelez94 <https://github.com/MVelez94> ... thanks for picking this up. If you need assistance please feel free to reach out! — Reply to this email directly, view it on GitHub <#12190 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AGG4MFEN33RCNLKA5B7FU53YVRLUXAVCNFSM6AAAAABDWOQ236VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRTG4ZDAMJYGA> . You are receiving this because you were mentioned.Message ID: ***@***.***>