Project

General

Profile

report #8379

Review strange specimen implementations

Added by Andreas Kohlbecker 12 months ago. Updated 13 days ago.

Status:
New
Priority:
Highest
Category:
cdm-dataportal
Target version:
Start date:
07/17/2019
Due date:
% Done:

0%

Severity:
major

Description

Please

I am repeating according comments from the original tickets here:


#8370

Important for the review is to understand why the bug was introduced with cdm-dataportal|e89c1115 this was for sure in the attempt to fix another problem.


#8368

After fixing the problems in this code i noticed that this code is not generating any output at all.

I attempted to base the code on existing functions and wrote the following lines:

        if(isset($specimen_or_observation->specimenTypeDesignations) && count($specimen_or_observation->specimenTypeDesignations) > 0){
          // $specimen_or_observation->specimenTypeDesignations are DTOs
          $specimen_type_designation_map = array();
          foreach($specimen_or_observation->specimenTypeDesignations as $stdDTO){
            if(!isset($specimen_type_designation_map[$stdDTO->typeStatus])){
              $specimen_type_designation_map[$stdDTO->typeStatus] = array();
            }
            $specimen_type_designation_map[$stdDTO->typeStatus][] = $stdDTO;
          }
          $items['data'] = compose_specimen_type_designations($specimen_type_designation_map);
        }

I removed them again and left the original code as it is. This code was introduced in cdm-dataportal|b3ac03bf


Related issues

Copied from Edit - bug #8368: problems in compose_cdm_specimen_or_observation_tree_entry() Closed 07/16/2019
Copied from Edit - bug #8370: feature blocks created by compose_description_element_individuals_association() are broken Closed 07/16/2019

History

#1 Updated by Andreas Kohlbecker 12 months ago

  • Copied from bug #8368: problems in compose_cdm_specimen_or_observation_tree_entry() added

#2 Updated by Andreas Kohlbecker 12 months ago

  • Copied from bug #8370: feature blocks created by compose_description_element_individuals_association() are broken added

#3 Updated by Katja Luther 10 months ago

  • Target version changed from Release 5.10 to Release 5.11

#4 Updated by Katja Luther 8 months ago

  • Target version changed from Release 5.11 to Release 5.12

#5 Updated by Katja Luther 6 months ago

  • Target version changed from Release 5.12 to Release 5.13

#6 Updated by Katja Luther 4 months ago

  • Target version changed from Release 5.13 to Release 5.14

#7 Updated by Katja Luther 3 months ago

  • Target version changed from Release 5.14 to Release 5.15

#8 Updated by Andreas Müller 3 months ago

  • Assignee changed from Katja Luther to Andreas Kohlbecker

I think we decided to move this issue to AK. Please correct me if wrong.

Please also decide which milestone it should go to, but definetely it should not be postponed to much as a good specimen is a more and more important requirement.

#9 Updated by Andreas Kohlbecker 3 months ago

  • Description updated (diff)

Andreas Müller wrote:

I think we decided to move this issue to AK. Please correct me if wrong.

Yes, exactly. But I will definitely need Katja to help shed more light on the original requirements and expectations.

Please also decide which milestone it should go to, but definetely it should not be postponed to much as a good specimen is a more and more important requirement.

Milestone 5.15 seems like realistic perspective to me

#10 Updated by Andreas Müller 13 days ago

  • Target version changed from Release 5.15 to Release 5.17

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)