Ticket #3900 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Allow spaces between group names

Reported by: Vickie Lynch Owned by: Karl Palmen
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Vickie Lynch

Description

Comma delimited group names in CreateCalFilesByNames and CreateGroupingWorkspace should ignore spaces after commas. An SNS user could only create one group because he was including spaces after the commas.

Change History

comment:1 Changed 9 years ago by Nick Draper

  • Owner set to Anyone
  • Status changed from new to assigned

comment:2 Changed 9 years ago by Karl Palmen

  • Owner changed from Anyone to Karl Palmen
  • Status changed from assigned to accepted

comment:3 Changed 9 years ago by Karl Palmen

  • Status changed from accepted to assigned

comment:4 Changed 9 years ago by Karl Palmen

I found that first algorithm referred to is actually called CreateCalFileByNames.

comment:5 Changed 9 years ago by Karl Palmen

  • Status changed from assigned to accepted

comment:6 Changed 9 years ago by Karl Palmen

Trim group names after separation re #3900

Also modified unit test to sometimes have spaces after commas.

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: 04814da72b7efb0752b16c5c9a2537ae59102e67

comment:7 Changed 9 years ago by Karl Palmen

I meant spaces after some commas in unit tests.

comment:8 Changed 9 years ago by Karl Palmen

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

comment:9 Changed 9 years ago by Vickie Lynch

  • Status changed from verify to verifying
  • Tester set to Vickie Lynch

comment:10 Changed 9 years ago by Vickie Lynch

  • Status changed from verifying to closed

Works with spaces.

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 4747

Note: See TracTickets for help on using tickets.