Project

General

Profile

Actions

bug #9365

closed

Reuse taxon does not work in new Taxon dialogue

Added by Andreas Müller almost 2 years ago. Updated almost 2 years ago.

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

90%

Estimated time:
Severity:
normal
Found in Version:

Description

partly copied from #5114.

If reuse taxon is selected still a new taxon is created.

Also the secundum field should be disabled as long as reuse taxon is selected (it is already filled with the correct secundum).


Related issues

Related to EDIT - task #9407: Reuse taxon should check classification before saveNewKatja Luther

Actions
Actions #2

Updated by Andreas Müller almost 2 years ago

  • Description updated (diff)
Actions #3

Updated by Katja Luther almost 2 years ago

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

Updated by Katja Luther almost 2 years ago

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

please review

Actions #5

Updated by Andreas Müller almost 2 years ago

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

The disabling of the secundum is still missing.

Actions #6

Updated by Andreas Müller almost 2 years ago

As you are now using a different method for createNewTaxonNode all the TaxonNode data like source and status are not persisted anymore.

I guess we should combine the 2 methods in a way that they use the same code except for all taxon related data.

So the reuse taxon method fetches the taxon and then runs the taxon node code and the DTO method creates the taxon and then runs the same code for taxon node data. Ofcourse the signature of the first method needs to be adapted, then.

Actions #7

Updated by Katja Luther almost 2 years ago

This is fixed with the last commit for #9393

Actions #8

Updated by Andreas Müller almost 2 years ago

  • Related to task #9407: Reuse taxon should check classification before save added
Actions #9

Updated by Katja Luther almost 2 years ago

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

please review.

Actions #10

Updated by Andreas Müller almost 2 years ago

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

Andreas Müller wrote:

As you are now using a different method for createNewTaxonNode all the TaxonNode data like source and status are not persisted anymore.

I guess we should combine the 2 methods in a way that they use the same code except for all taxon related data.

Has this also been fixed? I can't see the code.

Actions #11

Updated by Katja Luther almost 2 years ago

  • Status changed from Feedback to Resolved

Andreas Müller wrote:

Andreas Müller wrote:

As you are now using a different method for createNewTaxonNode all the TaxonNode data like source and status are not persisted anymore.

I guess we should combine the 2 methods in a way that they use the same code except for all taxon related data.

Has this also been fixed? I can't see the code.

this is fixed, too.

Actions #12

Updated by Katja Luther almost 2 years ago

  • Assignee changed from Katja Luther to Andreas Müller
Actions #13

Updated by Andreas Müller almost 2 years ago

  • Status changed from Resolved to Feedback
  • Assignee changed from Andreas Müller to Katja Luther
  • % Done changed from 50 to 90

I changed it a bit the way that first it is checked if a taxon is reused, if not a new taxon is created (with according params) and it is checked if a name is reused. If not everything is created a new.

If reuse taxon or name do not exist an exception is thrown (until now a new taxon was created which is unwanted behavior I guess).

Please review updated code. If ok, close ticket.

Actions #14

Updated by Katja Luther almost 2 years ago

  • Status changed from Feedback to Closed

the looks fine, close the ticket.

Actions

Also available in: Atom PDF