Fix DQT filter counts when no fields selected #10229
Conversation
github-actions
bot
added
the
Module: dataquery
There was a problem hiding this comment.
Pull request overview
This PR fixes a UI crash in the Data Query Tool that occurred when users removed the PSCID field from the "Define Fields" tab, causing the "Define Filters" tab to become unresponsive. The fix adds proper guards and error handling for the candidate count fetch operation.
Key Changes:
- Added a guard condition to prevent fetching candidate counts when no fields are selected
- Refactored the count-fetching logic from promise chains to async/await with proper abort handling
- Implemented cleanup mechanism using AbortController to cancel in-flight requests when the component unmounts
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| throw new Error('Error creating query.'); | ||
| } | ||
| return resp.json(); | ||
| } | ||
| ).then( | ||
| (data) => { | ||
| fetch( | ||
| '/dataquery/queries/' | ||
| + data.QueryID + '/count', | ||
|
|
||
| const data = await createResp.json(); | ||
| const countResp = await fetch( | ||
| `/dataquery/queries/${data.QueryID}/count`, | ||
| { | ||
| method: 'GET', | ||
| credentials: 'same-origin', | ||
| } | ||
| ).then((resp) => { | ||
| if (!resp.ok) { | ||
| throw new Error('Could not get count.'); | ||
| } | ||
| return resp.json(); | ||
| }).then((result) => { | ||
| signal: controller.signal, | ||
| }, | ||
| ); | ||
|
|
||
| if (!countResp.ok) { | ||
| throw new Error('Could not get count.'); |
There was a problem hiding this comment.
The error messages "Error creating query." and "Could not get count." are generic and don't provide helpful context for debugging. Consider including the HTTP status code or response details to help identify the root cause when these errors occur. For example: throw new Error(\Error creating query: ${createResp.status} ${createResp.statusText}`);`
|
@driusan, could you please assign someone to review this PR? I’d really appreciate it. |
1 participant
Brief summary of changes
tab.
Testing instructions
Descriptions”, open the Data Query Tool.
Filters” tab opens normally.
npm run lint:js(passes; existing repo warnings only).Link(s) to related issue(s)
Before Fix
After Fix
