PASTA-ELN / pasta-eln

PASTA-ELN with new frontend
https://pasta-eln.github.io/pasta-eln/
Other
7 stars 2 forks source link

Double clicking on empty cells in metadata table throws console errors #305

Open jmurugan-fzj opened 2 weeks ago

jmurugan-fzj commented 2 weeks ago

Describe the bug Double clicking any randomly selected cells in the metadata table of data hierarchy editor throws console errors. Reproducible: Occasional

Steps To Reproduce Steps to reproduce the behavior:

  1. Run the latest version of PASTA application.
  2. Go to System | Data hierarchy editor (or press F8)
  3. Switch items between "Data Type" combo-box (eg. Structure level 0, measurement, sample etc.)
  4. image
  5. Double click on any cells in the metadata table randomly (empty or non-empty)
  6. image
  7. See the following type of error been thrown in the console
  8. image

Expected behavior Double click on any empty or normal cell should not throw any console errors.

Screenshots Given above.

Desktop (please complete the following information):

Additional context Similar issue has been reported already related to specific pyqt versions: https://stackoverflow.com/questions/46383409/qaccessibletablechild-invalid-index-in-empty-directory-with-qtableview-and-qf https://forum.qt.io/topic/23872/5-0-1-cannot-create-accessible-interface-for-object-warnings/2

HTsybenko commented 1 day ago

This is indeed an occasional bug, and occurs rather rarely for me as well (2.6.0b1).

jmurugan-fzj commented 1 day ago

@HTsybenko I have noticed that the functionality of the tool is not affected by these errors, they are rather annoying, will try to investigate further as soon as I start the rework in data hierarchy tool!

HTsybenko commented 1 day ago

@jmurugan-fzj Yes, I agree. I would also prioritize the related issue #306, since it does not allow metadata entries in the affected fields.

jmurugan-fzj commented 1 day ago

@HTsybenko The fix for the issue #306 is currently in review now..