Ticket #4982 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

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

Note: See TracTickets for help on using tickets.