feature request #8030
closedCache updater service methods externalized and base class specific
100%
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
Updated by Andreas Kohlbecker about 5 years ago
- Copied from bug #7874: TeamOrPersonBase entity can become unusable due to replacement of the title caches on using getters added
Updated by Andreas Müller about 5 years ago
- Status changed from New to In Progress
- Priority changed from New to Highest
- Target version changed from Unassigned CDM tickets to Release 5.5
- % Done changed from 0 to 30
Updated by Andreas Müller about 5 years ago
- Status changed from In Progress to Resolved
- Assignee changed from Andreas Müller to Andreas Kohlbecker
I think this is fixed now. Please review.
Updated by Andreas Müller about 5 years ago
- Related to feature request #8096: Implement CacheUpdater as long running task with correct progress monitoring added
Updated by Andreas Kohlbecker about 5 years ago
- Status changed from Resolved to Closed
- Assignee changed from Andreas Kohlbecker to Andreas Müller
- % Done changed from 30 to 100
review completed, no issues found