Skip to content
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

Batch Edit: Some tree rank relationship fields throw errors in the QB #6315

Open
lexiclevenger opened this issue Mar 11, 2025 · 1 comment · May be fixed by #6196
Open

Batch Edit: Some tree rank relationship fields throw errors in the QB #6315

lexiclevenger opened this issue Mar 11, 2025 · 1 comment · May be fixed by #6196
Assignees
Labels
2 - Queries Issues that are related to the query builder or queries in general 2 - Trees Issues that are related to the tree system and related functionalities. Batch Edit
Milestone

Comments

@lexiclevenger
Copy link
Collaborator

Describe the bug
Batch Edit for Multiple Trees adds the ability to query on any field from a tree rank, fixing #6011. Some relationship fields added here throw errors after running the query.

To Reproduce
Steps to reproduce the behavior:

  1. Go to the Query Builder
  2. Choose a base table that has a tree (Taxon, Storage, Geography, etc.)
  3. Add a field from one of the following related tables (NOT (formatted) or (aggregated)): Definition, CreatedByAgent, ModifiedByAgent, Taxon Citations, Taxon Attributes. Most of the relationships that are hidden by default will throw an error.
  4. Run the Query
  5. See error

Expected behavior
Query results should be returned without errors.

Screenshots

Screen.Recording.2025-03-11.at.1.00.59.PM.mov

Crash Report

Specify 7 Crash Report - 2025-03-11T18_01_37.536Z.txt

Please fill out the following information manually:

@lexiclevenger lexiclevenger added 2 - Queries Issues that are related to the query builder or queries in general 2 - Trees Issues that are related to the tree system and related functionalities. Batch Edit labels Mar 11, 2025
@sharadsw
Copy link
Contributor

Thanks for writing up the issue! It looks like some code from #4929 was lost in my batch edit PRs. Trees are only supposed to have simple fields in the mapper

@sharadsw sharadsw added this to the 7.10.3 milestone Mar 12, 2025
@sharadsw sharadsw self-assigned this Mar 12, 2025
@sharadsw sharadsw linked a pull request Mar 12, 2025 that will close this issue
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2 - Queries Issues that are related to the query builder or queries in general 2 - Trees Issues that are related to the tree system and related functionalities. Batch Edit
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants