Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[data grid] Debugger pauses on caught exceptions: No row with id found #12704

Closed
SherenL opened this issue Apr 8, 2024 · 4 comments
Closed
Labels
component: data grid This is the name of the generic UI component, not the React module!

Comments

@SherenL
Copy link

SherenL commented Apr 8, 2024

Steps to reproduce

Link to live example: (required)

https://stackblitz.com/edit/github-v2hkyr?file=src%2FApp.js

Steps:

  1. Right click inspect and go to the sources tab
  2. Uncheck 'Pause on caught exceptions'
  3. Resume script execution (F8)
  4. Check 'Pause on caught exceptions'
  5. Click 'Update Rows' button

Current behavior

Depending on the user’s selection, different sets of rows are loaded into the DataGrid.

The program halts if the debugger’s “Pause on caught exceptions” option is enabled when these data rows are swapped with a new set of rows.

However, if this option is disabled, the program operates without any interruptions.

issues_race_condition

Expected behavior

Whether the "Pause on caught exceptions" option is ticked or not, the program should run without any issues.

Context

No response

Your environment

npx @mui/envinfo
 System:
    OS: Windows 10 10.0.19045
  Binaries:
    Node: 18.16.0 - C:\Program Files\nodejs\node.EXE
    npm: 9.5.1 - C:\Program Files\nodejs\npm.CMD
    pnpm: Not Found
  Browsers:
    Chrome: Not Found
    Edge: Chromium (123.0.2420.81)
  npmPackages:
    @emotion/react: ^11.11.1 => 11.11.1
    @emotion/styled: ^11.11.0 => 11.11.0
    @mui/base:  5.0.0-beta.21
    @mui/core-downloads-tracker:  5.14.15
    @mui/icons-material: ^5.14.15 => 5.14.15
    @mui/material: ^5.14.15 => 5.14.15
    @mui/private-theming:  5.14.15
    @mui/styled-engine:  5.14.15
    @mui/styled-engine-sc: ^6.0.0-alpha.3 => 6.0.0-alpha.3
    @mui/system: ^5.14.15 => 5.14.15
    @mui/types:  7.2.7
    @mui/utils:  5.15.11
    @mui/x-data-grid: latest => 6.19.5
    @mui/x-data-grid-premium: ^6.19.5 => 6.19.5
    @mui/x-data-grid-pro:  6.19.5
    @mui/x-license-pro:  6.10.2
    @mui/x-tree-view: ^6.0.0-beta.0 => 6.0.0-beta.0
    @types/react: 18.2.24 => 18.2.24
    react: 18.2.0 => 18.2.0
    react-dom: 18.2.0 => 18.2.0
    styled-components: ^6.1.0 => 6.1.0
    typescript: 5.1.3 => 5.1.3

Search keywords: datagrid

@SherenL SherenL added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Apr 8, 2024
@zannager zannager transferred this issue from mui/material-ui Apr 8, 2024
@zannager zannager added the component: data grid This is the name of the generic UI component, not the React module! label Apr 8, 2024
@michelengelen
Copy link
Member

Hey @SherenL ... why would the expected way to not pause on caught exceptions? That's the whole reason why this option exists.

I am not sure what your intention is. Could you elaborate?

@michelengelen michelengelen added status: waiting for author Issue with insufficient information and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Apr 8, 2024
@michelengelen michelengelen changed the title Debugger pauses on caught exceptions: No row with id found [data grid] Debugger pauses on caught exceptions: No row with id found Apr 8, 2024
@michelengelen
Copy link
Member

One more thing: I tried to replicate the error you mentioned, but for me it never showed No row with id found

@github-actions github-actions bot added status: waiting for maintainer These issues haven't been looked at yet by a maintainer and removed status: waiting for author Issue with insufficient information labels Apr 8, 2024
@SherenL
Copy link
Author

SherenL commented Apr 8, 2024

Thank you for your replies and for letting me know it wasn't reproducible. I've added the script name to the 'ignore list' in the debugger and am able to continue using the debugger.

@SherenL SherenL closed this as completed Apr 8, 2024
@github-actions github-actions bot removed the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Apr 8, 2024
Copy link

github-actions bot commented Apr 8, 2024

⚠️ 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.

@SherenL: 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: data grid This is the name of the generic UI component, not the React module!
Projects
None yet
Development

No branches or pull requests

3 participants