search

specify / specify7

Specify 7
https://www.specifysoftware.org/products/specify-7/
GNU General Public License v2.0
60 stars 36 forks source link

Saved Query Unable to get tree ranks #3337

Open carlosmbe opened 1 year ago

carlosmbe commented 1 year ago

Describe the bug Unable to see saved query. Error message is "Unable to get tree ranks for a LithoStrat table." I would prefer to get said error after requesting a query or count and not before.

To Reproduce Steps to reproduce the behavior:

  1. Go to NHMD Amber in NHMD Test as petrifiedroadkill
  2. Click on Queries
  3. Select down to VPalLabels
  4. See error

Expected behavior See Error after Query or Count. So user can fix Query. Or just not be able to save the broken query.

Screen Recording and Crash Report

https://user-images.githubusercontent.com/53784701/231522591-4f7c680f-9a89-465e-82c8-f07a7ee023de.mp4

Specify 7 Crash Report - 2023-04-12T16_26_00.150Z (1).txt

Desktop:

Database Name: NHMD Test

Reported By Carlos @carlosmbe

carlosmbe commented 1 year ago

Same error occurs with mappingVP Query. However, unlike VPalLabels. It's multiple errors and they're stacked. So dismissing takes time and it looks no change is happening. Unless you're looking at the error messages.

Unable to get tree ranks for a LithoStrat table Error: Unable to get tree ranks for a GeologicTimePeriod table

https://user-images.githubusercontent.com/53784701/231528373-aa262804-fa29-4ab6-8368-10b9efc57c55.mp4

carlosmbe commented 1 year ago

After taking a look at the query through Specify 6. Error does not happen in NHMD Vertebrate Palaeontology The error message is as follows in VPalLabels: Screenshot 2023-04-14 at 10 52 05 AM

grantfitzsimmons commented 1 year ago

Unable to get tree ranks for a LithoStrat table Error: Unable to get tree ranks for a GeologicTimePeriod table

Today we discovered that the crash occurs when querying a tree table (LithoStrat) which does not have any tree def items in the discipline. You can recreate this issue by querying LithoStrat or GeologicTimePeriod trees from a discipline that is not paleontology.

grantfitzsimmons commented 11 months ago

Encountered again today testing v7.9-dev

emenslin commented 1 month ago

Can recreate in edge (7.9.6), basically the same behavior except if you try the query mappingVP the errors do not stack

https://github.com/specify/specify7/assets/150940840/697245f4-d9e8-4d80-b653-e9bdaa0d6041

Specify 7 Crash Report - 2024-07-11T16_47_27.947Z.txt