Skip to content

Fix NPE when attempting to enable NextFlow in the root container#531

Merged
labkey-jeckels merged 1 commit intorelease24.11-SNAPSHOTfrom
24.11_fb_rootNPE
Mar 26, 2025
Merged

Fix NPE when attempting to enable NextFlow in the root container#531
labkey-jeckels merged 1 commit intorelease24.11-SNAPSHOTfrom
24.11_fb_rootNPE

Conversation

@labkey-jeckels
Copy link
Contributor

Rationale

java.lang.NullPointerException: Cannot invoke "org.labkey.api.data.Container.getId()" because "c" is null
org.labkey.api.data.PropertyCache.getCacheKey(PropertyCache.java:59)
org.labkey.api.data.PropertyCache.getProperties(PropertyCache.java:37)
org.labkey.api.data.AbstractPropertyStore.getProperties(AbstractPropertyStore.java:58)
org.labkey.api.data.AbstractPropertyStore.getProperties(AbstractPropertyStore.java:73)
org.labkey.api.data.PropertyManager.getProperties(PropertyManager.java:96)
org.labkey.nextflow.NextFlowManager.isEnabled(NextFlowManager.java:104)
org.labkey.nextflow.NextFlowController$BeginAction.getView(NextFlowController.java:201)
org.labkey.nextflow.NextFlowController$BeginAction.getView(NextFlowController.java:186)
org.labkey.api.action.FormViewAction.handleRequest(FormViewAction.java:110)

Changes

  • Don't try to get the inherited value when invoked at the root

@labkey-jeckels labkey-jeckels requested a review from a team March 24, 2025 18:36
@labkey-jeckels labkey-jeckels merged commit d6dda54 into release24.11-SNAPSHOT Mar 26, 2025
3 checks passed
@labkey-jeckels labkey-jeckels deleted the 24.11_fb_rootNPE branch March 26, 2025 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants