Ticket #8573 (closed: fixed)
Crash possible in 'Send To' tab of the preferences dialog
Reported by: | Russell Taylor | Owned by: | Russell Taylor |
---|---|---|---|
Priority: | trivial | Milestone: | Release 3.1 |
Component: | GUI | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Michael Reuter |
Description
Select one of the attributes of a program (e.g. target), hit edit and MantidPlot will crash. It's assuming you have the top-level tree item selected.
Also, when entering the name of the algorithm to use in the editing dialog, you get a warning log message from the AlgorithmManager for each letter typed until completion of the algorithm name. In fact, this code should call the AlgorithmFactory::exists method, which would be silent.
Change History
comment:2 Changed 7 years ago by Russell Taylor
Re #8573. Avoid a bunch of warnings while typing algorithm name.
Just ask the algorithm factory if the algorithm exists rather than trying to create it on every keystroke (which as well as being expensive gave a warning log message until the algorithm name was complete).
Changeset: 577d7634e028b366168c022b265088b0ab1a0729
comment:3 Changed 7 years ago by Russell Taylor
- Status changed from inprogress to verify
- Resolution set to fixed
Testing: Just check that the issues identified in the description have been remedied.
comment:4 Changed 7 years ago by Michael Reuter
- Status changed from verify to verifying
- Tester set to Michael Reuter
comment:5 Changed 7 years ago by Michael Reuter
- Status changed from verifying to closed
Merge remote branch 'origin/bugfix/8573_sendto_crash'
Full changeset: 99e00192b31267612f5b67aa12dc53eaf5e1dc2e
Re #8573. Avoid a crash by recursing up the tree if necessary.
Rather than just assuming it's the top level that's selected.
Changeset: a079a281305a8478a8d50441ad4ff62a64c89290