Ticket #3332 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Implement "Send To" functionality for workspaces

Reported by: Martyn Gigg Owned by: Robert Whitley
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: nick.draper@… Blocked By:
Blocking: Tester: Peter Parker

Description

We need to provide a simpler way for users to carry their data on to other programs. This will be implemented as a "Save To ..." option on the workspace context menu in MantidPlot.

Change History

comment:1 Changed 9 years ago by Martyn Gigg

  • Cc nick.draper@… added

comment:2 Changed 9 years ago by Robert Whitley

In [13106]:

Refs #3332. Added a method to ConfigService to retrieve subkeys from a configuration.

comment:3 Changed 9 years ago by Robert Whitley

  • Status changed from new to accepted

comment:4 Changed 9 years ago by Robert Whitley

In [13212]:

Refs #3332 Implemented a simple send to option on the workspace context menu in MantidPlot which saves the current workspace and opens it up on another program. The menu is populated by the programs defined by the user in the properties file.

comment:5 Changed 9 years ago by Robert Whitley

In [13213]:

Refs #3332 Fixing Linux Build

comment:6 Changed 9 years ago by Robert Whitley

In [13545]:

Refs #3332 Implemented a send to options tab on the preference dialog where a user can control what programs a workspace can be sent to. Add and Edit buttons open a seperate dialog containing a form with validation to make sure the user enters the correct information.

comment:7 Changed 9 years ago by Robert Whitley

In [13548]:

Refs #3332 Avoiding limitations of earlier Poco versions on Linux

comment:8 Changed 9 years ago by Martyn Gigg

In [13549]:

Refs #3332. Fix an syntax error for gcc with >> for nested templates.

comment:9 Changed 9 years ago by Martyn Gigg

In [13550]:

Refs #3332. Remove a function from the ConfigService that didn't belong there. It was only used in the GUI and QString already has similar functionality so use that instead.

comment:10 Changed 9 years ago by Martyn Gigg

In [13551]:

Refs #3332. Remove a parenthesis.

comment:11 Changed 9 years ago by Martyn Gigg

In [13552]:

Refs #3332. Missed the test reference.

comment:12 Changed 9 years ago by Robert Whitley

In [13554]:

Refs #3332 Minor edit to the add and edit dialogs to see what value they return and to do nothing if cancel is pressed.

comment:13 Changed 9 years ago by Mathieu Doucet

There are 10 or so new warnings that were added by the last few commits on this ticket.

comment:14 Changed 9 years ago by Robert Whitley

In [13557]:

Refs #3332 Fixed the warnings

comment:15 Changed 9 years ago by Mathieu Doucet

In [13569]:

Fix warnings Re #3332

comment:16 Changed 9 years ago by Robert Whitley

This is pretty much fixed, just need to implement workspace but it currently works without.

comment:17 Changed 9 years ago by Mathieu Doucet

In [14308]:

Expand environment variables. Re #3332

comment:18 Changed 9 years ago by Mathieu Doucet

In [14330]:

Expand environment variables. Re #3332

comment:19 Changed 9 years ago by Mathieu Doucet

In [14340]:

Expand environment variables. Re #3332

comment:20 Changed 9 years ago by Mathieu Doucet

In [14353]:

Don't consider item visible if it doesn't point to an executable. Re #3332

comment:21 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 30 to Iteration 31

Bulk move of tickets to iteration 31 at the iteration 30 code freeze

comment:22 Changed 9 years ago by Robert Whitley

  • Status changed from accepted to verify
  • Resolution set to fixed

This currently works but needs expanding further to allow for environmental variables. This will allow Mantid to have default programs that the workspaces can be sent to straight after the user has installed it on their computer.

Another ticket has been created for this task. See #3657.

comment:23 Changed 9 years ago by Peter Parker

  • Status changed from verify to verifying
  • Tester set to Peter Parker

comment:24 Changed 9 years ago by Peter Parker

  • Status changed from verifying to closed

Deleting a program from the "List of Current Programs" in preferences fails to update the list without a restart. Raised #4445.

Otherwise passed.

comment:25 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4179

Note: See TracTickets for help on using tickets.