bug #9556
closedoccurrence/{uuid}/fieldUnitDTOs web service adds duplicate units to response
100%
Description
see http://localhost:8080/occurrence/8585081c-b73b-440b-b349-582845cf3fb4/fieldUnitDTOs.json
the response data may be filtered with
http http://localhost:8080/occurrence/8585081c-b73b-440b-b349-582845cf3fb4/fieldUnitDTOs.json | jq '.' | grep 'XKCD MASKS 2X (Masks)'
to see duplicates easily
Related issues
Updated by Andreas Kohlbecker over 2 years ago
- Related to bug #9557: implement occurrence/{uuid}/rootUnitDTOs added
Updated by Andreas Kohlbecker over 2 years ago
This method is kind of misused: After solving #9557 it is only being used by .DerivedUnitFacadeController.doGetFieldObjectMediaDTO() which again is used by the data portal to retrieve the FieldUnit Media.
- First of all this method adds an extra overhead by creating DTOs for all intermediate units which are not being used at all.
- Secondly does this method not deliver media of all FieldUnits, only for the first one that is found, see #9253
Updated by Andreas Kohlbecker over 2 years ago
- Related to task #9253: revise OccurrenceServiceImpl.findFieldUnitDTO(..) added
Updated by Andreas Kohlbecker over 2 years ago
- Related to task #9430: revise OccurrenceServiceImpl._findRootUnitDTO(..) supports multiple origin paths added
Updated by Andreas Kohlbecker over 2 years ago
- Related to deleted (task #9430: revise OccurrenceServiceImpl._findRootUnitDTO(..) supports multiple origin paths)
Updated by Andreas Kohlbecker over 2 years ago
- Status changed from New to Rejected
- Target version deleted (
Release 5.22)
since the intermediate units are not used at all by consuming methods, they should be skipped completely.
Updated by Andreas Kohlbecker over 2 years ago
- Status changed from Rejected to Feedback
- Assignee changed from Andreas Kohlbecker to Katja Luther
- Target version set to Release 5.22
reopening this issue since loadFieldUnitDTO() is being used in the Taxeditor
@Katja: are the derivatives of the FieldUnitDTO being used in the TaxEditor?
Updated by Andreas Kohlbecker over 2 years ago
- Category changed from cdmlib-remote to cdmlib
DerivedUnitFacadeController.doGetFieldObjectMediaDTO() is no longer using loadFieldUnitDTO see 3309605f
therefore the Taxeditor is the only component which is using this method.
Updated by Katja Luther over 2 years ago
- Assignee changed from Katja Luther to Andreas Kohlbecker
Andreas Kohlbecker wrote:
reopening this issue since
loadFieldUnitDTO()is being used in the Taxeditor@Katja: are the derivatives of the FieldUnitDTO being used in the TaxEditor?
no, the derivatives are not used in the editor
Updated by Andreas Kohlbecker over 2 years ago
- Assignee changed from Andreas Kohlbecker to Katja Luther
so we should modify this service method so that it no longer add the derivatives or is there an alternative service method that can be used instead. In the latter case loadFieldUnitDTO could be removed.
Updated by Katja Luther over 2 years ago
- Assignee changed from Katja Luther to Andreas Kohlbecker
I removed that the derivatives are added to the fieldUnitDto in loadFieldUnitDto().
Updated by Andreas Kohlbecker over 2 years ago
- Status changed from Feedback to Resolved
- % Done changed from 0 to 50
after the 6d3c3fa9 which is rather related to listRootUnitDTOsByAssociatedTaxon() the web services should no longer include duplicate derivatives
Updated by Andreas Kohlbecker over 2 years ago
- Status changed from Resolved to Closed
- % Done changed from 50 to 100
can be closed now