bug #8767
closedWorkflow support for publish flag
100%
Description
When changing an unpublished synonym to a MAN aftwards the MAN is published. This is not correct. The publish flag should have the same value as the synonym before.
Please also check for similar operations.
These operations are:
- swap synonym and accepted taxon - works
- change synonym to MAN - works
- change MAN to synonym - works
- change accepted to synonym - works
- change synonym to accepted taxon - works
- move synonym/homotypic group to another taxon - works, but open issue: check synonyms of the homotypic group
- move accepted to new parent - works
Related issues
Updated by Katja Luther over 4 years ago
- Target version changed from Release 5.12 to Release 5.13
Updated by Katja Luther over 4 years ago
- Target version changed from Release 5.13 to Release 5.14
Updated by Andreas Müller over 4 years ago
- Target version changed from Release 5.14 to Release 5.15
Updated by Andreas Müller about 4 years ago
- Target version changed from Release 5.15 to Release 5.18
Updated by Katja Luther over 3 years ago
- Target version changed from Release 5.18 to Release 5.19
Updated by Andreas Müller over 3 years ago
- Target version changed from Release 5.19 to Release 5.21
Updated by Andreas Müller over 3 years ago
- Related to bug #9426: Set publish flag for Synonyms in accordance with accepted taxon publish flag added
Updated by Andreas Müller over 3 years ago
- Tags changed from euro+med to euro+med, fast
Updated by Katja Luther over 3 years ago
This is fixed, also for pp synonyms.
I also adapted the changeSynonymToAcceptedTaxon, the newly created taxon has the same publish state as the synonym, is this the wanted behaviour?
And the clone method of TaxonBase also clones the publish flag now.
Updated by Katja Luther over 3 years ago
- Status changed from New to Resolved
- Assignee changed from Katja Luther to Andreas Müller
- % Done changed from 0 to 60
Updated by Andreas Müller over 3 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
Katja Luther wrote:
I also adapted the changeSynonymToAcceptedTaxon, the newly created taxon has the same publish state as the synonym, is this the wanted behaviour?
Yes I think this is ok. Maybe only in case the publish Flag of the new taxon differs from the publish flag of the parent of the new taxon we could add a warning dialog saying that it differs and that it should be checked for correctnes.
Updated by Katja Luther over 3 years ago
- Status changed from Feedback to Resolved
- Assignee changed from Katja Luther to Andreas Müller
Andreas Müller wrote:
Katja Luther wrote:
I also adapted the changeSynonymToAcceptedTaxon, the newly created taxon has the same publish state as the synonym, is this the wanted behaviour?
Yes I think this is ok. Maybe only in case the publish Flag of the new taxon differs from the publish flag of the parent of the new taxon we could add a warning dialog saying that it differs and that it should be checked for correctnes.
now there is a message box when the publish flags differ.
Updated by Andreas Müller over 3 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
The message pops up 2x now
Updated by Andreas Müller over 3 years ago
Also swapping synonym with accepted has strange behavior: only the names swap but the publish flag stays. E.g. if old accepted is published the new accepted is also published and if the old synonym is unpublished the new synonym is also unpublished. This I think is unwanted.
In the rare case that the both status differ, we should keep the flag with the according taxon but warn that the flags differ.
Updated by Andreas Müller over 3 years ago
Accordingly we should check ALL operations to handle the publish flag correctly.
Updated by Katja Luther over 3 years ago
Andreas Müller wrote:
The message pops up 2x now
With the actual nightly build I can not reproduce this.
Updated by Andreas Müller over 3 years ago
Andreas Müller wrote:
The message pops up 2x now
I can not reproduce this anymore.
Updated by Andreas Müller over 3 years ago
Andreas Müller wrote:
Accordingly we should check ALL operations to handle the publish flag correctly.
E.g. also move synonym to another accepted taxon should do the same check as synonymToAccepted. If the old accepted taxon has a different publish flag then the new accepted taxon a warning should appear.
Updated by Andreas Müller over 3 years ago
By the way, currently this is handled as an info (i) but I think a warning would be better.
Updated by Katja Luther over 3 years ago
Changing an unpublished accepted to a synonym of an published accepted taxon, how should we handle this? Keep the publish flag from the old accepted taxon and warn?
Updated by Andreas Müller over 3 years ago
Katja Luther wrote:
Changing an unpublished accepted to a synonym of an published accepted taxon, how should we handle this? Keep the publish flag from the old accepted taxon and warn?
Yes, I think a warning is needed here. In general we should always warn if not all related taxa (taxon/syn itself, old parent/accepted, new parent/accepted) do have the same pub. status.
Updated by Katja Luther over 3 years ago
- Status changed from Feedback to Resolved
implemented the publish check also for accepted to syn.
Updated by Katja Luther over 3 years ago
- Assignee changed from Katja Luther to Andreas Müller
Updated by Andreas Müller over 3 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
Testing swap:
- if parent taxon has state 1 and old+new taxon state 2 no warning appears
- if warning appears it says mentions only 2 taxa not all 3. To not to have a warning for each different case we should have 1 warning saying "Parent taxon, old accepted taxon and new accepted taxon do not all have the same ..."
Updated by Andreas Müller over 3 years ago
Katja, can you please test correct functionality for each of the above operations and leave a message in this ticket what you have tested.
E.g.
swap: parent 0, taxon 0, syn 1 => message appears (OK)
swap: parent 1, taxon 0, syn 1 => no message (FALSE)
swap: parent 1, taxon 1, syn1 => no message (OK)
...
Updated by Katja Luther over 3 years ago
For operations like swap I do not see the need to test for the parent if the old and the new accepted have the same publish status, because the status of the child is still the same.
Updated by Andreas Müller over 3 years ago
Katja Luther wrote:
For operations like swap I do not see the need to test for the parent if the old and the new accepted have the same publish status, because the status of the child is still the same.
OK, you are right. In this special case it is not really needed to check all 3 related taxa.
Updated by Andreas Müller over 3 years ago
Tested Syn => MAN/ppSyn
acc 1, syn 0 => MAN 0 (OK)
acc 0, syn 1 => MAN 1 (OK)
acc 0, syn 0 => MAN 0 (OK)
Updated by Andreas Müller over 3 years ago
Tested MAN/ppSyn => Syn
acc 1, MAN 0 => Syn 0 (OK)
acc 1, MAN 1 => Syn 0 (FALSE)
acc 0, MAN 0 => Syn 0 (OK)
acc 0, MAN 1 => Syn 0 (FALSE)
The new synonym always looses its state when publish flag was checked. But it should keep the state (without warning in this case).
Updated by Andreas Müller over 3 years ago
Tested change accepted to synonym
Old parent 1, old acc 1, new acc 1 => new syn 0, w/o warning (FALSE, should be 1 w/o warning)
Old parent 1, old acc 1, new acc 0 => new syn 0, with warning (OK)
Old parent 1, old acc 0, new acc 1 => new syn 1, w/o warning (FALSE, should be 1 with warning)
Old parent 1, old acc 0, new acc 0 => new syn 0, with warning (FALS, should be 0 w/o warning)
Updated by Andreas Müller over 3 years ago
Andreas Müller wrote:
Tested change accepted to synonym
Old parent 1, old acc 1, new acc 1 => new syn 0, w/o warning (FALSE, should be 1 w/o warning)
Old parent 1, old acc 1, new acc 0 => new syn 0, with warning (OK)
Old parent 1, old acc 0, new acc 1 => new syn 1, w/o warning (FALSE, should be 1 with warning)
Old parent 1, old acc 0, new acc 0 => new syn 0, with warning (FALS, should be 0 w/o warning)
The evaluation assumes that the state of the old parent does not matter as discussed via email.
Updated by Andreas Müller over 3 years ago
change synonym to accepted taxon
Old acc 1, old syn 1, new parent 1 => new acc 1 w/o warning (OK)
Old acc 1, old syn 0, new parent 1 => new acc 0 with warning (OK)
Old acc 1, old syn 1, new parent 0 => new acc 1 with warning (OK)
Old acc 1, old syn 0, new parent 0 => new acc 0 w/o warning (OK)
Old acc 0, old syn 1, new parent 1 => new acc 1 w/o warning (OK), old acc should have no influence
This operation looks ok.
Updated by Katja Luther over 3 years ago
Andreas Müller wrote:
Tested MAN/ppSyn => Syn
acc 1, MAN 0 => Syn 0 (OK)
acc 1, MAN 1 => Syn 0 (FALSE)
acc 0, MAN 0 => Syn 0 (OK)
acc 0, MAN 1 => Syn 0 (FALSE)The new synonym always looses its state when publish flag was checked. But it should keep the state (without warning in this case).
this is fixed.
Updated by Andreas Müller over 3 years ago
Tested move synonym/homotypic group to another taxon
Old acc 1, old syn 1, new acc 1 => new syn 1 w/o warning (OK)
Old acc 1, old syn 0, new acc 1 => new syn 0 with warning (OK)
Old acc 1, old syn1 1, old syn2 0, new acc 1 => new syn1 1, new syn2 0 w/o warning (FALSE, the homotypic synonym syn2 should also be checked) - if any of the synonyms in the HG differs from state of the new acc taxon the warning should come
Old acc 1, old syn 1, new acc 0 => new syn 1 with warning (OK)
Old acc 1, old syn 0, new acc 0 => new syn 0 w/o warning (OK)
Generally correct, but not yet implemented for full homotypic group
Updated by Andreas Müller over 3 years ago
Katja, can you do the same check for "move accepted to new parent". I don't if this is already fully fixed, committed and build in nightly
Updated by Katja Luther over 3 years ago
Andreas Müller wrote:
Andreas Müller wrote:
Tested change accepted to synonym
Old parent 1, old acc 1, new acc 1 => new syn 0, w/o warning (FALSE, should be 1 w/o warning)
Old parent 1, old acc 1, new acc 0 => new syn 0, with warning (OK)
Old parent 1, old acc 0, new acc 1 => new syn 1, w/o warning (FALSE, should be 1 with warning)
Old parent 1, old acc 0, new acc 0 => new syn 0, with warning (FALS, should be 0 w/o warning)The evaluation assumes that the state of the old parent does not matter as discussed via email.
I think the warnings should come when the new accepted taxon and the synonym have different publish flags and I checked it again and it looks like this:
old acc 1, new acc 1 => new syn 1
old acc 1, new acc 0 => new syn 1 with warning
old acc 0, new acc 1 => new syn 0 with warning
old acc 0, new acc 0 => new syn 0
Updated by Katja Luther over 3 years ago
Andreas Müller wrote:
Katja, can you do the same check for "move accepted to new parent". I don't if this is already fully fixed, committed and build in nightly
moved acc 0, new parent 1 => warning
moved acc 0, new parent 0 => no warning
moved acc 1, new parent 1 => warning -> false
Updated by Katja Luther over 3 years ago
Katja Luther wrote:
Andreas Müller wrote:
Katja, can you do the same check for "move accepted to new parent". I don't if this is already fully fixed, committed and build in nightly
moved acc 0, new parent 1 => warning
moved acc 0, new parent 0 => no warning
moved acc 1, new parent 1 => warning -> false
this is fixed, now:
moved acc 0, new parent 1 => warning
moved acc 0, new parent 0 => no warning
moved acc 1, new parent 1 => no warning
moved acc 1, new parent 0 => warning
Updated by Andreas Müller over 3 years ago
- Subject changed from Change Synonym to Misapplied Name handles publish flag not correctly to Workflow support for publish flag
Updated by Andreas Müller over 3 years ago
Katja Luther wrote:
Andreas Müller wrote:
Tested MAN/ppSyn => Syn
acc 1, MAN 0 => Syn 0 (OK)
acc 1, MAN 1 => Syn 0 (FALSE)
acc 0, MAN 0 => Syn 0 (OK)
acc 0, MAN 1 => Syn 0 (FALSE)The new synonym always looses its state when publish flag was checked. But it should keep the state (without warning in this case).
this is fixed.
for me it does not work yet
Updated by Andreas Müller over 3 years ago
Katja Luther wrote:
Andreas Müller wrote:
Andreas Müller wrote:
Tested change accepted to synonym
Old parent 1, old acc 1, new acc 1 => new syn 0, w/o warning (FALSE, should be 1 w/o warning)
Old parent 1, old acc 1, new acc 0 => new syn 0, with warning (OK)
Old parent 1, old acc 0, new acc 1 => new syn 1, w/o warning (FALSE, should be 1 with warning)
Old parent 1, old acc 0, new acc 0 => new syn 0, with warning (FALS, should be 0 w/o warning)The evaluation assumes that the state of the old parent does not matter as discussed via email.
I think the warnings should come when the new accepted taxon and the synonym have different publish flags and I checked it again and it looks like this:
old acc 1, new acc 1 => new syn 1
old acc 1, new acc 0 => new syn 1 with warning
old acc 0, new acc 1 => new syn 0 with warning
old acc 0, new acc 0 => new syn 0
I agree about the handling of the warning and I tested again and everything works, except for 1x for the second case (no warning appeared) but I could not reproduce, so I guess in general it works. However, we should watch it a bit.
P.S: Looks like I mixed case 2 and 3 somehow in my original message
Updated by Katja Luther over 3 years ago
Andreas Müller wrote:
Katja Luther wrote:
Andreas Müller wrote:
Tested MAN/ppSyn => Syn
acc 1, MAN 0 => Syn 0 (OK)
acc 1, MAN 1 => Syn 0 (FALSE)
acc 0, MAN 0 => Syn 0 (OK)
acc 0, MAN 1 => Syn 0 (FALSE)The new synonym always looses its state when publish flag was checked. But it should keep the state (without warning in this case).
this is fixed.
for me it does not work yet
maybe I missed to commit, I try again, when jenkins is ready.
Updated by Katja Luther over 3 years ago
Katja Luther wrote:
Andreas Müller wrote:
Katja Luther wrote:
Andreas Müller wrote:
Tested MAN/ppSyn => Syn
acc 1, MAN 0 => Syn 0 (OK)
acc 1, MAN 1 => Syn 0 (FALSE)
acc 0, MAN 0 => Syn 0 (OK)
acc 0, MAN 1 => Syn 0 (FALSE)The new synonym always looses its state when publish flag was checked. But it should keep the state (without warning in this case).
this is fixed.
for me it does not work yet
maybe I missed to commit, I try again, when jenkins is ready.
tested again with nightly on test server:
acc 1, MAN 0 => Syn 1 (FALSE)
acc 1, MAN 1 => Syn 1 (OK)
acc 0, MAN 0 => Syn 0 (OK)
acc 0, MAN 1 => Syn 0 (FALSE)
but with local server with same TaxonServiceImpl it is:
acc 1, MAN 0 => Syn 0 (OK)
acc 1, MAN 1 => Syn 1 (OK)
acc 0, MAN 0 => Syn 0 (OK)
acc 0, MAN 1 => Syn 1 (OK)
Updated by Katja Luther over 3 years ago
Now also MAN -> Synonym works as expected:
acc 1, MAN 0 => Syn 0 (OK)
acc 1, MAN 1 => Syn 1 (OK)
acc 0, MAN 0 => Syn 0 (OK)
acc 0, MAN 1 => Syn 1 (OK)
Updated by Katja Luther over 3 years ago
- Description updated (diff)
- Status changed from Feedback to Resolved
- Assignee changed from Katja Luther to Andreas Müller
I think we can close this ticket, I checked all operations mentioned above.
Updated by Katja Luther over 3 years ago
- Description updated (diff)
create follow up ticket for the publish check of the whole homotypic group
Updated by Katja Luther over 3 years ago
- Related to bug #9516: Check publish flag of all homotypic synonyms when moving added
Updated by Andreas Müller over 3 years ago
- Status changed from Resolved to Closed
- Assignee changed from Andreas Müller to Katja Luther
- Priority changed from New to Highest
- % Done changed from 60 to 100
Yes this can be closed.
Updated by Andreas Müller about 3 years ago
- Tags changed from euro+med, fast to euro+med