Project

General

Profile

feature request #5833

task #4338: [MASTER] Handle abbreviated title and protected cache correctly in TaxEditor

Implement single cache strategy for references

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

Status:
Closed
Priority:
Highest
Category:
cdmlib
Target version:
Start date:
05/25/2016
Due date:
07/11/2016
% Done:

100%

Severity:
normal

Description

The new single default cache strategy for Reference.

As we do have only one Reference class left which implements multiple interfaces, we may also only need 1 single cache strategy. However, care must be taken as the formatting differs dependent on the type an the in-reference structure.

Generally the cache strategy allows 3 formattings:

  • for bibliographic references (stored in {@link Reference#getTitleCache() titleCache}).

  • for nomenclatural references (stored in {@link Reference#getAbbrevTitleCache() abbrevTitleCache}), but without micro reference (detail).

  • for nomenclatural references with micro reference, but not stored anywhere as the micro reference is part of the name, not of the reference

  • for short citation (e.g. Author 2009) as defined in {@link IReferenceBaseCacheStrategy#getCitation(Reference)}

  • the fullAbbrevTitleString which is the same as the titleCache but using the abbrevTitle instead of titleCache if possible, not stored in any cache

Also remove the rectifyCacheStrateggy method in Reference as it sets the old cache strategy.

duplicated by: #2862 (but more detailed)


Related issues

Related to Edit - bug #6960: Improve reference cachestrategy New 09/20/2017

Associated revisions

Revision 28085 (diff)
Added by Andreas Müller almost 3 years ago

Implement single cache strategy for references #5833

  • Implemented new class NewDefaultReferenceCacheStrategy and helper class TitleWithoutYearAndAuthorHelper
  • Moved all functionality of old ReferenceCacheStrategies to new class(es)
  • Moved all old strategies to package .old
  • Moved all tests into one test class
  • Removed all old tests
  • Renamed abbrevTitleCache to fullAbbrevTitleString in cache strategies
  • Added new method getNomenclaturalCache to cache strategy

Revision 9db62d49 (diff)
Added by Andreas Müller almost 3 years ago

Implement single cache strategy for references #5833

  • Implemented new class NewDefaultReferenceCacheStrategy and helper class TitleWithoutYearAndAuthorHelper
  • Moved all functionality of old ReferenceCacheStrategies to new class(es)
  • Moved all old strategies to package .old
  • Moved all tests into one test class
  • Removed all old tests
  • Renamed abbrevTitleCache to fullAbbrevTitleString in cache strategies
  • Added new method getNomenclaturalCache to cache strategy

History

#1 Updated by Andreas Müller almost 3 years ago

  • Status changed from New to In Progress

#2 Updated by Andreas Müller almost 3 years ago

implemented the following by cdmlib 9db62d494989

  • Implemented new class NewDefaultReferenceCacheStrategy and helper

class TitleWithoutYearAndAuthorHelper

  • Moved all functionality of old ReferenceCacheStrategies to new

class(es)

  • Moved all old strategies to package .old

  • Moved all tests into one test class

  • Removed all old tests

  • Renamed abbrevTitleCache to fullAbbrevTitleString in cache strategies

  • Added new method getNomenclaturalCache to cache strategy

#3 Updated by Andreas Müller almost 3 years ago

[28085:28090]

#4 Updated by Andreas Müller almost 3 years ago

  • Target version changed from Release 4.1 to Release 4.2

#5 Updated by Andreas Müller almost 3 years ago

  • Target version changed from Release 4.2 to Release 4.3

#6 Updated by Andreas Müller almost 3 years ago

  • Status changed from In Progress to Resolved
  • Target version changed from Release 4.3 to Release 4.1

#7 Updated by Andreas Müller almost 3 years ago

  • Status changed from Resolved to Closed
  • Resolution set to fixed
  • % Done set to 100

Seems to be fixed. However, we definetly still need to improve the cache strategy. But this is handled in separate tickets.

#8 Updated by Andreas Müller almost 3 years ago

  • Priority changed from New to Highest

#10 Updated by Andreas Müller over 1 year ago

  • Description updated (diff)
  • Private changed from Yes to No

#11 Updated by Andreas Müller over 1 year ago

  • Related to bug #6960: Improve reference cachestrategy added

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)