bug #9825
closedSelection wizards for areas in mulitlanguage instances show every vocabulary twice
100%
Description
The selection wizard for areas for the distribution editor shows the vocabularies twice in cuba, this is a multilanguage instance.
Related issues
Updated by Katja Luther about 2 years ago
- Related to bug #9598: Deduplicate NamedAreas with >1 representations and include label in search added
Updated by Andreas Müller about 2 years ago
- Status changed from New to In Progress
- Assignee changed from Katja Luther to Andreas Müller
- Priority changed from New to Highest
- Target version changed from Release 5.44 to Release 5.28
- % Done changed from 0 to 20
First implementation with d955087e . However, it may not yet work due to generics (the current DefinedTermDaoImpl.deduplicateResult() method maybe works only for CdmBase objects) but we work with DTOs here. Once this is tested we may also adapt other DTO based methods in TermVocabularyDaoImpl as there are lots of methods being DTO based but still loading the representation(s) as model objects.
Updated by Andreas Müller about 2 years ago
- Assignee changed from Andreas Müller to Katja Luther
The method used is a DTO method VocabularyService.findVocabularyDtoByTermType() or finally on DTO level TermVocabularyDaoImpl.findVocabularyDtoByTermTypes(). This method already implements deduplication server side as the new test (259c70d2e105) shows.
The DTO stores both representations in the result. So I guess the problem is somewhere on client side. Katja, can you check this.
Updated by Katja Luther about 2 years ago
- Status changed from In Progress to Feedback
- Assignee changed from Katja Luther to Andreas Müller
Andreas Müller wrote:
The method used is a DTO method VocabularyService.findVocabularyDtoByTermType() or finally on DTO level TermVocabularyDaoImpl.findVocabularyDtoByTermTypes(). This method already implements deduplication server side as the new test (259c70d2e105) shows.
The DTO stores both representations in the result. So I guess the problem is somewhere on client side. Katja, can you check this.
Sorry it was the wrong method, it is the find(UUID) which gives back two vocabularies for the uuid.
Updated by Andreas Müller about 2 years ago
- Status changed from Feedback to Resolved
- % Done changed from 20 to 50
Applied in changeset cdmlib|e3538034fba60c336b797d1ba58fbca89db817f8.
Updated by Andreas Müller about 2 years ago
- Assignee changed from Andreas Müller to Katja Luther
Should be solved now. Can you please review?
Updated by Andreas Müller about 2 years ago
- Status changed from Resolved to Closed
- Assignee changed from Katja Luther to Andreas Müller
- % Done changed from 50 to 100
I tested on rem_conf_am and everything seems to work as expected.