bug #9556
closed
- Related to bug #9557: implement occurrence/{uuid}/rootUnitDTOs added
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
- Related to task #9253: revise OccurrenceServiceImpl.findFieldUnitDTO(..) added
- Related to task #9430: revise OccurrenceServiceImpl._findRootUnitDTO(..) supports multiple origin paths added
- Related to deleted (task #9430: revise OccurrenceServiceImpl._findRootUnitDTO(..) supports multiple origin paths)
- 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.
- 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?
- 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.
- 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
- 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.
- Assignee changed from Katja Luther to Andreas Kohlbecker
I removed that the derivatives are added to the fieldUnitDto in loadFieldUnitDto().
- 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
- Status changed from Resolved to Closed
- % Done changed from 50 to 100
Also available in: Atom
PDF