Skip to content

LIMS-2016: Only check snapshot images for data collections#1017

Merged
ndg63276 merged 3 commits intopre-release/2026-R1.1from
fix/LIMS-2016/only-check-images-for-data-collections
Jan 6, 2026
Merged

LIMS-2016: Only check snapshot images for data collections#1017
ndg63276 merged 3 commits intopre-release/2026-R1.1from
fix/LIMS-2016/only-check-images-for-data-collections

Conversation

@ndg63276
Copy link
Copy Markdown
Collaborator

@ndg63276 ndg63276 commented Dec 15, 2025

JIRA ticket: LIMS-2016

Summary:

When viewing a list of data collections, the list of id's is sent to the backend to see whether they have snapshots available on disk. Unfortunately, when there are robot actions in the list, the robot action ids are added to the list of ids to check.

The backend fails to verify the ids are relevant, so checks for snapshots assuming each id is for a data collection, but as robot action ids are smaller numbers than data collection ids, it often ends up looking at years old data collections from other proposals and beamlines. Sometimes these have snapshots in invalid locations, leading to a PHP error.

Changes:

  • In the frontend, change the logic so that only "data" and "grid" id's are sent
  • In the backend, verify any id's received belong to the proposal currently being looked at
  • Send in a project id if viewing a project to bypass the backend proposal check (as projects can view multiple proposals)

To test:

@gfrn
Copy link
Copy Markdown
Collaborator

gfrn commented Dec 18, 2025

Had to add myself to the project to be able to view it, but LGTM

@ndg63276 ndg63276 changed the base branch from master to pre-release/2026-R1.1 January 6, 2026 10:20
@ndg63276 ndg63276 merged commit 9618abb into pre-release/2026-R1.1 Jan 6, 2026
2 checks passed
@ndg63276 ndg63276 deleted the fix/LIMS-2016/only-check-images-for-data-collections branch January 6, 2026 10:21
ndg63276 added a commit that referenced this pull request Jan 19, 2026
* LIMS-1986: Show main menu on click (#1013)

* LIMS-1987: Add a 'Download CSV' button to the Summary page (#1016)

* LIMS-2016: Only check snapshot images for data collections (#1017)

* Fix download of Dispensing CSV button
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants