Project

General

Profile

Actions

feature request #3472

closed

Update name in source data when changing an accepted taxon into a synonym

Added by Andreas Kohlbecker almost 11 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Highest
Assignee:
Category:
cdmlib
Target version:
Start date:
Due date:
% Done:

30%

Estimated time:
Severity:
major

Description

name in source information is not always entered. DescriptionElementSources which are having an empty nameUsedInSource field are considered to refer to the name of the accepted taxon. When changing an accepted taxon into a synonym all DescriptionElementSources where nameUsedInSource == null should get updated with name of the the former accepted taxon.

affected methods:

  • ITaxonService.swapSynonymWithAcceptedTaxon

  • ITaxonNodeService.makeTaxonNodeASynonymOfAnotherTaxonNode

  • more ?

the same feature is also required when moving descriptions: #3982 (update name in source data when moving a Description to another Taxon)

NOTE: This is a critical feature as it is maybe not wanted or creates incorrect data, please read comments on #3982 before implementing

Alternatively we could ask the user to fill the nameInSource in case it is not yet filled


Related issues

Related to EDIT - feature request #3982: Update name in source data when moving a Description to another TaxonNewKatja Luther

Actions
Related to EDIT - feature request #8478: Follow up ticket for update empty name in source handling in TaxEditorNewKatja Luther

Actions
Actions #1

Updated by Andreas Müller almost 11 years ago

Please have in mind that OriginalSourceBase also allows the String attribute originalNameString. This is a more generic field that does not only apply for names. It is also possible that the original name is stored in this field. In E+M this may be the case as E+M in Berlin Model also allows foreign keys as well as Strings for original name information.

Actions #2

Updated by Andreas Müller almost 11 years ago

Also we have to have in mind that missing original name information does not necessarily mean that the name used in the source is the one of the the accepted taxon. It may also be left out or in the Berlin Model Editor when you moved a taxon you may have forgot to adapt the information accordingly. In this case automatic addition of the name will lead to wrong data.

Actions #3

Updated by Andreas Kohlbecker over 10 years ago

adding a snipped from my notes just for not to lose it:

"this behaviour whether name in source is automatically filled or not, should be configurable in the dialogue : checkbox!"

Actions #4

Updated by Andreas Müller over 10 years ago

  • Severity changed from critical to major
Actions #5

Updated by Andreas Kohlbecker about 10 years ago

  • Keywords set to Euro+Med,Migration
  • Target version changed from Euro+Med Migration to cdmlib RELEASE 3.3.0

moving resolved tickets to release milestone

Actions #6

Updated by Andreas Kohlbecker about 10 years ago

  • Target version changed from cdmlib RELEASE 3.3.0 to Euro+Med Migration
Actions #7

Updated by Andreas Müller over 6 years ago

  • Related to feature request #3982: Update name in source data when moving a Description to another Taxon added
Actions #8

Updated by Andreas Müller over 6 years ago

  • Description updated (diff)
  • Priority changed from New to Priority13
  • Private changed from Yes to No
Actions #9

Updated by Andreas Müller over 6 years ago

  • Tracker changed from bug to feature request
  • Subject changed from update name in source data when changing an accepted taxon into a synonym to Update name in source data when changing an accepted taxon into a synonym
Actions #10

Updated by Andreas Müller over 5 years ago

  • Priority changed from Priority13 to Priority10
Actions #14

Updated by Andreas Müller over 5 years ago

  • Description updated (diff)
  • Target version changed from Euro+Med Migration to Euro+Med post migration
Actions #15

Updated by Andreas Müller almost 5 years ago

  • Assignee changed from Andreas Müller to Katja Luther
  • Priority changed from Priority10 to Highest
  • Target version changed from Euro+Med post migration to Euro+Med TaxEditor issues

We finally decided to implement it such for E+M, see comments in #3982

Actions #16

Updated by Katja Luther over 4 years ago

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

this is implemented now.

The setting of the name in source is configurable, but in editor a new wizard would be needed, therefore the actual implementation works with two menu items.
Please review and please have a look at the labels, I am not sure whether they are understandable or not.

Actions #17

Updated by Andreas Müller over 4 years ago

  • Target version changed from Euro+Med TaxEditor issues to Release 5.9
Actions #18

Updated by Andreas Müller over 4 years ago

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

Currently both methods do end in an unexpected state. After running the operation the old taxon is dirty (and unsaved) and the new taxon does not yet contain the new data. Only by saving the old taxon this seems to happen.

I don't know if this is related to this ticket but it should be fixed urgently (to me seems to be a regression).

Actions #19

Updated by Andreas Müller over 4 years ago

The method with setting the name should be moved down in menu

Actions #20

Updated by Andreas Müller over 4 years ago

  • % Done changed from 0 to 30

Please open a new ticket for the final implementation with a new wizard including the question if the name in sources should be taken from the accepted taxon if being empty and removing the second label.

Actions #21

Updated by Andreas Müller over 4 years ago

The labels of the 2 menus should be equal except for the addition about the Name in Source. Is there a reason why you use selected facts once and only fact(s) other time? Use the shorter as "selected" is not necessary.
This is correct for Descriptions but not for DescriptionELements.

"... Other Taxon ..." => "other taxon" for all 4 menu entries.

"... and set Name in Source" => "... and set Name in Source if empty" (for the 2 menu entries)

Actions #22

Updated by Andreas Müller over 4 years ago

Andreas Müller wrote:

Please open a new ticket for the final implementation with a new wizard including the question if the name in sources should be taken from the accepted taxon if being empty and removing the second label.

What about checking all elements to move (including the description itself?) if they have sources with empty name in source. If yes have a dialog and ask if the empty name in sources should be set to old accepted taxon automatically and allow yes, no, cancel. Only then open the ordinary dialog/assistant.
Maybe only check sources of type primaryXXX, no import sources.

Actions #23

Updated by Katja Luther over 4 years ago

Andreas Müller wrote:

Currently both methods do end in an unexpected state. After running the operation the old taxon is dirty (and unsaved) and the new taxon does not yet contain the new data. Only by saving the old taxon this seems to happen.

I don't know if this is related to this ticket but it should be fixed urgently (to me seems to be a regression).

But this was a requirement we fixed a few weeks ago. The moving should be done after saving.(#8234)

Actions #24

Updated by Andreas Müller over 4 years ago

Ok, in generall this probably makes sense. Maybe I just wasn't used to it yet.
However, if this is the expected behavior I do not understand why I get a warning that my taxon is not saved and it will be saved if I proceed with the command. I would expect this is not necessary at all for the old taxon and only necessary for the new taxon (and probably only for the save action and not yet during the move action)

Actions #25

Updated by Katja Luther over 4 years ago

  • Related to feature request #8478: Follow up ticket for update empty name in source handling in TaxEditor added
Actions #26

Updated by Katja Luther over 4 years ago

  • Status changed from Feedback to Closed

the new ticket is #8478 and the labels are adapted.

Actions

Also available in: Atom PDF