Ticket #4982 (closed: fixed)
GenericDialog: Dynamically hidden properties cause MantidPlot to crash on Mac/Linux
| Reported by: | Janik Zikovsky | Owned by: | Janik Zikovsky |
|---|---|---|---|
| Priority: | blocker | Milestone: | Release 2.1 |
| Component: | MantidPlot | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Tester: | Owen Arnold |
Description
E.g. BinMD, and then check/uncheck AxisAligned. This is probably related to my recent changes to GenericDialog.
Some unit tests for these dialogs would be nice.
Change History
comment:1 Changed 9 years ago by Janik Zikovsky
- Status changed from new to accepted
- Summary changed from GenericDialog: Dynamically hidden properties cause MantidPlot to crash on Mac to GenericDialog: Dynamically hidden properties cause MantidPlot to crash on Mac/Linux
comment:2 Changed 9 years ago by Janik Zikovsky
Refs #4982 disable the validateInputs() called for now to fix
the crash
Changeset: ee07d63b242582beb9977a064330706ed01f7280
comment:3 Changed 9 years ago by Janik Zikovsky
Refs #4982: fix crash in MantidPlot related to ValidateInputs()
Very annoying 9-character fix that took a long time to find, caused by the AlgorithmProxy, which is always causing trouble!
Changeset: bfaccc8247c6d82923bd732409a3a7592d1b7809
comment:4 Changed 9 years ago by Janik Zikovsky
Refs #4982 re-enable validateInputs()
Changeset: 8ba2794ff57b3f9f4907027ce40e84ec21b75a9d
comment:5 Changed 9 years ago by Janik Zikovsky
- Status changed from accepted to verify
- Resolution set to fixed
Note to testers: please try on all platforms to make sure it works!
comment:6 Changed 9 years ago by Janik Zikovsky
Refs #4982 disable the validateInputs() called for now to fix
the crash
Changeset: ee07d63b242582beb9977a064330706ed01f7280
comment:7 Changed 9 years ago by Janik Zikovsky
Refs #4982: fix crash in MantidPlot related to ValidateInputs()
Very annoying 9-character fix that took a long time to find, caused by the AlgorithmProxy, which is always causing trouble!
Changeset: bfaccc8247c6d82923bd732409a3a7592d1b7809
comment:8 Changed 9 years ago by Janik Zikovsky
Refs #4982 re-enable validateInputs()
Changeset: 8ba2794ff57b3f9f4907027ce40e84ec21b75a9d
comment:9 Changed 8 years ago by Owen Arnold
- Status changed from verify to verifying
- Tester set to Owen Arnold
comment:10 Changed 8 years ago by Owen Arnold
- Status changed from verifying to closed
Cannot reproduce the original problem on Linux.
comment:11 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 5828
