Project

General

Profile

bug #4390

Titlecache of field unit does not update when changing field unit data via derived unit facade

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

Status:
Closed
Priority:
Highest
Assignee:
Category:
cdmlib
Target version:
Start date:
09/02/2014
Due date:
% Done:

50%

Severity:
normal
Found in Version:

Description

What I did in TaxEditor

  • for some existing derived unit with field unit attached I updated the "locality" in the details view.

  • this automatically updates the title cache in the details view and also the specimen entry in the bulk editor

  • after saving and running new search I expect that also the field unit in the bulk editor has an updated title cache but this is not the case, only in details view it is updated

  • checking the database shows that the titleCache didn't update

The same is true for using the Derivate Search in TaxEditor.


Related issues

Related to Edit - bug #6138: fix test testNoRecursiveChangeEvents() in DerivedUnitFacadeTest Closed 10/20/2016
Related to Edit - bug #6448: Update of field unit in details view of derived unit does not update field unit record New 03/01/2017

Associated revisions

Revision 5d6b7e01 (diff)
Added by Katja Luther almost 2 years ago

fix #4390: update titlecache for all merged identifiableEntities

Revision 867bd81d (diff)
Added by Katja Luther almost 2 years ago

fix #4390: update titlecache only for SpecimenOrObservationBase entities

Revision e8b30a54 (diff)
Added by Katja Luther almost 2 years ago

fix #4390: add missing check for protected titleCache

Revision 958adbe2 (diff)
Added by Andreas Müller almost 2 years ago

ref #4390 adapt boolean value for protected

History

#1 Updated by Andreas Müller over 4 years ago

  • Target version set to cdmlib RELEASE 3.3.7
  • Priority changed from Priority08 to Highest

#2 Updated by Andreas Kohlbecker over 3 years ago

  • Target version changed from cdmlib RELEASE 3.5.0 to cdmlib RELEASE 3.5.1

moving tickets to next milestone

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

  • Target version changed from cdmlib RELEASE 3.5.1 to cdmlib RELEASE 3.5.2

move open 3.5.1 tickets to next milestone after release

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

  • Target version changed from cdmlib RELEASE 3.5.2 to Unassigned CDM tickets

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

  • Related to bug #6138: fix test testNoRecursiveChangeEvents() in DerivedUnitFacadeTest added

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

  • Description updated (diff)
  • Assignee changed from Andreas Müller to Katja Luther
  • Target version changed from Unassigned CDM tickets to Release 4.6

#7 Updated by Katja Luther almost 2 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50

#8 Updated by Katja Luther almost 2 years ago

  • Assignee changed from Katja Luther to Andreas Müller

please review

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

This second parameter in this code to me does not make sense. Is always false IMO.

if (!specimen.isProtectedTitleCache()){
      specimen.setTitleCache(specimen.generateTitle(), specimen.isProtectedTitleCache());
}

Changed this with 958adbe25 please check for correctness.

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

  • Related to bug #6448: Update of field unit in details view of derived unit does not update field unit record added

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

  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Katja Luther

The issue itself seems to be fixed and can be closed.
I opened a related ticket for refreshing the field unit + sibblings in the bulk editor: #6448

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

  • Private changed from Yes to No

#13 Updated by Katja Luther over 1 year ago

  • Status changed from Feedback to Closed

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)