Project

General

Profile

Actions

task #9595

closed

Use DerivedUnit or DnaSample for DnaSampleGeneralDetailElement

Added by Andreas Müller almost 3 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Priority14
Assignee:
Category:
taxeditor
Target version:
Start date:
Due date:
% Done:

50%

Estimated time:
Severity:
normal

Description

same reason as for #9594

This is also a step forward to not use DerivedUnitFacade in SpecimenTreeEditor details views anymore.


Related issues

Related to EDIT - task #9594: Use MediaSpecimen as input for MediaSpecimenGeneralDetailElementClosedAndreas Müller

Actions
Related to EDIT - feature request #5575: DNA Sample requires CacheStrategyClosedAndreas Müller

Actions
Related to EDIT - bug #9609: Use DerivedUnit or FieldUnit for DetailsView of tree editorNewKatja Luther

Actions
Actions #1

Updated by Andreas Müller almost 3 years ago

  • Related to task #9594: Use MediaSpecimen as input for MediaSpecimenGeneralDetailElement added
Actions #2

Updated by Andreas Müller almost 3 years ago

  • Description updated (diff)
  • Status changed from New to Resolved
  • Assignee changed from Andreas Müller to Katja Luther
  • % Done changed from 0 to 50

This is implemented. However it does not work similar to MediaSpecimen as DnaSample somehow always gets the DerivedUnitFacadeCacheStrategy injected when data comes from the server.
This is probably because initDefaultCacheStrategy() does not fully work yet . It tries to solve the fact that also TissueSample uses DnaSample as class often and therefore the cache strategy can be defined only after setting basisOfRecord.

But this is only related to this issue. => #5575#note-10

Please review.

Actions #3

Updated by Andreas Müller almost 3 years ago

Actions #4

Updated by Katja Luther almost 3 years ago

  • Related to bug #9609: Use DerivedUnit or FieldUnit for DetailsView of tree editor added
Actions #5

Updated by Katja Luther almost 3 years ago

  • Status changed from Resolved to Closed

this works as expected.

Actions #6

Updated by Andreas Müller almost 3 years ago

  • Target version changed from Release 5.25 to Release 5.23
Actions

Also available in: Atom PDF