[data grid] Nested grouping and sort by aggregation does not sort the second level

[mdoverhag]

Steps to reproduce

Link to live example: https://stackblitz.com/edit/github-zzppol?file=src%2Fdemo.tsx

Steps:

1. Group by Commodity
2. Group by Trader name
3. Add size aggregation to Trader name
4. Sort by Group ascending
5. Expand the row Corn
6. Do secondary Sort by Trader name descending (CMD-click)

Current behavior

Trader names under Corn are not sorted by the aggregation.

Expected behavior

Trader names under Corn gets sorted by number of orders (Most at the top)

Context

We're grouping by Year-Month (have a separate string column for that). Then we group by sales person. We'd like to order the expanded Year-Month by the sales person with the most sales at the top.

Your environment

npx @mui/envinfo

  System:
    OS: macOS 14.4.1
  Binaries:
    Node: 20.11.1 - ~/.asdf/installs/nodejs/20.11.1/bin/node
    npm: 10.2.4 - ~/.asdf/plugins/nodejs/shims/npm
    pnpm: Not Found
  Browsers:
    Chrome: 123.0.6312.107
    Edge: Not Found
    Safari: 17.4.1
  npmPackages:
    @emotion/react: ^11.11.4 => 11.11.4
    @emotion/styled: ^11.11.5 => 11.11.5
    @mui/base:  5.0.0-beta.41
    @mui/core-downloads-tracker:  5.15.15
    @mui/icons-material: ^5.15.15 => 5.15.15
    @mui/material: ^5.15.15 => 5.15.15
    @mui/private-theming:  5.15.14
    @mui/styled-engine:  5.15.14
    @mui/system:  5.15.15
    @mui/types:  7.2.14
    @mui/utils:  5.15.14
    @mui/x-data-grid:  7.1.0
    @mui/x-data-grid-premium: ^7.1.0 => 7.1.0
    @mui/x-data-grid-pro:  7.1.0
    @mui/x-date-pickers:  7.1.0
    @mui/x-date-pickers-pro: ^7.1.0 => 7.1.0
    @mui/x-license:  7.0.0
    @types/react: ^18.2.66 => 18.2.72
    react: ^18.2.0 => 18.2.0
    react-dom: ^18.2.0 => 18.2.0
    typescript: ^5.4.2 => 5.4.3

Search keywords: group sort
Order ID: 37c0051b8b2fa563b59fcf5f2406f346Tz04Njg1OSxFPTE3NDI1NjE3ODcwMDAsUz1wcmVtaXVtLExNPXN1YnNjcmlwdGlvbixLVj0y

[mdoverhag]

Hi, first bug report here. Thanks for a cool product.

We just recently started using X Data Grid Premium and I was going to try and update from version 6.19.8 to 7.1.0.

In 6.19.8 I seem to be able to get above to work as expected. But the sorting does no longer seem to work correctly in 7.0.0 or later.

[michelengelen]

Hey @mdoverhag and thanks for raising an issue here.
I am glad you like the product! ✨

The use case you have there is pretty advanced.
I just tried it and you are right. The second sorting does not work.

This clearly is a regression, so I will add this to our board

v6: https://stackblitz.com/edit/github-zzppol-p1ndqj

Screen.Recording.2024-04-05.at.10.55.09.mov

v7: https://stackblitz.com/edit/github-zzppol-vhewiz

Screen.Recording.2024-04-05.at.10.52.58.mov

[mdoverhag]

Thanks for the prompt reply @michelengelen 👍 I'll keep a look out here then!

[cherniavskii]

Hi @mdoverhag
I checked both v6 and v7 (added links to demos in #12684 (comment)) and it seems to me that the behavior in v7 is correct.
In v6, sorting on the grouping column was only applied to the first level of the rows:

v6	v7

This was fixed #9679

We're grouping by Year-Month (have a separate string column for that). Then we group by sales person. We'd like to order the expanded Year-Month by the sales person with the most sales at the top.

Did you consider using multiple grouping columns for your use case?
Here's a working demo with Data Grid v7 https://stackblitz.com/edit/github-zzppol-u2w6mf

[mdoverhag]

Hi @cherniavskii,

Thank you for making the videos comparing v6 and v7. I get it, considering we're sorting on Grouping, the second level is also technically part of that column and it should be sorted alphabetically. 👍

Thanks for the suggestion on multiple grouping. We'll give that ago!

[github-actions]

⚠️ This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

@mdoverhag: How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey.