Project

General

Profile

task #9595

Use DerivedUnit or DnaSample for DnaSampleGeneralDetailElement

Added by Andreas Müller about 2 months ago. Updated 28 days ago.

Status:
Closed
Priority:
Priority14
Assignee:
Category:
taxeditor
Target version:
Start date:
04/29/2021
Due date:
% Done:

50%

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 MediaSpecimenGeneralDetailElement Closed 04/29/2021
Related to Edit - feature request #5575: DNA Sample requires CacheStrategy Feedback 02/18/2016
Related to Edit - bug #9609: Use DerivedUnit or FieldUnit for DetailsView of tree editor New 05/06/2021

Associated revisions

Revision 05bdc44f (diff)
Added by Andreas Müller about 2 months ago

ref #9595 move xxxPreferredDetermination from facade to SpecimenOrObservationBase

Revision a1dce9c6 (diff)
Added by Andreas Müller about 2 months ago

ref #9595 add SampleDesignation uuid to DefinedTerm

Revision 16419c4d (diff)
Added by Andreas Müller about 2 months ago

ref #9595 load empty DnaSamples with DnaSampleDefaultCacheStrategy

Revision 4899e12f (diff)
Added by Andreas Müller about 2 months ago

ref #9595 load sample designation preferred to other identifiers for DNA Sample

Revision bad2d5b5 (diff)
Added by Andreas Müller about 2 months ago

ref #9595 shortcut to load single identifier

Revision 8aaabf50 (diff)
Added by Andreas Müller about 1 month ago

ref #9595 , ref #5575 use DnaSampleCacheStrategy in DerivateLabelProvider

History

#1 Updated by Andreas Müller about 2 months ago

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

#2 Updated by Andreas Müller about 1 month 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.

#3 Updated by Andreas Müller about 1 month ago

#4 Updated by Katja Luther about 1 month ago

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

#5 Updated by Katja Luther about 1 month ago

  • Status changed from Resolved to Closed

this works as expected.

#6 Updated by Andreas Müller 28 days ago

  • Target version changed from Release 5.25 to Release 5.23

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)