[DataGrid] Clean params of onCellEditCommit

[valenfv]

Fixes #3442.

The issue

The params sent toonCellEditCommit callback function either when clicking outside or pressing Enter / Tab, were inconsistent.

The solution

Clean the params to only contain {id, field, value}

When clicking outside of a cell in editing state, the params sent to the publishEvent are now the same as the ones sent when tabbing or pressing enter.

• Given the following instance:
  <DataGrid onCellEditCommit={e => console.log(e)} rows={rows} columns={columns} />

1. Edit a row and click outside
2. Edit a row and press Enter/Tab

Actual results: id, field and value are the fields received in the params

Solution provided by @alexfauquette

[mui-bot]

These are the results for the performance tests:

Test case	Unit	Min	Max	Median	Mean	σ
Filter 100k rows	ms	181.1	392.1	312.2	300.02	83.367
Sort 100k rows	ms	348.7	872.2	746.5	698.04	184.667
Select 100k rows	ms	161.2	381.3	302.8	279.16	74.487
Deselect 100k rows	ms	132.5	380.2	200.2	236.36	87.395

Generated by 🚫 dangerJS against 112e666

[alexfauquette]

Thanks for your contribution :)

@DanailH The code looks good to me, but it is a solution I proposed in an issue. Could you check that it is not a problem to remove some information here?

[flaviendelangle]

ping @m4theushw

[m4theushw]

Looks good to me. I merged with master to fix argos.

[zigang93]

is this props onCellEditCommit been removed in v6?

[m4theushw]

@zigang93 Yes, you can use processRowUpdate for similar behavior. See https://mui.com/x/react-data-grid/editing/#persistence