Conversation
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
emenslin
left a comment
There was a problem hiding this comment.
This does not appear to be working unless I am misunderstanding the instructions. I query on any rank -> name and set equal to a taxon that has synonyms but when I check 'search synonyms' only the one node appears.
If I check 'search synonyms' in any tree aside from taxon I get an error.
11-07_15.21.mp4
grantfitzsimmons
left a comment
There was a problem hiding this comment.
The fundamental issue here is that 'Search Synonyms' is used most often for searching Collection Objects rather than tree names via a tree base table. The inadequate request this PR is based on unfortunately did not clarify this well enough.
See #752 (comment):
Hey guys, this one is quite important. If I create a collection object query and search for a preferred taxon name, it does not return collection objects with the original taxon name that are different (i.e. junior synonyms) to the preferred taxon name. The functionality is as expected in Specify 6.
Also related to this is that when querying collection objects from a taxon name in the taxon tree, the behaviour differs from Specify 6. In 6 you get records where either the taxon name or the preferred taxon name match the searched name, but in 7 it only searches on taxon name, so searches for a senior synonym don't return collection objects identified as the junior synonyms.
See that if I search for a synonym in a database (e.g. Opsanus tau) with Collection Object as the base table, I can still check "Search Synonyms":

This same behavior is expected in 7.
Using a copy of herb_rbge, I see the following:
Left: v7.11.2.1, Right: issue-752

Same query, exported and imported in both: Taxon Synonyms.json
On the v7.11.2.1, only results where synonyms do have an ID that isn't empty are returned (67532). On issue-752, all taxon records are returned (370611).
In neither scenario is "Search Synonyms" checked.
Searching directly for a Full Name for a synonymized node (e.g. Ericaceae) does yield correct results when querying Taxon directly:
Triggered by a44165a on branch refs/heads/issue-752
Triggered by defac6a on branch refs/heads/issue-752
|
@acwhite211 Is this a draft or can it be tested? |
Triggered by 9db7a80 on branch refs/heads/issue-752
Triggered by b9ac633 on branch refs/heads/issue-752
emenslin
left a comment
There was a problem hiding this comment.
- verify that the node is being return as well as its synonym
General functionality seems good but the testing instructions seem to be lacking after Grants comment (i.e. now search synonyms appear with CO as a base table but that is not mentioned in the testing instructions at all), I don't want to approve without further clarification on the testing instructions in case I miss anything.
Should the Search Synonyms checkbox work similarly to the Series checkbox, so that it only appears when there is some sort of tree information in the query? Not necessarily an issue, since it's the same in 6, I just wanted to mention it



Fixes #752
Checklist
and self-explanatory (or properly documented)
Testing instructions