Ticket #3595 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Consolidate FieldData to MetaData conversions.

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: minor Milestone: Iteration 30
Component: VATES Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

These are used widely and often defined/implemented more than once. i.e findExistingWorkspaceLocation, which crops-up a lot. It's getting hard to understand the code-base with code snippets like this occuring all over the place. Each implementation is also untested.

These should be implemented once, and should be used with the minimum amount of setup on the part of the client-code. Also provide tests for these conversions.

Change History

comment:1 Changed 9 years ago by Owen Arnold

In [14131]:

refs #3595. New conversion types implemented and unit tested. Next step is to go through the codebase do replacements.

comment:2 Changed 9 years ago by Owen Arnold

In [14133]:

refs #3595. Replace individual methods/duplicates with common implementation via the new types.

comment:3 Changed 9 years ago by Owen Arnold

In [14134]:

refs #3595. Clear out unused cpp file.

comment:4 Changed 9 years ago by Owen Arnold

In [14141]:

refs #3595. Correct runtime issue whereby attempting to provide workspace location when not required.

comment:5 Changed 9 years ago by Owen Arnold

  • Status changed from new to accepted

comment:6 Changed 9 years ago by Owen Arnold

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

comment:7 Changed 9 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:8 Changed 9 years ago by Michael Reuter

  • Status changed from verifying to closed

This looks to have been done correctly.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4442

Note: See TracTickets for help on using tickets.