Project

General

Profile

Actions

task #9678

closed

Remove Derived- and FieldUnitFacadeCacheStrategies

Added by Andreas Müller over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Highest
Category:
cdmlib
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Severity:
normal

Description

Cache strategies for FieldUnits and DerivedUnits are currently using the DerivedUnitFacadeCacheStrategy or DerivedUnitFacadeFieldUnitCacheStrategy.

This is problematic because

  1. these cache strategies are only available in cdmlib-service and therefore can be loaded only with workaround within the model classes.
  2. for units that have data that are against the facade contract an exception is thrown when trying to compute the titleCaches

We should implement cache strategies within cdmlib-model doing more or less the same as the facade cachestrategies. At some places the code can be much more simple then in the facade as the facade is intened to also be used for writing data which makes it much more complex.

This ticket is also a prerequisit for implementing the "identityCache" or "shortCache" for SpecimenOrObservationBase.


Related issues

Related to EDIT - feature request #5952: Second cache field for specimen (identityCache)ClosedAndreas Müller

Actions
Related to EDIT - feature request #5951: Add collector, field number and collection to specimen search labelNewKatja Luther

Actions
Related to EDIT - task #9679: Formatter and update script for SpecimenOrObservationBase short cacheNewAndreas Müller

Actions
Related to EDIT - bug #8517: SpecimenOrObservation: data from protected titleCache missig in portal pagesClosedAndreas Kohlbecker

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

Actions
Related to EDIT - feature request #5573: Cache Strategy for Media Specimen requiredClosedAndreas Müller

Actions
Related to EDIT - bug #9683: Auto-initialize root beanClosedAndreas Müller

Actions
Related to EDIT - bug #7904: DerivedUnitFacadeFieldUnitCacheStrategy.getCollectorAndFieldNumber() creates temporary Team objects which can not be garbage collected. ClosedAndreas Müller

Actions
Related to EDIT - feature request #9894: Make primary collector editable for field unit ClosedKatja Luther

Actions
Related to EDIT - bug #9978: Change of CacheStrategy from DerivedUnitFacade to DerivedUnitDefaultCacheStrategy results in different caches for MediaSpecimenIn ProgressKatja Luther

Actions
Actions #1

Updated by Andreas Müller over 1 year ago

Actions #2

Updated by Andreas Müller over 1 year ago

  • Related to feature request #5951: Add collector, field number and collection to specimen search label added
Actions #3

Updated by Andreas Müller over 1 year ago

  • Related to task #9679: Formatter and update script for SpecimenOrObservationBase short cache added
Actions #4

Updated by Andreas Müller over 1 year ago

  • Tags set to formatting
Actions #5

Updated by Andreas Müller over 1 year ago

  • Related to bug #8517: SpecimenOrObservation: data from protected titleCache missig in portal pages added
Actions #6

Updated by Andreas Müller over 1 year ago

Actions #7

Updated by Andreas Müller over 1 year ago

Actions #8

Updated by Andreas Müller over 1 year ago

  • Tags deleted (formatting)
  • Status changed from New to In Progress

Mostly implemented but there is still a test in vaadin that fails due to a lazy initialization exception when calling specimen.getTitleCache().

Actions #9

Updated by Andreas Müller about 1 year ago

  • Related to bug #9683: Auto-initialize root bean added
Actions #10

Updated by Andreas Müller about 1 year ago

  • Status changed from In Progress to Resolved

The failing test in vaadin is related to incorrect autoinitialization (#9683) which is fixed now. So this ticket should be resolved and is up for review.

Actions #11

Updated by Andreas Müller about 1 year ago

  • Status changed from Resolved to Closed
  • % Done changed from 50 to 100

I think we can close this ticket now. Also the failing test in dataportal is fixed now (mediaspecimen are shown there in typification context with short label as the MediaSpecimenDefaultCacheStrategy is used there now, which is correct).

However, we need to adapt all SpecimenOrObservationBase cachestrategies in a way that they have a long version and a short version which is related to the "identityCache" (#5952).

If anybody wants to review this ticket, please feel free to reopen.

Actions #12

Updated by Andreas Müller about 1 year ago

  • Related to bug #7904: DerivedUnitFacadeFieldUnitCacheStrategy.getCollectorAndFieldNumber() creates temporary Team objects which can not be garbage collected. added
Actions #13

Updated by Andreas Müller 10 months ago

Actions #14

Updated by Andreas Müller 6 months ago

  • Related to bug #9978: Change of CacheStrategy from DerivedUnitFacade to DerivedUnitDefaultCacheStrategy results in different caches for MediaSpecimen added
Actions

Also available in: Atom PDF