bug #9365
Reuse taxon does not work in new Taxon dialogue
90%
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
Associated revisions
fix #9365: fix reuse taxon in newTaxonNodeWizard
ref #9393: disable sec selection if reused taxon is used
merge the two create taxon methods
ref #9365 update code for reuse taxon (change order for check name and check taxon and throw exceptions if not exists)
History
#2 Updated by Andreas Müller 6 months ago
- Description updated (diff)
#3 Updated by Katja Luther 5 months ago
- Status changed from New to Resolved
- % Done changed from 0 to 50
Applied in changeset taxeditor|de6b6edbe71e3f03039fbc6436ad7edba7651e3b.
#4 Updated by Katja Luther 5 months ago
- Assignee changed from Katja Luther to Andreas Müller
- Severity changed from normal to blocker
please review
#5 Updated by Andreas Müller 5 months 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.
#6 Updated by Andreas Müller 5 months 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.
#7 Updated by Katja Luther 5 months ago
This is fixed with the last commit for #9393
#8 Updated by Andreas Müller 5 months ago
- Related to task #9407: Reuse taxon should check classification before save added
#9 Updated by Katja Luther 5 months ago
- Status changed from Feedback to Resolved
- Assignee changed from Katja Luther to Andreas Müller
please review.
#10 Updated by Andreas Müller 5 months 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.
#11 Updated by Katja Luther 5 months 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.
#12 Updated by Katja Luther 5 months ago
- Assignee changed from Katja Luther to Andreas Müller
#13 Updated by Andreas Müller 5 months 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.
#14 Updated by Katja Luther 5 months ago
- Status changed from Feedback to Closed
the looks fine, close the ticket.
