Ticket #7265 (closed: invalid)

Opened 7 years ago

Last modified 5 years ago

Copy workspace button doesn't always work

Reported by: Peter Peterson Owned by: Keith Brown
Priority: critical Milestone: Release 3.0
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Mathieu Doucet

Description

In algorithm dialogs there are several places where the "copy" button next to the output workspace puts the wrong name or fails to write anything into the output workspace text box.

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner changed from Nick Draper to Peter Peterson

Pete, This really needs at least one concrete reproducible example

comment:2 Changed 7 years ago by Nick Draper

  • Status changed from assigned to new

comment:3 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:4 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.6 to Backlog

Moved to the Backlog after R2.6

comment:5 Changed 7 years ago by Peter Peterson

Found an example: DiffractionFocussing with event data

comment:6 Changed 7 years ago by Nick Draper

  • Owner changed from Peter Peterson to Keith Brown
  • Component changed from Framework to User Interface
  • Milestone changed from Backlog to Release 3.0

comment:7 Changed 7 years ago by Keith Brown

  • Status changed from new to inprogress

comment:8 Changed 7 years ago by Keith Brown

I've discovered a likely cause.

In any instance where it doesn't copy it's likely there is an optional combo-box on the same dialogue which is also has Direction::Input set. As the widgets are stored in a QHash they're not ordered in any way and so if the other combo-box with Direction::Input set comes first in the QHash the copy workspace button will attempt to copy from there instead of the InputWorkspace combo-box.

comment:9 Changed 7 years ago by Keith Brown

Looks like #7541 was a duplicate of this, but it's been fixed first.

comment:10 Changed 7 years ago by Keith Brown

  • Status changed from inprogress to verify
  • Resolution set to invalid

#7541 did indeed fix this

comment:11 Changed 7 years ago by Mathieu Doucet

  • Status changed from verify to verifying
  • Tester set to Mathieu Doucet

comment:12 Changed 7 years ago by Mathieu Doucet

  • Status changed from verifying to closed

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8111

Note: See TracTickets for help on using tickets.