Project

General

Profile

Actions

feature request #9515

closed

Add DescriptionType to TaxonDescription of TaxonAssociations in ABCD import

Added by Katja Luther over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
New
Assignee:
Category:
cdmadapter
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Severity:
normal
Tags:

Description

copied from #9512:

Probably more such descriptions are created by the ABCD import. Katja, can you please check if ABCD already sets the status? If not create a ticket with additivity tag.
There are some other issues related to this status. I will create new tickets for them.

Open issues:

  • search for a description of type "TaxonAssociation" and use this description or create a new one

Related issues

Related to EDIT - bug #9512: Descriptions being created for individual associations in matrix should have description status "Individual association"ClosedKatja Luther

Actions
Related to EDIT - feature request #9519: Add posibilty to add IndividualAssociation descriptions in Facts ViewNewKatja Luther

Actions
Actions #1

Updated by Katja Luther over 1 year ago

  • Related to bug #9512: Descriptions being created for individual associations in matrix should have description status "Individual association" added
Actions #2

Updated by Andreas Müller over 1 year ago

  • Tags set to additivity, fast
  • Target version changed from Unassigned CDM tickets to Release 5.22
Actions #3

Updated by Andreas Müller over 1 year ago

  • Related to feature request #9519: Add posibilty to add IndividualAssociation descriptions in Facts View added
Actions #4

Updated by Andreas Müller over 1 year ago

  • Subject changed from Add DescriptionType to TaxonDescription of TaxonAssociations to Add DescriptionType to TaxonDescription of TaxonAssociations in ABCD import
Actions #5

Updated by Katja Luther over 1 year ago

Now a description with type IndivialsAssociation is created or if already existing used to add the individual associations.

Actions #6

Updated by Katja Luther over 1 year ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50
Actions #7

Updated by Katja Luther over 1 year ago

  • Category changed from cdmadapter to taxeditor
  • Assignee changed from Katja Luther to Andreas Müller
  • Severity changed from normal to blocker

please review

Actions #8

Updated by Andreas Müller over 1 year ago

  • Category changed from taxeditor to cdmadapter
  • Severity changed from blocker to normal
Actions #9

Updated by Andreas Müller over 1 year ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Katja Luther

Have you tested with a real import that it works?

Actions #10

Updated by Andreas Müller over 1 year ago

  • % Done changed from 50 to 70

I have renamed the method to set the description to the import state to get/setIndividualsAssociationDescriptionPerTaxon to make semantics clear. (push will come with the model update branch)

There is one thing I wonder about: in about line 1253 you call this method again. I it should be removed here and either added to the loop (before break) or generally be put before "//PREPARE REFERENCE QUESTIONS" and called only 1x (so being removed in the if-statement with "TaxonDescription.NewInstance(taxon, false);" (the later is preferred as it is simpler)

Can you adapt the code accordingly (once I have pushed my changes) and run a final import test on nightly. Then we can close this ticket.

Actions #11

Updated by Katja Luther over 1 year ago

  • Status changed from Feedback to Closed

adapted the code, tested and worked as expected.

Actions #12

Updated by Katja Luther over 1 year ago

  • % Done changed from 70 to 100
Actions #13

Updated by Andreas Müller over 1 year ago

  • Tags changed from additivity, fast to additivity
Actions

Also available in: Atom PDF