Project

General

Profile

bug #9395

DerivedUnit.collectFieldUnits() and findFieldUnitDTOByAssociatedTaxon() and findByAccessionNumber() in OccurrenceService to return all root units

Added by Andreas Kohlbecker about 2 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Highest
Category:
cdmlib
Target version:
Start date:
01/18/2021
Due date:
% Done:

100%

Severity:
normal
Found in Version:
Tags:

Description

DerivedUnit.collectFieldUnits() needs to returns also DerivedUnits as root unit which do not have a FieldUnit. This is required for #7704

same accounts for the IOccurrenceService methods:

  • FieldUnitDTO findByAccessionNumber(String accessionNumberString, List orderHints);
  • List findFieldUnitDTOByAssociatedTaxon(Set includedRelationships,
  • Pager pageFieldUnitsByAssociatedTaxon(Set includeRelationships,

Related issues

Related to Edit - feature request #7704: Dataportal specimen table should be able to show DerivedUnits without a FieldUnit Closed 08/30/2018
Related to Edit - bug #9397: Derivate tree view must show DerivedUnits even if the FieldUnit is missing Closed 01/18/2021
Related to Edit - bug #9415: DNA and other Derivatives missing in DerivedUnitDTO Closed 01/22/2021

Associated revisions

Revision 485da8a9 (diff)
Added by Andreas Kohlbecker about 2 months ago

fix #9395 DerivedUnit.collectFieldUnits() as DerivedUnit.collectRootUnits() to return all types of root units

Revision 26f72855 (diff)
Added by Andreas Kohlbecker about 2 months ago

fix #9395 DerivedUnit.collectFieldUnits() as DerivedUnit.collectRootUnits() to return all types of root units - removig debug code

Revision 9fcfd180 (diff)
Added by Andreas Kohlbecker about 1 month ago

ref #9395 adapting services and web-services to return all root units

Revision 70e9fc0d (diff)
Added by Andreas Kohlbecker about 1 month ago

ref #9395 adapting services and web-services to return all root units - adapting taxeditor

Revision 25b933c4 (diff)
Added by Andreas Kohlbecker about 1 month ago

ref #7704, ref #9395 using web-services now returning all root units

Revision 408979d3 (diff)
Added by Andreas Kohlbecker about 1 month ago

ref #9395 adapting services and web-services to return all root units - adapting taxeditor - generic type fix

Revision 2edb5aa1 (diff)
Added by Andreas Kohlbecker about 1 month ago

ref #9395 fully implementing maxDepth for derivative assembly and fixing OccurrenceServiceImpl._findRootUnitDTO()

Revision 8195f008 (diff)
Added by Andreas Kohlbecker about 1 month ago

ref #9395 findByAccessionNumber() -> findByGeneticAccessionNumber(), preserving old method as deprecated

Revision c5c00803 (diff)
Added by Andreas Kohlbecker about 1 month ago

ref #9395 simplifying findByGeneticAccessionNumber() as extra code has no imact on performance

Revision 5e0162fb (diff)
Added by Andreas Kohlbecker about 1 month ago

ref #7599 ref #9395 documenting requirements

Revision 021cc0fe (diff)
Added by Andreas Kohlbecker about 1 month ago

ref #9395 solving problem in derivation-dto-graph assembly (replaced field units)

Revision b2217f44 (diff)
Added by Andreas Kohlbecker about 1 month ago

ref #9395 adapting test to modified service method

History

#1 Updated by Andreas Kohlbecker about 2 months ago

  • Related to feature request #7704: Dataportal specimen table should be able to show DerivedUnits without a FieldUnit added

#2 Updated by Andreas Kohlbecker about 2 months ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50

#3 Updated by Andreas Kohlbecker about 2 months ago

Hi, for your information:

to solve this ticket I renamed and extended DerivedUnit.collectFieldUnits() as DerivedUnit.collectRootUnits(). I don't expect any probelms for other parts of the library.

Best
Andreas

#4 Updated by Andreas Kohlbecker about 2 months ago

  • Related to bug #9397: Derivate tree view must show DerivedUnits even if the FieldUnit is missing added

#5 Updated by Andreas Kohlbecker about 1 month ago

  • Related to bug #9415: DNA and other Derivatives missing in DerivedUnitDTO added

#6 Updated by Andreas Kohlbecker about 1 month ago

  • Subject changed from DerivedUnit.collectFieldUnits() to return all root units to DerivedUnit.collectFieldUnits() and findFieldUnitDTOByAssociatedTaxon() and findByAccessionNumber() in OccurrenceService to return all root units
  • Description updated (diff)

#7 Updated by Andreas Kohlbecker about 1 month ago

  • Status changed from Resolved to Closed
  • % Done changed from 50 to 100

completed

#8 Updated by Andreas Kohlbecker about 1 month ago

  • Status changed from Closed to In Progress
  • % Done changed from 100 to 50

#9 Updated by Andreas Kohlbecker about 1 month ago

  • Status changed from In Progress to Resolved

#10 Updated by Andreas Kohlbecker about 1 month ago

  • Status changed from Resolved to Closed
  • % Done changed from 50 to 100

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)