New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add group number support in Taxon queries #4724
base: production
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
@@ -22,7 +22,7 @@ | |||
DATE_PART_RE = re.compile(r'(.*)((NumericDay)|(NumericMonth)|(NumericYear))$') | |||
|
|||
# Pull out author or groupnumber field from taxon query fields. | |||
TAXON_FIELD_RE = re.compile(r'(.*) ((Author)|(GroupNumber))$') | |||
TAXON_FIELD_RE = re.compile(r'(.*) ((Author)|(groupNumber))$') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI: from what Ben told me, the only reason we limit what fields are accessible in tree queries is because that's what sp6 did
that reason might be less good today compared to how it was in 2013.
@realVinayak interested in tackling this at some point?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing instructions
Note: To properly test some of these, you will need to use Specify 6 locally to configure such a query or you can import this query that was created in Specify 6:
GroupNumber 6 Query.json
- Run a query that had the field "Group Number" added at the Family level in Specify 6
- Verify the output is consistent with Specify 6 and that all query functions work as expected
- Add "Group Number" to various ranks in the Taxon tree and ensure it can be searched on and results are displayed properly
- Verify that hiding and unhiding "Group Number" in the schema configuration make it appear as expected
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing instructions
- Run a query that had the field "Group Number" added at the Family level in Specify 6
- Verify the output is consistent with Specify 6 and that all query functions work as expected
- Add "Group Number" to various ranks in the Taxon tree and ensure it can be searched on and results are displayed properly
- Verify that hiding and unhiding "Group Number" in the schema configuration make it appear as expected
Everything looks to be working correctly, used it between a local specify 6 DB and used the query Grant posted for instructions on sdnhmherps as well.
And the query I made for the local DB (worked on sdnhmherps as well):
GroupNumberTest.json
Good work!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Run a query that had the field "Group Number" added at the Family level in Specify 6
- Verify the output is consistent with Specify 6 and that all query functions work as expected
- Add "Group Number" to various ranks in the Taxon tree and ensure it can be searched on and results are displayed properly
- Verify that hiding and unhiding "Group Number" in the schema configuration make it appear as expected
Query results are different between SP 6 and 7.
With one of the group number lines removed SP 7 matches.
And this is how it looks on v7
(without the GroupNumber lines)
Fixes #4723
This has been tested between both Specify 6 and 7. Existing queries that previously were non-functional created in 6 work now as well!
The previous configuration expected
<RankName> GroupNumber
, but Specify 6 constructs the string ID for these query items as<RankName> groupNumber
.Checklist
and self-explanatory (or properly documented)
Testing instructions
Note: To properly test some of these, you will need to use Specify 6 locally to configure such a query or you can import this query that was created in Specify 6:
GroupNumber 6 Query.json