Skip to content

LIMS-1923: Fix downstream accordion if files missing from disk#987

Merged
ndg63276 merged 1 commit intopre-release/2025-R4.4from
fix/LIMS-1923/downstream-broken-if-ligandfit-file-missing
Oct 14, 2025
Merged

LIMS-1923: Fix downstream accordion if files missing from disk#987
ndg63276 merged 1 commit intopre-release/2025-R4.4from
fix/LIMS-1923/downstream-broken-if-ligandfit-file-missing

Conversation

@ndg63276
Copy link
Copy Markdown
Collaborator

@ndg63276 ndg63276 commented Oct 1, 2025

JIRA ticket: LIMS-1923

Summary:

If the results files for the LigandFit or Shelxt pipelines are no longer on disk, the Downstream Processing accordion will no longer open. We should check the file exists before trying to read it.

Changes:

  • Do a file_exists check on files before reading them

To test:

  • Go to a data collection with a LigandFit result no longer on disk eg /dc/visit/nt37477-28/id/15841906
  • Click the Downstream Processing bar, check it opens, even if there are no results for LigandFit
  • Go to a data collection with a Shelxt result eg /dc/visit/cm40638-4/id/19599724
  • The result file still exists on disk, so temporarily change the filename on line 48 of Shelxt.php to simulate a missing file
  • Click the Downstream Processing bar, check it opens, even if there are no results for Shelxt

@gfrn
Copy link
Copy Markdown
Collaborator

gfrn commented Oct 8, 2025

It "locks up" for a second or two, but that's because I assume it's waiting on the filesystem, and it happens in prod too, so I guess it's fine

@ndg63276 ndg63276 changed the base branch from master to pre-release/2025-R4.4 October 14, 2025 09:09
@ndg63276 ndg63276 merged commit de1e91c into pre-release/2025-R4.4 Oct 14, 2025
2 checks passed
@ndg63276 ndg63276 deleted the fix/LIMS-1923/downstream-broken-if-ligandfit-file-missing branch October 14, 2025 09:10
ndg63276 added a commit that referenced this pull request Oct 22, 2025
* LIMS-1924: Fix update of container owner (#988)

* LIMS-1923: Fix downstream accordion if files missing from disk (#987)

* LIMS-1876: Fix URL parameters on fault report list (#979)

* LIMS-1841: Link downstream jobs to their upstream counterpart (#985)

* LIMS-1836: Improvements to reprocessing (#978)
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