Allow use of space/tab in SaveAscii

[Peter Parker]

Currently, whitespace is not parsed and the default of a comma is used instead. Also, tabs cannot be inputted since it simply moves focus to the next field.

[Peter Parker]

Refs #4309 - Only SaveAscii modified to allow spaces and tabs.

Drop down menu added (via a ListValidator property) to allow user to pick from same options found in LoadAscii. If they pick a choice of "User Defined", then they can also input a custom separator.

Changeset: 941d66c61fedb5a3d6dbdda7a8cbc64fcf082243

[Peter Parker]

Refs #4309 - Only SaveAscii modified to allow spaces and tabs.

Drop down menu added (via a ListValidator property) to allow user to pick from same options found in LoadAscii. If they pick a choice of "User Defined", then they can also input a custom separator.

Changeset: 941d66c61fedb5a3d6dbdda7a8cbc64fcf082243

[Russell Taylor]

The above commit was reverted in [c35036af] (trac hook failed).

Revert "Refs #4309 - Only SaveAscii modified to allow spaces and tabs."

This reverts commit [941d66c6].

Mathieu says "Sorry for the revert but that commit broke the SANS reduction. Let's discuss what needs to be done."

[Peter Parker]

"Un-reverts" a revert made to SaveAscii.

This reverts commit c35036affa527e2ab0d41a4a3a38ab220873ef98, as well as changing the two places where SaveAscii is called with a separator specified. This should ensure ORNL SANS reduction still works. Refs #4309.

Changeset: eaece9d766930d0c227a5c62ad074563cad6a279

[Nick Draper]

Moved to iteration 33 at iteration 32 code freeze

[Russell Taylor]

Is this complete? If so it should just be marked fixed. If not, a follow up ticket should be written and this one closed.

The reason is that this changes the algorithm signature in a way that will break any scripts that use it (that aren't in our repository). In general this is something we try to avoid, so at minimum this needs to be mentioned in the release notes - that's why I've moved it back to Iteration 32.

[Peter Parker]

This is complete - my mistake for not marking it as such.

The decision was made to change the algorithm signature as it was the most sensible way to implement the previously missing space/tab functionality via the UI (though "/t" was working elsewhere). SaveAscii now works in a more synergistic and uniform manner with its sister algorithm, LoadAscii.

Updated Wiki: http://www.mantidproject.org/SaveAscii.

[Stuart Campbell]

This ticket has been transferred to github issue ​5156