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

unhandled error in database action #2430

Closed
targos opened this issue Jun 12, 2023 · 4 comments
Closed

unhandled error in database action #2430

targos opened this issue Jun 12, 2023 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@targos
Copy link
Member

targos commented Jun 12, 2023

Cannot read properties of null (reading 'ranges')

{
  source: 'webpack://_N_E/node_modules/nmrium/lib/component/reducer/Reducer.js',
  line: 402,
  column: 67,
  name: 'draft'
}
{
  source: 'webpack://_N_E/node_modules/nmrium/lib/component/reducer/actions/DatabaseActions.js',
  line: 9,
  column: 18,
  name: 'ranges'
}
@targos targos added the bug Something isn't working label Jun 12, 2023
@hamed-musallam hamed-musallam linked a pull request Jun 20, 2023 that will close this issue
@hamed-musallam
Copy link
Member

@jobo322

Resurrect spectrum from ranges failed, rangesToXY give me this error only in the negatives values
image

@jobo322
Copy link
Member

jobo322 commented Jun 25, 2023

@jobo322

Resurrect spectrum from ranges failed, rangesToXY give me this error only in the negatives values image

yes, it is a problem in the simulation of the spectrum, I am working on it

@jobo322
Copy link
Member

jobo322 commented Jun 25, 2023

Cannot read properties of null (reading 'ranges')

{
source: 'webpack://_N_E/node_modules/nmrium/lib/component/reducer/Reducer.js',
line: 402,
column: 67,
name: 'draft'
}
{
source: 'webpack://_N_E/node_modules/nmrium/lib/component/reducer/actions/DatabaseActions.js',
line: 9,
column: 18,
name: 'ranges'
}

seems it fixed, currently if there is not ranges in the element database it is excluded from the list presented to the user.

@jobo322 jobo322 linked a pull request Jun 26, 2023 that will close this issue
@hamed-musallam
Copy link
Member

fixed by #2475

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging a pull request may close this issue.

3 participants