Project

General

Profile

Actions

feature request #8030

closed

Cache updater service methods externalized and base class specific

Added by Andreas Kohlbecker about 5 years ago. Updated about 5 years ago.

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

100%

Estimated time:
Severity:
normal

Description

copied from #7874

Generally at the moment I do not fully understand how far we need the lines after line 439. The call

entitiesToUpdate.add(entity);

is not really required I think as the entities all belong to the session already. If not, are the .getTitleCache or .getAbbrevTitleCace calls important here? Or can we completely delete this part. If not I wonder why the calls are alternative, e.g. if olcTitleCache <> newTitleCache the entity instanceof Team part is not called anymore. Is this on purpose?

Also the whole method is only called if entity.isProtectedTitleCache() == false which might be unwanted e.g. for Teams. If the titleCache is protected but nomenclaturalTitleCache is not protected we may still want to update it.

Generally we need to refactor this part and move away the specific class handling from IdentifiableServiceBase (e.g. we could move the update routines to the model classes as the knowledge about the model classes is best here)


Related issues

Related to EDIT - feature request #8096: Implement CacheUpdater as long running task with correct progress monitoringClosedKatja Luther

Actions
Copied from EDIT - bug #7874: TeamOrPersonBase entity can become unusable due to replacement of the title caches on using gettersClosedAndreas Müller

Actions
Actions

Also available in: Atom PDF