Ticket #3588 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Eliminate the EventWorkspaceValidator

Reported by: Russell Taylor Owned by: Russell Taylor
Priority: minor Milestone: Iteration 30
Component: Mantid Keywords:
Cc: zikovskyjl@… Blocked By:
Blocking: Tester: Janik Zikovsky

Description

It's redundant - the correct way to handle this is to make the WorkspaceProperty template type (I)EventWorkspace. This then takes care of the validation.

Change History

comment:1 Changed 9 years ago by Russell Taylor

  • Cc zikovskyjl@… added
  • Status changed from new to accepted

comment:2 Changed 9 years ago by Russell Taylor

In [14129]:

Eliminate the EventWorkspace validator and re-type workspace properties as EventWorkspace instead. Re #3588.

comment:3 Changed 9 years ago by Russell Taylor

In [14132]:

Too much auto-replacing. Re #3588.

comment:4 Changed 9 years ago by Russell Taylor

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

comment:5 Changed 9 years ago by Janik Zikovsky

  • Status changed from verify to verifying
  • Tester set to Janik Zikovsky

comment:6 Changed 9 years ago by Janik Zikovsky

  • Status changed from verifying to closed

It's gone!

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4435

Note: See TracTickets for help on using tickets.