Project

General

Profile

task #8379

Review strange specimen implementations

Added by Andreas Kohlbecker about 2 years ago. Updated 22 days ago.

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

0%

Severity:
major
Tags:

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

TODO: check if it is feasible to simplify and unify the code by using compose_specimen_type_designations(). (low priority)


Related issues

Related to Edit - task #8419: harmonization of the different specimen representations In Progress 07/25/2019
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 about 2 years ago

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

#2 Updated by Andreas Kohlbecker about 2 years ago

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

#3 Updated by Katja Luther almost 2 years ago

  • Target version changed from Release 5.10 to Release 5.11

#4 Updated by Katja Luther over 1 year ago

  • Target version changed from Release 5.11 to Release 5.12

#5 Updated by Katja Luther over 1 year ago

  • Target version changed from Release 5.12 to Release 5.13

#6 Updated by Katja Luther over 1 year ago

  • Target version changed from Release 5.13 to Release 5.14

#7 Updated by Katja Luther over 1 year ago

  • Target version changed from Release 5.14 to Release 5.15

#8 Updated by Andreas Müller over 1 year 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 over 1 year 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 about 1 year ago

  • Target version changed from Release 5.15 to Release 5.18

#11 Updated by Andreas Kohlbecker about 1 year ago

  • Related to task #8419: harmonization of the different specimen representations added

#12 Updated by Andreas Kohlbecker 10 months ago

  • Description updated (diff)

#13 Updated by Andreas Kohlbecker 8 months ago

  • Target version changed from Release 5.18 to Release 5.19

TODO: final check

#14 Updated by Andreas Kohlbecker 6 months ago

  • Target version changed from Release 5.19 to Release 5.21

#15 Updated by Andreas Müller 5 months ago

  • Target version changed from Release 5.21 to Release 5.22

#16 Updated by Andreas Kohlbecker 4 months ago

  • Target version changed from Release 5.22 to Release 5.25

#17 Updated by Andreas Müller about 1 month ago

  • Tags set to specimen
  • Tracker changed from report to task
  • Status changed from New to Resolved

According to note 13 this only needs a final check. So I put it on resolved in case it is fixed and only needs review.

#18 Updated by Andreas Kohlbecker 22 days ago

  • Target version changed from Release 5.25 to Release 5.27

moving to next milestone to speed up review phase of 5.25.0

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)